Removing unused jobs #1040
Removing unused jobs #1040
Conversation
|
|
||
| // it would be slick if we could trigger based on dockerhub lsstsqre/centos | ||
| // notifications but AFAIK, this can't be restricted by tag | ||
| upstream('sqre/infra/build-layercake', 'SUCCESS') |
There was a problem hiding this comment.
I think this means that build-layercake can go away as well.
| scm { | ||
| git { | ||
| remote { | ||
| github('lsst-sqre/sqre-codekit') |
There was a problem hiding this comment.
I think Build Engineering needs to take over ownership of this. SQuaRE isn't going to use it for anything. We should then replace this trigger with a GHA.
There was a problem hiding this comment.
We have taken over this and have replaced it with GHA
| scm { | ||
| git { | ||
| remote { | ||
| github('lsst-sqre/tag-monger') |
There was a problem hiding this comment.
Looking at what this repo does, I think something like it may be necessary. We can't keep all eups package artifacts forever. But we do want to keep official releases, so a simple time-based lifecycle policy won't work.
That doesn't mean we need to keep this job, but we need to plan to replace this code somehow.
| @@ -1,10 +0,0 @@ | |||
| import util.Plumber | |||
|
|
|||
| def p = new Plumber(name: 'sqre/ci-ci/ghslacker', dsl: this) | |||
There was a problem hiding this comment.
This has been replaced by SQuaRE's checkerboard service that Build Engineering is not responsible for.
Removal is correct.
| scm { | ||
| git { | ||
| remote { | ||
| github('lsst-sqre/git-lfs-s3-server') |
There was a problem hiding this comment.
Again, SQuaRE owns git-lfs. Removal is correct.
| @@ -1,38 +0,0 @@ | |||
| import util.Plumber | |||
|
|
|||
| def p = new Plumber(name: 'release/codekit/github-tag-release', dsl: this) | |||
There was a problem hiding this comment.
This one is used in the nightly/weekly/official release jobs. It can't be removed.
There was a problem hiding this comment.
Actually, it looks like this job is not used, as the release jobs invoke the same code directly, not via the job. Removal is OK.
| @@ -1,29 +0,0 @@ | |||
| import util.Plumber | |||
|
|
|||
| def p = new Plumber(name: 'release/codekit/github-tag-teams', dsl: this) | |||
There was a problem hiding this comment.
Similarly, this is used in the nightly/weekly/official releases for auxiliary repos. Don't remove it.
There was a problem hiding this comment.
Actually, it looks like this job is not used, as the release jobs invoke the same code directly, not via the job. Removal is OK.
| @@ -1,66 +0,0 @@ | |||
| node('jenkins-manager') { | |||
There was a problem hiding this comment.
Actually, it looks like this pipeline is not used, as the release jobs invoke the same code directly, not via the pipeline. Removal is OK.
| @@ -1,32 +0,0 @@ | |||
| node('jenkins-manager') { | |||
There was a problem hiding this comment.
Actually, it looks like this pipeline is not used, as the release jobs invoke the same code directly, not via the pipeline. Removal is OK.
roceb
force-pushed
the
tickets/DM-47324
branch
from
10c563a to
b32f57f
Compare
No description provided.