-
Notifications
You must be signed in to change notification settings - Fork 60
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
Test PR 225 (install/pre-install: Update containerd) #226
Test PR 225 (install/pre-install: Update containerd) #226
Conversation
When building the pre-install-payload image for CI it needs to pull/push the image from a local registry that is not protected. The `docker manifest` commands (e.g. create) refuses to connect in an unsecure registry by default, therefore the pre-install-payload build fail. That can be solved by passing the --insecure flag to `docker manifest` thus this change allow to pass extra flags to that command. Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Currently changes on install/pre-install-payload directory aren't tested because the scripts aren't re-building the pre-install-payload image. With this change the image will always be built and used. It was added more two dependencies: - kustomize: used to edit the kustomization file so to update the pre-install-payload image - qemu-user-static: used by docker buildx to build the pre-install-payload image for multiple architectures. It also needs to pass the `--insecure` to `docker manifest` commands because the image is pushed/pulled to a local insecure registry, otherwise `docker manifest` fails Fixes confidential-containers#177 Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
/test-kata-qemu |
On both kata-clh and kata-qemu jobs we are getting connectivety issues to build the pre-install image:
|
This might be a long shot but can you replace the repo url with: Otherwise (if that doesn't help) lets do like you suggested: fetch kubectl with curl directly. |
ddeec6f
to
2142f5e
Compare
hmmm... trying this solution first: 91e2c8a |
/test-kata-qemu |
2142f5e
to
9c35fe3
Compare
Forgot to install gpg in the container... let's see: /test-kata-qemu |
Currently we have installed the kubectl package from google cloud but it has been problematic due to many networking and certification issues like in below. Instead let's pull and install the kubectl binary (no packaged). ``` 10:37:14 confidential-containers#9 9.663 Err:2 https://packages.cloud.google.com/apt kubernetes-xenial InRelease 10:37:14 confidential-containers#9 9.663 The following signatures couldn't be verified because the public key is not available: NO_PUBKEY B53DC80D13EDEF05 10:37:14 confidential-containers#9 9.681 Hit:5 http://archive.ubuntu.com/ubuntu focal-backports InRelease 10:37:15 confidential-containers#9 9.824 Reading package lists... 10:37:15 confidential-containers#9 10.81 W: GPG error: https://packages.cloud.google.com/apt kubernetes-xenial InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY B53DC80D13EDEF05 10:37:15 confidential-containers#9 10.81 E: The repository 'https://apt.kubernetes.io/ kubernetes-xenial InRelease' is not signed. ``` Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Update containerd version now we have a new release Signed-off-by: stevenhorsman <steven@uk.ibm.com>
9c35fe3
to
32683e7
Compare
Updated this PR to include the commit cf3aaef which changed the pre-install Dockerfile to pull the kubectl binary, instead of installing via package which has been a nightmare. /test-kata-qemu |
/test |
SEV tests have failed with:
and
I've requeued, so hoping for third time lucky |
The third sev attempt failed with:
again, so I'm not going to try and re-do it a fourth time. @UnmeshDeodhar @ryansavino - do you have any ideas? Any reason why the node can't built the new container image? |
It is failing to push the image to a local registry. I've almost sure I saw that error before but can't remember the reason. As I commented on PR #225 I think we can disregard the sev results, merge that PR then test again with the new built image. |
I am going to close this as the PR #225 is merged already. |
I'm trying to merge #179 before PR #225 so that the later would be tested before merge. However, PR #179 is blocked due to what seems network issues on SEV and TDX machine when it comes to build the pre-install image.
As we are approaching the 0.7.0 release I think it is more prudent to unblock the merge of #225. So this PR, where I pilled up #225 on #179 then I will run the non-TEE tests jobs. After the merge of #225 and when a new pre-install image is built then we will have the opportunity to test the updated containerd on SEV and TDX (as the pre-install image won't be built on those machines I expect the test suite to just run).
That's my rationale for this PR. Marking as do-not-merge.