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

Add Declarative to aggregator #16

Merged
merged 3 commits into from
Feb 1, 2017
Merged

Conversation

abayer
Copy link
Member

@abayer abayer commented Feb 1, 2017

Note that this is deliberately not using workflow-multibranch 2.10,
since that depends on branch-api 2.0.0 and that's a problematic
release.

Dependency versions bumped to match those in
pipeline-model-definition's 1.0 release, along with reasonable updates
of plugins that aren't Declarative dependencies. This does require
moving to core 2.7.1 as well.

cc @reviewbybees esp @jglick

Note that this is deliberately not using workflow-multibranch 2.10,
since that depends on branch-api 2.0.0 and that's a problematic
release.

Dependency versions bumped to match those in
pipeline-model-definition's 1.0 release, along with reasonable updates
of plugins that aren't Declarative dependencies. This does require
moving to core 2.7.1 as well.
pom.xml Outdated
@@ -63,23 +63,23 @@
</pluginRepository>
</pluginRepositories>
<properties>
<jenkins.version>1.642.3</jenkins.version>
<jenkins.version>2.7.1</jenkins.version>
Copy link
Member

Choose a reason for hiding this comment

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

I think 2.7.3 would be more appropriate.

Copy link
Member

Choose a reason for hiding this comment

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

That would require Blue Ocean to bump their minimum (which is 2.7.1 currently)

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe BO should require 2.7.3?

Copy link
Member Author

Choose a reason for hiding this comment

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

Why not just stick with 2.7.1? That's the first LTS release of 2.x, it's what Declarative and BO depend on, we generally stick to depending on the earliest possible LTS release rather than a later one, etc. I know of no reason to depend on 2.7.3 rather than 2.7.1.

Copy link
Member

Choose a reason for hiding this comment

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

I also would prefer 2.7.3. For Pipeline plugins we generally depend on the most recent update in a given LTS line. This ensures that tests etc. are incorporating known fixes.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok, I'll bump it to 2.7.3 - neither BO nor Declarative depend on workflow-aggregator, so they're not going to get messed up by bumping the aggregator core dependency.

@rsandell
Copy link
Member

rsandell commented Feb 1, 2017

🐝

It depends on git 3.0.2, which is part of the suite of blocked
releases from the branch-api 2.0 fallout.
@abayer
Copy link
Member Author

abayer commented Feb 1, 2017

Just realized that pipeline-github-lib 1.0 brought in git 3.0.2 and that whole mess, so I removed that plugin completely.

@jglick
Copy link
Member

jglick commented Feb 1, 2017

I would also like to bump up dependencies in the demo but that can be done separately.

@jglick
Copy link
Member

jglick commented Feb 1, 2017

pipeline-github-lib 1.0 brought in git 3.0.2 and that whole mess

Yes, #13 was a problem. Really should have waited for a (non-beta) release.

@abayer
Copy link
Member Author

abayer commented Feb 1, 2017

The demo was actually more up to date than the aggregator. =)

@rsandell
Copy link
Member

rsandell commented Feb 1, 2017

🐝

@jglick jglick merged commit 53dc5c9 into jenkinsci:master Feb 1, 2017
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.

5 participants