Skip to content

SAT-45983: Remove hammer insights cloud-connector enable command#20

Merged
chris1984 merged 1 commit into
theforeman:masterfrom
jnagare-redhat:remove-cloud-connector-hammer-command
Jul 6, 2026
Merged

SAT-45983: Remove hammer insights cloud-connector enable command#20
chris1984 merged 1 commit into
theforeman:masterfrom
jnagare-redhat:remove-cloud-connector-hammer-command

Conversation

@jnagare-redhat

@jnagare-redhat jnagare-redhat commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Cloud connector setup moved to foremanctl deploy --add-feature cloud-connector. The enable_connector API is removed in foreman_rh_cloud #1214.
Card: https://redhat.atlassian.net/browse/SAT-45983
Raised separate PR for translation changes #21

@jnagare-redhat jnagare-redhat changed the title Remove hammer insights cloud-connector enable command SAT-45983: Remove hammer insights cloud-connector enable command Jul 1, 2026

@jeremylenz jeremylenz left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we please put the translation changes in a separate PR? (.po and .pot files)

@chris1984

chris1984 commented Jul 2, 2026

Copy link
Copy Markdown
Member

@jnagare-redhat I tested and it works fine, I agree with Jeremy. Not sure why my comment didn't save when I acked it (I blame windows) but We need this in a separate change, if you need help let me know or Claude can do it as well if you have it setup on your machine.

Cloud connector setup moved to foremanctl deploy --add-feature
cloud-connector. The enable_connector API is removed in
foreman_rh_cloud #1214.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jnagare-redhat

jnagare-redhat commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Can we please put the translation changes in a separate PR? (.po and .pot files)

Raised separate PR for translation changes #21

@chris1984 chris1984 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for moving the translations out

@chris1984 chris1984 merged commit 2e0dd2c into theforeman:master Jul 6, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants