Unable to set the SSH key via UI in PMM Server

Description

Problem:

I was unable to set the SSH key via UI on the PMM Server Settings UI. 

Steps to reproduce:

This is what you see in `pmm-managed.log`:

How to test

None

How to document

None

Attachments

1

Smart Checklist

Activity

Show:

Aaditya Dubey July 24, 2023 at 11:56 AM
Edited

Hi , ,

Thank you for the report.

Since this feature not meant for docker then user should have either option disabled or proper error should be seen.  Improvement request being tracked here , Marking the report OPEN for reference.

Roma Novikov July 20, 2023 at 10:44 AM

I have created to track improvements. Let's keep the current one to ensure SSH works in the supported cases.

Alex Demidoff July 20, 2023 at 8:56 AM
Edited

I can tell, this is no longer a bug, I'm not sure when it was fixed

FYI: this feature is meant to work only for our AMI and OVF distributions, not for docker. However, you will see this screen in docker as well, which may be super confusing for some users.

Solution? It's easy to detect the PMM distribution (the backend can read the contents of /srv/distribution) and hide the feature for docker. Simple.

Can we convert this ticket into an improvement?

Aaditya Dubey July 19, 2023 at 5:00 PM

Hi ,

Sure, thanks!

Denys Kondratenko October 26, 2022 at 10:19 AM

  if I login to the container there is no "admin" user.

 

https://github.com/percona/pmm/blob/main/build/ansible/roles/cloud-node/tasks/main.yml#L82

 

looks like such user only created for the VMs ?

 

`admin` user is hard coded:

https://github.com/percona/pmm/blob/4e5131b276e816c9783f676936a1117818d2e946/managed/services/server/server.go#L828

https://jira.percona.com/browse/PMM-4317

So the question is - is the doc confusing or we need that ansible role work for all the images?

 

Looks like not. Doc also mentions appliance:

https://docs.percona.com/percona-monitoring-and-management/how-to/configure.html#ssh-key

 

So it doesn't looks like a bug unless you see this issue in VM, but rather improvement to detect env and don't show this setting, or improve doc.

Details

Assignee

Reporter

Priority

Components

Needs QA

Yes

Needs Doc

No

Regression Issue

Yes

Affects versions

Smart Checklist

Created October 24, 2022 at 7:24 PM
Updated March 6, 2024 at 12:30 AM