Skip to content

test/mon: FreeBSD returns ENOENT on resolv error#38026

Merged
tchaikov merged 1 commit intoceph:masterfrom
wjwithagen:wjw-fix-test-mon-build_initial
Nov 15, 2020
Merged

test/mon: FreeBSD returns ENOENT on resolv error#38026
tchaikov merged 1 commit intoceph:masterfrom
wjwithagen:wjw-fix-test-mon-build_initial

Conversation

@wjwithagen
Copy link
Copy Markdown
Contributor

Tracker: https://tracker.ceph.com/issues/48183

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

@wjwithagen wjwithagen added bug-fix FreeBSD Modifications specific for (only) FreeBSD tests labels Nov 11, 2020
Copy link
Copy Markdown
Contributor

@tchaikov tchaikov left a comment

Choose a reason for hiding this comment

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

could you please use

test/mon: return ENOENT on resolv error on FreeBSD

as the title of the commit message?

@wjwithagen wjwithagen changed the title Mon: FreeBSD returns ENOENT on resolv error test/mon: FreeBSD returns ENOENT on resolv error Nov 14, 2020
@wjwithagen wjwithagen force-pushed the wjw-fix-test-mon-build_initial branch from 323a432 to 8ad8b1d Compare November 14, 2020 17:02
@wjwithagen wjwithagen requested a review from tchaikov November 14, 2020 19:35
@tchaikov
Copy link
Copy Markdown
Contributor

@wjwithagen i still don't like the title, which claims that "FreeBSD returns ENOENT". this is a fact, not the change you made in this patch. so could you put it in the body of the commit message, as the reason why we need your change? and use

return ENOENT on resolv error on FreeBSD

as the title instead?

FreeBSD returns ENOENT on resolv error

Tracker: https://tracker.ceph.com/issues/48183
Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>
@wjwithagen wjwithagen force-pushed the wjw-fix-test-mon-build_initial branch from 8ad8b1d to 5ae4827 Compare November 15, 2020 09:10
@wjwithagen
Copy link
Copy Markdown
Contributor Author

@tchaikov
Used a slightly different wording from yours, but it is descriptive.

@tchaikov tchaikov merged commit 6120bb2 into ceph:master Nov 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix FreeBSD Modifications specific for (only) FreeBSD needs-qa tests wip-kefu-testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants