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

[feature] add describe command #2055

Merged
merged 3 commits into from
Aug 14, 2024
Merged

Conversation

facchettos
Copy link
Contributor

@facchettos facchettos commented Aug 13, 2024

What issue type does this pull request address? (keep at least one, remove the others)
/kind feature

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
resolves eng-4313

Please provide a short message that should be published in the vcluster release notes
Fixed an issue where vcluster ...

What else do we need to know?

Output example :

Backing Store: embedded-database
Distro: k8s
Image tags:
  apiServer: registry.k8s.io/kube-apiserver:v1.29.6
  controllerManger: registry.k8s.io/kube-controller-manager:v1.29.6
  scheduler: registry.k8s.io/kube-scheduler:v1.29.6
  syncer: ghcr.io/loft-sh/vcluster-pro:0.20.0-beta.15
Version: 0.20.0-beta.15

Copy link

netlify bot commented Aug 13, 2024

Deploy Preview for vcluster-docs canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit ff2d81e
🔍 Latest deploy log https://app.netlify.com/sites/vcluster-docs/deploys/66bcb93d438ee80008e8cc78

@facchettos facchettos force-pushed the describe branch 2 times, most recently from 9209896 to 4e9027d Compare August 14, 2024 12:02
@facchettos facchettos marked this pull request as ready for review August 14, 2024 12:13
@facchettos facchettos added the backport-to-v0.20 backport this PR to v0.20 branch label Aug 14, 2024
@facchettos facchettos changed the title Describe [feature] add describe command Aug 14, 2024
@facchettos facchettos force-pushed the describe branch 2 times, most recently from 81d595a to 72df660 Compare August 14, 2024 12:55
namespace = flags.Namespace
}

secretName := "vc-config-" + name
Copy link
Member

Choose a reason for hiding this comment

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

Should we use find vcluster here?

_, err = output.Write(b)
return err
}
fVcluster, err := find.GetVCluster(ctx, rawConfig.CurrentContext, name, namespace, log.Discard)
Copy link
Member

Choose a reason for hiding this comment

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

why not get this earlier?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

because for the yaml and json output we don't need it

pkg/cli/describe_helm.go Outdated Show resolved Hide resolved
@FabianKramm FabianKramm merged commit 450e528 into loft-sh:main Aug 14, 2024
61 checks passed
loft-bot pushed a commit that referenced this pull request Aug 14, 2024
* [feat] added describe command

* fixed conditions

* now uses camelcase

(cherry picked from commit 450e528)
@loft-bot
Copy link

💚 All backports created successfully

Status Branch Result
v0.20

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

FabianKramm added a commit that referenced this pull request Aug 14, 2024
[v0.20] [feature] add describe command (#2055)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-v0.20 backport this PR to v0.20 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants