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

Action fix #133

Merged
merged 1 commit into from
Feb 1, 2024
Merged

Action fix #133

merged 1 commit into from
Feb 1, 2024

Conversation

QuanMPhm
Copy link
Contributor

@QuanMPhm QuanMPhm commented Jan 12, 2024

Addressing #91, the Github action file is changed to use the public image on ghcr. The unit tests should pass if the related PR in acct-manager is accepted first.

closes #91

@joachimweyl
Copy link

@QuanMPhm do you know the next steps for resolving the microshift test failure?

@QuanMPhm
Copy link
Contributor Author

Apparently it takes a lot longer to pull the account manager image from ghcr. I'll need to increase the timeout to 16 minutes

@knikolla
Copy link
Collaborator

Apparently it takes a lot longer to pull the account manager image from ghcr. I'll need to increase the timeout to 16 minutes

Isn't it because you need to update the pin, rather than the timeout? https://github.com/nerc-project/coldfront-plugin-cloud/blob/ea54a6af62da5301619ec6e4108b314d1f32f8e1/.github/workflows/test-py39-functional-microshift.yaml#L12C22-L12C62

@QuanMPhm
Copy link
Contributor Author

@knikolla You are right. The pin was the problem. I've updated it

@jtriley jtriley merged commit 15d4d7a into nerc-project:main Feb 1, 2024
4 checks passed
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.

Use published openshift-acct-mgt image rather than building our own
4 participants