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

fix: Repository resource handles PermissionDenied status #200

Conversation

KasnocknDave
Copy link
Contributor

Description of your changes

Fixes #193

This code change handles the PermissionDenied error that gets returned by Argo CD when fetching a repository resource that doesn't exist yet. This is actually an issue within Argo CD, reported here: argoproj/argo-cd#20005.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Unit tests are adjusted and it has been tested by following the local testing steps via kind cluster.

@MisterMX MisterMX force-pushed the fix/repository-not-found-status branch from f6bc098 to 0acaa90 Compare October 1, 2024 15:23
Copy link
Collaborator

@MisterMX MisterMX left a comment

Choose a reason for hiding this comment

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

LGMT. Thank you very much @KasnocknDave!

@MisterMX MisterMX merged commit a224394 into crossplane-contrib:main Oct 1, 2024
7 of 8 checks passed
Copy link

github-actions bot commented Oct 1, 2024

Successfully created backport PR for release-0.8:

@MisterMX
Copy link
Collaborator

MisterMX commented Oct 1, 2024

/backport

Copy link

github-actions bot commented Oct 1, 2024

Backport failed for release-0.8, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git fetch origin release-0.8
git worktree add -d .worktree/backport-200-to-release-0.8 origin/release-0.8
cd .worktree/backport-200-to-release-0.8
git checkout -b backport-200-to-release-0.8
ancref=$(git merge-base 1568f5c6aff887376617da09e55305736370d3de 0acaa902cc8c6a0a6c8f7803d744ad98921046ae)
git cherry-pick -x $ancref..0acaa902cc8c6a0a6c8f7803d744ad98921046ae

Copy link

github-actions bot commented Oct 1, 2024

Successfully created backport PR #203 for release-0.9.

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.

Create repository returns PermissionDenied
2 participants