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

[GEN-2294] Odigos set config cli command #2346

Merged
merged 250 commits into from
Jan 30, 2025

Conversation

alonkeyval
Copy link
Collaborator

This pull request introduces a new command to manage Odigos configuration settings and updates the Helm chart for the UI deployment to use a configuration property from a ConfigMap. The most important changes include adding a new config command with subcommands to set configuration properties and updating the UI deployment to reference the new configuration property.

New command for managing configuration:

  • Added a new config command in cli/cmd/config.go to manage Odigos configuration settings, including a set subcommand to update specific properties.
  • Implemented the updateConfigProperty function to update configuration properties in the odigos-config ConfigMap.
  • Implemented the restartUIPod function to restart the UI pod when the central-backend-url property is updated.

Helm chart update:

  • Updated the helm/odigos/templates/ui/deployment.yaml file to use the centralBackendURL property from the odigos-config ConfigMap for the CENTRAL_BACKEND_WS_URL environment variable.

Copy link

@BenElferink BenElferink added the enhancement New feature or request label Jan 29, 2025
@alonkeyval alonkeyval merged commit b89523d into odigos-io:main Jan 30, 2025
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants