Skip to content

Fix config macro TAG for thinkst and fix the default value for the okta main log event stream config macros#274

Merged
keith-smiley-gravwell merged 3 commits intogravwell:mainfrom
kris-watts-gravwell:main
Mar 24, 2026
Merged

Fix config macro TAG for thinkst and fix the default value for the okta main log event stream config macros#274
keith-smiley-gravwell merged 3 commits intogravwell:mainfrom
kris-watts-gravwell:main

Conversation

@kris-watts-gravwell
Copy link
Copy Markdown
Contributor

This PR addresses no specific issue

This PR proposes...

  • Fixing the config macro for the thinkst kit to be a TAG
  • update the default value for okta config macro to match what the hosted-ingester is producing

PR Tasks

  • e2e and/or unit tests included. If not, please provide an explanation.
  • Bug fixes only: minimal repro steps included on the issue (for PR QA + Release QA).

Reviewer Tasks

  • e2e or unit tests are present to test the proposed changes.
  • Code is sufficiently documented.
  • Code meets quality and correctness expectations.

Copy link
Copy Markdown
Contributor

@keith-smiley-gravwell keith-smiley-gravwell left a comment

Choose a reason for hiding this comment

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

LGTM

@keith-smiley-gravwell keith-smiley-gravwell merged commit eb5aabf into gravwell:main Mar 24, 2026
1 check 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.

2 participants