Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sync_config_files issue when supplying optional file option #85

Closed
koomie opened this issue Apr 26, 2016 · 1 comment
Closed

sync_config_files issue when supplying optional file option #85

koomie opened this issue Apr 26, 2016 · 1 comment
Assignees
@koomie
Labels
bug
Milestone
v0.54.0

Comments

@koomie
Copy link
Contributor

koomie commented Apr 26, 2016 •
edited
Loading 

# /admin/losf/sync_config_files /etc/passwd
** Syncing configuration files (discovery:knl7)
Can't use an undefined value as a HASH reference at /admin/losf/utils/sync_config_utils.pl line 237.
@koomie koomie self-assigned this Apr 26, 2016
@koomie koomie added the bug label Apr 26, 2016
koomie added a commit that referenced this issue Apr 27, 2016
@koomie
bug fix for sync_name_file, updated to include use of replace_vars (#85)
d4ba000
@koomie
Copy link
Contributor Author

koomie commented Apr 27, 2016

Thanks to Tommy@TACC for the catch and fix.

@koomie koomie mentioned this issue Jun 1, 2017
Closed
@koomie koomie added this to the v0.54.0 milestone Jun 1, 2017
koomie added a commit that referenced this issue Jun 1, 2017
@koomie
add unit test for sync_config_files (#85)
6628844
@koomie koomie closed this as completed Jun 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@koomie koomie

Labels
bug
Projects
None yet
Milestone
v0.54.0
Development

No branches or pull requests
1 participant
@koomie