Skip to content

pretesting https://github.com/jenkinsci/jenkins/pull/23873 #6031

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
Draft
jtnord wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

pretesting https://github.com/jenkinsci/jenkins/pull/23873 #6031

jtnord wants to merge 3 commits into from

Conversation

@jtnord
Copy link
Member

@jtnord jtnord commented Dec 1, 2025

pretesting jenkinsci/jenkins#23873

Testing done

None

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@MarkEWaite
Copy link
Contributor

MarkEWaite commented Dec 1, 2025

I added the weekly-test label and started the build on ci.jenkins.io so that plugin tests will run

@timja
Copy link
Member

timja commented Dec 1, 2025

Few tests are failing due to the change in behaviour:

Expecting message to be:
  "anonymous is missing the Overall/Administer permission"
but was:
  "anonymous is missing the Overall/Manage permission"

@jtnord jtnord mentioned this pull request Dec 2, 2025
Merged
6 tasks
jtnord added a commit to jtnord/config-file-provider-plugin that referenced this pull request Dec 2, 2025
@jtnord
Adapt to enablement of Overall/Manage permission
dc511de 
Adapts to jenkinsci/jenkins#23873 so the tests pass on the current Jenkins version as well as a version containing the PR

discovered by jenkinsci/bom#6031 in preparation for jenkinsci/jenkins#23873

tested with mvn test -Djenkins.version=2.540-rc37747.87da_150079a_1
@jtnord jtnord mentioned this pull request Dec 2, 2025
Merged
6 tasks
jtnord added a commit to jtnord/kubernetes-plugin that referenced this pull request Dec 2, 2025
@jtnord
Adapt to enablement of Overall/Manage permission
11fb911 
Adapts to jenkinsci/jenkins#23873 so the tests pass on the current Jenkins version as well as a version containing the PR

discovered by jenkinsci/bom#6031 in preparation for jenkinsci/jenkins#23873

tested with mvn test -Djenkins.version=2.540-rc37747.87da_150079a_1
@jtnord jtnord mentioned this pull request Dec 2, 2025
Merged
6 tasks
jtnord added a commit to jtnord/mailer-plugin that referenced this pull request Dec 2, 2025
@jtnord
Adapts to jenkinsci/jenkins#23873 so the tests pass on the current Je…
2cb2c35 
…nkins version as well as a version containing the PR

discovered by jenkinsci/bom#6031 in preparation for jenkinsci/jenkins#23873

tested with mvn test -Djenkins.version=2.540-rc37747.87da_150079a_1
@jtnord jtnord mentioned this pull request Dec 2, 2025
Merged
6 tasks
jtnord added a commit to jtnord/role-strategy-plugin that referenced this pull request Dec 2, 2025
@jtnord
Adapt to enablement of Overall/Manage permission
f3b0ac8 
Adapts to jenkinsci/jenkins#23873 so the tests pass on the current Jenkins version as well as a version containing the PR

discovered by jenkinsci/bom#6031 in preparation for jenkinsci/jenkins#23873

tested with mvn test -Djenkins.version=2.540-rc37747.87da_150079a_1
@jtnord jtnord mentioned this pull request Dec 2, 2025
Merged
6 tasks
@jtnord
Copy link
Member Author

jtnord commented Dec 2, 2025 •
edited
Loading

Few tests are failing due to the change in behaviour:

Expecting message to be:
  "anonymous is missing the Overall/Administer permission"
but was:
  "anonymous is missing the Overall/Manage permission"

PRs should have been filed for all plugins impacted in the BOM:

The gitlab-kubernetes plugin failure seems to be an incompatibility with a different plugin unrelated to this change.

java.lang.NoSuchMethodError: 'void io.jenkins.plugins.gitlabserverconfig.credentials.PersonalAccessTokenImpl.<init>(com.cloudbees.plugins.credentials.CredentialsScope, java.lang.String, java.lang.String, java.lang.String)'
	at io.jenkins.plugins.gitlabcredentialsk8s.GitLabCredentialConverter.convert(GitLabCredentialConverter.java:67)
	at io.jenkins.plugins.gitlabcredentialsk8s.GitLabCredentialConverterTest.canConvertAValidSecret(GitLabCredentialConverterTest.java:116)

@timja
Copy link
Member

timja commented Dec 2, 2025

Gitlab issue should be resolved according to jenkinsci/gitlab-branch-source-plugin#479 (comment)

@alecharp
Copy link
Member

alecharp commented Dec 2, 2025

Can this be re-run with Mailer version 525.v2cb_2c35b_15c2? (incremental from James' pull request before I merge / release mailer plugin)

github-merge-queue bot pushed a commit to jenkinsci/mailer-plugin that referenced this pull request Dec 2, 2025
@jtnord
Adapts to jenkinsci/jenkins#23873 so the tests pass on the current Je…
2458bd8 
…nkins version as well as a version containing the PR (#401)

discovered by jenkinsci/bom#6031 in preparation for jenkinsci/jenkins#23873

tested with mvn test -Djenkins.version=2.540-rc37747.87da_150079a_1
@jtnord
Copy link
Member Author

jtnord commented Dec 2, 2025

Can this be re-run with Mailer version 525.v2cb_2c35b_15c2? (incremental from James' pull request before I merge / release mailer plugin)

13a7961

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

Reviewers

No reviews

Assignees

No one assigned

Labels

do-not-merge weekly-test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jtnord @MarkEWaite @timja @alecharp