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

feat: added posthog events for cloud provider and version #4443

Merged
merged 5 commits into from
Jan 22, 2024

Conversation

ashishdevtron
Copy link
Contributor

@ashishdevtron ashishdevtron commented Dec 21, 2023

Description

Fixes #4478

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

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?


Copy link

sonarcloud bot commented Dec 21, 2023

Quality Gate Failed Quality Gate failed

Failed conditions

37.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

vikramdevtron
vikramdevtron previously approved these changes Jan 18, 2024
@vikramdevtron vikramdevtron self-requested a review January 18, 2024 12:16
Copy link

sonarcloud bot commented Jan 22, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

37.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@ashishdevtron ashishdevtron merged commit c3987b8 into main Jan 22, 2024
3 of 5 checks passed
@ashishdevtron ashishdevtron deleted the posthog-v2 branch January 22, 2024 07:35
Ash-exp added a commit that referenced this pull request Jan 22, 2024
* stage 1

* fix: Image-Scanner status for failed request (#4513)

* retry envs added

* json naming

* migration for image scan plugin and retry env fix

* fix

* image scan migration script updated - added new input variables

* migration updated

* migration updated

* name changed

* refactor

* added default value and change script no

* chore: added sql-validator in git-hub action (#4255)

* added sql-validator in git-hub action

* removed exit commands

* edited the grep with whole word

* modified

* modified comments

---------

Co-authored-by: Badal Kumar Prusty <badalkumar@Badals-MacBook-Pro.local>

* feat: Adds Copacetic plugin to patch vulnerable images  (#4566)

* Adds copacetic plugin

* copacetic plugin: adds plugin icon and updates url in migration file

* Update 211_copacetic_plugin_v1_0_0.up.sql

adds copa patch command exit condition

* Update 211_copacetic_plugin_v1_0_0.up.sql

adds exit 1 if patching multi architecture image

* dummy commit 211_copacetic_plugin_v1_0_0.up.sql

dummy commit

* chore: added sql-validator in git-hub action (#4255)

* added sql-validator in git-hub action

* removed exit commands

* edited the grep with whole word

* modified

* modified comments

---------

Co-authored-by: Badal Kumar Prusty <badalkumar@Badals-MacBook-Pro.local>

* updates migration no.

---------

Co-authored-by: Badal Kumar <130441461+badal773@users.noreply.github.com>
Co-authored-by: Badal Kumar Prusty <badalkumar@Badals-MacBook-Pro.local>

* feat: Create Dockerslim plugin and Create EKS Cluster plugin (#4525)

* Plugins icon

* eks and dockerslim plugin sql up

* eks and dockerslim plugin down script

* dockerslim plugin after review PR

* Changes after review dockerslim

* image changes

* new image dockerslim

* eks creation plugin

* Final PR commit, eks and dockerslim plugins

* feat: making pre,post,deploy triggers flows idempotent (#4486)

* update common-lib

* added idempotency code and common-lib version bump

* added the optionsal validations pubsub msg pre-processing-hooks logic and added duplicate trigger check as a validation for pre,post,deploy trigger flows

* bump common-lib

* added callback logger func

* sql scripts added

* query fix

* query fix

* fix

* refactoring

* remove dag exececutor dependency from ciEeventHalndler Sevice

* bump common-lib

* move context to trigger context

* add logs

* migration version fix

* remove test data

* update script number

---------

Co-authored-by: Kripansh <kripansh@devtron.ai>

* chore: panic middleware and proxy handler (#4546)

* initial commit

* removed from hyperion

* changes

* added trace

* pr comments

* pg timeouts

* panic message constant

* cleaning up

* common lib version

* argo assets revert

* adding image scanner proxy route

* common-lib-update

* refactoring: App Store deployment services

* fix: sql script added for putting Identifier type=0 for all null values present in generic_note (#4568)

* script for initializing null identifier type col to 0(cluster type)

* script fix

* empty script for 216

* 216 up script revert

* fix: 500 fixes second iteration (#4464)

* FetchChartNotes return 4xx when installed app or installed app versions not found

* GetDeploymentHistory and GetInstalledAppVersion to return 4xx when installed app or installed app versions not found

* fixes

* adding pg:",discard_unknown_columns" in tables where doesn't exist

* fix for Git Sensor response

* fix for inception

* fix for k8s resource delete

* fix for pipeline already exists

* fix for pipeline already exists

* fix

* fix

* fix

* fix

* go mod version changes

* merged

* common lib main version sync

* main merge

---------

Co-authored-by: ShashwatDadhich <dadhichshashwat1808@gmail.com>

* feat: added posthog events for cloud provider and version (#4443)

* added cloud provider event

* added logger for testing

* logger removed

* common-lib version upgraded

* introduced CASBIN_DATABASE env (#4547)

* added --reuse-values in troubleshoot (#4577)

Co-authored-by: Badal Kumar Prusty <badalkumar@Badals-MacBook-Pro.local>

---------

Co-authored-by: nishant <nishant@devtron.ai>
Co-authored-by: ashishdevtron <141303172+ashishdevtron@users.noreply.github.com>
Co-authored-by: Badal Kumar <130441461+badal773@users.noreply.github.com>
Co-authored-by: Badal Kumar Prusty <badalkumar@Badals-MacBook-Pro.local>
Co-authored-by: tayalrishabh96 <135199635+tayalrishabh96@users.noreply.github.com>
Co-authored-by: Rupin Solanki <44662963+rupinSec@users.noreply.github.com>
Co-authored-by: Gireesh Naidu <111440205+gireesh-devtron@users.noreply.github.com>
Co-authored-by: Kripansh <kripansh@devtron.ai>
Co-authored-by: subhashish-devtron <129164905+subhashish-devtron@users.noreply.github.com>
Co-authored-by: Prakash <prakash.kumar@devtron.ai>
Co-authored-by: ShashwatDadhich <dadhichshashwat1808@gmail.com>
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.

Feature: Posthog - serverVersion with every event and cluster provider
2 participants