Skip to content

Conversation

lemeurherve
Copy link
Member

This draft PR is checking the incremental produced from the LTS 2.528.1 RC backport and is not intended to be reviewed nor merged.

Refs:

Testing done

CI

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

@lemeurherve lemeurherve added the full-test Test all LTS lines in this PR and do not halt upon first error. label Oct 1, 2025
MarkEWaite added a commit to MarkEWaite/bom that referenced this pull request Oct 1, 2025
Configuration as code plugin 1985.vdda_32d0c4ea_b_ includes the 'allow
empty map' fix from pull request:

* jenkinsci/configuration-as-code-plugin#2712

That fix needed changes in the tests of 3 plugins:

* azure-vm-agents: AdvancedConfigAsCodeTest.exportExportConfiguration
  and BasicConfigAsCodeTest.exportBasicConfiguration
* github-branch-source: GitHubAppCredentialsJCasCCompatibilityTest.should_support_configuration_export
* ldap: CascSecurityRealmTest.export_ldap_no_secret

All 3 of those plugins have been released with updated tests to
adapt to the improved behavior of Configuration as code plugin
1985.vdda_32d0c4ea_b_, but the azure-vm-agents and ldap releases both
require 2.516.x.  Exclude those tests until we drop plugin BOM support
for the 2.504.x line.

Detected in tests of 2.528.1-rc by @lemeurherve in pull request:

* jenkinsci#5770

Was not detected with last week's BOM release because in order to save
money we only test the weekly, oldest, and newest LTS versions when
performing a plugin BOM release.  Last week, 2.504.x was not included
in that test because we tested 2.492.x, 2.516.x, and weekly.  This week
we dropped the 2.492.x line and added the 2.528..x line, so we will test
2.504.x, 2.528.x, and weekly.
@MarkEWaite
Copy link
Contributor

Thanks for running this full-test! It detected a problem that I missed last week. The problem is not related to 2.528.x. This pull request shows that 2.528.1-rc is passing tests in the plugin BOM. I've submitted a pull request to fix the problem that I missed last week:

MarkEWaite added a commit that referenced this pull request Oct 1, 2025
Configuration as code plugin 1985.vdda_32d0c4ea_b_ includes the 'allow
empty map' fix from pull request:

* jenkinsci/configuration-as-code-plugin#2712

That fix needed changes in the tests of 3 plugins:

* azure-vm-agents: AdvancedConfigAsCodeTest.exportExportConfiguration
  and BasicConfigAsCodeTest.exportBasicConfiguration
* github-branch-source: GitHubAppCredentialsJCasCCompatibilityTest.should_support_configuration_export
* ldap: CascSecurityRealmTest.export_ldap_no_secret

All 3 of those plugins have been released with updated tests to
adapt to the improved behavior of Configuration as code plugin
1985.vdda_32d0c4ea_b_, but the azure-vm-agents and ldap releases both
require 2.516.x.  Exclude those tests until we drop plugin BOM support
for the 2.504.x line.

Detected in tests of 2.528.1-rc by @lemeurherve in pull request:

* #5770

Was not detected with last week's BOM release because in order to save
money we only test the weekly, oldest, and newest LTS versions when
performing a plugin BOM release.  Last week, 2.504.x was not included
in that test because we tested 2.492.x, 2.516.x, and weekly.  This week
we dropped the 2.492.x line and added the 2.528..x line, so we will test
2.504.x, 2.528.x, and weekly.
@MarkEWaite
Copy link
Contributor

Closing since the 2.528.1 release candidate is announced and available. Changelog and upgrade pull request is available as well. Release checklist is in progress.

@MarkEWaite MarkEWaite closed this Oct 4, 2025
@lemeurherve lemeurherve deleted the test-2.528.1-rc branch October 4, 2025 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-test Test all LTS lines in this PR and do not halt upon first error.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants