Skip to content
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

Add warning while creating keytab entry #1958

Merged

Conversation

mjivraja
Copy link
Contributor

@mjivraja mjivraja commented Feb 1, 2023

In creating keytab entry during the configuration of direct AD integration with GSS-Proxy, we can warn the end-user about connectivity between hosts and domain. This will save time for end-users.

https://bugzilla.redhat.com/show_bug.cgi?id=2159686

Please cherry-pick my commits into:

  • Foreman 3.5/Katello 4.7 (planned Satellite 6.13)
  • Foreman 3.4/Katello 4.6 (EL8 only)
  • Foreman 3.3/Katello 4.5 on EL7 & EL8 (Satellite 6.12 on EL8 only)
  • Foreman 3.2/Katello 4.4 on EL7 & EL8
  • Foreman 3.1/Katello 4.3 on EL7 & EL8 (Satellite 6.11 EL7/8, orcharhino 6.1 on EL7, orcharhino 6.2 on EL7/8)
  • For Foreman 3.0 or older, please create a separate PR.
  • We do not accept PRs for Foreman 2.3 or older.

Copy link
Contributor

@adamlazik1 adamlazik1 left a comment

Choose a reason for hiding this comment

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

Slightly reworded.

@mjivraja mjivraja force-pushed the 15428-Warning-about-keytab-master branch from 070f865 to 6908a84 Compare March 3, 2023 12:08
In creating keytab entry during the configuration of direct AD
integration with GSS-Proxy, we can warn the end-user about connectivity
between hosts and domain. This will save the time for end-users.

https://bugzilla.redhat.com/show_bug.cgi?id=2159686
@mjivraja mjivraja force-pushed the 15428-Warning-about-keytab-master branch from 6908a84 to 906f13a Compare March 3, 2023 14:52
@adamlazik1
Copy link
Contributor

Waiting for technical review and even waiting to know if it is absolutely required.

Are we ready to move this forward?

@adamlazik1 adamlazik1 merged commit 018cb2e into theforeman:master Mar 7, 2023
@adamlazik1
Copy link
Contributor

Merged and cherry-picked:
2840e84..0f62a1d 3.1 -> 3.1
1513934..ebf9884 3.2 -> 3.2
7b596ab..9198868 3.3 -> 3.3
5ef92de..d355815 3.4 -> 3.4
1335926..928f819 3.5 -> 3.5
62308a4..3de9e02 3.6 -> 3.6

@mjivraja mjivraja deleted the 15428-Warning-about-keytab-master branch March 7, 2023 09:26
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.

None yet

5 participants