-
Notifications
You must be signed in to change notification settings - Fork 929
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
Implement schemaValidation
mode handling in dataconnect:sql:migrate
and deploy
#7639
Conversation
dataconnect:sql:migrate
and deploy
dataconnect:sql:migrate
and deploy
schemaValidation
mode handling in dataconnect:sql:migrate
and deploy
@@ -34,6 +34,7 @@ export const command = new Command("dataconnect:sql:migrate [serviceId]") | |||
options, | |||
schema: serviceInfo.schema, | |||
validateOnly: true, | |||
schemaValidation: serviceInfo.dataConnectYaml.schema.datasource.postgresql?.schemaValidation, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quick sanity check - does this get broken by our move from datasource -> datasources? @fredzqm might know off the top of his head.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Chatted offline -- based on go/fdc:multi-data-source it looks like the datasource
field will still be supported, though we should also support datasources
in #7589.
…e` and `deploy` (#7639) * Support handling schema validation options in `diff`. * Format. * Don't output second set of diffs if they're equal to the first. * Update dataconnect:sql:migrate * Fix my failed rebase lol * Finish handling schemaValidation mode in `dataconnect:sql:migrate`. * Fix setSchemaValidationMode. * Concat follow-up strict diffs, instead of replacing compat diffs. * Don't display purely optional changes with flag. * Pass `schemaValidation` into `deploy` command. * Address review comments.
Description
Updates
firebase dataconnect:sql:migrate
andfirebase deploy
per go/fdc-compat-mode-cli-api.Scenarios Tested
schemaValidation
unset, both required and optional changes:firebase deploy
firebase deploy --force
firebase deploy --non-interactive
firebase deploy --non-interactive --force
firebase dataconnect:sql:migrate
firebase dataconnect:sql:migrate --force
firebase dataconnect:sql:migrate --non-interactive
firebase dataconnect:sql:migrate --non-interactive --force
schemaValidation
unset, only required changes:firebase deploy
firebase deploy --force
firebase deploy --non-interactive
firebase deploy --non-interactive --force
firebase dataconnect:sql:migrate
firebase dataconnect:sql:migrate --force
firebase dataconnect:sql:migrate --non-interactive
firebase dataconnect:sql:migrate --non-interactive --force
schemaValidation
unset, only optional changes:firebase deploy
firebase deploy --force
firebase deploy --non-interactive
firebase deploy --non-interactive --force
firebase dataconnect:sql:migrate
firebase dataconnect:sql:migrate --force
firebase dataconnect:sql:migrate --non-interactive
firebase dataconnect:sql:migrate --non-interactive --force
schemaValidation=COMPATIBLE
, both required and optional changes:firebase deploy
firebase deploy --force
firebase deploy --non-interactive
firebase deploy --non-interactive --force
firebase dataconnect:sql:migrate
firebase dataconnect:sql:migrate --force
firebase dataconnect:sql:migrate --non-interactive
firebase dataconnect:sql:migrate --non-interactive --force
schemaValidation=COMPATIBLE
, only optional changes:firebase deploy
firebase deploy --force
firebase deploy --non-interactive
firebase deploy --non-interactive --force
firebase dataconnect:sql:migrate
firebase dataconnect:sql:migrate --force
firebase dataconnect:sql:migrate --non-interactive
firebase dataconnect:sql:migrate --non-interactive --force
schemaValidation=STRICT
, both required and optional changes:firebase deploy
firebase deploy --force
firebase deploy --non-interactive
firebase deploy --non-interactive --force
firebase dataconnect:sql:migrate
firebase dataconnect:sql:migrate --force
firebase dataconnect:sql:migrate --non-interactive
firebase dataconnect:sql:migrate --non-interactive --force
Sample Commands
firebase deploy
, withschemaValidation
unset, both required and optional changes, and default options selected: