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

[WIP] Update pr_check.sh #733

Closed
wants to merge 1 commit into from
Closed

Conversation

jameswnl
Copy link

No description provided.

@awcrosby
Copy link
Contributor

We plan to soon move our CI bits to GitHub Actions, not sure if that can be added here instead: https://github.com/ansible/galaxy_ng/tree/master/.github/workflows

@jameswnl
Copy link
Author

We plan to soon move our CI bits to GitHub Actions, not sure if that can be added here instead: https://github.com/ansible/galaxy_ng/tree/master/.github/workflows

I assume Hub's current CI bits are centered around python unit test framework?

So what pr_check job can do that is not available (or difficult to do) in GitHub Actions workflow is to validate that the PR branch will build and is deployable to OSD4

  • the deployment is in an environment that is very similar to that of the production
  • have its own UI deployment
  • execute IQE smoke test suites
    This is especially useful and important for the c.r.c's app-interface CI/CD toolchain.

You can have the regular GitHub Actions doing its unit test coverage (and e.g. linting etc) and pr_check job focusing on the CI/CD aspect. (FYI, Analytics is doing just like that)

@awcrosby
Copy link
Contributor

You can have the regular GitHub Actions doing its unit test coverage (and e.g. linting etc) and pr_check job focusing on the CI/CD aspect. (FYI, Analytics is doing just like that)

Sorry for the late reply. Yes that sounds good, we can only put things in GH Actions where it makes sense.

@jameswnl
Copy link
Author

Replaced by #767

@jameswnl jameswnl closed this May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants