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

migrating Pipeline jenkinsio-cn from trusted to infra.ci #3086

Open
6 tasks
smerle33 opened this issue Aug 4, 2022 · 12 comments
Open
6 tasks

migrating Pipeline jenkinsio-cn from trusted to infra.ci #3086

smerle33 opened this issue Aug 4, 2022 · 12 comments

Comments

@smerle33
Copy link
Contributor

smerle33 commented Aug 4, 2022

Service(s)

infra.ci.jenkins.io

Summary

as this project evolve slowly, as per daniel demand, it could be migrate to infra.ci.jenkins.io

definition of done :

  • duplicate the job on infra.ci.jenkins.io
  • add a node "docker&&linux" as used in the pipeline
  • duplicate the credentials for this pipeline on infra.ci.jenkins.io (BLOBXFER_STORAGEACCOUNTKEY)
    • decode BLOBXFER_STORAGEACCOUNTKEY from trusted
    • add it to infra
  • when up and running on infra (with the blobxfer working), delete the entry on trusted

Reproduction steps

No response

@smerle33 smerle33 added the triage Incoming issues that need review label Aug 4, 2022
@smerle33 smerle33 self-assigned this Aug 5, 2022
@smerle33
Copy link
Contributor Author

smerle33 commented Aug 5, 2022

@dduportal WDYT ?

@dduportal
Copy link
Contributor

Lgtm

@smerle33 smerle33 assigned dduportal and unassigned smerle33 Aug 5, 2022
@dduportal dduportal added this to the infra-team-sync-2022-08-09 milestone Aug 5, 2022
@dduportal dduportal added cn.jenkins.io trusted.ci and removed triage Incoming issues that need review labels Aug 5, 2022
@dduportal
Copy link
Contributor

Handing over as @smerle33 is going to holidays.

Ping @daniel-beck : I understand that the job on trusted.ci.jenkins.io is responsible to deploy the site in production when there is change on the master branch.
Hence the request to move it out of trusted.ci as there has been no changes since months.

Is there a reason to not also migrate jenkins.io's deployment job out of trusted.ci.jenkins.io since they share the same production bucket (and crerdential)?

I wonder if (cn)jenkins.io websites deployment should not be on release.ci.jenkins.io, with Core, remoting (and Docker images of core).

@timja
Copy link
Member

timja commented Aug 5, 2022

I wonder if (cn)jenkins.io websites deployment should not be on release.ci.jenkins.io, with Core, remoting (and Docker images of core).

I don't think it needs to be on release.ci.jenkins.io. but yes docker images should move there

@dduportal
Copy link
Contributor

I wonder if (cn)jenkins.io websites deployment should not be on release.ci.jenkins.io, with Core, remoting (and Docker images of core).

I don't think it needs to be on release.ci.jenkins.io. but yes docker images should move there

Make sense. My question could be simplified to:

1 - Should we allow infra.ci.jenkins to deploy cn.jenkins.io ?
2 - Should we also do the same for jenkins.io ? If no why ?

@timja
Copy link
Member

timja commented Aug 5, 2022

  1. yes
  2. yes
    makes sense to me

@daniel-beck
Copy link

Is there a reason to not also migrate jenkins.io's deployment job out of trusted.ci.jenkins.io since they share the same production bucket (and crerdential)?

I need to be able to disable, enable, and trigger builds during publishing. Agents should not take minutes to be available. Otherwise should be fine.

@timja
Copy link
Member

timja commented Aug 5, 2022

What's the need for disable / enable?

infra.ci has webhooks so will trigger on commit.

@daniel-beck
Copy link

daniel-beck commented Aug 5, 2022

I don't want the site to update between taking ci.j.io back online (even more so once we remove this stupid dependency) and pushing the advisory, potentially already advertising new stuff with incomplete changelog (also as RSS) etc.

@timja
Copy link
Member

timja commented Aug 5, 2022

even more so once we remove this stupid dependency

I couldn't see an issue for this so filed #3087

@dduportal
Copy link
Contributor

For info: disabled the job on trusted.ci.jenkins.io (see screenshot below) with an explanation in the description, which point to this issue:

Capture d’écran 2022-09-20 à 12 40 36

@lemeurherve
Copy link
Member

All of these elements should be cleaned up, cf #3379

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants