org-formation / org-formation-cli

Better than landingzones!
MIT License
1.39k stars 129 forks source link

Command `print-tasks` does not generate yaml nor directory #464

Open mgamsjager opened 1 year ago

mgamsjager commented 1 year ago

Subject of the issue

Today I saw that on one of our customers CI that the print-tasks command no longer generates the ./.printed-stacks/ folder nor generates any data at all. The command exits successfully. The issue is reproducible locally on OSX and on Github with all current ubuntu images. Other customer with Gitlab and a very old nodejs version (10.x) does not have the same issue.

Org-formation versions are all the same.

5 days ago it did work (with https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230129.2) which lead me to believe it had to do with https://github.com/actions/runner-images/pull/7053 but using an older image also resulted in the same outcome.

Your environment

Steps to reproduce

$ org-formation print-tasks ./organization-tasks.yml $ ls -al ./.printed-stacks/ -> ls: ./.printed-stacks/: No such file or directory

Expected behaviour

Generate the print-stack (used by static code tool)

Actual behaviour

print-tasks exits successfully but no files are being generated.

mgamsjager commented 1 year ago

Kinda missed this warning:

WARN: Accounts were added to the organization model.
WARN: Tasks might depend on updating the organization.
WARN: Validation and printing of tasks will be skipped.

So this simplified the issue a lot. But is there a way to to run print-tasks when adding a new account?

OperationalFallacy commented 5 months ago

@mgamsjager did you ever figure it out? This would be very helpful to reviewing changes in generated CF templates.

OlafConijn commented 5 months ago

i think the "issue" is that print-tasks cannot print task if there were accounts added to your organization. after you have performed tasks (updated the organization + state) it will be able to print tasks.

not sure whether this is of importance to your usecase @OperationalFallacy