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

Revert ssh-credentials 342.x #3404

Merged

Conversation

MarkEWaite
Copy link
Contributor

Revert ssh-credentials 342.x

Plugin loading fails in several BOM tests, including:

  • LINE=2.440.x PLUGINS=git-server TEST=InjectedTest bash local-test.sh

The test report lists the other plugins with failures as:

  • workflow-basic-steps with 2.440.x
  • git with 2.440.x and 2.452.x
  • pipeline-model-definition with 2.452.x
  • docker-workflow with 2.440.x and 2.452.x
  • blueocean plugin with 2.440.x and 2.452.x

This reverts commit 63a6418.

Testing done

LINE=2.440.x PLUGINS=git-server TEST=InjectedTest bash local-test.sh passes with this change

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 demonstrates feature works or fixes the issue

…enkinsci#3402)"

Plugin loading fails in several BOM tests, including:

* `LINE=2.440.x PLUGINS=git-server TEST=InjectedTest bash local-test.sh`

https://ci.jenkins.io/job/Tools/job/bom/view/change-requests/job/PR-3374/10/testReport/
lists the other plugins with failures as:

* workflow-basic-steps with 2.440.x
* git with 2.440.x and 2.452.x
* pipeline-model-definition with 2.452.x
* docker-workflow with 2.440.x and 2.452.x
* blueocean plugin with 2.440.x and 2.452.x

This reverts commit 63a6418.
@MarkEWaite MarkEWaite requested a review from a team as a code owner July 25, 2024 19:53
@MarkEWaite MarkEWaite added the bug Something isn't working label Jul 25, 2024
@MarkEWaite MarkEWaite merged commit c55f5fe into jenkinsci:master Jul 25, 2024
5 checks passed
@MarkEWaite MarkEWaite deleted the revert-ssh-credentials-latest branch July 25, 2024 20:09
MarkEWaite added a commit to MarkEWaite/ssh-credentials-plugin that referenced this pull request Jul 25, 2024
The plugin compatibility tester mistakenly uses the version of optional
plugins that is provided by the pom of the tested plugin rather than
using the version of plugin that is listed in the pom file of the plugin
bill of materials.

Because of that issue, the 2.426.x plugin bill of materials version
needs to remain on an older version in order to not include an optional
dependency on a newer version of the trilead API plugin.

3080.vfa_b_e4a_a_39b_44 is the version immediately before a trilean API
plugin upgrade in the BOM.

jenkinsci/bom#3404 has more details

jenkinsci#211 includes
some discussion as well.
MarkEWaite added a commit that referenced this pull request Jul 25, 2024
…3402)" (#3404)

Plugin loading fails in several BOM tests, including:

* `LINE=2.440.x PLUGINS=git-server TEST=InjectedTest bash local-test.sh`

https://ci.jenkins.io/job/Tools/job/bom/view/change-requests/job/PR-3374/10/testReport/
lists the other plugins with failures as:

* workflow-basic-steps with 2.440.x
* git with 2.440.x and 2.452.x
* pipeline-model-definition with 2.452.x
* docker-workflow with 2.440.x and 2.452.x
* blueocean plugin with 2.440.x and 2.452.x

This reverts commit 63a6418.
olamy pushed a commit to jenkinsci/ssh-credentials-plugin that referenced this pull request Jul 26, 2024
The plugin compatibility tester mistakenly uses the version of optional
plugins that is provided by the pom of the tested plugin rather than
using the version of plugin that is listed in the pom file of the plugin
bill of materials.

Because of that issue, the 2.426.x plugin bill of materials version
needs to remain on an older version in order to not include an optional
dependency on a newer version of the trilead API plugin.

3080.vfa_b_e4a_a_39b_44 is the version immediately before a trilean API
plugin upgrade in the BOM.

jenkinsci/bom#3404 has more details

#211 includes
some discussion as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant