Skip to content

Conversation

@kartik-579
Copy link
Member

Description

Fixes #

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR requires documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested it for all user roles.
  • I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?


vikramdevtron and others added 13 commits May 30, 2025 11:38
* fix upgrade feedback issue

* Refactor installer module path handling for configurability.

Replaced hardcoded installer module paths with configurable environment variables in `ServerEnvConfig`. This improves flexibility and maintainability, allowing paths to be customized without altering the codebase. Removed redundant constant `INSTALLER_MODULES_HELM_KEY`.

* Refactor module key generation to use basePath configuration.

Updated functions to include `basePath` in module enable key generation, ensuring consistency and flexibility when constructing keys. Adjusted related calls across services to pass the `basePath` from environment configuration.

* Update env_gen.json placeholder content

Revised the content of env_gen.json to modify the placeholder or default configuration. This ensures alignment with the expected environment setup and eliminates any outdated references.

* Refactor server status mapping logic for Helm applications

Simplified and standardized the server status mapping by reinstating Helm release-based logic, removing redundant code, and eliminating the `mapApplicationStatusToServerStatus` function. This enhances maintainability and aligns with existing Helm release workflows.

* installation sleep

* Fix syntax error in installation script sleep function

Replaced incorrect usage of `sleep(180);` with `sleep 180` to align with proper syntax. This ensures the installation script runs without errors during execution.

* Fix syntax for sleep function in installation script

Replaced incorrect usage of `sleep` with the proper function call `sleep(180)`. This ensures the script executes correctly without syntax errors.

* Update installation script to replace sleep with shellScript

Replaced the fixed sleep duration with repeated shellScript calls to improve the script's flexibility and manageability. This change aims to enhance the installation process by providing clearer execution control and better

* Update installation script to replace sleep with shellScript

Replaced the fixed sleep duration with repeated shellScript calls to improve the script's flexibility and manageability. This change aims to enhance the installation process by providing clearer
* notification for failed Deployment

* notification for failed Deployment IN ARGO

* review comments

* async func

* fix

* wire
* external argo fixes

* removing unwanted param

* nil handling

* updating rest config

---------

Co-authored-by: Asutosh Das <[email protected]>
* fixed user status function inauthroser

* fix: correct function name for user retrieval in UserService
@gitguardian
Copy link

gitguardian bot commented Jun 4, 2025

⚠️ GitGuardian has uncovered 3 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
9257608 Triggered Base64 AWS keys 000bd42 scripts/devtron-reference-helm-charts/deployment-chart_4-21-0/test-values.json View secret
2763127 Triggered Generic High Entropy Secret 000bd42 scripts/devtron-reference-helm-charts/deployment-chart_4-21-0/secrets-test-values.yaml View secret
141558 Triggered Generic High Entropy Secret 000bd42 scripts/devtron-reference-helm-charts/reference-chart_5-1-0/env-values.yaml View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

vivek-devtron
vivek-devtron previously approved these changes Jun 4, 2025
@kartik-579 kartik-579 merged commit 2643db0 into main Jun 4, 2025
8 checks passed
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 4, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants