Skip to content
This repository has been archived by the owner on May 30, 2022. It is now read-only.

fix CI issues #406

Merged
merged 4 commits into from
Nov 2, 2021
Merged

Conversation

stefanotorresi
Copy link
Member

@stefanotorresi stefanotorresi commented Oct 28, 2021

fixes #389 and #354
it also make the ansible linting and python checking in parallel with the go testing, which should speed up things considerably.

@stefanotorresi stefanotorresi added the DX Developer Experience label Oct 28, 2021
@stefanotorresi stefanotorresi linked an issue Oct 28, 2021 that may be closed by this pull request
Copy link
Member

@fabriziosestito fabriziosestito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

Copy link
Contributor

@arbulu89 arbulu89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@@ -0,0 +1,5 @@
ansible==4.6.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea!
Most probably, we could even have a prod requirements file, that is used in runner container creation.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, I thought about this! Probably in another PR though.

@stefanotorresi stefanotorresi merged commit 3f21fbc into trento-project:main Nov 2, 2021
@stefanotorresi stefanotorresi deleted the ci-issues branch November 2, 2021 09:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DX Developer Experience
Development

Successfully merging this pull request may close these issues.

Limit concurrency to 1 in the CI Restore manual triggering of the CI/CD
3 participants