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

[WIP] [JENKINS-38307] Stop installing detached plugins #191

Closed
wants to merge 2 commits into from

Conversation

recampbell
Copy link
Member

We should just let the war behave as normal instead of hacking the plugins to avoid fixing some WithPlugin lists.

@recampbell
Copy link
Member Author

Noting that this will likely break a lot of tests, but since I'd rather not cause a brown out by running the ATH locally, I'll let the PR builder handle it.

@recampbell recampbell closed this Sep 17, 2016
@recampbell recampbell reopened this Sep 17, 2016
@recena
Copy link
Contributor

recena commented Sep 28, 2016

@recampbell If we do that, what happens whether we want to run the ATH with previous releases of 2.x? /cc @olivergondza

@olivergondza
Copy link
Member

@recena, Good point, that needs to be retested.

The change itself did not break much actually.

jglick added a commit to jglick/acceptance-test-harness that referenced this pull request Dec 1, 2016
…cent versions of plugins, not whatever old stuff happened to be in the detached list.
@olivergondza olivergondza self-assigned this Jan 3, 2017
@olivergondza
Copy link
Member

The fact we do this seems to be cause of plugin installations problems or an extra work at least. We unbundle outdated credentials plugin, and dynamic deploy of needed plugins fails because credentials plugins is too old. If we quit doing that, the correct plugin version can be installed right away.

My initial concern about pre-2.0 jenkins is not valid as we have started to do this after 2.0 - the older version will unbundle the plugins correctly.

I am running tests against 1.609.3 and 2.19.3 before I merge this.

olivergondza added a commit to olivergondza/acceptance-test-harness that referenced this pull request Jan 4, 2017
[WIP] [JENKINS-38307] Stop installing detached plugins
@olivergondza
Copy link
Member

olivergondza commented Jan 4, 2017

There seems to be ~5 ATH testcases affected by JENKINS-37545. Since the bug is and will be present in releases we want to support in ATH for a while and plugin installation is sort of crucial. I lean towards updating the testcases so the detached dependencies are requested explicitly. It will cause no harm to Jenkins 1.X and will make Jenkins 2.X work.

@olivergondza
Copy link
Member

Continuing in #251.

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

Successfully merging this pull request may close these issues.

None yet

3 participants