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

Ubuntu 20.04. CI/CD Pipeline as GitHub Action #1697

Merged

Conversation

udosson
Copy link
Contributor

@udosson udosson commented Sep 14, 2021

This PR solves #1678 and #1679 and provides the GHA workflow for the CI and the CD pipeline of Indy-Node.
Also, this PR contains an update of the ubuntu-20.04-upgrade feature branch with the most recent commits of master.
The PR contains:

  • CI workflow based on master branch and Indy-Plenum
  • CD workflow based on master branch and Indy-Plenum
  • Fixed linting issues with newer version of flake8 (flake8==3.8.4) and python 3.8
  • updated the version of Indy-SDK to python3-indy==1.15.0-dev-1625, the same version as used in Indy-Plenum which contains the freeze leger support.
  • adjusted runner.py to be compatible with the newer version of pytest (pytest>=6.2.2)
  • install_requires in setup.py is based on indy-plenum==1.13.0.dev135 which was built and published from the ubuntu-20.04-upgrade branch of Indy-Plenum in this GHA run
  • copied the uploading structure of the debian file to jfrog artifactory from master branch

A successful run of the workflow can be found at: https://github.com/udosson/indy-plenum/actions/runs/1186783441

Thanks to @WadeBarnes for all your support!

Toktar and others added 30 commits September 19, 2019 01:15
adds GitHub Actions and build containers
Signed-off-by: anton.denishchenko <[email protected]>
Signed-off-by: anton.denishchenko <[email protected]>
Signed-off-by: anton.denishchenko <[email protected]>
Signed-off-by: anton.denishchenko <[email protected]>
UP-34: Add LEDGERS_FREEZE and GET_FROZEN_LEDGERS transactions
WadeBarnes and others added 9 commits March 11, 2021 05:23
- The actions don't have access to publish test results to commits on a PR.
- This should not cause the job to fail.

Signed-off-by: Wade Barnes <[email protected]>
…ublishing

Allow test report publishing to fail
- Following the pattern and enhancements developed for the plenum workflows.
- Update Sovrin signing keys.
- Update test runner (`runner.py`) to automatically report the test results for each set of tests.
  - Enhancements from hyperledger/indy-plenum@06d9406
  - Add an option for suppressing the automated junitxml results.
    - Add `--nojunitxml` flag to allow junitxml results to be suppressed.
    - Suppress junitxml output when running the Jenkins Pipelines
      - Generation of junitxml output for the `indy_node/test/auth_rule` tests causes the tests to crash and fail when run via Jenkins.
    - The GHA workflows use the junitxml results for reporting, and are not affected by the same issue seen with Jenkins.
- Address feedback from @udosson

Signed-off-by: Wade Barnes <[email protected]>
added building and publishing of python artifacts
@sovbot
Copy link

sovbot commented Sep 14, 2021

Can one of the admins verify this patch?

@WadeBarnes
Copy link
Member

@udosson, Looking good. Last thing to do is to squash/rebase the CI/CD updates onto this commit:
image

So the merge from the main branch stays intact and all of the CI/CD updates get included as one commit on top of that commit.

@udosson udosson force-pushed the ubuntu-20.04-ci-cd-updates branch from 3c079c6 to 71c190b Compare September 14, 2021 14:47
@udosson
Copy link
Contributor Author

udosson commented Sep 14, 2021

@WadeBarnes, thanks for your review.
I squashed all commits from 6f2d3b6 on into one commit.
image

@udosson udosson force-pushed the ubuntu-20.04-ci-cd-updates branch 2 times, most recently from 96d399e to abc6a9f Compare September 14, 2021 15:02
@udosson udosson force-pushed the ubuntu-20.04-ci-cd-updates branch from abc6a9f to 0ba79fe Compare September 15, 2021 07:54
Copy link

@Echsecutor Echsecutor left a comment

Choose a reason for hiding this comment

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

\n @ EOF might be nice ;)

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

Successfully merging this pull request may close these issues.