Skip to content

Conversation

@halkeye
Copy link
Member

@halkeye halkeye commented Jan 5, 2023

jenkins-infra/jenkins.io#5668 (comment)

makes sure property is set right for components
also don't include report a problem / improve this page (but easy to add if desired)

Copy link
Contributor

@daniel-beck daniel-beck left a comment

Choose a reason for hiding this comment

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

Thanks for the adaptation.

@halkeye
Copy link
Member Author

halkeye commented Jan 5, 2023

@daniel-beck it'll be at least a day before i can poke at this again (work then personal meeting), do you want to make the changes you requested yourself?

otherwise i'll try to fix things up after everything tonight, but likely not till tomorrow afternoon for me

@daniel-beck
Copy link
Contributor

it'll be at least a day before i can poke at this again (work then personal meeting), do you want to make the changes you requested yourself?

otherwise i'll try to fix things up after everything tonight, but likely not till tomorrow afternoon for me

I don't think I'll have time before then. Thanks for checking!

* Switch to cat | sed to make it more osx portable
* Document with usage when bad parameters
@halkeye
Copy link
Member Author

halkeye commented Jan 6, 2023

Cleanup generate jenkins io template

  • Switch to cat | sed to make it more osx portable
  • Document with usage when bad parameters

I opted out of having Improve This Page and Report a Problem links on the footer but they are simple html attributes if we want them.

Copy link
Contributor

@daniel-beck daniel-beck left a comment

Choose a reason for hiding this comment

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

https://github.com/jenkins-infra/update-center2/blob/master/src/main/java/io/jenkins/update_center/JenkinsIndexTemplateProvider.java needs to be changed as well for pages like https://updates.jenkins.io/download/war/ or https://updates.jenkins.io/download/plugins/matrix-auth/

Called via

echo "--key $SECRET/update-center.key --certificate $SECRET/update-center.cert --root-certificate $( dirname "$0" )/../resources/certificates/jenkins-update-center-root-ca-2.crt --index-template-url https://www.jenkins.io/templates/downloads/ $EXTRA_ARGS $*" >> "$MAIN_DIR"/tmp/args.lst
@Option(name = "--index-template-url", usage = "Url of a template for index files")
public String indexTemplate = null;
public void build(MavenRepository repo) throws IOException {
IndexTemplateProvider indexTemplateProvider = indexTemplate == null ? new IndexTemplateProvider() : new JenkinsIndexTemplateProvider(indexTemplate);
try (LatestLinkBuilder latestLinks = prepareLatestLinkBuilder(indexTemplateProvider)) {

@daniel-beck daniel-beck added the needs-fix This PR needs to be fixed before it can be considered mergable. label Feb 16, 2023
@daniel-beck daniel-beck added the wrapper PR affects wrapper behavior and will be effective immediately when merged. label Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-fix This PR needs to be fixed before it can be considered mergable. 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