Skip to content

Add hammer-cli-foreman-salt package #1216

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bastian-src
Copy link

Adds Hammer CLI for foreman_salt: https://github.com/theforeman/hammer_cli_foreman_salt

@bastian-src
Copy link
Author

@ekohl can you help me debugging the issue maybe?

I'd say the errors seem to be unrelated to the changes of my PR. I've checked the logs here and it seems like the error origins in rh_cloud tests?

2025-02-21T15:08:58.7882600Z ##[error]Scenario: install foreman with rh_cloud behaves like the foreman application Command "curl -s --cacert /etc/foreman-certs/certificate.pem https://almalinux9-64-puppet7.example.com -w '%{redirect_url}' -o /dev/null" stdout is expected to eq "https://almalinux9-64-puppet7.example.com/users/login"

Failure/Error: its(:stdout) { is_expected.to eq("https://#{host_inventory['fqdn']}#{params.fetch(:expected_login_url_path, '/users/login')}") }
  
  expected: #<Encoding:UTF-8> "https://almalinux9-64-puppet7.example.com/users/login"
       got: #<Encoding:US-ASCII> ""
  
  (compared using ==)

The same applies to my second PR which introduces the same changes, but for Hammer CLI for SCC Manager #1215

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.

1 participant