search

plus3it / amigen7

Set of tools to provide automation of tasks for creating STIG-partitioned EL7 AMIs
Apache License 2.0
34 stars 27 forks source link

Ensure /var/run is link to /run #99

Closed ferricoxide closed 3 years ago

ferricoxide commented 4 years ago

Fixes issue where /var/run doesn't get set up correctly (necessary for environments that don't use spel)

ferricoxide commented 4 years ago

I assume you've tested this for your current use case, but have you tested it with spel, or do you need someone else to handle that test (which is fine, just doublechecking...)?

My assumption is/was that spel had a graceful handler for the missing /var/run → /run link. However, I haven't explicitly tested it in the spel context nor did I find anything other than the pytest references w/in the spel project-code. So, "yeah: wants testing w/in spel (with spel being adjusted for the corrected AMIgen7 behavior if issues are found)"

lorengordon commented 3 years ago

My assumption is/was that spel had a graceful handler for the missing /var/run → /run link. However, I haven't explicitly tested it in the spel context nor did I find anything other than the pytest references w/in the spel project-code.

Ok, can you update from latest amigen7/master? I'll test using spel, but latest spel requires a patch that is not present in your branch.