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

doc: 12.1.1 & 12.1.2 release notes #16377

Merged
merged 3 commits into from Aug 2, 2017

Conversation

Projects
None yet
5 participants
@theanalyst
Member

theanalyst commented Jul 17, 2017

Adding the changes from 12.1.0, still needs a major changes section for highlighting the important feature additions between RCs

Ceph Luminous (v12.2.0) will be the foundation for the next long-term
stable release series. There have been major changes since Kraken
(v11.2.z) and Jewel (v10.2.z), and the upgrade process is non-trivial.
Please read these release notes carefully.
Notable Changes since v12.1.1 (RC1)
-----------------------------------

This comment has been minimized.

@liewegas

liewegas Jul 17, 2017

Member

I'd rather put this section down below and keep all the top-level luminous release info at the top, so that this is treated more like a draft for the final luminous release notes. That way people still see (at the top) the major changes, upgarde procedure, etc.

This comment has been minimized.

@theanalyst
* The 'rados df' JSON output now prints numeric values as numbers instead of
strings.
* There was a bug introduced in Jewel (#19119) that broke the mapping behavior

This comment has been minimized.

@theanalyst

theanalyst Jul 17, 2017

Member

@liewegas do we still need to mention this somewhere in the release notes, we had an entry when we released jewel 10.2.8, we could add the same caveat in Luminous if it is still relevant

This comment has been minimized.

@liewegas

liewegas Jul 18, 2017

Member

only needed for jewel; can leave it off here!

This comment has been minimized.

@theanalyst
* rgw: S3 lifecycle now supports expiration date (`pr#15807 <https://github.com/ceph/ceph/pull/15807>`_, Zhang Shaowen)
* rgw: silence compile warning from -Wmaybe-uninitialized (`pr#15996 <https://github.com/ceph/ceph/pull/15996>`_, Jiaying Ren)
* rgw: silence warning from -Wmaybe-uninitialized (`pr#15949 <https://github.com/ceph/ceph/pull/15949>`_, Jos Collin)
* rgw: Tested-by: Yuri Weinstein <yweinste@redhat.com> (`issue#20107 <http://tracker.ceph.com/issues/20107>`_, `pr#15912 <https://github.com/ceph/ceph/pull/15912>`_, Jens Rosenboom)

This comment has been minimized.

@yehudasa

yehudasa Jul 17, 2017

Member

@theanalyst this line can be edited to reflect issues fixed better

@yehudasa

@theanalyst minor non issue, otherwise lgtm

* tests: qa/tasks/radosbench: increase timeout (`pr#15885 <https://github.com/ceph/ceph/pull/15885>`_, Sage Weil)
* tests: qa/workunits/ceph-helpers: enable experimental features for osd (`pr#16319 <https://github.com/ceph/ceph/pull/16319>`_, Kefu Chai)
* tests: qa/workunits/ceph-helpers: test wait_for_health_ok differently (`pr#16317 <https://github.com/ceph/ceph/pull/16317>`_, Kefu Chai)
* tests: Reviewed By: Brad Hubbard <bhubbard@redhat.com> (`pr#15902 <https://github.com/ceph/ceph/pull/15902>`_, Jos Collin)

This comment has been minimized.

@tchaikov

tchaikov Jul 18, 2017

Contributor

this line could be improved

This comment has been minimized.

@theanalyst

theanalyst Jul 18, 2017

Member

yeah I'm fixin the script to drop these messages, it is because we have a match only for "Reviewed-By" and not other patterns :)

@theanalyst

This comment has been minimized.

Member

theanalyst commented Jul 18, 2017

@tchaikov @yehudasa updated to reflect all the lines which were showing the incorrect messages

@theanalyst theanalyst changed the title from wip: doc: 12.1.1 release notes to doc: 12.1.1 release notes Jul 18, 2017

@theanalyst

This comment has been minimized.

Member

theanalyst commented Jul 19, 2017

test this please

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Jul 25, 2017

needs rebase

@theanalyst theanalyst changed the title from doc: 12.1.1 release notes to doc: 12.1.1 & 12.1.2 release notes Jul 25, 2017

@theanalyst

This comment has been minimized.

Member

theanalyst commented Jul 25, 2017

@liewegas added 12.1.2 and cleaned up pending release notes again

@theanalyst

This comment has been minimized.

Member

theanalyst commented Aug 1, 2017

need to add a note if #15801 gets in

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Aug 2, 2017

@alfredodeza @ktdreyer we want to get this PR merged before releasing v12.1.2, don't we?

@theanalyst

This comment has been minimized.

Member

theanalyst commented Aug 2, 2017

@tchaikov needs some more updates since the release notes were luminous as of last week and we need to add the diff, I'll rebase and add those

@theanalyst

This comment has been minimized.

Member

theanalyst commented Aug 2, 2017

@jcsp

jcsp approved these changes Aug 2, 2017

@theanalyst

This comment has been minimized.

Member

theanalyst commented Aug 2, 2017

jenkins test this please

theanalyst added some commits Jul 17, 2017

doc: 12.1.1 release notes
Adding the changelog from 12.1.0 -> 12.1.1

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
doc: 12.1.1 release notes add major changes
Also cleanup PendingReleasenotes to an empty file so that only newer
changes are tracked, adding the relevant section back to
RC1 where relevant. Moving all the RC1 announcements back to RC2, when
we go to 12.2.0 we'll collapse all of these back to the release
announcments

Signed-off-by: Abhishek Lekshmanan <alekshmanan@suse.com>
doc: release notes for v12.1.2
Signed-off-by: Abhishek Lekshmanan <alekshmanan@suse.com>

@liewegas liewegas merged commit abe4099 into ceph:master Aug 2, 2017

1 of 4 checks passed

Unmodified Submodules Build triggered. sha1 is merged.
Details
make check Build triggered. sha1 is merged.
Details
make check (arm64) running make check
Details
Signed-off-by all commits in this PR are signed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment