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.

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

3.1.1 prep #85

Merged
merged 3 commits into from
Jan 8, 2021
Merged

3.1.1 prep #85

merged 3 commits into from
Jan 8, 2021

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Jan 8, 2021

No description provided.

nkoenig and others added 3 commits January 8, 2021 10:26
* Fix env parsing by placing it before executable parsing

Signed-off-by: Nate Koenig <[email protected]>

* Add tests to #81 (#82)

This adds a simple check to verify that the env directives apply
to everything in the ignition namespace, as fixed in #81.

The test itself is a bit of a workaround. We don't have a mechanism
for checking the manager output, and executable command directives
cannot expand environment variables.

This writes a script that is executed by the test to verify that a
file specified by an environment variable is touch-ed

Signed-off-by: Michael Carroll <[email protected]>

Co-authored-by: Nate Koenig <[email protected]>
Co-authored-by: Michael Carroll <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>

Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
@github-actions github-actions bot added the 🔮 dome Ignition Dome label Jan 8, 2021
@codecov
Copy link

codecov bot commented Jan 8, 2021

Codecov Report

Merging #85 (c0f4199) into ign-launch3 (39efc58) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           ign-launch3      #85   +/-   ##
============================================
  Coverage        30.24%   30.24%           
============================================
  Files                3        3           
  Lines              734      734           
============================================
  Hits               222      222           
  Misses             512      512           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39efc58...c0f4199. Read the comment docs.

@chapulina chapulina changed the title 3 3 1 prep 3.1.1 prep Jan 8, 2021
@nkoenig nkoenig merged commit 8ad92df into ign-launch3 Jan 8, 2021
@nkoenig nkoenig deleted the 3-3-1-prep branch January 8, 2021 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants