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

Hwh v17.2.6 #57468

Closed
wants to merge 2,975 commits into from
Closed

Hwh v17.2.6 #57468

wants to merge 2,975 commits into from

Conversation

advance2016
Copy link

Contribution Guidelines

  • To sign and title your commits, please refer to Submitting Patches to Ceph.

  • If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.

  • When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an x between the brackets: [x]. Spaces and capitalization matter when checking off items this way.

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows
  • jenkins test rook e2e

adk3798 and others added 30 commits February 17, 2023 10:45
Otherwise, it could remain in the list and cephadm could think
there is an offline host in the cluster when said host has
actually already been removed.

Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit beea26f)
quincy: mgr/orchestrator: fix upgrade status help message

Reviewed-by: Redouane Kachach <rkachach@redhat.com>
quincy: cephadm: only pull host info from applied spec, don't try to parse yaml

Reviewed-by: Redouane Kachach <rkachach@redhat.com>
quincy: cephadm: don't overwrite cluster logrotate file

Reviewed-by: Teoman ONAY <tonay@ibm.com>
quincy: mgr/cephadm: iscsi username and password defaults to admin

Reviewed-by: Nizamudeen A <nia@redhat.com>
quincy: cephadm: add `ip_nonlocal_bind` to haproxy deployment

Reviewed-by: Redouane Kachach <rkachach@redhat.com>
Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit 7c6c715)
* drop the blank FS Menu when there is no FS
* cleanup display_fs_menu() code

Signed-off-by: Jos Collin <jcollin@redhat.com>
(cherry picked from commit a386400)
This commit intends to add:
- sort-by field value feature to cephfs-top.
- feature to limit number of clients displayed

Fixes: https://tracker.ceph.com/issues/55121
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
(cherry picked from commit 27cad62)

 Conflicts:
	src/tools/cephfs/top/cephfs-top:Removing the cherry-picked
	conflict (size=0) from line number 651 in this commit as it's
	position is changed as part of another PR.
Fixes: https://tracker.ceph.com/issues/55121
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
(cherry picked from commit bc360a4)
Fixes: https://tracker.ceph.com/issues/58031
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
(cherry picked from commit fd40f76)
Sorting is not working properly in case of change
in the number of clients or after removing and
creating the same filesystem again.

Fixes: https://tracker.ceph.com/issues/58028
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
(cherry picked from commit 2da139d)

 Conflicts:
	src/tools/cephfs/top/cephfs-top:Added argument `x_coord_map` in the function call
	for `create_client()` inside create_clients() method.
quincy: qa/suites/rbd: fix sporadic "rx-only direction" test failures

Reviewed-by: Ramana Raja <rraja@redhat.com>
quincy: librbd: avoid EUCLEAN error after "rbd rm" is interrupted

Reviewed-by: Ramana Raja <rraja@redhat.com>
quincy: ceph-crash: drop privleges to run as "ceph" user, rather than root (CVE-2022-3650)
quincy: common: notify all when max backlog reached in OutputDataSocket

Reviewed-by: Laura Flores <lflores@redhat.com>
quincy: ceph.spec.in: disable system_pmdk on s390x for SUSE distros

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
quincy: make-dist: don't set Release tag in ceph.spec for SUSE distros

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
quincy: mgr/telemetry: add `basic_pool_options_bluestore` collection
Fixes: https://tracker.ceph.com/issues/57786

Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit 9ee9c94)
Add definition for "period" to glossary.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit bd9bc3a)
…rt-50154-to-quincy

quincy: doc/glossary: add "Period" to glossary
… 'tmpfs'

When the device variable is rebound and become 'tmpfs', lsblk() shouldn't be called.

Fixes: https://tracker.ceph.com/issues/58784

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit c0369fb)
quincy: ceph-volume/tests: add allowlist_externals to tox.ini
Replace "execute" with "run" in fourteen out of fifteen instances in
multisite.rst. "Execute" is plain old too grand in most of these
instances.

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit b54b1b6)
…rt-50164-to-quincy

quincy: doc/radosgw: s/execute/run/ in multisite.rst
When a snapshot sync is resumed and the get_replay_status function
is called before handle_copy_image_progress, the syncing_percent
value may be greater than 100 as the m_local_object_count is still
set to zero. This commit sets the syncing_percent to 0 in such cases.

Fixes: https://tracker.ceph.com/issues/58706

Signed-off-by: N Balachandran <nibalach@redhat.com>
(cherry picked from commit c7ae0f6)
Fixes: https://tracker.ceph.com/issues/58779
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
(cherry picked from commit a235be2)
A monitor does support two kinds of addresses when binding msgr:
  1) `bind_addrs` which are basically passed down the stack to
     e.g. the `bind()` syscall -- they are physical, low-level ones;
  2) `public_addrs` which are exposed to peers over Ceph's on-wire
protocols.

This differentation is crucial for Rook and K8S use scenarios
where, due to ephemeral nature of node's physical IPs, virtual
_cluster IPs_ must be used instead for mon accesses.
Although monitors still bind to physical ones, virtuals are exposed.
Then, thankfully to NAT, the latter are converted to the former ones.

Unfortunately, the current implmentation of this feature is based
on the idea to 1) bind-as-usually and then 2) change-the-addrs
via powerful `set_addrs()` setter. This setter is too powerful
but the main issue is the `rebind()` sceario. It doesn't happen
in monitors but OSDs may rebind when they get marked down by
a monitor.

Fixes: https://tracker.ceph.com/issues/58779
Signed-off-by: Radosław Zarzyński <rzarzyns@redhat.com>
(cherry picked from commit 1b24588)
Fixes: https://tracker.ceph.com/issues/58779
Signed-off-by: Radosław Zarzyński <rzarzyns@redhat.com>
(cherry picked from commit b207f9b)
Copy link

This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved

@anthonyeleven
Copy link
Contributor

I suspect this PR is unintentional.

Copy link
Contributor

@anthonyeleven anthonyeleven left a comment

Choose a reason for hiding this comment

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

I suspect this was not intentional, as it isn't vaguely a valid PR

@batrick batrick closed this May 17, 2024
@ceph ceph locked as spam and limited conversation to collaborators May 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Done