Loïc Dachary <loic@dachary.org>于2021年4月28日 周三18:12写道:
Hi Nathan,

Josh noticed that one line could be removed[0] from the test script. I did it and repushed[1]. Would you be so kind as to push the change to GitHub?

Loïc and Nathan, when testing the change, I ran into an error like:

Traceback (most recent call last):
  File "/home/kchai/teuthology/virtualenv/bin/teuthology-suite", line 33, in <module>
    sys.exit(load_entry_point('teuthology', 'console_scripts', 'teuthology-suite')())
  File "/home/kchai/teuthology/scripts/suite.py", line 189, in main
    return teuthology.suite.main(args)
  File "/home/kchai/teuthology/teuthology/suite/__init__.py", line 143, in main
    run.prepare_and_schedule()
  File "/home/kchai/teuthology/teuthology/suite/run.py", line 397, in prepare_and_schedule
    num_jobs = self.schedule_suite()
  File "/home/kchai/teuthology/teuthology/suite/run.py", line 615, in schedule_suite
    self.args.newest, job_limit)
  File "/home/kchai/teuthology/teuthology/suite/run.py", line 467, in collect_jobs
    self.package_versions
  File "/home/kchai/teuthology/teuthology/suite/util.py", line 394, in get_package_versions
    distro_version=os_version,
  File "/home/kchai/teuthology/teuthology/suite/util.py", line 274, in package_version_for_hash
    sha1=hash,
  File "/home/kchai/teuthology/teuthology/packaging.py", line 853, in __init__
    super(ShamanProject, self).__init__(project, job_config, ctx, remote)
  File "/home/kchai/teuthology/teuthology/packaging.py", line 462, in __init__
    self._init_from_config()
  File "/home/kchai/teuthology/teuthology/packaging.py", line 497, in _init_from_config
    OS.version_codename(self.os_type, self.os_version)
  File "/home/kchai/teuthology/teuthology/orchestra/opsys.py", line 200, in version_codename
    (version_or_codename, name))
KeyError: '8.3 not a ubuntu version or codename'

I think the root cause is that the rados/standalone test suite includes it’s own faces for choosing a random distro, and my test happened to pick rhel 8.3 for testing, but the distro name was overridden by the one specified by c2c.yaml. That’s why I had a combination of Ubuntu 8.3. I just took the liberty to push another commit to the pull request in hope to test sooner. If it looks sane to you, could you include it in your commit? Or I can do this with your permission.


Thanks for your help!

[0] https://github.com/ceph/ceph/pull/41014#pullrequestreview-645521549
[1] https://lab.fedeproxy.eu/ceph/ceph/-/tree/wip-mempool-cacheline-49781

On 25/04/2021 17:10, Loïc Dachary wrote:
> Great! Thank you :-)
>
> On 25/04/2021 10:39, Nathan Cutler wrote:
>> On Sat, Apr 24, 2021 at 11:13:39PM +0200, Loïc Dachary wrote:
>>> Thanks for pushing the branch. I amended it a little and the teuthology run now passes[0]. There are still issues, I'm sure, but it's probably good enough for a pull request. Would you be so kind as to create one based on my branch[1] with the following cover? Thanks a again for your help
> :-)
>> Sure, here you go:
>>
>> https://github.com/ceph/ceph/pull/41014
>
> _______________________________________________
> Dev mailing list -- dev@ceph.io
> To unsubscribe send an email to dev-leave@ceph.io

--
Loïc Dachary, Artisan Logiciel Libre
_______________________________________________
Dev mailing list -- dev@ceph.io
To unsubscribe send an email to dev-leave@ceph.io
--
Regards
Kefu Chai