Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

test: Simple App to test legacy errors #14679

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

Merged
merged 6 commits into from
May 23, 2025

Conversation

purva9413
Copy link
Contributor

This is a simple app which will help us debug and visualize legacy errors that Compute throws in its client library. We can then draw comparisons to the same error scenario from the Compute APIs accessed through different platforms like gcloud or the GCP portal and check how we can better propagate these errors to the customer in our client library.

The error scenario reproduced by this app is trying to create an already existing disk resource in GCP. The disk can easily be created beforehand, before running this app.

Commands:

## Use default auth from terminal of vscode
gcloud auth application-default login

## Run command
dotnet run project-id={your-project-id} zone={your-zone} disk-name={your-disk-name}

@purva9413 purva9413 requested a review from a team as a code owner April 28, 2025 19:35
@purva9413 purva9413 force-pushed the legacy-errors-test-app branch from 2b3f419 to 44f33f7 Compare April 28, 2025 20:08
@purva9413 purva9413 changed the title Simple App to test legacy errors test: Simple App to test legacy errors Apr 28, 2025
@amanda-tarafa amanda-tarafa self-requested a review April 28, 2025 22:03
@amanda-tarafa amanda-tarafa self-assigned this Apr 28, 2025
issues/TestLegacyErrors.sln Outdated Show resolved Hide resolved
issues/TestLegacyErrors/Program.cs Outdated Show resolved Hide resolved
issues/TestLegacyErrors/TestLegacyErrors.csproj Outdated Show resolved Hide resolved
issues/TestLegacyErrors/Program.cs Outdated Show resolved Hide resolved
Copy link
Contributor

@amanda-tarafa amanda-tarafa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but you need to rebase this PR on upstream/main into a single commit. The Diff Checker failures should go away then.

@purva9413 purva9413 force-pushed the legacy-errors-test-app branch from 6695079 to 2d96885 Compare May 23, 2025 23:26
@purva9413 purva9413 merged commit 7bf0bff into googleapis:main May 23, 2025
9 checks passed
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.

3 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.