Skip to content
Snippets Groups Projects

Removal entry for Runner Helm chart fields

Merged Darren Eastman requested to merge removal-runner-helm-chart-fields into master
All threads resolved!

/label release post release post item Technical Writing ~devops:: ~group:: release post itemremoval /label typemaintenance /milestone % /assign @EM/PM (choose the DRI; remove backticks here, and below)

Be sure to link this MR to the relevant issues.

  • Deprecation issue:
  • Removal issue: #379064 (closed)
  • MR that removed (or will remove) the feature:

If there is no relevant deprecation issue, hit pause and:

Removals must be announced as deprecations at least 2 milestones in advance of the planned removal date.

If the removal creates a breaking change, it can only be removed in a major "XX.0" release.

By the 10th: Assign this MR to these team members as reviewers, and for approval:

  • Required:
  • Optional:
    • Product Designer(s): @ProductDesigners
    • Product Marketing: @PMM

By 7:59 PM UTC 15th (11:59 AM PT): EM/PM assigns this MR to the TW reviewer for final review and merge: @EM/PM

By 7:59 AM UTC 18th (11:59 PM PT 17th): TW Reviewer updates Docs by merging this MR to master: @TW


Please review:

EM/PM release post item checklist

  • Set yourself as the Assignee, meaning you are the DRI.
  • If the removal is a breaking change, add label breaking change.
  • Follow the process to create a removal YAML file.
  • Make sure that the milestone dates are based on the dates in Product milestone creation.
  • Add reviewers by the 10th.
  • When ready to be merged and not later than the 15th, add the ~ready label and @ message the TW for final review and merge.
    • Removal notices should not be merged before the code is removed from the product. Do not mark ~ready until the removal is complete, or you are certain it will be completed within the current milestone and released. If PMs are not sure, they should confirm with their Engineering Manager.

Reviewers

When the content is ready for review, the Technical Writer and Engineering Manager must review it. Optional reviewers can include Product Marketing, Product Design, and the Product Leaders for this area. Use the reviewers feature for all reviews. Reviewers will approve the MR and remove themselves from the reviewers list when their review is complete.

Tech writer review

The TW should review according to the criteria listed below. Review a removal MR with the same process as regular docs MRs. Add suggestions as needed, @ message the PM to inform them the first review is complete, and remove yourself as a reviewer if it's not yet ready for merge.

Removal notices should not be merged before the code is removed from the product.

Expand for Details
  • Title:
    • Length limit: 7 words (not including articles or prepositions).
    • Capitalization: ensure the title is sentence cased.
  • Dates:
  • Consistency:
    • Ensure that all resources (docs, removal, etc.) refer to the feature with the same term / feature name.
  • Content:
    • Make sure the removal is accurate based on your understanding. Look for typos or grammar mistakes. Work with PM and PMM to ensure a consistent GitLab style and tone for messaging, based on other features and removals.
    • Review use of whitespace and bullet lists. Will the removal item be easily scannable when published? Consider adding line breaks or breaking content into bullets if you have more than a few sentences.
    • Make sure there aren't acronyms readers may not understand per https://about.gitlab.com/handbook/communication/#writing-style-guidelines.
  • Links:
    • All links must be full URLs, as the removal YAML files are used in multiple projects. Do not use relative links. The generated doc is an exception to the relative link rule and currently uses absolute links only.
    • Make sure all links and anchors are correct. Do not link to the H1 (top) anchor on a docs page.
  • Code. Make sure any included code is wrapped in code blocks.
  • Capitalization. Make sure to capitalize feature names. Stay consistent with the Documentation Style Guidance on Capitalization.
  • Blank spaces. Remove unnecessary spaces (end of line spaces, double spaces, extra blank lines, and lines with only spaces).

When the PM indicates it is ready for merge and all issues have been addressed, start the merge process.

Technical writer merge process

The removals doc's .md file must be updated before this MR is merged:

  1. Check out the MR's branch (in the gitlab-org/gitlab project).
  2. From the command line (in the branch), run bin/rake gitlab:docs:compile_removals. If you want to double check that it worked, you can run bin/rake gitlab:docs:check_removals to verify that the doc is up to date.
  3. Commit the updated file and push the changes.
  4. Set the MR to merge when the pipeline succeeds (or merge if the pipeline is already complete).

If you have trouble running the rake task, check the troubleshooting steps.

Edited by Darren Eastman

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Fiona Neill marked the checklist item Title: as completed

    marked the checklist item Title: as completed

  • Fiona Neill marked the checklist item Dates: as completed

    marked the checklist item Dates: as completed

  • Fiona Neill marked the checklist item Consistency: as completed

    marked the checklist item Consistency: as completed

  • Fiona Neill marked the checklist item Links: as completed

    marked the checklist item Links: as completed

  • Fiona Neill marked the checklist item Code. Make sure any included code is wrapped in code blocks. as completed

    marked the checklist item Code. Make sure any included code is wrapped in code blocks. as completed

  • Fiona Neill marked the checklist item Capitalization. Make sure to capitalize feature names. Stay consistent with the Documentation Style Guidance on Capitalization. as completed

    marked the checklist item Capitalization. Make sure to capitalize feature names. Stay consistent with the Documentation Style Guidance on Capitalization. as completed

  • Fiona Neill marked the checklist item Blank spaces. Remove unnecessary spaces (end of line spaces, double spaces, extra blank lines, and lines with only spaces). as completed

    marked the checklist item Blank spaces. Remove unnecessary spaces (end of line spaces, double spaces, extra blank lines, and lines with only spaces). as completed

  • Hey @DarrenEastman 👋 I added a suggestion 🏓

  • Darren Eastman resolved all threads

    resolved all threads

  • Darren Eastman added 1 commit

    added 1 commit

    Compare with previous version

  • added Ready label

  • Darren Eastman marked this merge request as ready

    marked this merge request as ready

  • Fiona Neill marked the checklist item Required review and approval: Technical Writer designated to the corresponding DevOps stage/group. as completed

    marked the checklist item Required review and approval: Technical Writer designated to the corresponding DevOps stage/group. as completed

  • Fiona Neill approved this merge request

    approved this merge request

  • requested review from @nicolewilliams

  • Darren Eastman marked the checklist item Set yourself as the Assignee, meaning you are the DRI. as completed

    marked the checklist item Set yourself as the Assignee, meaning you are the DRI. as completed

  • Darren Eastman marked the checklist item Follow the process to create a removal YAML file. as completed

    marked the checklist item Follow the process to create a removal YAML file. as completed

  • Darren Eastman marked the checklist item Make sure that the milestone dates are based on the dates in Product milestone creation. as completed

    marked the checklist item Make sure that the milestone dates are based on the dates in Product milestone creation. as completed

  • Romuald Atchadé approved this merge request

    approved this merge request

  • Fiona Neill added 1 commit

    added 1 commit

    • ff6e50a4 - Fixed lint error and updated md

    Compare with previous version

  • A deleted user added documentation label

    added documentation label

  • Fiona Neill resolved all threads

    resolved all threads

  • Fiona Neill added 565 commits

    added 565 commits

    Compare with previous version

  • Fiona Neill resolved all threads

    resolved all threads

  • Fiona Neill enabled an automatic merge when the pipeline for 9023ac22 succeeds

    enabled an automatic merge when the pipeline for 9023ac22 succeeds

  • merged

  • Fiona Neill mentioned in commit b0fd6020

    mentioned in commit b0fd6020

  • mentioned in merge request gitlab-runner!4084 (merged)

  • added workflowstaging label and removed workflowcanary label

  • John Gaughan mentioned in merge request !152455 (merged)

    mentioned in merge request !152455 (merged)

  • Please register or sign in to reply
    Loading