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

feat(provider/google): Added all cloudrun commands in halyard #2180

Merged
merged 2 commits into from
Sep 27, 2024

Conversation

sanopsmx
Copy link
Contributor

@sanopsmx sanopsmx commented Aug 28, 2024

This is part of spinnaker/governance#302

Added all cloudrun commands in halyard.

halyard cloudrun commands:

  1. hal config provider appengine [parameters] [subcommands]
    account: Manage and view Spinnaker configuration for the cloudrun provider’s account
    disable: Set the cloudrun provider as disabled
    edit: Edit cloudrun’s app engine configuration.
    enable: Set the cloudrun provider as enabled

  2. hal config provider cloudrun account ACCOUNT [parameters] [subcommands]

    add: Add an account to the cloudrun provider.
    delete: Delete a specific cloudrun account by name.
    edit: Edit an account in the cloudrun provider.
    get: Get the specified account details for the cloudrun provider.
    list: List the account names for the cloudrun provider.

  3. hal config provider cloudrun account delete ACCOUNT [parameters]

    ACCOUNT: The name of the account to operate on.

  4. hal config provider cloudrun account get ACCOUNT [parameters]

    ACCOUNT: The name of the account to operate on.

  5. hal config provider cloudrun disable [parameters]

  6. hal config provider cloudrun account list [parameters]

  7. hal config provider cloudrun enable [parameters]

Parameters
--deployment: If supplied, use this Halyard deployment. This will not create a new deployment.
--no-validate: (Default: false) Skip validation.

@sanopsmx
Copy link
Contributor Author

Added a video of all the halyard cloudrun commands. It is working as expected. Pls let me know if you have any more issues..

cloudrun-cmds.mp4

Copy link
Contributor

@kirangodishala kirangodishala left a comment

Choose a reason for hiding this comment

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

Can you list all the commands in the PR description?

@sanopsmx
Copy link
Contributor Author

sanopsmx commented Sep 17, 2024

Can you list all the commands in the PR description?

Listed out all commands in the PR description. Pls can you check once?

static final String LOCAL_REPOSITORY_DIRECTORY_DESCRIPTION =
"A local directory to be used to stage source files"
+ " for Cloud Run deployments within Spinnaker's Clouddriver microservice.";
static final String GIT_HTTPS_USERNAME_DESCRIPTION =
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it useful to cloudrun? What is the usecase with GIT?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is not useful for cloudrun. we need to remove.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kirangodishala Can you review once now?

Copy link
Contributor

Choose a reason for hiding this comment

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

looks fine.

@j-sandy j-sandy added the ready to merge Approved and ready for a merge label Sep 27, 2024
@mergify mergify bot added the auto merged label Sep 27, 2024
@mergify mergify bot merged commit a2ca14e into spinnaker:master Sep 27, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged ready to merge Approved and ready for a merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants