Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

Baseline filter generation #507

Closed
wants to merge 6 commits into from
Closed

Conversation

dooreelko
Copy link

Pretty much implementing #292

A new command "baseline" which will print yaml-ready resource filters (based on names or properties).

@svenwltr
Copy link
Member

Hello @dooreelko,

thank you for your contribution and sorry for the late response. I have two thoughts on this:

  1. There is no documentation, so we should add something to the README.
  2. Currently, it simply loops over all properties of an resource and adds each one to the config. This can render the output of some resources useless, because those filters are not grouped. An example are S3 objects, where it would basically filter all objects that are not versioned:
S3Object:
- property: "Key" 
  value: "my-s3-file"
- property: "VersionID" 
  value: "null"
- property: "IsLatest" 
  value: "true"
- property: "Bucket" 
  value: "some-bucket"

That does not sound like the desired behavior to me. What do you think?

@dooreelko
Copy link
Author

Hi @svenwltr,

I've added docs on the command, please have a look.
As for the second point, i'm not sure i understand completely. From the baseline perspective, we can only offer providing individual filters for each discovered resources. And based on my undersaanding of how the nuke processes filters, first filter wins, so i don't think there's a way to implement custom filter grouping by autogeneration.

Base automatically changed from master to main February 5, 2021 15:55
@svenwltr
Copy link
Member

Hello. Sorry for the very late answer.

And based on my undersaanding of how the nuke processes filters, first filter wins, so i don't think there's a way to implement custom filter grouping by autogeneration.

I guess that is the reason we did not implement it yet. This solution feels a bit non-intuitive to me, because the result looks random to me. Does @rebuy-de/prp-aws-nuke have an opinion on this?

@bjoernhaeuser
Copy link
Member

I think this command would really help users. Not sure how to remove the randomness though.

@ekristen
Copy link
Contributor

ekristen commented Oct 2, 2024

@dooreelko I love the idea, but since the fork has diverged so much I cannot pull this directly over. I've opened an issue to track this and will look more into this. There are other requests for config validation which might play nicely into this as well. ekristen/aws-nuke#361


Please see the copy of the notice from the README about the deprecation of this project. Sven was kind enough to grant me access to help triage and close issues and pull requests that have already been addressed in the actively maintained fork. Some additional information is located in the welcome issue for more information.

Caution

This repository for aws-nuke is no longer being actively maintained. We recommend users to switch to the actively maintained fork of this project at ekristen/aws-nuke.
We appreciate all the support and contributions we've received throughout the life of this project. We believe that the fork will continue to provide the functionality and support that you have come to expect from aws-nuke.
Please note that this deprecation means we will not be addressing issues, accepting pull requests, or making future releases from this repository.
Thank you for your understanding and support.

@ekristen ekristen closed this Oct 2, 2024
@dooreelko
Copy link
Author

Dude, it's been four years. I don't think I have enough motivation to pick this up :)

@ekristen
Copy link
Contributor

ekristen commented Oct 2, 2024

Totally understand. I had no control over this until recently. I am looking at a feature like this for the fork. If you want to contribute there, please do, if not I totally understand. Cheers.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants