This doesn't do anything as a default is provided.
Maybe have this be an alloy variable which is documented in the README as well? See the comment in the README, if we move this config to the group vars then the this variable can be kept as is.
Thanks for implementing the changes, some small comments.
Since alloy_config_default has a default configuration set, it should be listed under Optional Arguments. I guess another option would be to truly have this as a required configuration and then…
The tag is still here, we already have the alloy tag in the playbook, so it can just be removed.
Nice improvement! I think this won't cause too much additional effort really and makes stuff just safer.
This can be moved into the next step or is there a reason to have this in a separate step?