Skip to content
This repository was archived by the owner on Dec 20, 2025. It is now read-only.

fix(serviceAccounts): Delete managed service account when pipeline is deleted (bp #769) - #803

Merged
ethanfrogers merged 1 commit into
release-1.20.xfrom
mergify/bp/release-1.20.x/pr-769
May 8, 2020
Merged

ethanfrogers merged 1 commit into
release-1.20.xfrom
mergify/bp/release-1.20.x/pr-769

Conversation

@mergify

@mergify mergify Bot commented May 6, 2020

Copy link
Copy Markdown
Contributor

This is an automated backport of pull request #769 done by Mergify

Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the simulator.

Finally, you can contact us on https://mergify.io/

… deleted (#769)

Sometime ago, we noticed the sync operation of Fiat was very slow (it was
taking between 2 and 3 minutes) and in some cases, the Fiat instance went out
of memory. After investigations, we found out that we had about 2000 managed
service accounts not associated to any pipelines, deleting them solved our
issue with Fiat.

This change deletes the managed service account liked to the pipeline currently
being deleted.

Co-Authored-By: Greg Jones <greg.jones@gmail.com>
Co-Authored-By: Mark Vulfson <markvu@live.com>

Co-authored-by: Adrien Gentil <adrien.gentil@adevinta.com>
Co-authored-by: Greg Jones <greg.jones@gmail.com>
Co-authored-by: Mark Vulfson <markvu@live.com>
(cherry picked from commit c721dfa)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants