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
common/options: validate see-also #42845
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested locally, lgtm!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shaman failure looks related,
Traceback (most recent call last):
File "/home/jenkins-build/build/workspace/ceph-dev-new-build/ARCH/arm64/AVAILABLE_ARCH/arm64/AVAILABLE_DIST/centos8/DIST/centos8/MACHINE_SIZE/gigantic/release/17.0.0-7149-g46305a89/rpm/el8/BUILD/ceph-17.0.0-7149-g46305a89/src/common/options/y2c.py", line 366, in <module>
main()
File "/home/jenkins-build/build/workspace/ceph-dev-new-build/ARCH/arm64/AVAILABLE_ARCH/arm64/AVAILABLE_DIST/centos8/DIST/centos8/MACHINE_SIZE/gigantic/release/17.0.0-7149-g46305a89/rpm/el8/BUILD/ceph-17.0.0-7149-g46305a89/src/common/options/y2c.py", line 360, in main
translate(opts)
File "/home/jenkins-build/build/workspace/ceph-dev-new-build/ARCH/arm64/AVAILABLE_ARCH/arm64/AVAILABLE_DIST/centos8/DIST/centos8/MACHINE_SIZE/gigantic/release/17.0.0-7149-g46305a89/rpm/el8/BUILD/ceph-17.0.0-7149-g46305a89/src/common/options/y2c.py", line 249, in translate
with open(opts.input) as infile, \
FileNotFoundError: [Errno 2] No such file or directory: '/home/jenkins-build/build/workspace/ceph-dev-new-build/ARCH/arm64/AVAILABLE_ARCH/arm64/AVAILABLE_DIST/centos8/DIST/centos8/MACHINE_SIZE/gigantic/release/17.0.0-7149-g46305a89/rpm/el8/BUILD/ceph-17.0.0-7149-g46305a89/aarch64-redhat-linux-gnu/src/common/options/mds-client.yaml'
https://shaman.ceph.com/builds/ceph/wip-deepika-testing-2021-08-23-1343/
no, the build failure is not related
|
sorry, i was wrong. it is related. but i still don't get why. |
9455b84
to
c08bcc1
Compare
@ideepika could you take another look? |
@tchaikov make check failure is related, can you fix it:
|
y2c.py is like a compiler which translates .yaml to .cc and .h files, it does not have access to all .yaml files. to validate the dangling see-also issue, we need to do this with a "linker". in this change, validate-options.py is introduced to check if any of option name included by the see-also property is valid. Fixes: https://tracker.ceph.com/issues/51483 Signed-off-by: Kefu Chai <kchai@redhat.com>
as FindPython3.cmake only sets Python3_EXECUTABLE for us, we should stick with Python3_EXECUTABLE instead of Python_EXECUTABLE. Signed-off-by: Kefu Chai <kchai@redhat.com>
@ideepika fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Fixes: https://tracker.ceph.com/issues/51483
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