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

[INFRA-2615] Deprecate fixed tiers, don't generate for 2.249.x+ #418

Merged

Conversation

daniel-beck
Copy link
Contributor

This implements the suggestion in jenkinsci/docker#954 (comment) and will result in the gradual retirement of fixed tiers.

Currently, all five latest fixed tiers are still generated for tools and scripts accessing them directly. With 2.249.x coming in ~5 weeks, 2.176.x will be retired, but 2.249.x would not be added with this change. This would continue until, after about a year, no fixed tiers are generated anymore.

jenkinsci/docker#954 should be implemented so the Docker image doesn't require these directories anymore. CC @slide

@daniel-beck daniel-beck requested a review from slide August 3, 2020 18:01
@daniel-beck daniel-beck added enhancement This is an enhancement for the tool or wrapper scripts, typically adding features. wrapper PR affects wrapper behavior and will be effective immediately when merged. labels Aug 3, 2020
@timja
Copy link
Member

timja commented Aug 4, 2020

This implements the suggestion in jenkinsci/docker#954 (comment) and will result in the gradual retirement of fixed tiers.

Currently, all five latest fixed tiers are still generated for tools and scripts accessing them directly. With 2.249.x coming in ~5 weeks, 2.176.x will be retired, but 2.249.x would not be added with this change. This would continue until, after about a year, no fixed tiers are generated anymore.

jenkinsci/docker#954 should be implemented so the Docker image doesn't require these directories anymore. CC @slide

jenkinsci/docker#954 has been implemented in the plugin installation manager tool, I doubt there's a plan to update the install-plugins script as well.

@daniel-beck
Copy link
Contributor Author

I doubt there's a plan to update the install-plugins script as well.

Isn't that what jenkinsci/docker#964 does?

@timja
Copy link
Member

timja commented Aug 4, 2020

isn't there still issues with it? and ref: jenkinsci/docker#964 (comment)

@daniel-beck
Copy link
Contributor Author

For now we'll continue to support the part that relies on the existence of tiered latest/ dirs, tracked as jenkinsci/docker#953

@oleg-nenashev oleg-nenashev self-requested a review August 10, 2020 13:15
@daniel-beck daniel-beck merged commit 4a421f2 into jenkins-infra:master Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This is an enhancement for the tool or wrapper scripts, typically adding features. wrapper PR affects wrapper behavior and will be effective immediately when merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants