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.

Sign up for GitHub

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

Jump to bottom

fix CI concurrency settings #4514

Merged
merged 1 commit into from
Aug 18, 2022
Merged

fix CI concurrency settings #4514

merged 1 commit into from
Aug 18, 2022

Conversation

mbien
Copy link
Member

@mbien mbien commented Aug 17, 2022

add base_ref to the group key so that two parallel branch sync PRs won't cancel each other

@mbien
fix CI concurrency settings
89ef85f 
 add base_ref to the group so that two parallel branch sync PRs
 won't cancel each other
@mbien mbien added the CI Continuous Integration label Aug 17, 2022
@mbien mbien added this to the NB16 milestone Aug 17, 2022
neilcsmith-net
neilcsmith-net approved these changes Aug 18, 2022
View reviewed changes
Copy link
Member

@neilcsmith-net neilcsmith-net left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@mbien mbien merged commit b138405 into apache:master Aug 18, 2022
@mbien
Copy link
Member Author

mbien commented Aug 18, 2022

might be a github issue, but the merge run errored the workflow.

Error: .github#L1
An unexpected error has occurred and we've been automatically notified. Errors are sometimes temporary, so please try again.

If the problem persists, please check whether the Actions service is operating normally at https://githubstatus.com/. If not, please try again once the outage has been resolved.

Should you need to contact Support, please visit https://support.github.com/contact and include request ID: 877A:841C:88794C:9811F2:62FE4A01

@mbien
Copy link
Member Author

mbien commented Aug 18, 2022

might be a random github anomaly. Another PR merged fine to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@neilcsmith-net neilcsmith-net neilcsmith-net approved these changes

Assignees
No one assigned
Labels
CI Continuous Integration
Projects
None yet
Milestone
NB16
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mbien @neilcsmith-net