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

DAOS-15918 ucx: Update UCX to 1.17 and build RPMs with logging. #11

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

jgmoore-or
Copy link
Contributor

No description provided.

@jgmoore-or jgmoore-or requested a review from a team as a code owner June 11, 2024 21:10
@@ -1,3 +1,9 @@
ucx (1.17.0-1) unstable; urgency=medium

* Update to 1.17
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's make sure we indicate that this is a pre-release of 1.17.

@@ -327,6 +327,9 @@ library internals, protocol objects, transports status, and more.
%endif

%changelog
* Tue Jun 11 2024 Joseph Moore <[email protected]> - 1.17.0-1
- Update to 1.17
Copy link
Contributor

Choose a reason for hiding this comment

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

Ditto on indicating that this is a pre-release.

Copy link
Contributor

Choose a reason for hiding this comment

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

I guess we are going to have to override line 36 of this file with:

Source: https://github.com/openucx/ucx/archive/770b5a6.tar.gz

ucx.spec Outdated
%global bugrelease 0
#%%global prerelease
#%%global prerelease a1
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
#%%global prerelease a1
%global prerelease a1

@brianjmurrell
Copy link
Contributor

We don't support centos7 any more. You will want to remove that from the Jenkinsfile.

Copy link
Contributor

Choose a reason for hiding this comment

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

You will need to change line 111 to:

%autosetup -p1 -n ucx-770b5a6700fc7fa920a90a4c8b44b8f655c3a301

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.

2 participants