-
Notifications
You must be signed in to change notification settings - Fork 10
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
[WIP]create deployments commands #455
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
return nil, fmt.Errorf("serviceId, serviceName and clusterName can not be null") | ||
} | ||
if serviceId != nil { | ||
result, err := c.client.UpdateServiceDeployment(c.ctx, *serviceId, attributes) |
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.
we should be able to merge these mutations i believe
…458) * Small style tweaks and show repository health in `repositories list` Pull status is definitely nice to be able to see here. * show pull errors as well
Shows the provider, and shows repo by url instead of id in some places where useful
michaeljguarino
force-pushed
the
deployments
branch
from
November 5, 2023 17:27
82f4d0f
to
fc6df10
Compare
michaeljguarino
approved these changes
Nov 11, 2023
michaeljguarino
added a commit
that referenced
this pull request
Aug 28, 2024
* create deployments commands * add repositories commands * update client auth * add deployments repositories create command * add deployments repositories list command * refactor * bump go client * fix list deployments * use go client model * create and update service deployment * Small style tweaks and show repository health in `repositories list` (#458) * Small style tweaks and show repository health in `repositories list` Pull status is definitely nice to be able to see here. * show pull errors as well * describe service * install agent helm method * create agent namespace * enable CAPI feature flag for e2e * add services delete command * Extend clusters list display (#460) Shows the provider, and shows repo by url instead of id in some places where useful * describe cluster + code refacor * add flag validation * support handle for plural deployments services list/describe * add required flags * add handle for create CD service * add wait flag for agent installation * update services by handle * add console client mocks * improve describe cluster and service * read kubeconfig from env var if present * respect KUBECONFIG env var in plural kube client too * force update helm repos on install to prevent stale chart installations * use correct release name * don't swallow helm errors * extend update cluster for kubeconfig * add plural cd providers list command * create/delete provider credentials * pipeline create command * fix some repositories commands * add deployments clusters get-credentials command (#467) * Add a persistent config file for console logins on cli * Implement `plural cd clusters delete` (#468) When the console-client-go pr is merged, we can implement the `--soft` flag too. * fix pointer bug on config updates * add create cluster command * create azure cluster * fix after schema changes * bump console client * bump console client * update arg name * refactor cluster commands * refactor provider command * remove provider id param * don't wait in cd install command * add clusters bootstrap command * add operator uninstall, cluster tags on bootstrap * more tweaks to agent installer * some more agent install tweaks * update gcp provider * update provider create * fix profile list cmd * filter out existing providers * remove gcp credenitials b64 encode logic * BYOK installer for plural console * wrap up console installer * fix linter * refactor cd structure * fix unit tests * fix cd install command * add kustomize support * init eject command * fix token name for cd install * make cd clusters kubeconfig more robust * add kas_dns to server context * add eject command and fix gcp capi provider bootstrapping * modify control plane installer * add service clone support --------- Co-authored-by: Marcin Maciaszczyk <[email protected]> Co-authored-by: michaeljguarino <[email protected]> Co-authored-by: Sebastian Florek <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
New CLI commands for CD deployments
Set env variables:
PLURAL_CONSOLE_URL=https://console.cd-demo.onplural.sh
PLURAL_CONSOLE_TOKEN=xxxx
Labels
Test Plan
Checklist