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

NEWS: 1.13.0 update for RC2 #8327

Merged
merged 1 commit into from
Jun 27, 2022
Merged

Conversation

evgeny-leksikov
Copy link
Contributor

What

Update NEWS file for RC2

Why ?

Full list of changes since RC1:

  • ff64ffa AZP: remove HWI hosts from testing
  • 7e2df91 UCT/CUDA: use right driver macro to log errors with drv fxns
  • 28b0f30 UCT/IB/DC: Always schedule DCI allocation during FC_HARD_REQ progress
  • f92b172 TEST/UCT: Remove atomic access from invalidate tests
  • 5cec390 UCT/IB: Fix indirect key registration by using correct MR type
  • a6c5a47 UCT/IB/DC: Fix managing FC_HARD_REQ resend time

@yosefe
Copy link
Contributor

yosefe commented Jun 21, 2022

  1. Need to update date to June 26 :)
  2. pls add "Fixes in ROCm memory detection and performance estimation" @edgargabriel

@evgeny-leksikov
Copy link
Contributor Author

  1. Need to update date to June 26 :)
  2. pls add "Fixes in ROCm memory detection and performance estimation" @edgargabriel

done

NEWS Outdated
@@ -11,7 +11,7 @@
### Features:
### Bugfixes:

## 1.13.0 (May 27, 2022)
## 1.13.0 (June 26, 2022)
Copy link
Contributor

Choose a reason for hiding this comment

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

it should be 1.13.0-rc2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

1.13.0-rc2

should we then also update extra version in configure.ac?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes, let's try this, it would be first time we use it in a release.

Copy link
Contributor

Choose a reason for hiding this comment

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

can we change NEWS only to v1.13.0-rc2 ?
release date - July 24

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hmm... then I'm not sure I understand relelase process completely. I don't see 1.12.X-rcY sections in NEWS. If we want to track changes for rc1, rc2 and so on, then I would expect to see 1.13.0-rc1, 1.13.0-rc2, ... and then 1.13.0 stable release.
If we squash all release candidates into single section when dropping stable release it requires more efforst because rc2 change log may be not relevant in the end since it's related only for new functionality

Copy link
Contributor

Choose a reason for hiding this comment

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

we don't have it in the NEWS now, because it was replaced it to v1.12.0 (for example) before the GA

Copy link
Contributor Author

Choose a reason for hiding this comment

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

then why need to track it before GA if it will be replaced in the end? IMHO it's simpler to merge it every RC. As you can see relevant change log is much shorter.

Copy link
Contributor

Choose a reason for hiding this comment

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

this is how it was done so far, so the diff reflects the changes vs. the previous GA.
@shamisp WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

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

We do this to track in case somebody picks up one of the RC instead of releases - this way it is actually reflected in news.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

NEWS Outdated
@@ -11,7 +11,7 @@
### Features:
### Bugfixes:

## 1.13.0 (May 27, 2022)
## 1.13.0.rc2 (June 26, 2022)
Copy link
Contributor

Choose a reason for hiding this comment

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

it should be "1.13.0-rc2"

configure.ac Outdated
define([ucx_ver_major], 1) # Major version. Usually does not change.
define([ucx_ver_minor], 13) # Minor version. Increased for each release.
define([ucx_ver_patch], 0) # Patch version. Increased for a bugfix release.
define([ucx_ver_extra], .rc2) # Extra version string. Empty for a general release.
Copy link
Contributor

Choose a reason for hiding this comment

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

-rc2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It does not work for RPM build on some systems:

The version Tag
The version tag defines the version of the software being packaged. The version specified should be as close as possible to the format of the original software's version. In most cases, there should be no problem specifying the version just as the software's original developer did. However, there is a restriction. There can be no dashes in the version. If you forget, RPM will remind you:

# rpmbuild -ba cdplayer-1.0.spec

* Package: cdplayer
Illegal '-' char in version: 1.0-a

http://rpm5.org/docs/max-rpm.html

this is why we decided to use . for int4

Copy link
Contributor

Choose a reason for hiding this comment

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

ok

Copy link
Contributor

Choose a reason for hiding this comment

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

@shamisp FYI

Copy link
Contributor

Choose a reason for hiding this comment

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

What about deb and other package managers ?

yosefe
yosefe previously approved these changes Jun 22, 2022
@yosefe
Copy link
Contributor

yosefe commented Jun 22, 2022

@shamisp @edgargabriel pls approve as well

edgargabriel
edgargabriel previously approved these changes Jun 22, 2022
NEWS Outdated
@@ -11,6 +11,15 @@
### Features:
### Bugfixes:

## 1.13.0-rc2 (July 24)
Copy link
Contributor

Choose a reason for hiding this comment

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

since it's 1.13 release need to update the 1.13.0 entry and set date to today

Copy link
Contributor Author

Choose a reason for hiding this comment

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

then maybe update 1.13.0 entry to 1.13.0-rc1 and add new one 1.13.0-rc2 (today)?

@yosefe yosefe merged commit dc07e04 into openucx:v1.13.x Jun 27, 2022
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.

4 participants