This repository has been archived by the owner on Aug 10, 2023. It is now read-only.
Adding a tutorial for Cloud Run using conditional IAM roles #1417
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
ToddKopriva
requested review from
ToddKopriva
and removed request for
parkjam4
|
Thanks, Mike. Has this tutorial received a technical review yet? |
|
Yes, I have had two others in my SA team run through the tutorial. I have
"failed build" and will update this later today to make sure it passes.
…On Wed, Sep 9, 2020 at 8:34 AM Todd Kopriva ***@***.***> wrote:
Thanks, Mike.
Has this tutorial received a technical review yet?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1417 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFSV6RUCSDY4QUBWGS2AJ3SE6OAZANCNFSM4RCRK2HA>
.
--
Mike Ensor
Cloud Solutions Architect
206-973-6621 Mobile
1021 Valley Street, Seattle, WA 98109
|
|
@ToddKopriva -- Looks like you've fixed the testing failures, what do I need to do next? |
|
Since you've already gotten a technical review internally, the next step is my editorial review and production pass. There are some other items ahead of this in the queue, so I estimate that I'll get to this on Friday. |
Thanks!! |
ToddKopriva
approved these changes
Sep 14, 2020
xiangshen-dk
pushed a commit
to xiangshen-dk/community
that referenced
this pull request
Jan 24, 2022
…oudPlatform#1417) * initial commit of outline and overview of steps * adding a diagram showing the architecture overview * exporting to png for diagram, specifying output for cloud bucket contents * finished step two and half way through step three * completing step 4 * Finished with steps 1-6, fully deployed with conditional IAM role bindings * adding a cleanup method * moving several sections to higher level * fixing the core project ID * updating gitlab URL to use https instead of unauthenticated ssh * Removing GSA and images * fixed frontmatter to pass CI tests * moved diagram to public bucket and began copy edit * check-in partway through line edit * cleaning up usage of 'service account' and some other copy-editing * Update index.md * finished first edit pass * Update index.md * breaking last block into steps Co-authored-by: Todd Kopriva <43478937+ToddKopriva@users.noreply.github.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
This tutorial comes from a customer interaction where they were looking for confirmation that Cloud Run can use a Google Service Account that has associated Conditional IAM role bindings.