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

Promotion 2023-05-31 (#5252) #5253

Merged
merged 21 commits into from
May 31, 2023
Merged

Promotion 2023-05-31 (#5252) #5253

merged 21 commits into from
May 31, 2023

Conversation

achave11-ucsc
Copy link
Member

@achave11-ucsc achave11-ucsc commented May 31, 2023

Connected issue: #5252

Checklist

Author

  • Target branch is prod
  • Name of PR branch matches promotions/yyyy-mm-dd
  • Title of connected issue matches Promotion yyyy-mm-dd
  • PR title references the connected issue
  • PR title starts with title of connected issue
  • PR is connected to issue via ZenHub
  • PR description links to connected issue

Author (reindex, API changes)

  • Added reindex label to PR or this PR does not require reindexing
  • Added API label to connected issue or this PR does not modify the Azul service API

Author (upgrading)

  • Added upgrade label to PR or this PR does not require upgrading

Primary reviewer (after approval)

  • Actually approved the PR
  • Labeled PR as no sandbox
  • Moved ticket to Approved column
  • Assigned PR to current operator

Operator (before pushing merge the commit)

  • Pushed PR branch to GitHub
  • Title of merge commit starts with title from this PR
  • Added PR reference to merge commit title
  • Added commit title tags to merge commit title
  • Pushed merge commit to GitHub

Operator (after pushing the merge commit)

  • Pushed merge commit to GitLab prod
  • Build passes on GitLab prod
  • Reviewed build logs for anomalies on GitLab prod
  • Deleted PR branch from GitHub
  • Deleted PR branch from GitLab prod
  • Moved connected issue to Merged prod column on ZenHub
  • Moved promoted issues from Merged to Merged prod column on ZenHub
  • Moved promoted issues from dev to prod column on ZenHub

Operator (reindex)

  • Deleted unreferenced indices in prod or this PR does not remove catalogs or otherwise causes unreferenced indices
  • Started reindex in prod or this PR does not require reindexing
  • Checked for and triaged indexing failures in prod or this PR does not require reindexing
  • Emptied fail queues in prod deployment or this PR does not require reindexing

Operator

  • Unassigned PR

Shorthand for review comments

  • L line is too long
  • W line wrapping is wrong
  • Q bad quotes
  • F other formatting problem

achave11-ucsc and others added 21 commits May 23, 2023 09:27
Bumps [requests](https://github.com/psf/requests) from 2.28.1 to 2.31.0.
- [Release notes](https://github.com/psf/requests/releases)
- [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md)
- [Commits](psf/requests@v2.28.1...v2.31.0)

---
updated-dependencies:
- dependency-name: requests
  dependency-type: direct:production
@github-actions github-actions bot added the orange [process] Done by the Azul team label May 31, 2023
@coveralls
Copy link

Coverage Status

Coverage: 84.435% (+0.06%) from 84.373% when pulling 1f28050 on promotions/2023-05-31 into 5093a46 on prod.

@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Merging #5253 (1f28050) into prod (be7830d) will increase coverage by 0.06%.
The diff coverage is 97.50%.

@@            Coverage Diff             @@
##             prod    #5253      +/-   ##
==========================================
+ Coverage   84.35%   84.41%   +0.06%     
==========================================
  Files         148      149       +1     
  Lines       18278    18305      +27     
==========================================
+ Hits        15418    15452      +34     
+ Misses       2860     2853       -7     
Impacted Files Coverage Δ
test/integration_test.py 0.00% <0.00%> (ø)
lambdas/indexer/app.py 95.11% <100.00%> (-0.04%) ⬇️
src/azul/openapi/params.py 100.00% <100.00%> (ø)
src/azul/openapi/spec.py 100.00% <100.00%> (ø)
test/service/test_app_logging.py 98.75% <100.00%> (ø)

@hannes-ucsc hannes-ucsc added the no sandbox [process] PR will not be tested in the sandbox label May 31, 2023
@achave11-ucsc achave11-ucsc merged commit 861b7af into prod May 31, 2023
@achave11-ucsc achave11-ucsc deleted the promotions/2023-05-31 branch June 1, 2023 02:19
@achave11-ucsc achave11-ucsc removed their assignment Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no sandbox [process] PR will not be tested in the sandbox orange [process] Done by the Azul team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants