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

Adjust CSV status descriptors for Vault to reflect new status block #142

Merged
merged 1 commit into from
Nov 3, 2017
Merged

Adjust CSV status descriptors for Vault to reflect new status block #142

merged 1 commit into from
Nov 3, 2017

Conversation

josephschorr
Copy link
Collaborator

Example status block:

status:
    clientPort: 8200
    initialized: true
    nodes:
      active: example-vault-1395307387-682n0
      available:
      - example-vault-1395307387-682n0
      - example-vault-1395307387-w7smw
      sealed: null
      standby:
      - example-vault-1395307387-w7smw
      updated:
      - example-vault-1395307387-682n0
      - example-vault-1395307387-w7smw
    serviceName: example-vault

Example status block:

```
status:
    clientPort: 8200
    initialized: true
    nodes:
      active: example-vault-1395307387-682n0
      available:
      - example-vault-1395307387-682n0
      - example-vault-1395307387-w7smw
      sealed: null
      standby:
      - example-vault-1395307387-w7smw
      updated:
      - example-vault-1395307387-682n0
      - example-vault-1395307387-w7smw
    serviceName: example-vault
```
path: serviceName
x-descriptors:
- 'urn:alm:descriptor:io.kubernetes:Service'
- description: The port at which the Vault cluster is running under the service.
Copy link
Member

Choose a reason for hiding this comment

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

doesn't this need x-descriptors?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Nope; they are optional and not necessary for simple displays of information. We should probably add some later, but for now, not necessary

Copy link
Member

@ecordell ecordell left a comment

Choose a reason for hiding this comment

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

LGTM

@josephschorr josephschorr merged commit 2236d32 into operator-framework:master Nov 3, 2017
@josephschorr josephschorr deleted the joseph.schorr/ALM-248/vault-csv branch November 3, 2017 17:21
njhale pushed a commit to njhale/operator-lifecycle-manager that referenced this pull request Sep 10, 2018
…horr/ALM-248/vault-csv

Adjust CSV status descriptors for Vault to reflect new status block
ecordell pushed a commit to ecordell/operator-lifecycle-manager that referenced this pull request Mar 8, 2019
…horr/ALM-248/vault-csv

Adjust CSV status descriptors for Vault to reflect new status block
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