Skip to content

Conversation

@scadu
Copy link
Contributor

@scadu scadu commented Sep 26, 2025

After merging
#1597, goss test started failing with missing refresh_authorized_keys.timer.

Looking closer, it looks like Packer overrides /tmp/conf directory with content of ../shared/conf.


Verifying base AMI built with this change, we got the timer back:

systemctl status refresh_authorized_keys.timer
○ refresh_authorized_keys.timer - Download ssh authorized_keys file from s3
     Loaded: loaded (/etc/systemd/system/refresh_authorized_keys.timer; disabled; preset: disabled)
     Active: inactive (dead)
    Trigger: n/a
   Triggers: ● refresh_authorized_keys.service

After merging
#1597, goss
test started failing with missing refresh_authorized_keys.timer.

Looking closer, it looks like Packer overrides `/tmp/conf` directory
with content of `../shared/conf`.
@scadu scadu requested a review from a team as a code owner September 26, 2025 09:56
Copy link
Contributor

@JoeColeman95 JoeColeman95 left a comment

Choose a reason for hiding this comment

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

I think just adding trailing slashes to the src and dest should merge but not going to nit-pick as I assume it's been tested and don't want to block

@scadu
Copy link
Contributor Author

scadu commented Sep 26, 2025

I think just adding trailing slashes to the src and dest should merge but not going to nit-pick as I assume it's been tested and don't want to block

@JoeColeman95 I hoped it'd worked that way. Tested it now – works fine! Thanks for flagging.

@scadu scadu merged commit 816513a into main Sep 26, 2025
1 check passed
@scadu scadu deleted the fix/missing_refresh_authorized_keys_SUP-4813 branch September 26, 2025 11:59
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