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

Allow VizierID to be used to find the Vizier's org #1650

Merged
merged 5 commits into from
Aug 9, 2023

Conversation

kpattaswamy
Copy link
Member

@kpattaswamy kpattaswamy commented Aug 3, 2023

Summary: This PR allows the VizierID to be used when getting the org as opposed to only relying on the DeployKey when adding feature flags.

Relevant Issues: #1632

Type of change: /kind bug

Test Plan: Another PR needs to come in on the operator side to retrieve/send the VizierID to the cloud. Getting this PR in on the cloud side will simplify the process of testing the integration between the two later.

…zier's org to set feature flags.

Signed-off-by: Kartik Pattaswamy <[email protected]>
@kpattaswamy kpattaswamy temporarily deployed to pr-actions-approval August 3, 2023 23:00 — with GitHub Actions Inactive
@ddelnano ddelnano requested a review from a team August 4, 2023 19:21
@kpattaswamy kpattaswamy temporarily deployed to pr-actions-approval August 4, 2023 22:50 — with GitHub Actions Inactive
@kpattaswamy kpattaswamy temporarily deployed to pr-actions-approval August 8, 2023 01:39 — with GitHub Actions Inactive
src/cloud/api/controllers/config_grpc_test.go Outdated Show resolved Hide resolved
Signed-off-by: Kartik Pattaswamy <[email protected]>
@kpattaswamy kpattaswamy temporarily deployed to pr-actions-approval August 9, 2023 17:54 — with GitHub Actions Inactive
@vihangm vihangm merged commit 1c328aa into pixie-io:main Aug 9, 2023
23 checks passed
kpattaswamy added a commit to kpattaswamy/pixie that referenced this pull request Aug 10, 2023
kpattaswamy added a commit to kpattaswamy/pixie that referenced this pull request Aug 11, 2023
kpattaswamy added a commit to kpattaswamy/pixie that referenced this pull request Aug 11, 2023
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.

4 participants