Comments (5)
I would suggest the logic should be:
use explicit VPC is a variable is set
use default VPC if it exists
create a vpc if neither
from deployment-automation.
We now support default vpc and provided vpc. I am leery of adding vpc creation as I feel that oversteps our boundary. Would appreciate thoughts.
from deployment-automation.
The issue that prompted me to create this ticket was that I had no default VPC. Maybe I deleted it for some reason, but I think it didn't exist by default in my sandbox account. For people in this same situation they would be unable to use the default settings of this project.
If we don't want to create a VPC for whatever reason without the user being explicit about it, then another approach could be to require the user to provide VPC details?
from deployment-automation.
Our current default action is to use the default VPC. If the user provides a VPC ID we build with that. I don't see a real need to force the user to create a VPC if they have a default one (which people do use... although they shouldn't) and haven't changed the necessary variables to add their own VPC into the module?
from deployment-automation.
Summarizing (lmk if I'm incorrect):
- The original issue is specific to the default VPC: if the default VPC does not exist and you try to use it with Ansible, this will fail
- There's an idea to create a default VPC if it does not exist -- I'd rather we fail hard if a person is trying to use this with a VPC and they deliberately deleted the default VPC behind the scenes
- A potential last idea is to require the user to provide VPC details, but IMO this would break any flows where people don't want to use a VPC at all
- It sounds like our current approach is the best compromise: use explicit VPC is a variable is set, use default VPC if it exists and variable is set, fail if variable is set and the VPC being used does not exist
closing this as done given ^^^ but please lmk if you disagree
from deployment-automation.
Related Issues (20)
- Terraform should tag resources HOT 2
- Ansible fails when enabling TLS due to safe restart
- excessive polling for geerlinguy.node_exporter can trigger Github ratelimits HOT 6
- Debug task referencing package_result.results appears to break ansible runs permanently.
- Ansible-lint: provision-node.yml cleanup HOT 1
- ansible: avoid end-user breakage for any roles changes HOT 1
- Stranded Compute in CI builds HOT 5
- Establish restart fails in some conditions because package_result isn't populated right HOT 4
- Set the Redpanda license HOT 1
- Always reconcile redpanda.yaml as it have node configuration HOT 1
- Validate/test if cluster configuration property of type string can be un-set HOT 3
- node_exporter only needed by grafana/prometheus, so move to grafana install playbook HOT 1
- Terraform for AWS should have a security group rule to allow intra-sg communication HOT 5
- Ansible setup for Redpanda-console HOT 3
- RP tuner fails to complete on RHEL HOT 1
- Test Server Side Schema Validation
- Use module and collection bundles copied onto client from cloudsmith
- Terraform outside of AWS us-west-2 improperly attempts to create AZ still inside us-west-2 HOT 4
- Add test on AZ & on instance types
- GCP Module Work HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from deployment-automation.