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

Add CI job that verifies sidecar images digests in devfile registry #19660

Closed
mkuznyetsov opened this issue Apr 22, 2021 · 5 comments
Closed
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system.

Comments

@mkuznyetsov
Copy link
Contributor

mkuznyetsov commented Apr 22, 2021

Is your task related to a problem? Please describe.

Occasionally, images that we list for sidecars may have outdated digests. If that happens, we would like to find out about it as early as possible (before the scheduled weekly releases), and decide whether we should update the images

Describe the solution you'd like

A github actions workflow that runs daily to check if all images have correct digest, if not - fetch a correct one (with skopeo or curl) and submit a PR to devfile-registry to update it

Describe alternatives you've considered

Additional context

@mkuznyetsov mkuznyetsov added the kind/task Internal things, technical debt, and to-do tasks to be performed. label Apr 22, 2021
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Apr 22, 2021
@nickboldt
Copy link
Contributor

Specific reason for this is the quarkus sidecar image, which keeps updating its floating tags and removing older images' digests.

@nickboldt nickboldt added area/ci CI build and releases, PR testing, & whitelabel/productization issues severity/P1 Has a major impact to usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Apr 22, 2021
@nickboldt
Copy link
Contributor

WIP in eclipse-che/che-devfile-registry#380 awaits revision and merge.

@nickboldt
Copy link
Contributor

nickboldt commented Apr 30, 2021

Job has failed in https://github.com/eclipse-che/che-devfile-registry/runs/2471789961?check_suite_focus=true (12 hrs ago)

 ./arbitrary-users-patch/check_sidecar_images_digests.sh
  shell: /usr/bin/bash -e {0}
/home/runner/work/_temp/b0039ca4-c0f6-46d5-80be-30920f8bb6fd.sh: line 4: ./arbitrary-users-patch/check_sidecar_images_digests.sh: No such file or directory
Error: Process completed with exit code 127.

@nickboldt
Copy link
Contributor

@nickboldt
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

3 participants