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

fix(test-composite-actions): use newer ghcr image #318

Conversation

sasakisasaki
Copy link
Contributor

@sasakisasaki sasakisasaki commented Sep 5, 2024

Description

Fix not to use outdated image in the workflow.

Related links

Not provided.

Tests performed (edited)

The target workflow is now functional. I believe this is enough as testing.

Notes for reviewers (edited)

Before this fix, the workflow test-get-self-packages-and-get-modified-packages was failing with following error.

  /usr/bin/docker --config /home/runner/work/_temp/.docker_5d3b7430-f2fa-4f8f-9a5c-a5d59f5f0360 login ghcr.io -u sasakisasaki --password-stdin
  /usr/bin/docker --config /home/runner/work/_temp/.docker_5d3b7430-f2fa-4f8f-9a5c-a5d59f5f0360 pull ghcr.io/autowarefoundation/autoware-universe:humble-latest
  Error response from daemon: denied
  Warning: Docker pull failed with exit code 1, back off 5.707 seconds before retry.
  /usr/bin/docker --config /home/runner/work/_temp/.docker_5d3b7430-f2fa-4f8f-9a5c-a5d59f5f0360 pull ghcr.io/autowarefoundation/autoware-universe:humble-latest
  Error response from daemon: denied
  Warning: Docker pull failed with exit code 1, back off 1.836 seconds before retry.
  /usr/bin/docker --config /home/runner/work/_temp/.docker_5d3b7430-f2fa-4f8f-9a5c-a5d59f5f0360 pull ghcr.io/autowarefoundation/autoware-universe:humble-latest
  Error response from daemon: denied
  Error: Docker pull failed with exit code 1

Interface changes

No interface change exists in this PR

ROS Topic Changes

No change exists in this PR.

ROS Parameter Changes

No change exists in this PR.

Effects on system behavior

The failing workflow test-composite-actions will succeed.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

  • The PR follows the pull request guidelines.
  • The PR has been properly tested.
  • The PR has been reviewed by the code owners.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.
  • The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

  * The last update of the image (ghcr.io/autowarefoundation/autoware-universe:humble-latest)
    was 6 months ago
  * Before this fix, the workflow was failing with following error:
```
Error response from daemon: Head "https://ghcr.io/v2/autowarefoundation/autoware-universe/manifests/humble-latest": unauthorized
```

Signed-off-by: Junya Sasaki <[email protected]>
@xmfcx xmfcx marked this pull request as ready for review September 5, 2024 09:10
@xmfcx xmfcx merged commit a173e01 into autowarefoundation:main Sep 5, 2024
18 checks passed
@@ -146,7 +146,8 @@ jobs:
matrix:
container:
- ros:humble
- ghcr.io/autowarefoundation/autoware-universe:humble-latest
- ghcr.io/autowarefoundation/autoware:latest-autoware-universe
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- ghcr.io/autowarefoundation/autoware:latest-autoware-universe
- ghcr.io/autowarefoundation/autoware:latest-universe-devel

is better from today.
autowarefoundation/autoware#5170

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you @youtalk for your proposal. The fix will be applied in this PR as this PR is merged.

sasakisasaki added a commit that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants