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

Create Github Workflow for Static & Spec Tests #958

Closed
sanfrancrisko opened this issue Mar 10, 2021 · 5 comments · Fixed by #1030
Closed

Create Github Workflow for Static & Spec Tests #958

sanfrancrisko opened this issue Mar 10, 2021 · 5 comments · Fixed by #1030
Assignees
Labels

Comments

@sanfrancrisko
Copy link
Contributor

sanfrancrisko commented Mar 10, 2021

Create a Github Workflow for the PR Tests that will cover:

On a hosted runner:

  • For each Ruby version (2.5, 2.7):
    • rubocop
    • spec:coverage
    • license_finder
    • test_pdk_as_library
    • spec
@sanfrancrisko sanfrancrisko self-assigned this Mar 10, 2021
@jpogran
Copy link
Contributor

jpogran commented Mar 10, 2021

We should target windows-latest and ubuntu-latest in hosted as a early warning system.

For acceptance testing, there is little difference (for our purposes) between 2016 and 2019, so we can limit to just that version and simplify things. For bonus points we could add a recent Windows 10 to test a desktop scenario, but I'm not sure what GCP images we have available.

I'm less certain about linux, I will check analytics but I think our most popular there is RHEL. I think ubuntu 20.04 should be good for the moment, and we should add as we get more info.

@sanfrancrisko
Copy link
Contributor Author

Sounds good. We'll go for parity with what we have at the moment, by the sounds of it.

I'll also update the description to plug in to Honeycomb with metrics.

Regarding Windows 10 (or any desktop Windows flavour), it seems to be missing from GCP's list of available images. The provision_service will be gaining support for AWS and Azure at some stage in the future, so I suspect the path of least resistance is to wait until that implementation and then add it in for ourselves.

@jpogran jpogran changed the title Move PR tests from TravisCI & AppVeyor to Github Workflow(s) w/ GCP Create Github Workflow(s) based on existing TravisCI & AppVeyor Mar 15, 2021
@jpogran
Copy link
Contributor

jpogran commented Mar 15, 2021

Updated ticket description with target implementation

@sanfrancrisko
Copy link
Contributor Author

An additional piece of work spurred from this was to refactor the validator in the provision_service to allow the pdk repo to provision resources in GCP.

See: https://github.com/puppetlabs/provision_service/pull/280

@sanfrancrisko sanfrancrisko changed the title Create Github Workflow(s) based on existing TravisCI & AppVeyor Create Github Workflow for Static & Spec Tests Mar 24, 2021
@sanfrancrisko
Copy link
Contributor Author

Given the complexities discovered during implementation for the Acceptance Test workflows, I'll raise separate tickets to cover:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants