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

[actions] Disable Perf PR action #1946

Merged
merged 1 commit into from
Jun 10, 2024

Conversation

JamesMBartlett
Copy link
Member

Summary: We need to rethink how we trigger perf on PRs.

Relevant Issues: N/A

Type of change: /kind cleanup

Test Plan: Disables the perf action, no test necessary.

Signed-off-by: James Bartlett <[email protected]>
@JamesMBartlett JamesMBartlett requested a review from a team as a code owner June 10, 2024 18:25
@JamesMBartlett JamesMBartlett merged commit 7359d40 into pixie-io:main Jun 10, 2024
20 checks passed
@JamesMBartlett JamesMBartlett deleted the james/disable_pr_perf branch June 10, 2024 18:29
ddelnano pushed a commit to ddelnano/pixie that referenced this pull request Sep 23, 2024
Summary: We need to rethink how we trigger perf on PRs.

Relevant Issues: N/A

Type of change: /kind cleanup

Test Plan: Disables the perf action, no test necessary.

Signed-off-by: James Bartlett <[email protected]>
GitOrigin-RevId: 7359d40
ddelnano added a commit to ddelnano/pixie that referenced this pull request Sep 23, 2024
* [actions] Disable Perf PR action (pixie-io#1946)

Summary: We need to rethink how we trigger perf on PRs.

Relevant Issues: N/A

Type of change: /kind cleanup

Test Plan: Disables the perf action, no test necessary.

Signed-off-by: James Bartlett <[email protected]>

* Parameterize create_cloud_secrets.sh to allow for specifying domain and environment

Signed-off-by: Dom Del Nano <[email protected]>

* Replace dev.withpixie.dev with dev.getcosmic.ai

Signed-off-by: Dom Del Nano <[email protected]>

* Squash with other create_cloud_secrets changes

Signed-off-by: Dom Del Nano <[email protected]>

* Ensure dev postgres uses postgres from k8s secret like the other dev services

Signed-off-by: Dom Del Nano <[email protected]>

* Changes to get initial testing cloud working

Signed-off-by: Dom Del Nano <[email protected]>

* Fix issues with cert-manager issued service-tls-certs

Signed-off-by: Dom Del Nano <[email protected]>

* WIP for patching testing environment's YAML for cosmic

Signed-off-by: Dom Del Nano <[email protected]>

* Start of separating the cosmic cloud changes out from the prior testing environment

Signed-off-by: Dom Del Nano <[email protected]>

* More separation of legacy test environment and cosmic's

Signed-off-by: Dom Del Nano <[email protected]>

* Revert changes to k8s/cloud/testing in favor of deletion patch

Signed-off-by: Dom Del Nano <[email protected]>

* Organize deletion patches into separate directory

Signed-off-by: Dom Del Nano <[email protected]>

* Add missing private/testing configs and remove hard coded oauth settings

Signed-off-by: Dom Del Nano <[email protected]>

* Remove changes not needed for Cosmic's testing cloud deployment

Signed-off-by: Dom Del Nano <[email protected]>

* Refactor bazel args so duplicate values can be reused

Signed-off-by: Dom Del Nano <[email protected]>

* Add missing cloud_deps configs

Signed-off-by: Dom Del Nano <[email protected]>

* Remove cert-manager files managed through terraform. Add deploy_secrets.sh script

Signed-off-by: Dom Del Nano <[email protected]>

* Add .sops.yaml with GCP KMS configuration

Signed-off-by: Dom Del Nano <[email protected]>

* Updates to get staging working without a security policy

Signed-off-by: Dom Del Nano <[email protected]>

* Remove extraneous sops comments

Signed-off-by: Dom Del Nano <[email protected]>

* Use Cosmic terms of service and privacy policy

Signed-off-by: Dom Del Nano <[email protected]>

* Fix linting issues

Signed-off-by: Dom Del Nano <[email protected]>

* Use react-cookie-consent
 in place of react-cookie-banner and add private cookie-banner overlay

Signed-off-by: Dom Del Nano <[email protected]>

* Fix email signing by using connection name rather than ID for PL_AUTH_EMAIL_PASSWORD_CONN

Signed-off-by: Dom Del Nano <[email protected]>

* Increase the length of users.profile_picture column

Signed-off-by: Dom Del Nano <[email protected]>

* Convert users.profile_picture to TEXT column

Signed-off-by: Dom Del Nano <[email protected]>
(cherry picked from commit 9c92b5d)

* Remove perf workflow

Signed-off-by: Dom Del Nano <[email protected]>

---------

Signed-off-by: James Bartlett <[email protected]>
Signed-off-by: Dom Del Nano <[email protected]>
Co-authored-by: James Bartlett <[email protected]>
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.

2 participants