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

Updated jenkins.war default path for linux #622

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

itsAftabAlam
Copy link
Contributor

@itsAftabAlam itsAftabAlam commented Dec 7, 2023

Fixes #427
Fixes #520

This PR changed the default jenkins.war location for OS other than windows to usr/share/java/jenkins.war

Operating System used

Linux: Ubuntu

Issue Reproduction Steps

Same as described in the issue description

Expected Result is Achieved

plugin installation manager tool finds jenkins.war at the default path and is able to get the Jenkins version from it.

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

@krisstern
Copy link
Member

Hi @itsAftabAlam, we already have an active PR at #520.

@timja
Copy link
Member

timja commented Dec 7, 2023

Hi @itsAftabAlam, we already have an active PR at #520.

the other one is a lot more complex for what is really a simple change, this one can be shipped

@timja timja added the enhancement New feature or request label Dec 7, 2023
@timja timja changed the title updated jenkins.war default path for linux Updated jenkins.war default path for linux Dec 7, 2023
@timja timja merged commit 24e1445 into jenkinsci:master Dec 7, 2023
15 checks passed
@krisstern
Copy link
Member

Okay

@MarkEWaite
Copy link
Contributor

the other one is a lot more complex for what is really a simple change, this one can be shipped

Won't this surprise users that were relying on the old location as the default? Do we need more text describing the change in the changelog?

@itsAftabAlam
Copy link
Contributor Author

itsAftabAlam commented Dec 9, 2023

Greetings @MarkEWaite. Just curious if updating PR heading updates changelog as well? If so, I can do so. I had updated the default war file path after taking a look on jenkinsci/packaging#266 & JENKINS-41218 mentioned in issue #427 , may we can mention that in the changelog?

@krisstern
Copy link
Member

Won't this surprise users that were relying on the old location as the default? Do we need more text describing the change in the changelog?

Yes @MarkEWaite I agree that this may come as a surprise to some users.

@MarkEWaite
Copy link
Contributor

Greetings @MarkEWaite. Just curious if updating PR heading updates changelog as well?

Once the release has been published, release drafter does not perform any additional updates.

I had updated the default war file path after taking a look on jenkinsci/packaging#266 & JENKINS-41218 mentioned in issue #427 , may we can mention that in the changelog?

I think that is a good idea. I've added a link to the Jira issue in the changelog. That provides some additional context for the change of default location

@dominykas
Copy link

The official Docker image for Jenkins has the WAR file in the old location. This means that interacting with the jenkins-plugin-cli now always prints War not found, installing all plugins: /usr/share/java/jenkins.war. Not sure exactly where this issues should be addressed, but I would guess that changing the path inside the official image would be quite a breaking change for people?

@timja
Copy link
Member

timja commented Jan 2, 2024

Looks like #520 should be re-evaluated. Unsure how much of a pain it would be to change the location in the image, it was changed for linux packages without any complaints I think.

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

Successfully merging this pull request may close these issues.

Default jenkins.war path wrong since 2.332.1
5 participants