Skip to content

Adding the test.yaml so that vendor pull works on quick start#392

Closed
thedarkwriter wants to merge 3 commits intocloudposse:mainfrom
thedarkwriter:add-required-temp-file-for-quickstart
Closed

Adding the test.yaml so that vendor pull works on quick start#392
thedarkwriter wants to merge 3 commits intocloudposse:mainfrom
thedarkwriter:add-required-temp-file-for-quickstart

Conversation

@thedarkwriter
Copy link

@thedarkwriter thedarkwriter commented Jun 21, 2023

what

  • I added a short command to add a temp.yaml file so that the quick-start guide will work. Otherwise when you are creating components and run atmos vendor pull --component infra/vpc-flow-logs-bucket the command will fail.

why

  • Getting started docs should work (even with a workaround)
  • Once the command is run, you can keep going in the quick-start.

references

  • All ref was in Sweetops slack Atmos channel

@thedarkwriter thedarkwriter requested review from a team as code owners June 21, 2023 14:57
@Nuru Nuru added the no-release Do not create a new release (wait for additional code changes) label Jun 22, 2023
@Gowiem
Copy link
Member

Gowiem commented Feb 23, 2024

@aknysh @osterman does this still being open mean that the quick-start is broken? Or was this fixed elsewhere?

@osterman
Copy link
Member

Thanks @thedarkwriter - we ended up fixing in #738 the expectation that stack configs are present. Vendoring now works without any files defined in the stack folder.

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

Labels

no-release Do not create a new release (wait for additional code changes)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants