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

add the doc for nats-server system_account credentials (#15381) #16400

Merged
merged 5 commits into from Dec 21, 2022

Conversation

raypinto
Copy link
Contributor

Description

Changes proposed in this pull request:

Related issue(s)

…15381)

* add the doc for nats-server system_account credentials

* restructuring according to template

* remove old texts

* remove typos

* Apply suggestions from code review

Co-authored-by: Nina Hingerl <76950046+NHingerl@users.noreply.github.com>

* add prerequisites

Co-authored-by: Nina Hingerl <76950046+NHingerl@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Dec 20, 2022

Deploy Preview for kyma-project-docs-preview ready!

Name Link
🔨 Latest commit 5c49658
🔍 Latest deploy log https://app.netlify.com/sites/kyma-project-docs-preview/deploys/63a321a103277d00080ae3e4
😎 Deploy Preview https://deploy-preview-16400--kyma-project-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@kyma-bot kyma-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Dec 20, 2022
@raypinto raypinto linked an issue Dec 20, 2022 that may be closed by this pull request
6 tasks
@raypinto raypinto added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. area/eventing Issues or PRs related to eventing area/documentation Issues or PRs related to documentation labels Dec 20, 2022
…-events.md

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>
nataliasitko
nataliasitko previously approved these changes Dec 20, 2022
@kyma-bot kyma-bot added the lgtm Looks good to me! label Dec 20, 2022
To acquire the credentials, run the following command:

```bash
kubectl get secrets -n kyma-system eventing-nats-secret -oyaml | grep -e accountsJson | awk '{print $2}' | base64 -d | grep {user: | awk '{$1=$1};1' | awk '{print substr($0, 2, length($0) - 2)}'
Copy link
Contributor

@k15r k15r Dec 20, 2022

Choose a reason for hiding this comment

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

Suggested change
kubectl get secrets -n kyma-system eventing-nats-secret -oyaml | grep -e accountsJson | awk '{print $2}' | base64 -d | grep {user: | awk '{$1=$1};1' | awk '{print substr($0, 2, length($0) - 2)}'
kubectl get secrets -n kyma-system eventing-nats-secret -ogo-template='{{.data.accountsJson|base64decode}}'| grep 'user:' | tr -d '{}'

nataliasitko
nataliasitko previously approved these changes Dec 21, 2022
@kyma-bot kyma-bot added the lgtm Looks good to me! label Dec 21, 2022

## Prerequisites

Kyma version `2.7` or later.
Copy link
Contributor

Choose a reason for hiding this comment

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

wrong version. 2.10 it is...
or just omit the version here. it will show up as soon as it is available

@kyma-bot kyma-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed lgtm Looks good to me! size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 21, 2022
@kyma-bot kyma-bot added the lgtm Looks good to me! label Dec 21, 2022
@raypinto raypinto removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 21, 2022
@kyma-bot kyma-bot merged commit 0afd642 into kyma-project:main Dec 21, 2022
@raypinto raypinto deleted the sys-account-docu branch December 21, 2022 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Issues or PRs related to documentation area/eventing Issues or PRs related to eventing lgtm Looks good to me! size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow using nats server info
5 participants