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

switch cluster access to API and Config map #114

Merged
merged 1 commit into from
Jun 9, 2024
Merged

switch cluster access to API and Config map #114

merged 1 commit into from
Jun 9, 2024

Conversation

jaxxstorm
Copy link
Member

@jaxxstorm jaxxstorm commented Jun 9, 2024

🚀 This description was created by Ellipsis for commit f0f562e

Summary:

Updated cluster access configuration to enable both API and ConfigMap based authentication.

Key points:

  • Updated NewCluster in provider/pkg/provider/cluster.go to set AuthenticationMode to API_AND_CONFIG_MAP.

Generated with ❤️ by ellipsis.dev

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested. Reviewed everything up to f0f562e in 55 seconds

More details
  • Looked at 15 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_AzYaUGmmwfB3oJAU


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

provider/pkg/provider/cluster.go Show resolved Hide resolved
@jaxxstorm jaxxstorm merged commit ad1aaeb into main Jun 9, 2024
14 checks passed
@jaxxstorm jaxxstorm deleted the api_access branch June 9, 2024 14:48
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.

1 participant