Skip to content

[v12] Tweak protogen to not change protos from cloud (#24688)#24739

Merged
espadolini merged 1 commit intobranch/v12from
espadolini/v12-prehog-protos-gen
Apr 18, 2023
Merged

[v12] Tweak protogen to not change protos from cloud (#24688)#24739
espadolini merged 1 commit intobranch/v12from
espadolini/v12-prehog-protos-gen

Conversation

@espadolini
Copy link
Copy Markdown
Contributor

Backport #24688 to branch/v12

* Tweak protogen to not change protos from cloud

* Use strategy: all whenever possible

* Sync protos from gravitational/cloud
// UIDiscoverIntegrationAWSOIDCConnectEvent is emitted when a user is finished with the step
// that asks user to setup aws integration or select from a list of existing
// aws integrations.
message UIDiscoverIntegrationAWSOIDCConnectEvent {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose you want those here, so we are piggybacking on the backport to bring the protos as well?

(No objections, just clarifying.)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just updated to the latest version from the tip of gravitational/cloud:master, I figured it was harmless enough.

@espadolini espadolini enabled auto-merge April 18, 2023 16:54
@espadolini espadolini added this pull request to the merge queue Apr 18, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 18, 2023
@espadolini espadolini added this pull request to the merge queue Apr 18, 2023
Merged via the queue into branch/v12 with commit cb7b0bb Apr 18, 2023
@espadolini espadolini deleted the espadolini/v12-prehog-protos-gen branch April 18, 2023 18:42
@kimlisa kimlisa mentioned this pull request May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants