Uploaded image for project: 'Puppet'
  1. Puppet
  2. PUP-8404

ssh_authorized_key should mention effective uid on debug output

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: PUP 5.4.0
    • Component/s: Types and Providers
    • Labels:
      None
    • Environment:

      linux

    • Template:
    • Acceptance Criteria:
      • debug output includes the additional information of which user the directory is being created with
    • Sub-team:
    • Team:
      Platform Core
    • Sprint:
      Platform Core KANBAN
    • Release Notes:
      Bug Fix
    • Release Notes Summary:
      Adds additional debug output when creating directories with the ssh authorized key parsed provider
    • QA Risk Assessment:
      No Action

      Description

      Puppet Version: 4.10.9
      Puppet Server Version: 2.8.0
      OS Name/Version: linux

      Trying to use ssh_authorized_key when the parent folder of .ssh doesn't belong to the target user leads to a slightly misleading "permission denied" error during mkdir. This happens because ssh_authorized_key runs mkdir as the target user, which might not be directly clear. Especially since it's not the same behaviour as an explicit file resource with owner set to the same target user (create as root than chown to target user).

      Desired Behavior:

      Output of puppet run with --debug:

      Debug: Creating /somefolder/.ssh as someuser
      Error: /Stage[main]/Main/Ssh_authorized_key[someuser]: Could not evaluate: Permission denied @ dir_s_mkdir - /somefolder/.ssh
      

      Actual Behavior:

      Output of puppet run with --debug:

      Debug: Creating /somefolder/.ssh
      Error: /Stage[main]/Main/Ssh_authorized_key[someuser]: Could not evaluate: Permission denied @ dir_s_mkdir - /somefolder/.ssh
      

        Attachments

          Activity

            jsd-sla-details-panel

              People

              • Assignee:
                melissa Melissa Stone
                Reporter:
                costela Leo Antunes
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: