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: add status printcolumn to ES CRD #318

Merged
merged 1 commit into from
Aug 10, 2021

Conversation

henninge
Copy link
Contributor

Fixes: #316

Copy link
Contributor

@paul-the-alien paul-the-alien bot left a comment

Choose a reason for hiding this comment

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

Greetings!
Thank you for contributing to this project!
If this is your first time contributing, please make
sure to read the Developer and Contributing Process guides.
Please also mind and follow our Code of Conduct.

@moolen
Copy link
Member

moolen commented Aug 10, 2021

/ok-to-test sha=cc7b0fb

@moolen
Copy link
Member

moolen commented Aug 10, 2021

Initial: empty

NAME      STORE        REFRESH INTERVAL   STATUS
example   sm-example   0

On error:

NAME      STORE        REFRESH INTERVAL   STATUS
example   sm-example   0                  SecretSyncedError

When synced:

NAME      STORE        REFRESH INTERVAL   STATUS
example   sm-example   0                  SecretSynced

@moolen
Copy link
Member

moolen commented Aug 10, 2021

/approve

@moolen
Copy link
Member

moolen commented Aug 10, 2021

/merge

@moolen moolen merged commit 328cf88 into external-secrets:main Aug 10, 2021
@paul-the-alien
Copy link
Contributor

This Pull Request Can not be merge currently

ElsaChelala added a commit to ElsaChelala/external-secrets that referenced this pull request Aug 31, 2021
ElsaChelala added a commit to ElsaChelala/external-secrets that referenced this pull request Aug 31, 2021
ElsaChelala added a commit to ElsaChelala/external-secrets that referenced this pull request Aug 31, 2021
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.

Add status to printed columns for kubectl get
2 participants