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.

Sign up for GitHub

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

Jump to bottom

os/bluestore: Enable writing data to HM-SMR hard drives. #35111

Merged
merged 1 commit into from May 27, 2020
Merged

os/bluestore: Enable writing data to HM-SMR hard drives. #35111

merged 1 commit into from May 27, 2020

Conversation

agayev
Copy link
Contributor

@agayev agayev commented May 18, 2020

Signed-off-by: Abutalib Aghayev agayev@cs.cmu.edu

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
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 backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

tchaikov
tchaikov reviewed May 20, 2020
View reviewed changes
src/os/bluestore/Allocator.cc Show resolved Hide resolved
src/os/bluestore/ZonedAllocator.h Outdated Show resolved Hide resolved
doc/dev/zoned-storage.rst Outdated Show resolved Hide resolved
src/os/CMakeLists.txt Outdated Show resolved Hide resolved
src/os/bluestore/Allocator.cc Show resolved Hide resolved
src/os/bluestore/BlockDevice.h Outdated Show resolved Hide resolved
tchaikov
tchaikov reviewed May 21, 2020
View reviewed changes
src/os/bluestore/HMSMRDevice.cc Outdated Show resolved Hide resolved
src/os/bluestore/HMSMRDevice.cc Show resolved Hide resolved
src/os/bluestore/HMSMRDevice.h Outdated Show resolved Hide resolved
src/os/bluestore/HMSMRDevice.h Outdated Show resolved Hide resolved
src/os/bluestore/HMSMRDevice.h Outdated Show resolved Hide resolved
src/os/bluestore/ZonedAllocator.h Outdated Show resolved Hide resolved
tchaikov
tchaikov reviewed May 21, 2020
View reviewed changes
doc/dev/zoned-storage.rst Outdated Show resolved Hide resolved
@tchaikov
Copy link
Contributor

@agayev thanks! could you squash all the commits into single one?

os/bluestore: Enable writing data to HM-SMR hard drives.
ab865ca 
Signed-off-by: Abutalib Aghayev <agayev@cs.cmu.edu>
@agayev
Copy link
Contributor Author

agayev commented May 22, 2020

@agayev thanks! could you squash all the commits into single one?

Done.

@tchaikov
Copy link
Contributor

the test failures are either tracked or fixed by

@tchaikov tchaikov merged commit 857f55d into ceph:master May 27, 2020
@agayev agayev deleted the hm-smr branch May 27, 2020 15:08
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 19, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
4e23f90 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

 Conflicts:
	src/vstart.sh

- bluestore-zoned pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 19, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
216cd2c 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

 Conflicts:
	src/vstart.sh

- bluestore-zoned pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 22, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
7f33439 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

Conflicts:
	src/vstart.sh

- bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 22, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
0e57031 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

Conflicts:
	src/vstart.sh

- bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 23, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
2385b6c 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

Conflicts:
	src/vstart.sh

- bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 23, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
2a3ba95 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

Conflicts:
	src/vstart.sh

- Removed bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 23, 2020
@jtlayton @varshar16
vstart: just use ganesha.nfsd that's in $PATH
4eeea85 
In an earlier patch, I added a --ganesha-path option, but that left out
ganesha-rados-grace, which also could be in an alternate path. Instead,
just change vstart to run whatever ganesha.nfsd it finds in $PATH. We
already do that for ganesha-rados-grace so it should be fine to do that
for ganesha.nfsd as well.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit eab8c81)

Conflicts:
	src/vstart.sh

- Removed bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 24, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
b8f6502 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

Conflicts:
	src/vstart.sh

- Removed bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jun 24, 2020
@jtlayton @varshar16
vstart: just use ganesha.nfsd that's in $PATH
47d75cd 
In an earlier patch, I added a --ganesha-path option, but that left out
ganesha-rados-grace, which also could be in an alternate path. Instead,
just change vstart to run whatever ganesha.nfsd it finds in $PATH. We
already do that for ganesha-rados-grace so it should be fine to do that
for ganesha.nfsd as well.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit eab8c81)

Conflicts:
	src/vstart.sh

- Removed bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jul 1, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
b4ff21a 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

Conflicts:
	src/vstart.sh

- Removed bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jul 1, 2020
@jtlayton @varshar16
vstart: just use ganesha.nfsd that's in $PATH
ec032d7 
In an earlier patch, I added a --ganesha-path option, but that left out
ganesha-rados-grace, which also could be in an alternate path. Instead,
just change vstart to run whatever ganesha.nfsd it finds in $PATH. We
already do that for ganesha-rados-grace so it should be fine to do that
for ganesha.nfsd as well.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit eab8c81)

Conflicts:
	src/vstart.sh

- Removed bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jul 8, 2020
@jtlayton @varshar16
vstart: allow specifying a different ganesha binary
1feed4e 
Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit 06d29a7)

Conflicts:
	src/vstart.sh

- Removed bluestore-zoned option as pr(ceph#35111) is not backported yet.
varshar16 pushed a commit to varshar16/ceph that referenced this pull request Jul 8, 2020
@jtlayton @varshar16
vstart: just use ganesha.nfsd that's in $PATH
ecc77f4 
In an earlier patch, I added a --ganesha-path option, but that left out
ganesha-rados-grace, which also could be in an alternate path. Instead,
just change vstart to run whatever ganesha.nfsd it finds in $PATH. We
already do that for ganesha-rados-grace so it should be fine to do that
for ganesha.nfsd as well.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(cherry picked from commit eab8c81)

Conflicts:
	src/vstart.sh

- Removed bluestore-zoned option as pr(ceph#35111) is not backported yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tchaikov tchaikov tchaikov approved these changes

Assignees
No one assigned
Labels
bluestore feature needs-qa wip-kefu-testing
Projects
No open projects
bluestore
Awaiting triage
Milestone
No milestone
3 participants
@agayev @tchaikov @athanatos