Skip to content

qa: retry cloning the ceph.git#40611

Merged
batrick merged 1 commit intoceph:masterfrom
lxbsz:tracker_50021
Apr 8, 2021
Merged

qa: retry cloning the ceph.git#40611
batrick merged 1 commit intoceph:masterfrom
lxbsz:tracker_50021

Conversation

@lxbsz
Copy link
Member

@lxbsz lxbsz commented Apr 6, 2021

The cloning may take a very long time, 2.5 hours for example, usally
it should be done in server minutes. If it really very slow, there
maybe something wrong with the current socket connections, retry it
maybe fixed.

This fixing will timeout the clone command after 30m, and retry it
by doubling the timeout value.

Fixes: https://tracker.ceph.com/issues/50021
Signed-off-by: Xiubo Li xiubli@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 api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

The cloning may take a very long time, 2.5 hours for example, usally
it should be done in server minutes. If it really very slow, there
maybe something wrong with the current socket connections, retry it
maybe fixed.

This fixing will timeout the clone command after 30m, and retry it
by doubling the timeout value.

Fixes: https://tracker.ceph.com/issues/50021
Signed-off-by: Xiubo Li <xiubli@redhat.com>
@lxbsz lxbsz requested review from a team and batrick April 6, 2021 08:06
@lxbsz lxbsz added cephfs Ceph File System needs-qa labels Apr 6, 2021
@github-actions github-actions bot added the tests label Apr 6, 2021
@mykaul
Copy link

mykaul commented Apr 6, 2021

I wonder if we can use '--depth 1' for shallow cloning.

@lxbsz
Copy link
Member Author

lxbsz commented Apr 6, 2021

I wonder if we can use '--depth 1' for shallow cloning.

This seems won't work for this test case, it also need the tag branches to do the snapshot test, please see qa/workunits/fs/snaps/snaptest-git-ceph.sh.

@batrick
Copy link
Member

batrick commented Apr 8, 2021

https://pulpito.ceph.com/pdonnell-2021-04-07_02:12:41-fs-wip-pdonnell-testing-20210406.213012-distro-basic-smithi/

@batrick batrick merged commit d06d66d into ceph:master Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants