-
Notifications
You must be signed in to change notification settings - Fork 17
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
Saving a report doesn't work when passing two catalogs into the CLI #37
Comments
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Just chiming in to the stale bot doesn't eat this issue. This is still a problem. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
<insert activity here> |
Hehe, don't mind the bot, he's not that much of a bad boy (though it could be controlled with some kind of ignore label) 😁 |
Ah, there was an issue open to implement it in the bot probot/stale#110 But the bot closed it 🤣 |
Rather than continuing this fight with a bot, I decided to just fix the problem myself :) |
This bit is actually useful 😃 |
When using the
--output_report
flag and passing in two catalogs, a report doesn't get saved.For example, this does not write a
report.json
:Looks like the problem is that the code responsible for writing a report only exists in the conditional branch for retrieving a catalog remotely:
https://github.com/camptocamp/puppet-catalog-diff/blob/v2.3.0/lib/puppet/face/catalog/diff.rb#L182-L187
The text was updated successfully, but these errors were encountered: