ci: Migrate metrics credentials to managed IAM #73811
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Migrates our credentials to upload metrics statistics to managed IAM credentials in order to make it easier to know where the credentials are coming from and to make it easier to add more permissions / less permissions later on Signed-off-by: Eli Uriegas <eliuriegas@fb.com> [ghstack-poisoned]
CI Flow Status⚛️ CI FlowRuleset - Version:
|
🔗 Helpful links
💊 CI failures summary and remediationsAs of commit 9d26f4e (more details on the Dr. CI page):
🕵️ 11 new failures recognized by patternsThe following CI failures do not appear to be due to upstream breakages:
|
seemethere
added a commit
that referenced
this pull request
Mar 4, 2022
Migrates our credentials to upload metrics statistics to managed IAM credentials in order to make it easier to know where the credentials are coming from and to make it easier to add more permissions / less permissions later on Signed-off-by: Eli Uriegas <eliuriegasfb.com> ghstack-source-id: 5699ca952488f4befe4b5e494957d28465efefd8 Pull Request resolved: #73811
kit1980
approved these changes
Mar 4, 2022
|
@pytorchbot merge this |
|
Merge failed due to PR 73811 does not match merge rules |
Migrates our credentials to upload metrics statistics to managed IAM credentials in order to make it easier to know where the credentials are coming from and to make it easier to add more permissions / less permissions later on. Relates to work done in [D34535827](https://www.internalfb.com/diff/D34535827) Signed-off-by: Eli Uriegas <eliuriegasfb.com> [ghstack-poisoned]
seemethere
added a commit
that referenced
this pull request
Mar 14, 2022
Migrates our credentials to upload metrics statistics to managed IAM credentials in order to make it easier to know where the credentials are coming from and to make it easier to add more permissions / less permissions later on Signed-off-by: Eli Uriegas <eliuriegasfb.com> ghstack-source-id: 721f50965297f07cec1d27d740d36a3170261064 Pull Request resolved: #73811
janeyx99
approved these changes
Mar 14, 2022
|
@pytorchbot merge this please |
|
Merge failed due to Too many checksuites for commit |
|
@pytorchbot merge this |
|
Merge failed due to Too many checksuites for commit |
|
@pytorchbot force merge this |
|
Merge failed due to Too many checksuites for commit |
|
@pytorchbot merge this |
|
Hey @seemethere. |
facebook-github-bot
pushed a commit
that referenced
this pull request
Mar 16, 2022
Summary: Migrates our credentials to upload metrics statistics to managed IAM credentials in order to make it easier to know where the credentials are coming from and to make it easier to add more permissions / less permissions later on Signed-off-by: Eli Uriegas <eliuriegasfb.com> Pull Request resolved: #73811 Approved by: https://github.com/kit1980, https://github.com/janeyx99 Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/c5bc9e122c762565ec756e2331481779df0cc918 Reviewed By: osalpekar Differential Revision: D34928749 Pulled By: seemethere fbshipit-source-id: 37dd962719d7c4c9f7a82abce712be33146aa599
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Stack from ghstack:
Migrates our credentials to upload metrics statistics to managed IAM
credentials in order to make it easier to know where the credentials are
coming from and to make it easier to add more permissions / less
permissions later on.
Relates to work done in D34535827
Signed-off-by: Eli Uriegas eliuriegas@fb.com