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

6.12 addendum for Okta and Entra ID #4220

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

nasirhussenm
Copy link

Description

Updates required to add the new docs version for Okta and Entra ID:

  • Adds the /sensu-go/6.12 doc set and updates for Okta provider
  • Adds the /sensu-go/6.12/doc set and updates for Entra ID provider

If you run the site locally, here are the links to use to view the content updates:

Motivation and Context

Support 6.12.0 release

Signed-off-by: nasirhussenm <[email protected]>
@jhenderson-pro
Copy link
Member

@hillaryfraley - As I mentioned in Slack, here's the PR.
@fguimond - This LGTM so far. 👍


Follow the steps in this section to create an Entra ID application and configure an Entra ID OIDC provider in Sensu.

### Create an Entra ID application
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider specifying where readers will follow these steps. For example, "In the Entra ID administrator console, create a user..." (or the correct name for the page or console, if "Entra ID administrator console" is incorrect).

6. In the API Permissions section, add the following Microsoft Graph API permissions:
- `Directory.Read.All`
- `User.Read`
7. The provider in the OIDC file should be set to `EntraID`.
Copy link
Contributor

Choose a reason for hiding this comment

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

If step 7 refers to the OIDC provider configuration in Sensu, I don't think you need to include this step.

@hillaryfraley
Copy link
Contributor

@jhenderson-pro the provider attribute needs to be added in a few more spots:

jhenderson-pro and others added 3 commits January 30, 2025 10:23
This looks great, thanks Hillary!

Co-authored-by: Hillary Fraley <[email protected]>
This looks great, thanks Hillary!

Co-authored-by: Hillary Fraley <[email protected]>
This looks great, thanks Hillary!

Co-authored-by: Hillary Fraley <[email protected]>
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