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.

Sign up for GitHub

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

Jump to bottom

Document using K9s on the seed #159

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from
Open

Document using K9s on the seed #159

wants to merge 1 commit into from

Conversation

MoteHue
Copy link
Contributor

@MoteHue MoteHue commented Jul 8, 2024

No description provided.

@MoteHue MoteHue requested review from mkjpryor and sd109 July 8, 2024 12:26
@MoteHue MoteHue mentioned this pull request Jul 8, 2024
Merged
mkjpryor
mkjpryor requested changes Jul 8, 2024
View reviewed changes
Copy link
Collaborator

@mkjpryor mkjpryor left a comment

Choose a reason for hiding this comment

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

Might be worth having this in a separate page, maybe with a screenshot or two, then noting that you can use it to look at resources on the seed or on the HA cluster.

@MoteHue MoteHue force-pushed the docs/k9s branch 3 times, most recently from 9309cf7 to 9139073 Compare July 8, 2024 16:01
mkjpryor
mkjpryor requested changes Jul 8, 2024
View reviewed changes
Copy link
Collaborator

@mkjpryor mkjpryor left a comment

Choose a reason for hiding this comment

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

This is pretty close - you just need to add the page to the index before it will appear in the rendered output.

docs/debugging/using-k9s.md Outdated Show resolved Hide resolved
docs/debugging/using-k9s.md

```console
$ ./bin/seed-ssh
ubuntu@azimuth-staging-seed:~$ export KUBECONFIG=./kubeconfig-azimuth-staging.yaml
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use generic names as in the "accessing the HA cluster page" I think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What do you mean by this? This is the same command as in "Accessing the HA cluster"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mkjpryor mkjpryor mkjpryor requested changes

@sd109 sd109 Awaiting requested review from sd109

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MoteHue @mkjpryor