search

ekristen / azure-nuke

Remove all resources from an Azure Tenant and it's Subscriptions.
https://ekristen.github.io/azure-nuke/
MIT License
32 stars 8 forks source link

Is --resource-id/AZURE_RESOURCE_ID still needed? #42

Closed abjoseph closed 5 months ago

abjoseph commented 7 months ago

The code snippet in the README.md file makes use of the cli flag --resource-id but upon reviewing the code at commands.go, it appears that the resource-id flag is no longer valid.

Can you confirm?

ekristen commented 7 months ago

@abjoseph thanks for asking. It is not. I'm in the process of entirely rewriting this tool, a beta should be out by the weekend and I will have updated docs and the works to go along with it. Azure constantly is changing their API and the golang sdks, so the past has been a bit unstable, should be better now.

abjoseph commented 7 months ago

@ekristen I completely understand. I also collaborated with an MS engineer on the MSGraph sdk and know the pains of a constantly shifting api surface first-hand.

If you need a beta tester, do let me know. I appreciate you making this tool available and would like to assist if I can.

ekristen commented 7 months ago

@abjoseph absolutely!

I'll try and drop a note here but you can always set your watch status on the repo to be notified of new releases. I spent Christmas break and the better part of the last couple of weeks writing libnuke and rewriting aws-nuke which I'm fully separated from the original over at rebuy-de for reasons.

Azure Nuke is up next to use the new framework with new tooling to make it easier for folks to help add resources and do testing.

ekristen commented 7 months ago

@abjoseph please checkout of the rewrite of this tool under the next branch. I have PR open with all the details you should need to know. https://github.com/ekristen/azure-nuke/pull/45, the readme has been updated.

abjoseph commented 7 months ago

@abjoseph please checkout of the rewrite of this tool under the next branch. I have PR open with all the details you should need to know. #45, the readme has been updated.

@ekristen , thanks for follow-up and letting me know. I'll give the PR a read and try it out in my tenant. Thanks again.

ekristen commented 7 months ago

@abjoseph any luck in testing?

abjoseph commented 7 months ago

@abjoseph any luck in testing?

Hey @ekristen, not as yet but I'll definitely have some time this weekend coming to test it out with some AKS clusters I plan to provision for another effort.

ekristen commented 6 months ago

hey @abjoseph just circling back around to this and following up with you. I'm getting ready to drop a big update.