Integrate generic oauth provider
Change summary:
- moved elasticsearch oidc realm def into a helper function to consolidate
- exposed default keycloak values as fields which can be overridden from the bb side
Closes #12 (closed).
Merge request reports
Activity
added elasticsearch logging statusreview labels
Relates to BB MR !417
- Resolved by Andrew Blanchard
Outstanding issue: the ES pods do not appear to restart when the underlying flux repository (this repo) is updated. The
logging/logging-ek-es-master-es-configsecret is updated correctly, but that doesn't cause the pod to restart and the changes aren't automatically applied.Not sure if this needs to be resolved prior to merging, but I'm open to suggestions..
Edited by Andrew Blanchard - Last reply by Andrew Blanchard
added 1 commit
- 2ec6092d - Updated the chart version to a reasonable value
added 10 commits
-
2ec6092d...870a1801 - 9 commits from branch
main - 8973e942 - merged master, resolved conflict - may need additional guidance on expected version update pattern
-
2ec6092d...870a1801 - 9 commits from branch
added 1 commit
- 4e982c18 - attempting to force the ES pods to restart with pod annotations on the values...
added 1 commit
- d542c2cc - Updated the annotation sha to check the oidc helper values
added 1 commit
- 399ed51a - Check for value before attempting to template
added 1 commit
- 066689a3 - Reverting the chart version back to a sensible value after testing/debugging.
added 1 commit
- 3b9fcf80 - Adjusted the 'default' value for 'optional' SSO params to check for a manual...
added 1 commit
- a65d315e - changed formatting on requested scope value expansion
added 1 commit
- 047a39ff - Trying the range iteration without the block quote?
added 1 commit
- b077e593 - Added a note in tkeycloak.md to address updating the role mapping to poitnt to...
added 4 commits
-
b077e593...318d0eaa - 3 commits from branch
main - 86b04dd2 - Merge branch 'main' into ablan/integrate-generic-oauth-provider
-
b077e593...318d0eaa - 3 commits from branch
added 1 commit
- 0a043f2a - Found a better way to handle the nil vs empty values upstream. Removed magic '-' value check.
added 1 commit
- db630bbd - updating the chart to support passing the sso.certificate_authorities param
mentioned in issue #12 (closed)
added statusdoing label and removed statusreview label
added 5 commits
-
5df13328...d84a935f - 4 commits from branch
main - 1500e115 - merged in main - incremented chart version (unsure if that is the correct...
-
5df13328...d84a935f - 4 commits from branch
changed milestone to %1.8.0
added statusreview label and removed statusdoing label
- Resolved by Andrew Blanchard
@ablanchard Can you write up a few points for this version bump in the CHANGELOG.md and let's get this merged!
- Last reply by Andrew Blanchard
added 1 commit
- d9343ad7 - Updated the chart version and added a summary to the CHANGELOG.
mentioned in commit e06f1646
