-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
qa/suites/rbd/nvmeof: change gateway-name to host-name #57522
base: main
Are you sure you want to change the base?
Conversation
ceph-nvmeof "listener add" cmd used arg "--gateway-name" in nvmeof:1.0.0 (see nvmeof_setup_subsystem.sh script). This has been changed to "--host-name" for nvmeof:1.2.5 Signed-off-by: Vallari Agrawal <val.agl002@gmail.com>
Is there someone looking into that? And separately, why isn't it happening automatically? |
I have informed Aviv and Barak that it needs to be changed - we are looking into second issue right now. |
I would be interested in a justification for a backwards incompatible change to the CLI post 1.0.0 too. |
This change was made more than a month ago. We're not backwards compatible to 7.0 TP. What is the concern @idryomov ? |
@caroav I'm asking from the upstream perspective. When the CLI overhaul was discussed, I thought it was decided that 1.0(.0) would be tagged and CLI would be declared stable, such that any future backwards incompatible change would need to pass a high bar. Do you have a different version for that base in mind? |
a new tag version is pushed automatically through our github action (CI) when creating new release here |
ceph-nvmeof "listener add" cmd used arg "--gateway-name" in nvmeof:1.0.0 (see nvmeof_setup_subsystem.sh script).
This has been changed to "--host-name" in nvmeof:1.2.5
Following thing need to be done before merging this PR:
Change quay.io/ceph/nvmeof-cli:latest to point to tag 1.2.5 (right now it points to tag 1.0.0)
Reason: Default nvmeof gateway image has been changed to 1.2.5 in mgr/cephadm: update default NVMEoF container image version #57182. Because teuthology tests use ceph's default gateway image + nvmeof-cli "latest" - there's a version mismatch of nvmeof gateway v1.2.5 and nvmeof-cli v1.0.0 in tests on main branch which causes error
HA must be enabled for subsystems
as seen in https://pulpito.ceph.com/dis-2024-05-16_08:07:49-rbd-wip-dis-testing-distro-default-smithi/7708811/(Tested the solution by running a test with both images using v1.2.5 and it was able to setup the subsystem)
Failing "nvme connect" test - "nvme list" it does not show any connected devices after doing a "nvme connect". It might be a bug in nvmeof, need to inspect.
Seen in: https://pulpito.ceph.com/vallariag-2024-05-16_13:41:16-rbd:nvmeof-main-distro-default-smithi/
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
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