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

Update ownership.yaml maintainer: jovel #88

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

andrewlayman
Copy link

What are you trying to accomplish?

Add a maintainer to the Service Catalog entry for this service.

Anything you want to highlight for special attention from reviewers?

Which environments does this change target?

  • Production
    • dotcom
    • proxima
    • GHES
  • Non-production
    • dsp-testing
    • docs

Risk Assessment

  • Low risk the change is fully under one or more Feature Flags OR the modifications are small, highly observable, and easily rolled back.
  • Medium risk changes that are isolated, reduced in scope or could impact few users and not bring the site down.
  • High risk changes are those that could impact our customers and SLOs, low or no test coverage, low observability, or slow to rollback.

Is there a Feature Flag?

  • Yes - Change can be disabled by feature flag, but will need to be rolled back if the feature flag does not work. Please document the feature flag name(s) below.
  • No - Change can only be disabled by rolling back via the merge group, or by deploying a revert commit.

How will you know you've accomplished your goal?

The maintainer name appears in the Service Catalog for this service.

@andrewlayman andrewlayman requested a review from a team as a code owner November 27, 2024 01:09
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.

1 participant