Skip to content

Conversation

@prkhrkat
Copy link
Contributor

@prkhrkat prkhrkat commented Jun 3, 2025

Description

Fixes https://github.com/devtron-labs/sprint-tasks/issues/2129

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?


@github-actions
Copy link

github-actions bot commented Jun 3, 2025

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP 404).\n

@github-actions
Copy link

github-actions bot commented Jun 4, 2025

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP 404).\n

@github-actions
Copy link

github-actions bot commented Jun 4, 2025

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP 404).\n

@github-actions
Copy link

github-actions bot commented Jun 4, 2025

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP 404).\n

@prkhrkat prkhrkat changed the title Posthog dev fix: Posthog events added Jun 4, 2025
@gitguardian
Copy link

gitguardian bot commented Jun 11, 2025

⚠️ GitGuardian has uncovered 2 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
2763127 Triggered Generic High Entropy Secret a43f58a scripts/devtron-reference-helm-charts/cronjob-chart_1-6-0/secrets-test-values.yaml View secret
141558 Triggered Generic High Entropy Secret a43f58a scripts/devtron-reference-helm-charts/gpu-workload-4-21-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.

posthog query
@sonarqubecloud
Copy link

@prkhrkat prkhrkat merged commit f4d311e into develop Jun 15, 2025
6 checks passed
@prkhrkat prkhrkat deleted the posthog-dev branch June 15, 2025 06:37
vikramdevtron added a commit that referenced this pull request Jun 19, 2025
* create cm in case of cluster event instead of secret

* bump cpmmon lib

* bump common lib

* bump common lib

* fix

* k8s supported label label

* small fix

* bump common lib

* bump common lib

* bump

* code review level 2

* added missing dependency (#6634)

* fix

* updates hash

* fixed waiting to start status in deployemnt (#6649)

* fix: Dependency upgrades (#6622)

* removed hardcoded dependencies

* k8s upgrade

* helm and argo upgrade

* helm and argo upgrade

* golang.org/x/net upgrade

* golang.org/x/net upgrade

* DeleteProject gitlab client changes

* common-lib sync

* common lib sync

* common-lib sync

* Merge pull request #6651 from devtron-labs/fix-panic-cd-trigger

fix: panic handling on cd trigger

* main merge

* make

* fix: Posthog events added  (#6640)

* refact

* app count events

* app count events

* app count events

* app count events

* query fix

* policies included

* wip

* query fix

posthog query

* removed extra dockerFile count queries

* fix

* fix

* fix: inconsistency in saving empty GitOps repository  (#6612)

* fix: gitops provider multiple save error

* updated const ArgoRepoSyncDelayErr as per new ArgoCd update

* fix: unable resolve 'HEAD' to a commit SHA

* fix: git clone issue for empty repository

* fix: ambiguous argument 'origin/HEAD': unknown revision or path not in the working tree.

* fix: SSH known host confirmation issue on cloning

* chore: updated default value of ARGO_REPO_REGISTER_RETRY_COUNT

* chore: incorporated review comments

* chore: removed dead code

* chore: minor refactorings

* chore: added integration test cases

* fix: added handling for argocd older versions

* sync: Main sync develop (#6669)

* panic fixes (#6657)

* fix-cd-pipeline-check (#6658)

* vendor update on develop

---------

Co-authored-by: Shivam Nagar <[email protected]>

* vendor update on rc 38 (#6671)

* added empty migration seq (#6672)

* fix down script for gp workload (#6678)

* sync: Main sync RC (#6679)

* panic fixes (#6657)

* fix-cd-pipeline-check (#6658)

* release: PR for v1.7.0 (#6667)

* Updated release-notes files

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* removed init-container from git-sensor (#6605)

Co-authored-by: Badal Kumar Prusty <[email protected]>

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* feat: added support for loadBalancerIP (#6617)

* added support for loadBalancerIP

* removed default value for loadBalancerIP

---------

Co-authored-by: Badal Kumar Prusty <[email protected]>

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* Updated release notes

* set fsGroup to allow non-root app to write to PVC (#6664)

Co-authored-by: Badal Kumar Prusty <[email protected]>

* misc: Shifted the dbConfigs under global (#6621)

* Shifted the dbConfigs under global

* updated the configurations for external database support

* handled casbin migration case for database name

* added support for custom annotations for ci-runner cd-runner and devtron

* switched the conditioning to handle devtron-bom.yaml

* removed check of values.global from postgresql.yaml

* Updated kubewatch to a4fc9044-419-33852 tag in values file

* Updated kubelink to a4fc9044-564-33855 tag in values file

* Updated devtron to c8e75fb-434-33854 tag in values file

* added support for nfs (#6665)

Co-authored-by: Badal Kumar Prusty <[email protected]>

* Updated lens to a4fc9044-333-33874 tag in values file

* Updated git-sensor to a4fc9044-200-33872 tag in values file

* Updated dashboard to a85f2624-690-33873 tag in values file

* Updated ci-runner to a4fc9044-138-33875 tag in values file

* Updated notifier to 19d654ff-372-33876 tag in values file

* Updated image-scanner to a4fc9044-141-33877 tag in values file

* Updated chart-sync to a4fc9044-836-33878 tag in values file

* Updated hyperion to c8e75fb-280-33879 tag in values file

* Updated the nfs condition

* Updated the version in scripts

* updated release-notes file

* Updated releasenotes.md

* Updated release-notes-v1.7.0.md

* finalized releasenotes

---------

Co-authored-by: Badal Kumar <[email protected]>
Co-authored-by: Badal Kumar Prusty <[email protected]>
Co-authored-by: akshatsinha007 <[email protected]>
Co-authored-by: akshatsinha007 <[email protected]>

* vendor update on rc

---------

Co-authored-by: Shivam Nagar <[email protected]>
Co-authored-by: systemsdt <[email protected]>
Co-authored-by: Badal Kumar <[email protected]>
Co-authored-by: Badal Kumar Prusty <[email protected]>
Co-authored-by: akshatsinha007 <[email protected]>
Co-authored-by: akshatsinha007 <[email protected]>

---------

Co-authored-by: Prakash Kumar <[email protected]>
Co-authored-by: prakhar katiyar <[email protected]>
Co-authored-by: kartik-579 <[email protected]>
Co-authored-by: kartik-579 <[email protected]>
Co-authored-by: Shivam Nagar <[email protected]>
Co-authored-by: Asutosh Das <[email protected]>
Co-authored-by: systemsdt <[email protected]>
Co-authored-by: Badal Kumar <[email protected]>
Co-authored-by: Badal Kumar Prusty <[email protected]>
Co-authored-by: akshatsinha007 <[email protected]>
Co-authored-by: akshatsinha007 <[email protected]>
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.

4 participants