Skip to content

build: deploy use of converter config in GCE BUILD rule + DIREGAPIC action#993

Merged
vchudnov-g merged 3 commits intomasterfrom
diregapic-config-deploy
Mar 14, 2025
Merged

build: deploy use of converter config in GCE BUILD rule + DIREGAPIC action#993
vchudnov-g merged 3 commits intomasterfrom
diregapic-config-deploy

Conversation

@vchudnov-g
Copy link
Copy Markdown
Contributor

@vchudnov-g vchudnov-g commented Mar 13, 2025

This will start working once the WORKSPACE file is updated to or past this one; the source-of-truth of that is Google-internal and gets propagated out.

"inlineSchemas" : [{
"schema": "",
"locations": {
"ErrorSet": [
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.

Please confirm this is a reasonable set for this inline schema that was causing the original problem.

@vchudnov-g vchudnov-g changed the title build: deploy uise of converter config in GCE BUILD rule + DIREGAPIC action build: deploy use of converter config in GCE BUILD rule + DIREGAPIC action Mar 13, 2025
@vchudnov-g vchudnov-g marked this pull request as ready for review March 13, 2025 19:04
Copy link
Copy Markdown
Contributor

@chingor13 chingor13 left a comment

Choose a reason for hiding this comment

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

Stamping this to unblock. I can't answer your question, so perhaps get confirmation before merging

@vchudnov-g vchudnov-g added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 14, 2025
@vchudnov-g
Copy link
Copy Markdown
Contributor Author

Adding do not merge until we get confirmation on the config from the GCE team.

@vchudnov-g
Copy link
Copy Markdown
Contributor Author

Thanks!

@vchudnov-g vchudnov-g added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 14, 2025
@vchudnov-g vchudnov-g force-pushed the diregapic-config-deploy branch from 324bfd3 to 25ea681 Compare March 14, 2025 19:01
@vchudnov-g vchudnov-g removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 14, 2025
@vchudnov-g vchudnov-g merged commit 1e6cb62 into master Mar 14, 2025
@vchudnov-g vchudnov-g deleted the diregapic-config-deploy branch March 14, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

owlbot:run Add this label to trigger the Owlbot post processor.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants