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

Deleting obsolete ExecutorStepTest.accessPermittedOnlyFromCurrentBuild #190

Merged
merged 2 commits into from
Dec 22, 2021

Conversation

jglick
Copy link
Member

@jglick jglick commented Dec 22, 2021

Test introduced in #177 begins failing after jenkinsci/jenkins#5885 and is rendered meaningless by that cleanup. Test coverage from #170 remains.

@car-roll car-roll merged commit 53373f0 into jenkinsci:master Dec 22, 2021
@basil
Copy link
Member

basil commented Dec 22, 2021

@car-roll Would be nice to get a release of this. Without a release, this test fails in PCT / bom-weekly for 2.326.

@car-roll
Copy link
Collaborator

car-roll commented Dec 22, 2021

@basil looks like there are some infra? issues preventing automated release atm

@basil
Copy link
Member

basil commented Dec 22, 2021

there are some issues

INFRA-3165 FTR

@jglick jglick deleted the accessPermittedOnlyFromCurrentBuild branch December 22, 2021 21:01
@basil
Copy link
Member

basil commented Dec 23, 2021

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants