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

Prep for 2.2.1 release #84

Merged
merged 1 commit into from
Jan 8, 2021
Merged

Prep for 2.2.1 release #84

merged 1 commit into from
Jan 8, 2021

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Jan 8, 2021

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

Signed-off-by: Nate Koenig <[email protected]>
@github-actions github-actions bot added the 🏰 citadel Ignition Citadel label Jan 8, 2021
@codecov
Copy link

codecov bot commented Jan 8, 2021

Codecov Report

Merging #84 (99e606e) into ign-launch2 (1870459) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           ign-launch2      #84   +/-   ##
============================================
  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 1870459...99e606e. Read the comment docs.

@nkoenig nkoenig merged commit c4433e7 into ign-launch2 Jan 8, 2021
@nkoenig nkoenig deleted the 2-2-1-prep branch January 8, 2021 21:03
nkoenig added a commit that referenced this pull request Jan 8, 2021
* Fix env parsing by placing it before executable parsing (#81)

* 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]>

* Prep for 2.2.1 release (#84)

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

Co-authored-by: Nate Koenig <[email protected]>

Co-authored-by: Nate Koenig <[email protected]>
Co-authored-by: Michael Carroll <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants