Skip to content

Conversation

@amaslenn
Copy link
Contributor

No description provided.

@jenkinsornl
Copy link

Build finished.

@swx-jenkins1
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ucx-pr/3166/ for details.

@mellanox-github
Copy link
Contributor

Test PASSed.
See http://hpc-master.lab.mtl.com:8080/job/hpc-ucx-pr/5267/ for details (Mellanox internal link).

@yosefe
Copy link
Contributor

yosefe commented Nov 28, 2017

@shamisp pls review as well

@shamisp
Copy link
Contributor

shamisp commented Nov 28, 2017

Overall I don't have a problem with the content. But I have feeling that we are not very consistent in the way we structure the new. @bbenton - can you take a look here.

Another question - once we merge it and update version to 1.2.2, where do we reflect the fact that it is 1.2.2-rc1, besides the tag ? Do we have to reflect the rc1 somewhere else ?

@yosefe
Copy link
Contributor

yosefe commented Nov 28, 2017

@shamisp @bbenton how should we reflect rc?

@shamisp shamisp added this to the v1.2.2 milestone Nov 28, 2017
@shamisp
Copy link
Contributor

shamisp commented Nov 29, 2017

👍

@shamisp shamisp merged commit fc625eb into openucx:v1.2.x Nov 29, 2017
@amaslenn amaslenn deleted the news-v1.2.2 branch November 30, 2017 07:40
@amaslenn amaslenn restored the news-v1.2.2 branch January 12, 2018 08:41
@amaslenn amaslenn deleted the news-v1.2.2 branch October 29, 2018 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants