Skip to content
This repository was archived by the owner on Aug 16, 2022. It is now read-only.

feat: FSx file-systems, aliases, DRAs#1273

Open
rocketbitz wants to merge 3 commits into
cloudquery:mainfrom
rocketbitz:feature/fsx-file-systems
Open

feat: FSx file-systems, aliases, DRAs#1273
rocketbitz wants to merge 3 commits into
cloudquery:mainfrom
rocketbitz:feature/fsx-file-systems

Conversation

@rocketbitz

@rocketbitz rocketbitz commented Jul 22, 2022

Copy link
Copy Markdown

Summary

cloudquery/cloudquery#1375
cloudquery/cloudquery#1376
cloudquery/cloudquery#1378


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests. Learn more about testing here 🧪
  • Update the docs by running go run ./docs/docs.go and committing the changes 📃
  • If adding a new resource, add relevant Terraform files in a separate PR 📂
  • Ensure the status checks below are successful ✅

@rocketbitz rocketbitz requested review from a team and zagronitay and removed request for a team July 22, 2022 19:24
@rocketbitz rocketbitz force-pushed the feature/fsx-file-systems branch from 0d00c6e to 739bb9b Compare July 22, 2022 19:28
@rocketbitz rocketbitz changed the title [WIP] FSx file-systems, aliases, DRAs FSx file-systems, aliases, DRAs Jul 22, 2022
@rocketbitz rocketbitz changed the title FSx file-systems, aliases, DRAs feat: FSx file-systems, aliases, DRAs Jul 23, 2022
@yevgenypats yevgenypats requested review from irmatov and roneli and removed request for zagronitay July 25, 2022 06:53
@zagronitay

Copy link
Copy Markdown
Contributor

Hi @rocketbitz .

Thank you for submitting this PR.
Looks great overall.

Few comments:

  1. Docs are missing.
  2. Mock tests are missing.
  3. FSX resource is not mapped in the provider.go
  4. Integration tests missing local + prod folders.
  5. resolveFsxFileSystemTags seems to be not implemented.

You may find explanation to the missing steps on https://github.com/cloudquery/cq-provider-aws/blob/main/docs/contributing/adding_a_new_resource.md#setting-up-the-resource

Feel free to comment on here or on our discord chat https://discord.gg/bAdBG5WU

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.

2 participants