Skip to content

Conversation

@NetDevAutomate
Copy link

AWS Deployment Enhancements

Changes

  • Added macOS support with new upload-images-to-aws-macos.sh script
  • Enhanced documentation with detailed comments in main.tf
  • Updated .gitignore with comprehensive patterns
  • Added configuration template in config.yml
  • Improved error handling and documentation in deployment scripts

Testing Done

  • Verified script functionality on macOS
  • Tested configuration template
  • Validated Terraform configurations
  • Ensured no sensitive data is exposed

Related Issue

[If there's a related issue, reference it here]

Additional Notes

  • All sensitive information is properly handled through environment variables
  • Documentation has been updated to reflect the changes

- Introduced a new `.editorconfig` file with standardized settings for line endings, indentation, and file types.
- Expanded `.gitignore` to include more editor and system-specific files, ensuring cleaner git status outputs.
- Updated README to inform users of the `prepare.sh` prerequisite step.
- Streamlined Terraform configurations with enhanced documentation.
@rschmied
Copy link
Member

Hey @NetDevAutomate -- I am not 100% sure what I should do with this PR... is this "FYI" only and should I pick and choose what I think is useful for our general audience? there's also quite a bit "Andy specific" information in there which should be removed. Please advise :D

@rschmied
Copy link
Member

@NetDevAutomate -- poking this PR again. Not sure what to do with it. This at least needs

  • clean-up as there's quite a few things in here which seem to be very specific to your particular use case
  • re-basing on current main branch
  • maybe a bit more explanation and also prioritization of the things that have been added and what should be preserved/merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants