Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

105 Add "module_acceptance_custom.yml" which allows for more flexible use cases. #107

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sooyean-hoo
Copy link

@sooyean-hoo sooyean-hoo commented Sep 3, 2024

Summary

Addressing problem or issue in #105 .
For .github/workflows/module_acceptance.yml, can we have a version which does not hardcoded to all the Litmus Tasks? This makes it harder to update some of old modules, whose dependencies/implementation/styles vary a lot.

Additional Context

Add any additional context about the problem here.

  • Make the task more flexible, given options for use to use tasks other than the Litmus's, while keeping the current breakdown of steps. Hence users can migrate off old workflow and into these new ones more easily.

Related Issues (if any)

#105
#103 - This included as also for continuity, as it is also on the same theme and for the same approach.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.

@sooyean-hoo
Copy link
Author

sooyean-hoo commented Sep 3, 2024

As for validity, it is tested in the field with migration of module https://github.com/puppetlabs/puppetlabs-servicenow_cmdb_integration , with success.
#105 (comment)

@sooyean-hoo
Copy link
Author

@sooyean-hoo sooyean-hoo force-pushed the 105-for-githubworkflowsmodule_acceptanceyml-can-we-have-a-version-which-does-not-hardcoded-to-all-the-litmus-tasks-this-makes-it-harder-to-update-some-of-old-modules-whose-dependencies-vary-a-lot branch from 69d673b to e982609 Compare September 4, 2024 04:46
@sooyean-hoo
Copy link
Author

FYI, I have my own tests running against my own forked version. https://github.com/sooyean-hoo/cat-github-actions/blob/dev_sooyean/.github/workflows/module_acceptance_custom.yml

This is where all the results above come from .

@sooyean-hoo sooyean-hoo force-pushed the 105-for-githubworkflowsmodule_acceptanceyml-can-we-have-a-version-which-does-not-hardcoded-to-all-the-litmus-tasks-this-makes-it-harder-to-update-some-of-old-modules-whose-dependencies-vary-a-lot branch from e982609 to 566e31e Compare September 4, 2024 05:12
generated by matrix_from_metadata_v2
Add new custom workflow

Add in a new input:  matrix_from_metadata default to
matrix_from_metadata_v2
Clean up some spaces in *custom.yaml

Signed-off-by: Hoo Sooyean (何書淵) <[email protected]>
Fix a mistake of the task name
@sooyean-hoo sooyean-hoo force-pushed the 105-for-githubworkflowsmodule_acceptanceyml-can-we-have-a-version-which-does-not-hardcoded-to-all-the-litmus-tasks-this-makes-it-harder-to-update-some-of-old-modules-whose-dependencies-vary-a-lot branch from 566e31e to 26d60a2 Compare September 5, 2024 04:12
@sooyean-hoo sooyean-hoo changed the title 105 for githubworkflowsmodule acceptanceyml can we have a version which does not hardcoded to all the litmus tasks this makes it harder to update some of old modules whose dependencies vary a lot 105 for "module acceptance.yml" can we have a version which does not hardcoded to all the litmus tasks this makes it harder to update some of old modules whose dependencies vary a lot Sep 6, 2024
@sooyean-hoo sooyean-hoo changed the title 105 for "module acceptance.yml" can we have a version which does not hardcoded to all the litmus tasks this makes it harder to update some of old modules whose dependencies vary a lot 105 for "module_acceptance.yml" can we have a version which does not hardcoded to all the litmus tasks this makes it harder to update some of old modules whose dependencies vary a lot Sep 6, 2024
@sooyean-hoo sooyean-hoo changed the title 105 for "module_acceptance.yml" can we have a version which does not hardcoded to all the litmus tasks this makes it harder to update some of old modules whose dependencies vary a lot 105 Add "module_acceptance.yml" which allows for more flexible use cases. Sep 11, 2024
@sooyean-hoo sooyean-hoo changed the title 105 Add "module_acceptance.yml" which allows for more flexible use cases. 105 Add "module_acceptance_custom.yml" which allows for more flexible use cases. Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
1 participant