Skip to content

mgr/volumes/nfs: use f-string for creating Exception#35609

Merged
tchaikov merged 1 commit intoceph:masterfrom
tchaikov:wip-mgr-volumes-nfs-fstring
Jun 17, 2020
Merged

mgr/volumes/nfs: use f-string for creating Exception#35609
tchaikov merged 1 commit intoceph:masterfrom
tchaikov:wip-mgr-volumes-nfs-fstring

Conversation

@tchaikov
Copy link
Contributor

@tchaikov tchaikov commented Jun 17, 2020

for better understanding the reason of the failure

Signed-off-by: Kefu Chai kchai@redhat.com

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
mgr/volumes/nfs: use f-string for creating Exception
5b8ba66 
for better understanding the reason of the failure

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov added cephfs Ceph File System mgr labels Jun 17, 2020
@tchaikov tchaikov requested review from ajarr, batrick and varshar16 June 17, 2020 04:35
@tchaikov
Copy link
Contributor Author

tchaikov commented Jun 17, 2020 •
edited
Loading

i came across it while looking at http://pulpito.ceph.com/kchai-2020-06-17_01:02:07-rados-wip-kefu-testing-2020-06-17-0030-distro-basic-smithi/5156251/

2020-06-17T01:31:27.820 INFO:teuthology.orchestra.run.smithi155.stderr:Error EINVAL: Failed to delete exports: {err} and {ret}
2020-06-17T01:31:27.821 DEBUG:teuthology.orchestra.run:got remote process result: 22
2020-06-17T01:31:27.821 INFO:teuthology.orchestra.run.smithi155:> sudo adjust-ulimits ceph-coverage /home/ubuntu/cephtest/archive/coverage timeout 120 ceph --cluster ceph --log-early log 'Ended test tasks
.cephfs.test_nfs.TestNFS.test_create_multiple_exports'
2020-06-17T01:31:28.145 INFO:ceph.mgr.a.smithi155.stdout:Jun 17 01:31:27 smithi155 bash[33746]: debug 2020-06-17T01:31:27.810+0000 7f5f4f325700 -1 mgr.server reply reply (22) Invalid argument Failed to delete exports: {err} and {ret}
2020-06-17T01:31:28.568 INFO:tasks.cephfs_test_runner:test_create_multiple_exports (tasks.cephfs.test_nfs.TestNFS) ... ERROR
2020-06-17T01:31:28.569 INFO:tasks.cephfs_test_runner:
2020-06-17T01:31:28.570 INFO:tasks.cephfs_test_runner:======================================================================
2020-06-17T01:31:28.570 INFO:tasks.cephfs_test_runner:ERROR: test_create_multiple_exports (tasks.cephfs.test_nfs.TestNFS)
2020-06-17T01:31:28.570 INFO:tasks.cephfs_test_runner:----------------------------------------------------------------------
2020-06-17T01:31:28.570 INFO:tasks.cephfs_test_runner:Traceback (most recent call last):
2020-06-17T01:31:28.571 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_ceph-c_wip-kefu-testing-2020-06-17-0030/qa/tasks/cephfs/test_nfs.py", line 119, in test_create_multiple_exports
2020-06-17T01:31:28.571 INFO:tasks.cephfs_test_runner:    self._test_delete_cluster()
2020-06-17T01:31:28.571 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_ceph-c_wip-kefu-testing-2020-06-17-0030/qa/tasks/cephfs/test_nfs.py", line 66, in _test_delete_cluster
2020-06-17T01:31:28.572 INFO:tasks.cephfs_test_runner:    self._nfs_cmd('cluster', 'delete', self.cluster_id)
2020-06-17T01:31:28.572 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_ceph-c_wip-kefu-testing-2020-06-17-0030/qa/tasks/cephfs/test_nfs.py", line 19, in _nfs_cmd
2020-06-17T01:31:28.572 INFO:tasks.cephfs_test_runner:    return self._cmd("nfs", *args)
2020-06-17T01:31:28.572 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_ceph-c_wip-kefu-testing-2020-06-17-0030/qa/tasks/cephfs/test_nfs.py", line 16, in _cmd
2020-06-17T01:31:28.573 INFO:tasks.cephfs_test_runner:    return self.mgr_cluster.mon_manager.raw_cluster_cmd(*args)
2020-06-17T01:31:28.573 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_ceph-c_wip-kefu-testing-2020-06-17-0030/qa/tasks/ceph_manager.py", line 1363, in raw_cluster_cmd
2020-06-17T01:31:28.573 INFO:tasks.cephfs_test_runner:    stdout=BytesIO(),
2020-06-17T01:31:28.574 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_git_teuthology_master/teuthology/orchestra/remote.py", line 206, in run
2020-06-17T01:31:28.574 INFO:tasks.cephfs_test_runner:    r = self._runner(client=self.ssh, name=self.shortname, **kwargs)
2020-06-17T01:31:28.574 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_git_teuthology_master/teuthology/orchestra/run.py", line 475, in run
2020-06-17T01:31:28.574 INFO:tasks.cephfs_test_runner:    r.wait()
2020-06-17T01:31:28.575 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_git_teuthology_master/teuthology/orchestra/run.py", line 162, in wait
2020-06-17T01:31:28.575 INFO:tasks.cephfs_test_runner:    self._raise_for_status()
2020-06-17T01:31:28.575 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_git_teuthology_master/teuthology/orchestra/run.py", line 184, in _raise_for_status
2020-06-17T01:31:28.575 INFO:tasks.cephfs_test_runner:    node=self.hostname, label=self.label
2020-06-17T01:31:28.576 INFO:tasks.cephfs_test_runner:teuthology.exceptions.CommandFailedError: Command failed on smithi155 with status 22: 'sudo adjust-ulimits ceph-coverage /home/ubuntu/cephtest/archive/coverage timeout 120 ceph --cluster ceph --log-early nfs cluster delete test'

@tchaikov
Copy link
Contributor Author

tchaikov commented Jun 17, 2020

tested at http://pulpito.ceph.com/kchai-2020-06-17_08:41:50-rados-wip-kefu-testing-2020-06-17-1349-distro-basic-smithi/5157584/.

tracker ticket created at https://tracker.ceph.com/issues/46046

@tchaikov tchaikov merged commit 7f8a21b into ceph:master Jun 17, 2020
@tchaikov tchaikov deleted the wip-mgr-volumes-nfs-fstring branch June 17, 2020 11:26
@varshar16 varshar16 mentioned this pull request Jun 23, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ajarr ajarr ajarr approved these changes

@batrick batrick Awaiting requested review from batrick

+1 more reviewer

@varshar16 varshar16 varshar16 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cephfs Ceph File System mgr needs-qa wip-kefu-testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tchaikov @ajarr @varshar16