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

test/e2e: kbs: Use the cached kbs-client #2121

Merged

Conversation

stevenhorsman
Copy link
Member

To save us storage space, time and the requirement
on the rust toolchain, we can pull down the cached
kbs-client rather than building it ourselves.

Update checkout_kbs.sh to use oras to pull the kbs-client
binary from the cache rather than building it.

@stevenhorsman
Copy link
Member Author

stevenhorsman commented Oct 16, 2024

Draft as this needs rebasing after #2099 is merged and depends on some changes in #2074

To save us storage space, time and the requirement
on the rust toolchain, we can pull down the cached
kbs-client rather than building it ourselves.

- Update checkout_kbs.sh to use oras to pull the kbs-client
binary from the cache rather than building it.

- Fix the kbs-client path now that it is pulled into `kbs`,
not build into target/releases

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
- Remove the rust toolchain install from the workflow
now it's no longer needed.
> [!NOTE]
> We need a follow up PR to remove rust from versions.yaml
> separately due to the pull_request_target issues
- Remove unused envs

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
@stevenhorsman stevenhorsman marked this pull request as ready for review October 22, 2024 15:40
@stevenhorsman stevenhorsman requested a review from a team as a code owner October 22, 2024 15:40
@stevenhorsman stevenhorsman merged commit aab207c into confidential-containers:main Oct 22, 2024
28 checks passed
@stevenhorsman stevenhorsman deleted the kbs-client-cache branch October 22, 2024 16:26
stevenhorsman added a commit to stevenhorsman/cloud-api-adaptor that referenced this pull request Oct 22, 2024
After confidential-containers#2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
stevenhorsman added a commit to stevenhorsman/cloud-api-adaptor that referenced this pull request Oct 23, 2024
After confidential-containers#2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
stevenhorsman added a commit to stevenhorsman/cloud-api-adaptor that referenced this pull request Oct 23, 2024
After confidential-containers#2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
stevenhorsman added a commit to stevenhorsman/cloud-api-adaptor that referenced this pull request Oct 24, 2024
After confidential-containers#2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
stevenhorsman added a commit to stevenhorsman/cloud-api-adaptor that referenced this pull request Oct 25, 2024
After confidential-containers#2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
stevenhorsman added a commit to stevenhorsman/cloud-api-adaptor that referenced this pull request Oct 25, 2024
After confidential-containers#2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
stevenhorsman added a commit to stevenhorsman/cloud-api-adaptor that referenced this pull request Oct 29, 2024
After confidential-containers#2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
stevenhorsman added a commit to stevenhorsman/cloud-api-adaptor that referenced this pull request Oct 29, 2024
After confidential-containers#2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
wainersm pushed a commit that referenced this pull request Oct 29, 2024
After #2121 the rust version is no longer needed

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test_e2e_libvirt Run Libvirt e2e tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants