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

🚧 target Jenkins v164.1 #790

Closed
wants to merge 4 commits into from
Closed

🚧 target Jenkins v164.1 #790

wants to merge 4 commits into from

Conversation

jetersen
Copy link
Member

Here is our checklist for contributors. No hard requirement here, just a reminder

  • Please describe what you did

Let's discuss targeting the v2.164.1 LTS, the project is pretty stable.
There are a few PRs we could get in and release before bumping.
The bump would be welcomed this is, after all, JCasC we should be able to safely bump core version without too much fuss from Jenkins users.
Would be great to know we can run on Java 11 runtime!

  • Link to issue you're working on if there's a relevant one

  • Did you provide a test-case to demonstrate feature actually works / issue is fixed ?

  • Please also consider adding a line to CHANGELOG.md

Changelog will be added later.

@jetersen jetersen changed the title Jenkins v164.1 chore: Jenkins v164.1 Mar 23, 2019
@jetersen jetersen changed the title chore: Jenkins v164.1 target Jenkins v164.1 Mar 24, 2019
@jetersen jetersen mentioned this pull request Mar 24, 2019
4 tasks
@jetersen
Copy link
Member Author

jetersen commented Mar 24, 2019

I honestly have given up on Jenkins CI, Travis at the moment does precisely the same as what Jenkins CI does and faster 😢 Except for Windows but appveyor shouldn't be too hard to add.

@jetersen
Copy link
Member Author

jetersen commented Mar 24, 2019

SpotBugs failure on Jenkins CI is due to spotbugs/spotbugs#756

@jonbrohauge
Copy link
Contributor

👍 for 🚀
We try to stay on the latest stable Jenkins + plugins, and as such we have a stable platform, so I see no reason not to bump to a newer required LTS for this plugin. There is obviously some maintenance, but it's at a minimum, and 99.9% of the maintenance is due to a plugin changing some logic we use.

.travis.yml Outdated
@@ -0,0 +1,18 @@
language: java
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not really related to this PR?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, it is kind of related. I was blocked by Jenkins CI trying to use JDK 11, and I needed to know if we could build with JDK 11!

Copy link
Member

@timja timja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM code wise...

@er1c
Copy link

er1c commented Mar 28, 2019

I just did a lot of work on: jenkinsci/github-plugin#209 would that then cause an issue to to get jenkinsci/github-plugin#210 working?

@jetersen
Copy link
Member Author

jetersen commented Mar 28, 2019

There should be no internal changes that should make your work obsolete. You can still use an older version for the plugin testing.

I think one of the main reason I want to bump is to avoid maintaining a lot of legacy/version matching dance to add some functionality like #788

Perhaps that is a lousy argument but an argument none the less.

Looking at our user installs by Jenkins version is currently not possible.
INFRA-2004

https://stats.jenkins.io/pluginversions/configuration-as-code.html
Let me chow some data 😄

@timja
Copy link
Member

timja commented Mar 29, 2019

Nice it does exist 🎉

@jetersen
Copy link
Member Author

jetersen commented Mar 29, 2019

Reading those numbers, it might make sense to do the version dance. 😢

1 Jenkins server with version 1.534 running JCasC that requires Java 8, DAFUQ 😖

@jonbrohauge
Copy link
Contributor

@Casz At some point we need to an upgrade of the target Jenkins, might as well be now, i.e. next release. Although v2.164.1 might be too far this time around, if you look at the stats.

@timja
Copy link
Member

timja commented Mar 29, 2019

Might be best to wait for next months numbers run and see how 2.164.1 is looking

@jetersen
Copy link
Member Author

We can just pin the PR for later and do the version dance in #788

@ewelinawilkosz ewelinawilkosz added the pinned Used to avoid stale[bot] marking a issue/PR stale label Apr 10, 2019
@jetersen jetersen changed the title target Jenkins v164.1 🚧 target Jenkins v164.1 Apr 21, 2019
@jetersen jetersen changed the title 🚧 target Jenkins v164.1 target Jenkins v164.1 Apr 21, 2019
@jetersen jetersen changed the title target Jenkins v164.1 🚧 target Jenkins v164.1 Apr 21, 2019
@oleg-nenashev
Copy link
Member

Would be great to know we can run on Java 11 runtime!

We can. It was a part of our smoke tests since October

Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure it is a good idea. If we ever need to extend the configurator API to offer more compatibility tools for plugins, it will require these plugins to be updated to the recent LTS. And it may be a non-starter for some plugin maintainers.

Unfortunately #279 was rejected, otherwise we could have maintained the majority of the codebase on the latest LTS.

Please do not merge right now, let's discuss it at the project meeting

@jetersen
Copy link
Member Author

it was already marked a WIP 😰 so no worries @oleg-nenashev

@oleg-nenashev
Copy link
Member

oleg-nenashev commented May 25, 2019 via email

@jetersen
Copy link
Member Author

Closing as we will most likely split into configurator API

@jetersen jetersen closed this Aug 25, 2019
@jetersen jetersen deleted the JenkinsV164.1 branch August 25, 2019 19:13
Copy link

@Pero110290 Pero110290 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pinned Used to avoid stale[bot] marking a issue/PR stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants