Skip to content

Feedback from customer on PDF#1065

Open
ranjinimn wants to merge 2 commits intosuse-edge:mainfrom
ranjinimn:PDF-customer-feedback
Open

Feedback from customer on PDF#1065
ranjinimn wants to merge 2 commits intosuse-edge:mainfrom
ranjinimn:PDF-customer-feedback

Conversation

@ranjinimn
Copy link
Collaborator

@ranjinimn ranjinimn commented Feb 24, 2026

Hi, I have accessed the PDF version of the Edge documentation, and there are some issues at least for the last few versions of the document. For example you can take a look at https://documentation.suse.com/suse-edge/3.5/pdf/edge_en.pdf:

Images that represent diagrams aren't correct, they are missing all the text, this text is displayed correctly on the HTML page, like PDF pages 21 (xxi), 22 (xxii), 24 (xxiv), 31 (3), 55 (27).
Text that represents code goes beyond document right margin, like in pages 76 (48), 80 (52), 82 (54), 170 (142), 174 (146), 176 (148). And sometimes beyond the bottom margin and overlap the page number like pages 199 (171), 200 (172), 202 (174), 203 (175).
Screenshots are larger than page size and go beyond the right margin or the bottom margin and you cannot really see the whole screenshot that is necessary, like in pages 178 (150), 180 (152).
The pages I mention are only examples. I guess the whole PDF should be reviewed in detail since there can be more issues.
I hope it helps.

https://suse.slack.com/archives/C072TF88KNU/p1771545274219749

Signed-off-by: Ranjini M N <ranjini.n@suse.com>
@ranjinimn ranjinimn self-assigned this Feb 24, 2026
@ranjinimn ranjinimn requested a review from hardys February 24, 2026 19:21
Comment on lines +214 to +218
userDataBase64: I2Nsb3VkLWNvbmZpZwpkaXNhYmxlX3Jvb3Q6IGZhbHNlCnNzaF9wd2F1dGg6IFRyd
WUKdXNlcnM6CiAgLSBkZWZhdWx0CiAgLSBuYW1lOiBzdXNlCiAgICBncm91cHM6IH
N1ZG8KICAgIHNoZWxsOiAvYmluL2Jhc2gKICAgIHN1ZG86ICBBTEw9KEFMTCkgTk9
QQVNTV0Q6QUxMCiAgICBsb2NrX3Bhc3N3ZDogRmFsc2UKICAgIHBsYWluX3RleHRf
cGFzc3dkOiAnc3VzZScK
Copy link
Collaborator Author

@ranjinimn ranjinimn Feb 24, 2026

Choose a reason for hiding this comment

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

@hardys This renders differently in the PDF version and runs off the page. Is there an option to replace this with anything more generic?

Copy link
Collaborator

Choose a reason for hiding this comment

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

This, and the strings below, are examples if I understand it correctly. I suggest to shorten them to something like
I2Nsb3VkLW[...]Anc3VzZScK
OK with you @hardys ?

Copy link
Contributor

Choose a reason for hiding this comment

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

They are examples, but we can't just truncate them IMO, as they serve a specific purpose which is to set the username/password for the SSH login which is described below (see password is "suse")

Another option is to take the unencoded data, e.g:

#cloud-config
disable_root: false
ssh_pwauth: True
users:
  - default
  - name: suse
    groups: sudo
    shell: /bin/bash
    sudo:  ALL=(ALL) NOPASSWD:ALL
    lock_passwd: False
    plain_text_passwd: 'suse'

We could say to create that file then in the kubectl apply say $(cat cloud-config.yaml)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thank you for this suggestion. 2 questions:
1 . This is one of the many examples of passwords. Do we need to add do the same action for every one of them?
2. Which section would be the most appropriate to add this step?

Copy link
Contributor

Choose a reason for hiding this comment

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

It's only referenced twice so I'd probably just add it to each example e.g:

$ cat <<EOF > user-data.yaml
#cloud-config
disable_root: false
ssh_pwauth: True
users:
  - default
  - name: suse
    groups: sudo
    shell: /bin/bash
    sudo:  ALL=(ALL) NOPASSWD:ALL
    lock_passwd: False
    plain_text_passwd: 'suse'
EOF
$ kubectl apply -f - <<EOF
apiVersion: kubevirt.io/v1
kind: VirtualMachine
metadata:
  name: ingress-example
  namespace: default
spec:
  runStrategy: Always
  template:
    metadata:
      labels:
        app: nginx
    spec:
      domain:
        devices: {}
        machine:
          type: q35
        memory:
          guest: 2Gi
        resources: {}
      volumes:
      - containerDisk:
          image: quay.io/containerdisks/opensuse-tumbleweed:1.0.0
        name: tumbleweed-containerdisk-0
      - cloudInitNoCloud:
          userDataBase64: $(cat user-data.yaml)
        name: cloudinitdisk

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@hardys Should it have the EOF at the end of this code block?

$ cat <<EOF > user-data.yaml
#cloud-config
disable_root: false
ssh_pwauth: True
users:
  - default
  - name: suse
    groups: sudo
    shell: /bin/bash
    sudo:  ALL=(ALL) NOPASSWD:ALL
    lock_passwd: False
    plain_text_passwd: 'suse'
EOF
$ kubectl apply -f - <<EOF
apiVersion: kubevirt.io/v1
kind: VirtualMachine
metadata:
  name: ingress-example
  namespace: default
spec:
  runStrategy: Always
  template:
    metadata:
      labels:
        app: nginx
    spec:
      domain:
        devices: {}
        machine:
          type: q35
        memory:
          guest: 2Gi
        resources: {}
      volumes:
      - containerDisk:
          image: quay.io/containerdisks/opensuse-tumbleweed:1.0.0
        name: tumbleweed-containerdisk-0
      - cloudInitNoCloud:
          userDataBase64: $(cat user-data.yaml)
        name: cloudinitdisk
EOF

[,console]
----
$6$G392FCbxVgnLaFw1$Ujt00mdpJ3tDHxEg1snBU3GjujQf6f8kvopu7jiCBIhRbRvMmKUqwcmXAKggaSSKeUUOEtCP3ZUoZQY7zTXnC1
$6$G392FCbxVgnLaFw$Ujt00mdpJ3tDHxEg1snBU3GjujQf6f8kvopu7jiCBIhRbRvMmKUqwcmXAKggaSSKeUUOEtCP3ZUoZQY7zTXnC1
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I had to remove the number from the password as it was not breaking the text correctly in the PDF rendition. Please let me know if it's okay.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Again, an example string

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@hardys Can we use the string as suggested by Frank in the earlier comment?
For example:

Suggested change
$6$G392FCbxVgnLaFw$Ujt00mdpJ3tDHxEg1snBU3GjujQf6f8kvopu7jiCBIhRbRvMmKUqwcmXAKggaSSKeUUOEtCP3ZUoZQY7zTXnC1
$6$G392FCbxVgn[...]Y7zTXnC1

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@hardys Can you please help here ?

users:
- username: root
encryptedPassword: $6$G392FCbxVgnLaFw1$Ujt00mdpJ3tDHxEg1snBU3GjujQf6f8kvopu7jiCBIhRbRvMmKUqwcmXAKggaSSKeUUOEtCP3ZUoZQY7zTXnC1
encryptedPassword: $6$G392FCbxVgnLaFw$Ujt00mdpJ3tDHxEg1snBU3GjujQf6f8kvopu7jiCBIhRbRvMmKUqwcmXAKggaSSKeUUOEtCP3ZUoZQY7zTXnC1
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

same comment as above

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.

3 participants