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

🌱 Delete identity associated with tracker #529

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

mguetta1
Copy link
Contributor

@mguetta1 mguetta1 commented Oct 18, 2023

Fixing resources cleanup at the end of the test, by sending the identity's ID to Identity.Delete instead of the tracker's ID

@mguetta1 mguetta1 changed the title Delete identity associated with tracker 🌱 Delete identity associated with tracker Oct 18, 2023
@jortel jortel requested a review from mansam October 18, 2023 17:59
@jortel
Copy link
Contributor

jortel commented Oct 18, 2023

This cannot be reviewed without at least something in the description.

Copy link
Member

@aufi aufi left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@jortel jortel merged commit 5aa81a9 into konveyor:main Oct 19, 2023
12 of 13 checks passed
aufi pushed a commit to aufi/tackle2-hub that referenced this pull request Oct 30, 2023
Fixing resources cleanup at the end of the test, by sending the
identity's ID to `Identity.Delete` instead of the tracker's ID

Signed-off-by: Maayan Hadasi <[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.

3 participants