Skip to content
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

qa/tests: added 'python3-cephfs','python3-rados' to excluded packadges #28301

Merged
merged 1 commit into from May 31, 2019

Conversation

yuriw
Copy link
Contributor

@yuriw yuriw commented May 29, 2019

Signed-off-by: Yuri Weinstein yweinste@redhat.com

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

@yuriw yuriw requested a review from jdurgin May 29, 2019 19:20
@yuriw
Copy link
Contributor Author

yuriw commented May 29, 2019

@tchaikov tchaikov self-assigned this May 30, 2019
@tchaikov
Copy link
Contributor

@yuriw lemme take a look

@yuriw
Copy link
Contributor Author

yuriw commented May 30, 2019

@tchaikov pls do, thx !

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>
@tchaikov tchaikov force-pushed the wip-yuriw-exl-python-mimic branch from d34b481 to 99caa61 Compare May 31, 2019 03:10
@tchaikov
Copy link
Contributor

@yuriw in /a/yuriw-2019-05-28_21:13:12-upgrade:mimic-p2p-mimic_v13.2.6_QE-distro-basic-smithi/3984806/teuthology.log, i found:

2019-05-28T22:15:53.697 DEBUG:teuthology.task.install:Package list is: {'deb': ['ceph', 'ceph-mds', 'ceph-mgr', 'ceph-common', 'ceph-fuse', 'ceph-test', 'radosgw', 'python-ceph', 'libcephfs2', 'libcephfs-dev', 'librados2', 'librbd1', 'rbd-fuse', 'python3-cephfs', 'python3-rados'], 'rpm': ['ceph-radosgw', 'ceph-test', 'ceph', 'ceph-mgr', 'ceph-fuse', 'libcephfs2', 'libcephfs-devel', 'librados2', 'librbd1', 'python-ceph', 'rbd-fuse', 'python3-cephfs', 'python3-rados']}

and after filtering the packages with exclude_packages black list, we translate python3-* packages to python36-* packages before installing them. that's why they snuck into the package list to be installed. i replaced all python36-* changes with python3-* in your change.

@tchaikov tchaikov removed their assignment May 31, 2019
@tchaikov tchaikov added the tests label May 31, 2019
@tchaikov tchaikov changed the title qa/tests: added 'python36-cephfs','python36-rados' to excluded packadges qa/tests: added 'python3-cephfs','python3-rados' to excluded packadges May 31, 2019
@yuriw
Copy link
Contributor Author

yuriw commented May 31, 2019

@tchaikov Wow, thx !

@yuriw yuriw merged commit 26da80a into ceph:mimic May 31, 2019
@yuriw yuriw deleted the wip-yuriw-exl-python-mimic branch May 31, 2019 14:55
@djgalloway
Copy link

we translate python3-* packages to python36-* packages before installing them

Where is this done? Just curious.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants