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

Various stability fixes discovered during testing #12

Merged
merged 8 commits into from
Nov 3, 2016
Merged

Various stability fixes discovered during testing #12

merged 8 commits into from
Nov 3, 2016

Conversation

jimmidyson
Copy link
Contributor

No description provided.

@jimmidyson
Copy link
Contributor Author

@bparees @gabemontero Wouldn't mind if you could take a look, especially around the dependent plugin upgrades?

</dependency>

<!-- testing -->
<dependency>
<groupId>org.jenkins-ci.plugins.pipeline-stage-view</groupId>
<artifactId>pipeline-stage-view</artifactId>
<version>1.0</version>
<version>2.1</version>
<scope>test</scope>
Copy link
Contributor

Choose a reason for hiding this comment

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

i think we're going to need to work with @tdawson to get all these plugin packages updated before our image can consume this new version of the plugin. :/

@tdawson
Copy link

tdawson commented Nov 2, 2016

Can someone put those plugin/rpm changes in a nice list. having everything in layered json isn't the easiest to determine everything that was changed.

Also, jenkins 1.651.3 ... gonna be hard to do .... just letting you know, it will take longer than that plugins.

@jimmidyson
Copy link
Contributor Author

Ah I thought @bparees told me I could safely upgrade the Jenkins version...

@bparees
Copy link
Contributor

bparees commented Nov 2, 2016

@jimmidyson our rhel image currently includes 1.651.2, is that sufficient? (our centos image has 1.651.3)

@jimmidyson
Copy link
Contributor Author

thanks for the tip - let me retest with 1.65.2 but I can't imagine any problems with that.

@bparees
Copy link
Contributor

bparees commented Nov 2, 2016

@tdawson sure thing, here's the full list being introduced by @jimmidyson. I did not double check to see if any of our plugin rpm packages are already at these levels, however:

git 3.0.0
git-client 2.0.0
workflow-api 2.5
workflow-step-api 2.4
workflow-durable-task-step 2.5
workflow-cps-global-lib 2.4
workflow-job 2.8
workflow-cps 2.21
workflow-basic-steps 2.2
workflow-support 2.10
pipeline-rest-api 2.1
structs 1.5
pipeline-stage-view 2.1

@jimmidyson
Copy link
Contributor Author

Thanks @bparees.

@jimmidyson
Copy link
Contributor Author

Note that pipeline-stage-view is only upgraded in test scope here for me to visualize stages while developing plugin - I guess it makes sense to upgrade anyway, but that is not a requirement of this sync plugin.

@jimmidyson
Copy link
Contributor Author

Retested at 1.651.2 & all works fine.

I'm going to move most of the plugins to test scope as they're not hard dependencies of the sync plugin, but rather versions that should be adopted to help with general pipeline stability & feature set.

@jimmidyson
Copy link
Contributor Author

I've just scoped dependencies properly. Required dependencies are:

git 3.0.0
git-client 2.0.0
workflow-api 2.5
workflow-job 2.8
workflow-cps 2.21
pipeline-rest-api 2.1

Recommended upgrades for pipeline jobs (but not direct runtime dependencies of this plugin) are:

workflow-step-api 2.4
workflow-durable-task-step 2.5
workflow-cps-global-lib 2.4
workflow-basic-steps 2.2
workflow-support 2.10
structs 1.5
pipeline-stage-view 2.1

@jimmidyson
Copy link
Contributor Author

I'm going to self-merge this once tests have passed so I can push some changes that require this PR to be merged.

LGTM

@jimmidyson jimmidyson merged commit c2ce9a0 into jenkinsci:master Nov 3, 2016
@bparees
Copy link
Contributor

bparees commented Nov 3, 2016

thanks @jimmidyson!

@tdawson i'd prefer to update everything @jimmidyson listed but since they aren't absolute requirements, feel free to manage your other priorities as needed.

@tdawson
Copy link

tdawson commented Nov 3, 2016

Just double checked and we'll have to update all of the ones listed.
I have time today so I'll get started. I'll start with the ones that have to get updated first and we'll see how far I can get.

@tdawson
Copy link

tdawson commented Nov 3, 2016

One quick question, looks like several of the packages are already newer than you have listed. Can I use the newer versions or would you like the specific ones you listed. Here is the list

pipeline-rest-api 2.1 (2.2)
pipeline-stage-view 2.1 (2.2)
workflow-basic-steps 2.2 (2.3)
workflow-cps 2.21 (2.22)
workflow-step-api 2.4 (2.5)

@jimmidyson
Copy link
Contributor Author

Damn those over-productive developers... Upgrading is fine.

@tdawson
Copy link

tdawson commented Nov 3, 2016

Done: plugins updated to

git 3.0.0
git-client 2.0.0
pipeline-rest-api 2.2
pipeline-stage-view 2.2
structs 1.5
workflow-api 2.5
workflow-basic-steps 2.3
workflow-cps 2.22
workflow-cps-global-lib 2.4
workflow-durable-task-step 2.5
workflow-job 2.8
workflow-step-api 2.5
workflow-support 2.10

puddles have been created, images have been built with new puddles.
Images are based of rhel 7.3
Both have been pushed to testing areas.

I'll work on getting the jenkins-1 rpm updated. That's going to be trickier to get it into puddles. For now, proceed as if I can't update it.

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.

3 participants