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

Calculate github actions job matrix from metadata.json; fix frozen-string modification in puppet_output #327

Merged
merged 3 commits into from
Oct 28, 2020

Conversation

DavidS
Copy link
Contributor

@DavidS DavidS commented Oct 28, 2020

The matrix_from_metadata executable supports creating a platform
matrix for the new cloud-ci project based off the metadata.json
in the current module.

@DavidS DavidS added the feature label Oct 28, 2020
@DavidS DavidS requested a review from a team as a code owner October 28, 2020 11:02
@DavidS DavidS force-pushed the matrix branch 4 times, most recently from d6543bb to 8765f3e Compare October 28, 2020 11:50
@DavidS DavidS marked this pull request as draft October 28, 2020 12:00
@DavidS
Copy link
Contributor Author

DavidS commented Oct 28, 2020

Used in puppetlabs/puppetlabs-testing#308

@codecov-io
Copy link

Codecov Report

Merging #327 into main will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #327   +/-   ##
=======================================
  Coverage   63.34%   63.34%           
=======================================
  Files           7        7           
  Lines         802      802           
=======================================
  Hits          508      508           
  Misses        294      294           
Impacted Files Coverage Δ
lib/puppet_litmus/puppet_helpers.rb 83.49% <100.00%> (ø)

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 6ba315e...b4dbf0b. Read the comment docs.

@DavidS DavidS added the bugfix label Oct 28, 2020
@DavidS DavidS changed the title Calculate github actions job matrix from metadata.json Calculate github actions job matrix from metadata.json; fix frozen-string modification in puppet_output Oct 28, 2020
@DavidS DavidS marked this pull request as ready for review October 28, 2020 16:36
exe/matrix_from_metadata Outdated Show resolved Hide resolved
The `matrix_from_metadata` executable supports creating a platform
matrix for the new cloud-ci project based off the `metadata.json`
in the current module.
To avoid various testing efforts colliding, puppet_litmus
always needs to test against the local PR's branch and remove
any other puppet_litmus declarations in MOTD's Gemfile.
`bolt_result` seems to be a frozen string now, so don't
unnecessarily try to modify it. Instead create a new string.
@carabasdaniel carabasdaniel merged commit 6fca854 into puppetlabs:main Oct 28, 2020
@DavidS DavidS deleted the matrix branch October 29, 2020 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants