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

doc: 12.2.0 major release announcements #16915

Merged
merged 2 commits into from Aug 29, 2017

Conversation

theanalyst
Copy link
Member

Reordered the RC releases sections back to their respective components,
added a ceph-mon section, added links to documentation wherever
possible, and a few forgotten RGW announcements.

Signed-off-by: Abhishek Lekshmanan abhishek@suse.com

@theanalyst
Copy link
Member Author

can everyone please review their respective sections (not only a part of the diff) to see that we've missed nothing out

will form the basis for API-based management of Ceph going forward.
- *ceph-mgr* also includes a `Prometheus exporter <../mgr/prometheus>`_
plugin, which can provide Ceph perfcounters to Prometheus.

Copy link
Member

Choose a reason for hiding this comment

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

This section is for high-level highlights and should stay where it is.

* The 'mon_warn_osd_usage_min_max_delta' config option has been
removed and the associated health warning has been disabled because
it does not address clusters undergoing recovery or CRUSH rules that do
not target all devices in the cluster.
Copy link
Member

Choose a reason for hiding this comment

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

This stuff shoudl go in the compatibility notes section below

containing high-level information of the Ceph cluster. This makes
it easy to monitor a Ceph cluster's status and send out
notifications in case of a malfunction.

Copy link
Member

Choose a reason for hiding this comment

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

this should go back above where it was

Copy link
Member Author

Choose a reason for hiding this comment

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

ack

@branch-predictor
Copy link
Contributor

Missing stuff form PendingReleaseNotes?

@theanalyst
Copy link
Member Author

@branch-predictor updated, thanks!

@theanalyst
Copy link
Member Author

Rendered version of docs at http://docs.ceph.com/docs/doc-12.2.0-major-changes/release-notes/

@@ -46,17 +42,24 @@ Major Changes from Kraken
allowing them to be used with RBD and CephFS.

* *ceph-mgr*:

Copy link
Contributor

Choose a reason for hiding this comment

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

do not delete this blank line. reStructedText needs it.

Copy link
Member Author

Choose a reason for hiding this comment

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

thanks

Copy link
Contributor

Choose a reason for hiding this comment

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

😄

``ceph osd pool stats``. Although these commands do not
self-update like ``ceph -w`` did, they do have the ability to
return formatted output by providing a ``--format=<format>``
option.
Copy link
Contributor

Choose a reason for hiding this comment

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

@theanalyst Please drop this entire "New ceph -w behavior" bullet point. The old behavior has been reinstated and "ceph pg stat" is giving me EINVAL.

Copy link
Member Author

Choose a reason for hiding this comment

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

ok

Copy link
Contributor

@smithfarm smithfarm Aug 14, 2017

Choose a reason for hiding this comment

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

So, I still don't quite know what is going on, but the ceph -w behavior in luminous is definitely different and, I think, deserves a release note. The current one isn't quite right, though, because - AFAICT - in luminous it is still possible to get real-time I/O information from ceph -w. So, let's change it to this:

+    - New "ceph -w" behavior - the "ceph -w" output no longer updates
+      available space, pg info, etc. in real time because these are no
+      longer logged to the central log (which is what "-w"
+      shows). Some of the information can be obtained by running ``ceph pg
+      stat``. Also, if the I/O information provided by "-w" is not fine-grained
+      enough, I/O rates per pool can be determined using
+      ``ceph osd pool stats``.

@jcsp What do you think? The current ceph -w behavior is a little confusing - it sits there, apparently listening to the central log, but in a quiescent cluster nothing appears. So far I tried creating and deleting rbd images, which I would think would create some I/O, but still nothing appeared. I have anecdotal evidence that ceph -w does produce self-updating I/O information during performance tests.

Copy link
Contributor

Choose a reason for hiding this comment

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

@theanalyst Now I know what's going on. In Luminous, ceph -w just displays the same info as ceph -s and then enters an infinite wait loop. It doesn't update anything. I opened http://tracker.ceph.com/issues/20995 to track that bug.

So, please ignore everything I have said up until now, and leave the release note as it is!

Copy link
Member Author

Choose a reason for hiding this comment

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

ok

@alfredodeza
Copy link
Contributor

jenkins test docs

@alfredodeza
Copy link
Contributor

this will need to be rebased to make use of all the fixes for docs that just went into master

@alfredodeza
Copy link
Contributor

jenkins test docs

@theanalyst
Copy link
Member Author

jenkins render docs

@theanalyst
Copy link
Member Author

jenkins render docs

* Added new configuration "public bind addr" to support dynamic
environments like Kubernetes. When set the Ceph MON daemon could
bind locally to an IP address and advertise a different IP address
``public addr`` on the network.
Copy link
Member

Choose a reason for hiding this comment

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

This is mixing in the detailed compat notes into the feature highlights section. This stuff goes in the lower section, 'Upgrade compatibility notes'

Copy link
Member

Choose a reason for hiding this comment

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

Actually, let's move the 'pg force-*' bits to teh CLi changes, and the choose-args encoding to the compat notes. Pool metadata can stay.

Copy link
Member Author

Choose a reason for hiding this comment

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

ok

@ceph-jenkins
Copy link
Collaborator

Doc render available at http://docs.ceph.com/ceph-prs/16915/

@theanalyst
Copy link
Member Author

jenkins render docs

1 similar comment
@theanalyst
Copy link
Member Author

jenkins render docs

@theanalyst
Copy link
Member Author

@liewegas made the changes, also added the changelog from post 12.1.4, and amalgamated all the previous RC/dev releases into one large 12.2.0 changelog, wdyt?

option.

- Added new commands ``pg force-recovery`` and
``pg-force-backfill``. Use them to boost recovery or backfill
Copy link
Member

Choose a reason for hiding this comment

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

s/pg-force-backfill/pg force-backfill/

Copy link
Member Author

Choose a reason for hiding this comment

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

fixing

Reordered the RC releases sections back to their respective components,
added a ceph-mon section, added links to documentation wherever
possible, and a few forgotten RGW announcements. Also cleared up the
PendingReleaseNotes upto this point

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
Copy link
Member Author

jenkins render docs

@theanalyst
Copy link
Member Author

@liewegas can you review again

@@ -90,34 +93,47 @@ Major Changes from Kraken
``osd_snap_trim_sleep``, and ``osd_scrub_sleep`` have been
reimplemented to work efficiently. (These are used in some cases
to work around issues throttling background work.)
* The crush ``choose_args`` encoding has been changed to make it
architecture-independent.
Copy link
Member

Choose a reason for hiding this comment

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

this compat note should go in the upgrade notes down below.

Copy link
Member Author

Choose a reason for hiding this comment

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

ok


Ceph versions are released through the regular FreeBSD ports and packages
Copy link
Member

Choose a reason for hiding this comment

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

eliminating this newline will (i think) make the blank lines in teh rendered output go away.

@theanalyst
Copy link
Member Author

jenkins render docs

@theanalyst
Copy link
Member Author

jenkins render docs

@ceph-jenkins
Copy link
Collaborator

Doc render available at http://docs.ceph.com/ceph-prs/16915/

@theanalyst
Copy link
Member Author

@liewegas docs build succeeded, does http://docs.ceph.com/ceph-prs/16915/release-notes/ look ok?

* msg: Initialize member variables in Infiniband (`pr#16781 <https://github.com/ceph/ceph/pull/16781>`_, amitkuma)
* msg: Remove unused variable perf_counter in RDMAStack (`pr#16783 <https://github.com/ceph/ceph/pull/16783>`_, amitkuma)
* msgr: messages: Initializing uninitialized members MMonGetVersion (`pr#16811 <https://github.com/ceph/ceph/pull/16811>`_, amitkuma)
* msgr: messages: Initializing uninitialized members MMonGet
Copy link
Contributor

Choose a reason for hiding this comment

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

@theanalyst this line does not look right.

* msg: Initialize member variables in Infiniband (`pr#16781 <https://github.com/ceph/ceph/pull/16781>`_, amitkuma)
* msg: Remove unused variable perf_counter in RDMAStack (`pr#16783 <https://github.com/ceph/ceph/pull/16783>`_, amitkuma)
* msgr: messages: Initializing uninitialized members MMonGetVersion (`pr#16811 <https://github.com/ceph/ceph/pull/16811>`_, amitkuma)
* msgr: messages: Initializing uninitialized members MMonGet
Copy link
Contributor

Choose a reason for hiding this comment

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

@theanalyst and this line does not look right.

* msg: Initialize member variables in Infiniband (`pr#16781 <https://github.com/ceph/ceph/pull/16781>`_, amitkuma)
* msg: Remove unused variable perf_counter in RDMAStack (`pr#16783 <https://github.com/ceph/ceph/pull/16783>`_, amitkuma)
* msgr: messages: Initializing uninitialized members MMonGetVersion (`pr#16811 <https://github.com/ceph/ceph/pull/16811>`_, amitkuma)
* msgr: messages: Initializing uninitialized members MMonGet
Copy link
Contributor

Choose a reason for hiding this comment

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

this line.

@theanalyst
Copy link
Member Author

jenkins render docs

@theanalyst
Copy link
Member Author

jenkins render docs

@ceph-jenkins
Copy link
Collaborator

Doc render available at http://docs.ceph.com/ceph-prs/16915/

Amalgamated all the dev & RC changelogs + the changelog we saw post
12.1.4 into one large 12.2.0 changelog.

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
Copy link
Member Author

@liewegas squashed the commits, this can be merged

@liewegas liewegas merged commit 1be5a85 into ceph:master Aug 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
10 participants