Evgeni Golov
Evgeni Golov
So, I played a bit a round with this, and this patch makes it a bit tidier: ```diff diff --git plugins/modules/content_export.py plugins/modules/content_export.py index a5d1b615..3811da38 100644 --- plugins/modules/content_export.py +++ plugins/modules/content_export.py @@...
> I _think_ it should be doable to have all this within a single module. We *could*. But from talking to Jeremy and reading the docs, the three workflows (library,...
Oooh, if it can look up the recent automatically, then perfect, leave it!
Shall we sit down on Monday or so together and talk tests?
Was the repository export added recently to Katello? The error says that there is no "repository" action on the exports controller. As the tests run against a recorded api, it...
https://github.com/Katello/katello/commit/4c1e72119659be50dd60682b1ef10e13c3ab682a seems to be in 4.5+, so yeah, "recent". Feel free to replace `katello.json` with a fresh 4.5 one (but vanilla Katello with ostree enabled please, no other plugins)
Here is the full diff I applied to the info module: ```diff diff --git plugins/modules/content_export_info.py plugins/modules/content_export_info.py index d9097595..6a17826a 100644 --- plugins/modules/content_export_info.py +++ plugins/modules/content_export_info.py @@ -34,26 +34,21 @@ options: - Export...
I am happy with the `content_export_library` module, but I think the others need a bit more work to eliminate the `cv_id` etc references. Shall we split this PR into four,...
the remaining sanity errors will be fixed in https://github.com/theforeman/foreman-ansible-modules/pull/1455 and are unrelated to this PR!
Just so that I understand it correctly, you'd want the same data structure as currently returned by the various `*_info` modules, but available through a lookup? And then you'd use...