docs: v.4 Tutorial fixes after testing#454
Open
izmalk wants to merge 26 commits intocanonical:mainfrom
Open
Conversation
commit 4752587 Author: Vladimir Izmalkov <48120135+izmalk@users.noreply.github.com> Date: Tue Jan 13 17:49:48 2026 +0000 docs: Home page remodeling (canonical#448) * Home page remodeling * Update tutorial enumeration * Fixed broken links * Resolve Kafka links errors * Minor update Update Applications management to Client connections page. Then realize we can simplify into Connections management. * Fix upstream docs link from 4.0 to 4.1 commit 638717e Author: Iman Enami <44609233+imanenami@users.noreply.github.com> Date: Tue Jan 13 10:06:27 2026 +0400 [DPE-9033] chore: update TF provider to v1.0+ and TLS to 1/stable (canonical#442)
Signed-off-by: Vladimir Izmalkov <48120135+izmalk@users.noreply.github.com>
izmalk
changed the title
marcoppenheimer
previously approved these changes
Jan 29, 2026
Contributor
marcoppenheimer
left a comment
marcoppenheimer
left a comment
There was a problem hiding this comment.
Approving to speed things along, left some todos that need addressing before merging.
|
|
||
| ```shell | ||
| juju show-secret --reveal cluster.kafka.app | yq '.. | ."admin-password"? // empty' | tr -d '"' | ||
| juju show-secret --reveal cluster.kafka.app | yq -r '.[].content["operator-password"]' |
Contributor
There was a problem hiding this comment.
question: Why change this? It's fine, just curious.
Contributor
Author
There was a problem hiding this comment.
- It's no longer
admin-password. - The "empty" thing didn't work on my system (Ubuntu 24.04 LTS).
- The older syntax produced weird results for me. I don't remember exactly what was wrong. I think there were some excessive line breaks, or maybe it failed to find the field due to nested levels.
|
|
||
| ```shell | ||
| bootstrap_address=$(juju show-unit kafka/0 | yq '.. | ."public-address"? // empty' | tr -d '"') | ||
| bootstrap_address=$(juju show-unit kafka/0 | yq '.. | ."public-address"? // ""' | tr -d '"' | tr -d '\r\n' | sed 's/^[[:space:]]*//;s/[[:space:]]*$//') |
Contributor
There was a problem hiding this comment.
question: Same here, why change this? The proposed change is much more complicated.
Contributor
Author
There was a problem hiding this comment.
I think there were problems with invisible characters and/or line breaks included in the field and failing because of that.
If you don't like the complexity, I can spend some more time to optimise it.
izmalk
commented
Feb 13, 2026
| First, create the Juju secret with the new password you wish to use: | ||
|
|
||
| ```shell | ||
| juju add-secret internal-kafka-users admin=mynewpassword |
Contributor
Author
There was a problem hiding this comment.
I've kept the admin here since I've tested the command myself and it worked that way. Do let me know if we need to change it here too.
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Kafka VM v.4 tutorial manual testing and review/rewrite.