ekohl
October 12, 2021, 12:35pm
#1
Since this PR was merged, nightly has been broken:
theforeman:develop ← ATIX-AG:remove_puppet_api_routes
opened 12:08PM - 23 Sep 21 UTC
Needed for seamless integration with the foreman_puppet plugin (https://github.c… om/theforeman/foreman_puppet/pull/196)
Prerequisite for this PR: https://github.com/theforeman/foreman_puppet/pull/201
<!---
Thank you for contributing to The Foreman, please read the
[following guide](https://www.theforeman.org/contribute.html), in short:
* [Create an issue](https://projects.theforeman.org/projects/foreman/issues)
* Reference the issue via `Fixes #1234` in the commit message
* Prefer present-tense, imperative-style commit messages
* Mark all strings for translation, see [1]
* Suggest prerequisites for testing and testing scenarios following example above.
* Prepend `[WIP]` for work in progress to prevent bots from triggering actions
* Be patient, we will do our best to take a look as soon as we can
* Explain the purpose of the PR, attach screenshots if applicable
* List all prerequisites for testing (e.g. VMware cluster, two smart proxies...)
* Reviewers often use extensive list of items to check, have a look prior submitting [2]
* Be nice and respectful
1: https://projects.theforeman.org/projects/foreman/wiki/Translating#Translating-for-developers
2: https://github.com/theforeman/foreman/blob/develop/developer_docs/pr_review.asciidoc
-->
Both Debian:
And RPM:
All of them fail on:
not ok 6 import motd puppet class
# (in test file fb-test-puppet.bats, line 64)
# `hammer proxy import-classes --name $(hostname -f)' failed with status 64
# Error: Unrecognised option '--name'.
#
# See: 'hammer proxy import-classes --help'.
The import-classes command needs this foreman_puppet PR to work:
committed 11:50AM - 23 Sep 21 UTC
The latest foreman_puppet version does not contain this commit, yet. Fix is in progress.
1 Like
The puppet failure seems to be fixed by the new plugin release (2.0.0.alpha.2)
1 Like
