-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Request AWS::EMR::Cluster resource to be added to cloudformation #7381
Comments
Hi @sli10, welcome to Moto! I'll mark it as an enhancement to add this feature. |
Hi @sli10, an initial implementation is now part of Are you able to try it out, and see whether this solves your issue? Not all properties for the CloudFormation-template are processed at the moment, so please let us know if there are any specific properties that you use that are missing. |
Hi @bblommers, we were able to try this out and it resolved our issue. On top of that we were encountering an issue with the stack deletion. We were wondering if Cloudformation implementation supports in order deletion of resources? |
Happy to hear that @sli10! I don't think we do anything special with stack deletion - we just delete all all resources in a random order, and re-try to delete them in case of failure, up to 5 times, to try and resolve any dependency problems. Can you share a bit more about the issues you're running into, ideally with an example template? |
Hi @bblommers , we are running into an issue with delete stack where the ordering of how the resources are deleted is causing an issue. When the cloudformation delete it's resources it gets deleted in reverse order it is created, so in our case the random order that moto deletes in gives us an error of Example of the resources we are using: |
With that particular example I can't quite reproduce it @sli10 - but I imagine that the actual setup is quite more complex. Having said that, I was able to improve the logic a bit around stack deletions in #7413. Are you able to test whether If that doesn't work, I would probably need a reproducible example to really understand what's going wrong. |
Hi @bblommers, the issue with delete stack is not producible with latest version any more, thank you for your timely response. We have a few more problems, one relating to Example of cf template: Another issue is related to the emr cluster resource mentioned above, when we create a resource using the latest docker image we get One more request we have regarding the |
Just to double check - do you mean the latest version (5.0.2), or I'll have a look at the other issues later this week. |
We are using |
Update on delete stack issue, sorry for going back and forth. Here is a reproducible example @bblommers , that's causing an issue when trying to delete stack. We were able to reproduce the ` Resources: |
Thank you for the example @sli10, that was very useful. I hadn't considered the scenario where a IAM role could be created/controlled outside of CloudFormation. Creating an The latest release (as of this morning) contains:
Yes - I'll try to get to that later this week. |
@bblommers Thank you for fixing the above. The latest release has resolved the issues you've listed above and I've also tried it out and it was working on our end. We realized a few more things that we want to request support for. Thank you so much for your help, we really appreciate it! |
Didn't mean to close this, so I'll reopen. But |
Thanks @bblommers for your help! Question about latest docker image, we are getting this error that didn't occur before pulling latest moto docker image. |
Happy to help @sli10! I can't reproduce that ValidationError though, I'm afraid. I just tried the following template, and that is created just fine (with the correct outputs) :
|
@bblommers Does moto cloudformation support conditions? For the role use case we have a condition check, if the parameter inputted is not empty then continue with creating the resource. |
@sli10 Yes, we do - we have some tests for that here:
|
Hi @bblommers, we have conditional checks in the outputs section. Here is an example that reproduces the error with Template Sample:
|
Hi @sli10, can you pull the |
I'll assume this is fixed - but feel free to open a new issue if you run into any (other) problems. |
Request for AWS::EMR::Cluster to be added to cloudformation. This is used for our testing and we would like to request this resource to be added.
The text was updated successfully, but these errors were encountered: