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

ci: tag CI-deployed resources #9043

Merged
merged 4 commits into from
Mar 23, 2024
Merged

ci: tag CI-deployed resources #9043

merged 4 commits into from
Mar 23, 2024

Conversation

dannysauer
Copy link
Contributor

Description

Add the owner and gh_team tags the new tagging policy requires Also add the long_running tag to the long-running instances

Test Plan

See what happens in CI; react accordingly.

Commentary (optional)

The terraform image used for doc publishing is switched from light to latest because the light tag is no longer maintained as of over a year ago.
hashicorp/terraform#33139

I've already imported, planned, and reapplied the updated Terraform files. So this should just work cleanly without needing to recreate objects, etc.

Supersedes #8969

Checklist

  • Changes have been manually QA'd
  • User-facing API changes need the "User-facing API Change" label.
  • Release notes should be added as a separate file under docs/release-notes/.
    See Release Note for details.
  • Licenses should be included for new code which was copied and/or modified from any external code.

Ticket

Add the owner and gh_team tags the new tagging policy requires
Also add the long_running tag to the long-running instances
Update the Makefile to work with current terraform
Update the CircleCI image to use one which still gets updates
Update the docs AWS resources with new label standard
 * To use aws default_labels, we need the 5.x AWS provider
 * To use the 5.x provider without a bunch of warnings, there were also a bunch
   of deprecated things to be updated
@dannysauer dannysauer requested a review from a team as a code owner March 23, 2024 03:51
@dannysauer dannysauer requested a review from dzhu March 23, 2024 03:51
@cla-bot cla-bot bot added the cla-signed label Mar 23, 2024
@determined-ci determined-ci requested a review from a team March 23, 2024 03:51
@determined-ci determined-ci added the documentation Improvements or additions to documentation label Mar 23, 2024
Copy link

netlify bot commented Mar 23, 2024

Deploy Preview for determined-ui canceled.

Name Link
🔨 Latest commit d85263f
🔍 Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/65fe575e867cc500089dc872

Copy link

codecov bot commented Mar 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.80%. Comparing base (1868723) to head (d85263f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9043      +/-   ##
==========================================
- Coverage   47.80%   47.80%   -0.01%     
==========================================
  Files        1161     1161              
  Lines      143646   143646              
  Branches     2371     2373       +2     
==========================================
- Hits        68674    68672       -2     
- Misses      74819    74821       +2     
  Partials      153      153              
Flag Coverage Δ
backend 42.94% <ø> (+<0.01%) ⬆️
harness 63.84% <ø> (-0.01%) ⬇️
web 40.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 4 files with indirect coverage changes

@dannysauer dannysauer enabled auto-merge (squash) March 23, 2024 03:59
@dannysauer dannysauer mentioned this pull request Mar 23, 2024
4 tasks
@dannysauer dannysauer merged commit f08b406 into main Mar 23, 2024
70 of 82 checks passed
@dannysauer dannysauer deleted the ci/dannys/update-ci-tagging branch March 23, 2024 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants