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

chore(pkg) templatize mirror-scripts on the 'pkgrepo' profile #3359

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Mar 29, 2024

Blocked by #3358 (and will need rebase before complete review or only check the last commit)

Follow up of jenkins-infra/helpdesk#3414

This PR is the second step of #3352 (after #3358).
It introduces the following changes:

The set of "mirror scripts" imported in https://github.com/jenkins-infra/jenkins-infra/pull/3358 have their content templatized to map to puppet hieradata configuration used by profiles.

Note: the profile updatesite has been updated (default param. value move to hieradata) to allow other profiles to lookup the value (updatesite and pkgrepo) are still highly coupled on the same VM).

Deployment process:

  • Disable the puppet agent
  • Merge
  • Run a noop and check the resulting scripts
  • Apply
  • Run a sync.sh (like the crontab) and validate full execution (~2 min)
  • Verify that update-center2 on trusted. does not fail a brand new build (~5 min)
  • Report change here

The set of "mirror scripts" imported in jenkins-infra#3358
have their content templatized to map to puppet hieradata configuration used by profiles.

Note: the profile `updatesite` has been updated (default param. value move to hieradata)
to allow other profiles to lookup the value (`updatesite` and `pkgrepo`` are still highly coupled on the same VM).

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
@dduportal dduportal force-pushed the chore/pkg/templatize-mirror-scripts branch from 96bed4d to 81d4bad Compare March 29, 2024 15:58
@dduportal dduportal marked this pull request as ready for review March 29, 2024 16:00
@dduportal dduportal requested a review from a team as a code owner March 29, 2024 16:00
@dduportal dduportal requested a review from smerle33 March 29, 2024 16:00
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Copy link
Contributor

@smerle33 smerle33 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@smerle33 smerle33 merged commit 0cd62ce into jenkins-infra:production Mar 29, 2024
2 checks passed
@dduportal dduportal deleted the chore/pkg/templatize-mirror-scripts branch March 29, 2024 16:41
@dduportal
Copy link
Contributor Author

Deployed with success during a pairing session with @smerle33 (hence the self-merged suggestions).

We confirmed that the scripts are running properly (as per the described procedure).

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.

None yet

2 participants