Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump base image #2465

Merged
merged 13 commits into from
Oct 14, 2021
Merged

Bump base image #2465

merged 13 commits into from
Oct 14, 2021

Conversation

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Sep 18, 2021
@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 18, 2021
@aojea
Copy link
Contributor Author

aojea commented Sep 18, 2021

/assign @BenTheElder
I can't build the node image :(

@aojea
Copy link
Contributor Author

aojea commented Sep 18, 2021

/hold

https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kind/2465/pull-kind-conformance-parallel-1-17/1439353716925796352
pull-kind-conformance-parallel-1-17

I have the same problem locally

> runtime/cgo: pthread_create failed: Operation not permitted
SIGABRT: abort
PC=0x7f03710d6603 m=0 sigcode=18446744073709551610
goroutine 0 [idle]:
runtime: unknown pc 0x7f03710d6603
...
gs     0x0
runtime/cgo: pthread_create failed: Operation not permitted
SIGABRT: abort
PC=0x7fbde81fc603 m=0 sigcode=18446744073709551610
...
Image build Failed! Failed to load images [command "docker exec --privileged -i kind-build-1632004227-417728918 ctr --namespace=k8s.io images import --all-platforms --no-unpack -" failed with error: exit status 2, write |1: broken pipe]

it has to be related to cross building

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 18, 2021
@BenTheElder
Copy link
Member

I get a different failure at HEAD now:

#17 0.108 Installing containerd ...
#17 0.162 curl: (6) getaddrinfo() thread failed to start
#17 0.162 
#17 1.164 curl: (6) getaddrinfo() thread failed to start
#17 1.164 
#17 3.166 curl: (6) getaddrinfo() thread failed to start
#17 3.166 
#17 7.170 curl: (6) getaddrinfo() thread failed to start
#17 7.170 
#17 15.18 curl: (6) getaddrinfo() thread failed to start
#17 15.18 
#17 31.19 curl: (6) getaddrinfo() thread failed to start

docker 20.10.2, kernel 5.10.40

@BenTheElder
Copy link
Member

docker/buildx#772 (comment) ?

@BenTheElder
Copy link
Member

BenTheElder commented Sep 24, 2021

Also reproduces on docker desktop currently, but I'm not at the latest version. Updating.

EDIT: not sure I can run docker desktop anymore at work actually ... forgot about that.

@qinqon
Copy link
Contributor

qinqon commented Sep 30, 2021

Looks like the fix is in place ? docker/buildx#772 (comment)

@aojea
Copy link
Contributor Author

aojea commented Oct 3, 2021

@qinqon
Copy link
Contributor

qinqon commented Oct 4, 2021

🤕 https://www.mail-archive.com/[email protected]/msg5953123.html

Another one?

@aojea
Copy link
Contributor Author

aojea commented Oct 8, 2021

/test all
something is wrong with 1.17

https://pastebin.com/wccujCtG

@aojea
Copy link
Contributor Author

aojea commented Oct 8, 2021

/test all

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 13, 2021
@aojea
Copy link
Contributor Author

aojea commented Oct 13, 2021

I think that the problem is that the krte image used in 1.17 is too old

networkop and others added 2 commits October 14, 2021 00:17
When pods are running at kind cluster their product_uuid and product_name
is the same since the share the kernel vfs,
this PR add a new mount to OCI spec to bind mount node's product_uuid
and product_name into pod's containers.

Signed-off-by: Quique Llorente <[email protected]>
@aojea
Copy link
Contributor Author

aojea commented Oct 14, 2021

@AkihiroSuda some of the building fails trying to use ctr , are you going to backport this in containerd?
containerd/containerd@55923da

@AkihiroSuda
Copy link
Member

@AkihiroSuda some of the building fails trying to use ctr , are you going to backport this in containerd? containerd/containerd@55923da

That commit has been already backported in v1.5.6
https://github.com/containerd/containerd/commits/v1.5.6

@AkihiroSuda
Copy link
Member

https://storage.googleapis.com/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kind/2465/pull-kind-conformance-parallel-1-17/1448552573672886272/build-log.txt

Image build Failed! Failed to load images [command "docker exec --privileged -i kind-build-1634197355-636563126 ctr --namespace=k8s.io images import --all-platforms --no-unpack -" failed with error: exit status 2, write |1: broken pipe]
runtime/cgo: pthread_create failed: Operation not permitted

The error looks like an issue of the host Docker rather than ctr.

The Docker (Moby) PR (moby/moby#42681) was already cherry-picked into to v20.10 branch (moby/moby#42836) targeting at v20.10.10, but v20.10.10 is not released yet.

@aojea
Copy link
Contributor Author

aojea commented Oct 14, 2021

indeed, we have so many nested layers that is hard to know 😄
let me try this , I think that the problem is in the image used to build the node

diff --git a/pkg/build/nodeimage/buildcontext.go b/pkg/build/nodeimage/buildcontext.go
index 5b409df0..f07ca217 100644
--- a/pkg/build/nodeimage/buildcontext.go
+++ b/pkg/build/nodeimage/buildcontext.go
@@ -360,6 +360,7 @@ func (c *buildContext) createBuildContainer() (id string, err error) {
                        "--entrypoint=sleep",
                        "--name=" + id,
                        "--platform=" + dockerBuildOsAndArch(c.arch),
+                       "--security-opt", "seccomp=unconfined", // ignore seccomp
                },
                []string{
                        "infinity", // sleep infinitely to keep the container around

@aojea
Copy link
Contributor Author

aojea commented Oct 14, 2021

The Docker (Moby) PR (moby/moby#42681) was already cherry-picked into to v20.10 branch (moby/moby#42836) targeting at v20.10.10, but v20.10.10 is not released yet.

I think most distros has already patched it, but I expect users hitting it in kind and reporting issues, we have to add this to the release notes

@aojea
Copy link
Contributor Author

aojea commented Oct 14, 2021

Awesome, it was that, now everything works, the job that was failing here pass now
And it can builds node images for all kubernetes versions since 1.14
https://github.com/aojea/kind-images/runs/3892473052?check_suite_focus=true
/hold cancel
@BenTheElder we are ready for release

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 14, 2021
Copy link
Member

@BenTheElder BenTheElder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 14, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aojea, BenTheElder

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@BenTheElder
Copy link
Member

@aojea please bump the default node image to one built with this base image X the latest k8s patch release

@rodnymolina
Copy link

And it can builds node images for all kubernetes versions since 1.14 ...

@BenTheElder, @aojea, could you please help me understand the typical procedure followed by KinD to have new node images being re-created when the base one is bumped? Can't find any info about this topic ...

I see that a new kindest/node:v1.22.2 image has been published, but what about other tags also supported by KinD's current v0.11.1 release (e.g. v1.21.x, v1.20.x, etc)? Are those images created only during release cycles? Sure, I can create them myself, but I was hoping to be able to make use of KinD's official images once that this PR got merged ...

Thanks.

@BenTheElder
Copy link
Member

I see that a new kindest/node:v1.22.2 image has been published, but what about other tags also supported by KinD's current v0.11.1 release (e.g. v1.21.x, v1.20.x, etc)? Are those images created only during release cycles? Sure, I can create them myself, but I was hoping to be able to make use of KinD's official images once that this PR got merged ...

There's a few issues tracking this. Currently pre-built images are primarily published upon releasing the binary.

@aojea
Copy link
Contributor Author

aojea commented Oct 18, 2021

@rodnymolina if is for testing you can use the ones I've created for testing, they are published in
https://quay.io/repository/aojea/kindnode?tab=tags , the tag is the stable version 1.19, 1.20, ...

@rodnymolina
Copy link

Thanks @BenTheElder and @aojea. More than testing I'm interested in having this issue fixed for the existing KinD-in-Sysbox users. As I said, I can build the images myself, but it's best for everyone to rely on the official ones.

ormergi added a commit to ormergi/kubevirtci that referenced this pull request Oct 20, 2021
The new KinD k8s-1.22 nodes includes a change [1] [2] that allows us
to stop using PodPresets and still support Kubevirt Live Migration.

[1] kubernetes-sigs/kind#2465
[2] kubernetes-sigs/kind#2321

Signed-off-by: Or Mergi <[email protected]>
ormergi added a commit to ormergi/kubevirtci that referenced this pull request Oct 20, 2021
The new KinD k8s-1.22 nodes includes a change [1] [2] that allows us
to stop using PodPresets and still support Kubevirt Live Migration.

[1] kubernetes-sigs/kind#2465
[2] kubernetes-sigs/kind#2321

Signed-off-by: Or Mergi <[email protected]>
kubevirt-bot pushed a commit to kubevirt/kubevirtci that referenced this pull request Oct 21, 2021
The new KinD k8s-1.22 nodes includes a change [1] [2] that allows us
to stop using PodPresets and still support Kubevirt Live Migration.

[1] kubernetes-sigs/kind#2465
[2] kubernetes-sigs/kind#2321

Signed-off-by: Or Mergi <[email protected]>
@felipecrs
Copy link
Contributor

Is there any ETA for the next version of KinD which will include this?

@BenTheElder
Copy link
Member

Probably later this week. We don't have any outstanding known regressions now, but Antonio and I have some other obligations early this week.

@felipecrs
Copy link
Contributor

That would be awesome. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WSL2 ERROR: failed to create cluster
10 participants