Remove `token` column from `ci_builds`
We're requiring the token for CI jobs to be encrypted all the time since #332284 (closed) and this means that it's using the token_encrypted
column to store the data.
We should drop the token
column because it's not used anymore and it will reduce the ci_builds
table size.
Progress
The column removal needs to follow a three step process to avoid downtime:
- Ignoring the column !91047 (merged)
in %15.2 - Dropping the column !92894 (merged)
in %15.3 - Removing the ignore rule !96258 (merged)
in %15.4
Designs
- Show closed items
Is blocked by
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Marius Bobin set weight to 1
set weight to 1
- Marius Bobin assigned to @mbobin
assigned to @mbobin
- 🤖 GitLab Bot 🤖 added typefeature label
added typefeature label
- Maintainer
Setting label(s) devopsverify sectionops based on grouppipeline execution.
- 🤖 GitLab Bot 🤖 added devopsverify sectionops labels
added devopsverify sectionops labels
- Developer
Adding triage
- Jackie Porter changed milestone to %Backlog
changed milestone to %Backlog
- 🤖 GitLab Bot 🤖 removed typefeature label
removed typefeature label
- Marius Bobin unassigned @mbobin
unassigned @mbobin
- 🤖 GitLab Bot 🤖 added [deprecated] Accepting merge requests label
added [deprecated] Accepting merge requests label
- Marius Bobin marked this issue as related to #339191 (closed)
marked this issue as related to #339191 (closed)
- Fabio Pitino mentioned in issue verify-stage#203 (closed)
mentioned in issue verify-stage#203 (closed)
- Mark Nuzzo changed milestone to %15.3
changed milestone to %15.3
- Contributor
Moving this into %15.3 to align with when the index removal is done so they are done at the same time per conversation with @mbobin
/cc @jheimbuck_gl
1 1 - James Heimbuck added workflowplanning breakdown label
added workflowplanning breakdown label
- James Heimbuck added workflowready for development label and removed workflowplanning breakdown label
added workflowready for development label and removed workflowplanning breakdown label
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#7753 (closed)
mentioned in issue gitlab-org/quality/triage-reports#7753 (closed)
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#7822 (closed)
mentioned in issue gitlab-org/quality/triage-reports#7822 (closed)
- James Heimbuck added workflowblocked label and removed workflowready for development label
added workflowblocked label and removed workflowready for development label
- James Heimbuck added backend label
added backend label
- Marius Bobin assigned to @mbobin
assigned to @mbobin
- Marius Bobin mentioned in merge request !91047 (merged)
mentioned in merge request !91047 (merged)
- Author Maintainer
Opened !91047 (merged) to address the first step from the column removal process.
- Fabio Pitino mentioned in issue #366333 (closed)
mentioned in issue #366333 (closed)
- Allison Browne added verify-hackathondelete-fest label
added verify-hackathondelete-fest label
- Maintainer
Thanks for working on this @mbobin! We've removed the Seeking community contributions label to avoid having multiple people working on the same issue.
- 🤖 GitLab Bot 🤖 removed [deprecated] Accepting merge requests label
removed [deprecated] Accepting merge requests label
- Marius Bobin mentioned in merge request !91258 (merged)
mentioned in merge request !91258 (merged)
- Contributor
Since this discussion in !91258 (merged) is still in-progress, I'm going to nudge this blocked issue to %15.4 for now. @jheimbuck_gl - We can always pull forward in %15.3 if things move quicker.
1 Collapse replies - Author Maintainer
The index removal is complete and we can continue with the next step in the column removal process: removing the column: !92894 (merged)
Edited by Marius Bobin 1
- Mark Nuzzo changed milestone to %15.4
changed milestone to %15.4
- Marius Bobin changed milestone to %15.3
changed milestone to %15.3
- Marius Bobin added workflowin dev label and removed workflowblocked label
added workflowin dev label and removed workflowblocked label
- Marius Bobin mentioned in merge request !92894 (merged)
mentioned in merge request !92894 (merged)
- Marius Bobin added workflowin review label and removed workflowin dev label
added workflowin review label and removed workflowin dev label
- Marius Bobin changed the description
Compare with previous version changed the description
- Author Maintainer
Collapse replies - Contributor
@mbobin are you actively working on this? I'm moving over to workflowin dev while I triage the workflowready for development list
1
- Marius Bobin changed milestone to %15.4
changed milestone to %15.4
- Marius Bobin added workflowready for development label and removed workflowin review label
added workflowready for development label and removed workflowin review label
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#8781 (closed)
mentioned in issue gitlab-org/quality/triage-reports#8781 (closed)
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#8844 (closed)
mentioned in issue gitlab-org/quality/triage-reports#8844 (closed)
- James Heimbuck added workflowin dev label and removed workflowready for development label
added workflowin dev label and removed workflowready for development label
- Marius Bobin mentioned in merge request !96258 (merged)
mentioned in merge request !96258 (merged)
- Marius Bobin changed the description
Compare with previous version changed the description
- Marius Bobin added workflowin review label and removed workflowin dev label
added workflowin review label and removed workflowin dev label
- Marius Bobin changed the description
Compare with previous version changed the description
- Author Maintainer
The column ignore rule was removed by !96258 (merged) and this is now complete!
- Marius Bobin added workflowproduction label and removed workflowin review label
added workflowproduction label and removed workflowin review label
- Marius Bobin closed
closed
- John Gaughan mentioned in merge request !120359 (merged)
mentioned in merge request !120359 (merged)
- Caroline Simpson mentioned in issue #458488 (closed)
mentioned in issue #458488 (closed)
- Caroline Simpson mentioned in issue #458494 (closed)
mentioned in issue #458494 (closed)