Skip to content

Build and workflow updates and fixes #1904

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

Merged
merged 5 commits into from
Feb 7, 2025

Conversation

WadeBarnes
Copy link
Member

  • Fix Plenum version format in the indy-node deb package.
  • Enable workflows on the ubuntu-22.04 branch.
  • Set concurrency rules on PR and Push workflows.
    • Cancel in progress runs when stating a new one.
  • Ensure tests are running on latest version of Ubuntu, and with Python 3.10 (the default on Ubuntu 22.04).

Copy link
Member

@swcurran swcurran left a comment

Choose a reason for hiding this comment

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

Is the use of “latest” make bring in 24.04 possible? I’m assuming not, and the change was intentional — perhaps it being “latest” within a release (20.04, 22.04, etc.).

Approving to not holding things up.

@WadeBarnes
Copy link
Member Author

Is the use of “latest” make bring in 24.04 possible? I’m assuming not, and the change was intentional — perhaps it being “latest” within a release (20.04, 22.04, etc.).

I did that on purpose to see the results. I was initially going to specify 22.04.

@GuillaumeBourque-QC
Copy link

Is the use of “latest” make bring in 24.04 possible? I’m assuming not, and the change was intentional — perhaps it being “latest” within a release (20.04, 22.04, etc.).

Approving to not holding things up.

If we look here: https://github.com/actions/runner-images we see that ubuntu-latest is already using 24.04

image

@WadeBarnes
Copy link
Member Author

Is the use of “latest” make bring in 24.04 possible? I’m assuming not, and the change was intentional — perhaps it being “latest” within a release (20.04, 22.04, etc.).
Approving to not holding things up.

If we look here: https://github.com/actions/runner-images we see that ubuntu-latest is already using 24.04

I am aware. I looked this up before making the changes. I want to see the affect of using 24.04 as the runner.

Copy link
Member

@swcurran swcurran left a comment

Choose a reason for hiding this comment

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

LGTM — I’ll leave it to you to merge.

- Fix Plenum version format in the indy-node deb package.
- Enable workflows on the `ubuntu-22.04` branch.
- Set concurrency rules on PR and Push workflows.
  - Cancel in progress runs when stating a new one.
- Ensure tests are running on latest version of Ubuntu, and with Python 3.10 (the default on Ubuntu 22.04).

Signed-off-by: Wade Barnes <[email protected]>
@WadeBarnes
Copy link
Member Author

Pushed a fix for the version conversion. I had added some whitespace to one of the sed statements that was causing an error.

@WadeBarnes
Copy link
Member Author

Ok, the plenum reference is fixed. But a couple other packages can't be found. So we need to look at those next.

@GuillaumeBourque-QC
Copy link

Since in plenum the version is [1, 14, 0, "rc", 0] should we use the same version in indy-node ?

@WadeBarnes
Copy link
Member Author

Since in plenum the version is [1, 14, 0, "rc", 0] should we use the same version in indy-node ?

That will come in time when we trigger an rc release.

@WadeBarnes
Copy link
Member Author

WadeBarnes commented Feb 7, 2025

The two packages that can't be found, python3-timeout-decorator 0.5.0, and python3-distro 1.7.0, are built during the Build Packages / Build 3rd Party Dependencies stage of the build. Since PRs don't have the permissions to publish the artifacts, they don't get published and therefore can't be found when indy-node is being installed by the test runners. I'll need to publish those manually, to seed the package repository.

@GuillaumeBourque-QC
Copy link

GuillaumeBourque-QC commented Feb 7, 2025

But these should be from the OS ? Because we do a sed here to get the OS version

if we search we get these:

apt search python3-timeout-decorator
...

python3-timeout-decorator/jammy 0.5.0-1 all

The same is true for the second one.

python3-distro/jammy,now 1.7.0-1 all [installed,automatic]

Are we just missing an apt update before the package install ? Or do we want to provide a copy of those pakages ?

@GuillaumeBourque-QC
Copy link

GuillaumeBourque-QC commented Feb 7, 2025

if we specify this version apt install python3-timeout-decorator=0.5.0-1 it will work and the same is true for the other one.

So maybe something in the regex is cutting the last part of the debian package name

@WadeBarnes
Copy link
Member Author

But these should be from the OS ?

if we search we get these:

apt search python3-timeout-decorator
...

python3-timeout-decorator/jammy 0.5.0-1 all

The same is true for the second one.

python3-distro/jammy,now 1.7.0-1 all [installed,automatic]

Are we just missing an apt update before the package install ? Or do we want to provide a copy of those pakages ?

The fpm package build in the build-indy_node.sh build script extracts the dependencies (by default) from the indy-node python dependencies. The build-3rd-parties.sh script builds the packages to match those dependencies. The prepare-package.sh script, does some alignment between available package(s) from the OS and the listed indy-node dependencies.

So we might just be able to adapt the dependencies in the prepare-package.sh script.

The indy-plenum prepare-package.sh script provides some better examples of how this is done.

I'll try fixing the dependencies this way.

@swcurran swcurran merged commit 311689d into hyperledger:ubuntu-22.04 Feb 7, 2025
28 of 39 checks passed
@swcurran
Copy link
Member

swcurran commented Feb 7, 2025

Merged with failing tests because this will not be the last PR — allows others to work on the latest changes to fix more things.

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.

3 participants