Skip to content
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

OCM-4186 | Feat | Changed marketplace-gcp-terms error message for non-interactive mode #569

Merged
merged 2 commits into from
Oct 10, 2023

Conversation

tirthct
Copy link
Contributor

@tirthct tirthct commented Oct 10, 2023

Changed

  • Changed marketplace-gcp-terms error message for non-interactive mode

Tested

  • Non-interactive mode should throw an error "Error: Review and accept Google Terms and Agreements on https://console.cloud.google.com/marketplace/agreements/redhat-marketplace/red-hat-openshift-dedicated. Set the flag --marketplace-gcp-terms to true once agreed in order to proceed further." : tested

@cristianoveiga
Copy link
Contributor

LGTM

@cristianoveiga cristianoveiga merged commit 0d3f4c3 into openshift-online:main Oct 10, 2023
4 checks passed
@tirthct tirthct deleted the ocm-4186 branch October 10, 2023 19:38
tirthct added a commit to tirthct/ocm-cli that referenced this pull request Oct 13, 2023
-4dad47f OCM-3510 | fix: allow clusters to edit ingresses
-57855bf fixed setting isGcpMarketplaceSubscriptionType for non interactive mode
-557a66b making GCP service account file mandatory for CCS clusters (openshift-online#565)
-1f5481b Add GCP marketplace terms and conditions for marketplace GCP clusters
-47cd35c showing error and re-prompting service-account-file question if one is not provided (openshift-online#567)
-0d3f4c3 OCM-4186 | Feat | Changed marketplace-gcp-terms error message for non-interactive mode (openshift-online#569)
-455f27e OCM-4184 | Feat | Convert relative path containing tilde for service account file (openshift-online#568)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants