-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add example of metadata.json file that is referenced by metadata2gha #107
Comments
When I wrote the documentation I thought the structure would be way more important than the actual data. So I showed how it's generated and then made the output readable. Since then the output has changed because of GitHub internals. Yesterday I was thinking about writing a description of what it would do. So describe the operating systems it found and whether it will (or won't run tests on them). Perhaps that's more useful than this. |
It would be most useful if the documentation described what OSes will and won't run. I was surprised to find out that
did not output anything for Fedora when I added the following to my metadata.json
Based on #106, I'm learning that some OSes don't run but again, as an amateur, it's been a difficult experience trying to learn the 'whys' and 'why nots' of how this entire process behaves. |
That surprises me, because the whole idea is that you pass in a metadata and it parses that. That's the whole concept of the gem: abstractions on top of your What was your expectation that it was operating on? |
Here's the metadata.json that I used. It's highly likely that I'm doing something wrong...hence the/my need for an example metadata.json file to be added. I understand that the $operatingsystemrelease of Fedora isn't valid, but no value of $operatingsystemrelease seems to work. |
My completely amateur/noob expectation was that any value for $operatingsystem would be considered valid and that value be parsed and output for each $operatingsystemrelease value as well. Obviously now, I was incorrect in that assumption. Somewhere (and possibly still related to #106) there is code that filters out Fedora and other 'non-supported' OSes. I've not found documentation yet that gives a list of supported OSes. Again, probably this is in the code but the info should be documented somewhere. |
There is no known working build for Fedora 8.
Correct: it only outputs combinations that we know we can test on. That's a core design principle. You take So we first have to distinguish unit tests and acceptance tests. For the unit tests this gem doesn't look at the operating systems. Instead it assumes you solve that yourself, either by doing it manually or relying on https://github.com/voxpupuli/rspec-puppet-facts. Then for acceptance tests we we on beaker. If we can test depends on several factors. It needs an OS image to work with. In GHA we now rely on containers and beaker.rb generates input for beaker-hostgenerator to generate the Beaker configuration. Note we can also work with other hypervisors and locally I tell beaker-hostgenerator to work with libvirt instead. It's just that in GHA we only have access to Docker, so that's why we're tied to containers. Then it needs the availability of a Puppet build. We mostly rely on official builds for operating systems, encoded in aio.rb. For some we've added distro Puppet package support and both #104 & #102 expand that to fill the gap of missing official Puppet packages. And combining all of the above into a configuration that can be used in GHA is what github_actions.rb does. We the have gha-puppet that uses this configuration to dynamically drive the matrices. The end result is that you can add an OS to |
Generating Github Actions outputs shows output of parsing a metadata.json file but the puppet_metadata projects doesn't include a metatadata.json file that was used to generate said output.
It'd be nice to have a metadata.json or metadata.json.EXAMPLE included in the project to be able to reproduce the output provided in the README.md.
The text was updated successfully, but these errors were encountered: