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

Remove concurrency options from ghcr.yml #3457

Closed
wants to merge 3 commits into from

Conversation

tobitege
Copy link
Collaborator

@tobitege tobitege commented Aug 19, 2024

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?

  • Removed concurrency option from ghcr.yml to avoid skipping of crucial workflows.

Attempting to get the Test Runtime (eventstream) get unstuck, for testing :

  • split ghcr_build_runtime into 2 separate platform jobs for amd64 and arm64
  • changed needs for Test Runtime (eventstream) to just the amd64 job

@tobitege tobitege added the backend Related to backend label Aug 19, 2024
@tobitege tobitege requested review from li-boxuan and mamoodi August 19, 2024 09:13
iFurySt
iFurySt previously approved these changes Aug 19, 2024
@tobitege tobitege enabled auto-merge (squash) August 19, 2024 10:59
@tobitege tobitege disabled auto-merge August 19, 2024 10:59
@tobitege tobitege dismissed iFurySt’s stale review August 19, 2024 12:19

more changes

@tobitege tobitege marked this pull request as draft August 19, 2024 12:19
@@ -149,7 +216,7 @@ jobs:
test_runtime:
Copy link
Collaborator

Choose a reason for hiding this comment

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

When you say you are getting it unstuck, do you mean the job never runs sometimes? Or the job itself gets stuck?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It just doesn't start, ever. Across multiple PR's.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah interesting. It may be just because of the concurrency. Did you try just removing that at first?

In the builds that it never starts, do you see the ghcr_build_runtime job pass successfully?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

in this one from yesterday it all ran fine:
#3446

this one with latest "ok" commit from 7 hours ago is stuck:
#3410

idk what's going on tbh lol

Copy link
Collaborator

Choose a reason for hiding this comment

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

Close this PR now?

@tobitege
Copy link
Collaborator Author

This was good for testing, but closing as there'll be other PR's for it. 😄

@tobitege tobitege closed this Aug 19, 2024
@tobitege tobitege deleted the ghcr-no-concurrency branch August 19, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants