*** lixiaoy1 has joined #openstack-cinder | 00:20 | |
*** sdake has joined #openstack-cinder | 00:26 | |
*** erlon has joined #openstack-cinder | 00:45 | |
*** sdake has quit IRC | 00:58 | |
*** phuongnh has joined #openstack-cinder | 01:01 | |
*** markvoelker has joined #openstack-cinder | 01:10 | |
*** sdake has joined #openstack-cinder | 01:16 | |
*** whoami-rajat has joined #openstack-cinder | 01:22 | |
*** sdake has joined #openstack-cinder | 01:22 | |
*** erlon has quit IRC | 01:38 | |
*** sdake has quit IRC | 01:42 | |
*** markvoelker has quit IRC | 01:44 | |
*** sdake has joined #openstack-cinder | 01:44 | |
*** sdake has quit IRC | 01:54 | |
*** sdake has joined #openstack-cinder | 02:00 | |
openstackgerrit | Chuck Short proposed openstack/cinder master: Drop gpfs commands from rootwrap https://review.openstack.org/637661 | 02:02 |
---|---|---|
*** Dinesh_Bhor has joined #openstack-cinder | 02:10 | |
*** sapd1_ has quit IRC | 02:13 | |
*** sdake has quit IRC | 02:22 | |
*** _alastor_ has joined #openstack-cinder | 02:33 | |
*** _alastor_ has quit IRC | 02:38 | |
*** sapd1 has joined #openstack-cinder | 02:38 | |
*** markvoelker has joined #openstack-cinder | 02:41 | |
*** bhagyashris has joined #openstack-cinder | 02:50 | |
*** psachin has joined #openstack-cinder | 02:59 | |
*** Dinesh_Bhor has quit IRC | 03:03 | |
*** Dinesh_Bhor has joined #openstack-cinder | 03:06 | |
*** markvoelker has quit IRC | 03:13 | |
*** dklyle has joined #openstack-cinder | 03:23 | |
*** sapd1 has quit IRC | 03:34 | |
*** dave-mccowan has joined #openstack-cinder | 03:46 | |
*** psachin has quit IRC | 03:50 | |
*** sapd1 has joined #openstack-cinder | 03:51 | |
*** udesale has joined #openstack-cinder | 03:54 | |
*** psachin has joined #openstack-cinder | 04:00 | |
*** Dinesh_Bhor has quit IRC | 04:03 | |
*** andreaf has quit IRC | 04:08 | |
*** andreaf has joined #openstack-cinder | 04:09 | |
*** _alastor_ has joined #openstack-cinder | 04:35 | |
*** _alastor_ has quit IRC | 04:39 | |
*** BlackDex has quit IRC | 04:43 | |
*** vishakha has joined #openstack-cinder | 04:45 | |
*** zzzeek has quit IRC | 04:49 | |
*** zzzeek has joined #openstack-cinder | 04:52 | |
*** irclogbot_0 has quit IRC | 04:55 | |
*** BlackDex has joined #openstack-cinder | 04:56 | |
*** dave-mccowan has quit IRC | 05:13 | |
*** Dinesh_Bhor has joined #openstack-cinder | 05:21 | |
*** udesale has quit IRC | 05:33 | |
*** markvoelker has joined #openstack-cinder | 05:41 | |
*** udesale has joined #openstack-cinder | 05:43 | |
*** itlinux has quit IRC | 05:45 | |
*** itlinux has joined #openstack-cinder | 06:09 | |
*** markvoelker has quit IRC | 06:14 | |
*** itlinux has quit IRC | 06:16 | |
*** e0ne has joined #openstack-cinder | 06:25 | |
*** e0ne has quit IRC | 06:26 | |
*** jackyzhu has joined #openstack-cinder | 06:35 | |
*** jackyzhu has quit IRC | 06:37 | |
*** dswebb has quit IRC | 06:41 | |
*** lemko has joined #openstack-cinder | 06:46 | |
*** Luzi has joined #openstack-cinder | 06:48 | |
*** zhubx has joined #openstack-cinder | 06:49 | |
*** zhubx has left #openstack-cinder | 06:49 | |
*** zhubx has joined #openstack-cinder | 06:50 | |
*** zhubx has quit IRC | 06:51 | |
*** zhubx has joined #openstack-cinder | 06:52 | |
*** itlinux has joined #openstack-cinder | 07:02 | |
*** itlinux has quit IRC | 07:06 | |
*** markvoelker has joined #openstack-cinder | 07:11 | |
*** itlinux has joined #openstack-cinder | 07:13 | |
*** itlinux has quit IRC | 07:17 | |
*** e0ne has joined #openstack-cinder | 07:23 | |
*** rcernin has quit IRC | 07:25 | |
*** takamatsu has joined #openstack-cinder | 07:39 | |
*** ociuhandu has joined #openstack-cinder | 07:40 | |
*** markvoelker has quit IRC | 07:43 | |
*** ociuhandu has quit IRC | 07:47 | |
*** udesale has quit IRC | 07:58 | |
*** udesale has joined #openstack-cinder | 08:03 | |
*** udesale has quit IRC | 08:07 | |
*** udesale has joined #openstack-cinder | 08:08 | |
*** pcaruana has joined #openstack-cinder | 08:11 | |
*** tkajinam has quit IRC | 08:12 | |
*** e0ne has quit IRC | 08:13 | |
*** sdake has joined #openstack-cinder | 08:27 | |
*** kholkina has joined #openstack-cinder | 08:30 | |
*** _alastor_ has joined #openstack-cinder | 08:36 | |
*** sdake has quit IRC | 08:37 | |
*** _alastor_ has quit IRC | 08:40 | |
*** markvoelker has joined #openstack-cinder | 08:40 | |
*** itlinux has joined #openstack-cinder | 08:41 | |
*** itlinux has quit IRC | 08:42 | |
*** sdake has joined #openstack-cinder | 08:45 | |
*** lixiaoy1 has quit IRC | 08:50 | |
*** lemko has quit IRC | 08:56 | |
*** ociuhandu has joined #openstack-cinder | 09:02 | |
*** itlinux has joined #openstack-cinder | 09:03 | |
*** itlinux has quit IRC | 09:03 | |
*** sapd1 has quit IRC | 09:09 | |
*** markvoelker has quit IRC | 09:14 | |
*** sapd1 has joined #openstack-cinder | 09:22 | |
*** ociuhandu has quit IRC | 09:23 | |
*** sdake has quit IRC | 09:25 | |
*** sdake has joined #openstack-cinder | 09:27 | |
*** Florian has quit IRC | 09:28 | |
*** ociuhandu has joined #openstack-cinder | 09:31 | |
*** ociuhandu has quit IRC | 09:35 | |
*** zhubx has quit IRC | 09:36 | |
*** davidsha has joined #openstack-cinder | 09:49 | |
*** sdake has quit IRC | 09:49 | |
*** e0ne has joined #openstack-cinder | 09:53 | |
*** bhdn has quit IRC | 09:53 | |
*** bhagyashris has quit IRC | 09:55 | |
*** ociuhandu has joined #openstack-cinder | 09:56 | |
*** ociuhandu has quit IRC | 09:57 | |
*** ociuhandu has joined #openstack-cinder | 09:58 | |
*** markvoelker has joined #openstack-cinder | 10:11 | |
*** ganso has joined #openstack-cinder | 10:22 | |
*** markvoelker has quit IRC | 10:43 | |
*** lpetrut has joined #openstack-cinder | 10:46 | |
*** phuongnh has quit IRC | 10:49 | |
*** carlos_silva has joined #openstack-cinder | 10:50 | |
*** luizbag has joined #openstack-cinder | 11:01 | |
*** kholkina has quit IRC | 11:06 | |
*** udesale has quit IRC | 11:10 | |
*** Dinesh_Bhor has quit IRC | 11:14 | |
*** markvoelker has joined #openstack-cinder | 11:40 | |
*** thgcorrea has joined #openstack-cinder | 11:41 | |
*** rosmaita has joined #openstack-cinder | 12:04 | |
*** enriquetaso has joined #openstack-cinder | 12:07 | |
*** mvkr has quit IRC | 12:09 | |
*** erlon has joined #openstack-cinder | 12:14 | |
*** markvoelker has quit IRC | 12:14 | |
*** erlon has quit IRC | 12:14 | |
*** erlon has joined #openstack-cinder | 12:15 | |
*** erlon has quit IRC | 12:16 | |
*** erlon has joined #openstack-cinder | 12:16 | |
*** dpawlik has quit IRC | 12:51 | |
*** udesale has joined #openstack-cinder | 12:55 | |
kevko | smcginnis: hi, yes ..it is related to get_fsid ... | 13:10 |
kevko | smcginnis: it has to be fixed in glance store ..cinder and nova ..now i know how to fix .. | 13:10 |
*** markvoelker has joined #openstack-cinder | 13:11 | |
kevko | smcginnis: i will send changes to review ..just one technicall question ...i filled a bug in launchpad ..in git commit message i want to define link to BUG .... How can be this achieved ? Three changes ...three partial bugs and last one closes bug ? | 13:11 |
kevko | smcginnis: or depends on ? | 13:11 |
*** itlinux has joined #openstack-cinder | 13:12 | |
*** mvkr has joined #openstack-cinder | 13:13 | |
*** sri_ has quit IRC | 13:18 | |
*** sri_ has joined #openstack-cinder | 13:18 | |
*** itlinux has quit IRC | 13:21 | |
*** lemko has joined #openstack-cinder | 13:21 | |
*** itlinux has joined #openstack-cinder | 13:22 | |
*** dpawlik has joined #openstack-cinder | 13:23 | |
*** sdake has joined #openstack-cinder | 13:23 | |
*** itlinux has quit IRC | 13:28 | |
*** vishakha has quit IRC | 13:31 | |
*** itlinux has joined #openstack-cinder | 13:32 | |
*** itlinux has quit IRC | 13:37 | |
*** zul has joined #openstack-cinder | 13:40 | |
*** jmlowe has quit IRC | 13:41 | |
*** sri_ has quit IRC | 13:42 | |
*** markvoelker has quit IRC | 13:43 | |
*** Dinesh_Bhor has joined #openstack-cinder | 13:45 | |
smcginnis | kevko: I think that actually needs to be fixed in the librados library. It shouldn't be on each consumer of that library to have to know they need to handle the output differently depending on the runtime. | 13:46 |
*** sri_ has joined #openstack-cinder | 13:47 | |
openstackgerrit | Lee Yarwood proposed openstack/cinder master: docs: Add os-migrate_volume_completion api-ref https://review.openstack.org/637847 | 13:49 |
*** dave-mccowan has joined #openstack-cinder | 13:50 | |
*** eharney has joined #openstack-cinder | 13:55 | |
openstackgerrit | Lee Yarwood proposed openstack/cinder master: docs: Add os-migrate_volume_completion api-ref https://review.openstack.org/637847 | 14:03 |
*** sdake has quit IRC | 14:05 | |
*** srini_ has joined #openstack-cinder | 14:07 | |
*** jmlowe has joined #openstack-cinder | 14:07 | |
*** sri_ has quit IRC | 14:10 | |
*** Dinesh_Bhor has quit IRC | 14:10 | |
kevko | smcginnis: yes, i agree that it has to be fixed in librados library, but , we want to have cinder working also in python3 , don't we ? ... so i will fix it inside glance_store , cinder, nova ...by using oslo_utils.encodeutils .... and will report bug to librados library ... in both cases it will fix the issue | 14:11 |
kevko | smcginnis: if they will fix it ..encodeutils working correct in py2 and in py3 also ...so ..if they will fix , or no ...in openstack code it can be ..because it will fix both cases | 14:12 |
kevko | smcginnis: agree ? | 14:12 |
*** sdake has joined #openstack-cinder | 14:13 | |
smcginnis | It will fix it, but not the root cause. It doesn't make sense to scatter workarounds all over the place rather than just taking care of the problem at the source to me. | 14:14 |
kevko | smcginnis: in openstack projects there are hundreds of encodeutils and six usages :D | 14:19 |
*** sdake has quit IRC | 14:19 | |
kevko | smcginnis: and i will report bug to librados ... , and i can promise that i will watch this bug report and when they will fix i can remove it from a code :P | 14:20 |
*** itlinux has joined #openstack-cinder | 14:22 | |
*** zul has quit IRC | 14:25 | |
*** mriedem has joined #openstack-cinder | 14:25 | |
*** zul has joined #openstack-cinder | 14:26 | |
*** itlinux has quit IRC | 14:30 | |
jbernard | smcginnis: i think the argument is one of pragmatism - as the rados and rbd bindings are system packages on stable distros, the time to update is considerably larger which leave py3 users without a solution for a potentially long time. While we can push these changes now and revert them when the bindings update and propogate into the supported distros. It's not ideal but I think it is reasonably | 14:35 |
jbernard | prudent | 14:35 |
*** sdake has joined #openstack-cinder | 14:38 | |
*** _alastor_ has joined #openstack-cinder | 14:38 | |
*** mriedem has left #openstack-cinder | 14:40 | |
*** mriedem has joined #openstack-cinder | 14:40 | |
*** markvoelker has joined #openstack-cinder | 14:41 | |
*** _alastor_ has quit IRC | 14:43 | |
openstackgerrit | Chuck Short proposed openstack/cinder master: Remove 'ploop' from rootwrap https://review.openstack.org/637864 | 14:50 |
smcginnis | jbernard: Fair enough. kevko, if you could put comments in those patches referencing the librados bug, that will help us make sure we clean that up when it's no longer needed. | 14:51 |
openstackgerrit | Eric Harney proposed openstack/cinder master: Disable etcd3 for lio-barbican job https://review.openstack.org/632773 | 14:51 |
kevko | smcginnis: yes, i can of course do it for you guys, also the bug i filled i wanted to have very detailistic to be able track the issue to the past .... | 14:52 |
smcginnis | Thanks, I appreciate that. | 14:52 |
*** itlinux has joined #openstack-cinder | 14:52 | |
kevko | smcginnis: ok, thanks, i'm going to implement .. | 14:53 |
*** sdake has quit IRC | 14:53 | |
*** udesale has quit IRC | 14:53 | |
*** itlinux has quit IRC | 14:56 | |
*** itlinux has joined #openstack-cinder | 14:57 | |
*** itlinux has quit IRC | 14:57 | |
*** gnufied has joined #openstack-cinder | 14:57 | |
openstackgerrit | Sofia Enriquez proposed openstack/cinder stable/ocata: ScaleIO Driver - adding cache and refactoring tests https://review.openstack.org/633570 | 14:58 |
*** sdake has joined #openstack-cinder | 14:59 | |
openstackgerrit | Sofia Enriquez proposed openstack/cinder stable/ocata: ScaleIO Driver - adding cache and refactoring tests https://review.openstack.org/633570 | 15:00 |
*** whoami-rajat has quit IRC | 15:01 | |
*** markvoelker has quit IRC | 15:14 | |
*** whoami-rajat has joined #openstack-cinder | 15:18 | |
*** Luzi has quit IRC | 15:19 | |
*** jmlowe has quit IRC | 15:23 | |
*** jmlowe has joined #openstack-cinder | 15:25 | |
kevko | smcginnis: already in progress :) https://tracker.ceph.com/issues/38381#change-129656 | 15:25 |
*** sdake has quit IRC | 15:28 | |
geguileo | jungleboyj: when you have a minute, the patch to create cinderlib independent project is waiting for PTL's +1 https://review.openstack.org/#/c/637613/3 | 15:29 |
geguileo | jungleboyj: also there's the governance patch https://review.openstack.org/#/c/637614/1 | 15:29 |
jungleboyj | geguileo: Thanks for working on that. Let me take a look. | 15:30 |
geguileo | jungleboyj: thanks! | 15:30 |
jungleboyj | geguileo: Oh, you will use Storyboard. So, that is another place that we will need to put some processes in place. | 15:31 |
geguileo | jungleboyj: I think it would bad not to start with storyboard | 15:32 |
geguileo | it's a "new" project... | 15:32 |
geguileo | jungleboyj: though I didn't see the cinder group created in storyboard yet | 15:32 |
geguileo | I saw neutron and other groups there | 15:33 |
jungleboyj | Right. It will be less active but I think it will want to have the same layout as what we do for Cinder. | 15:33 |
smcginnis | I could see a "bug" in cinderlib actually being something in a cinder driver, so not sure if I really like the idea of all cinder projects being under launchpad and this one being under storyboard. | 15:34 |
*** jmlowe has quit IRC | 15:34 | |
jungleboyj | smcginnis: True, but Cinder is going to have to move its components soon. | 15:35 |
* jungleboyj hopes to not start this argument again today. | 15:35 | |
smcginnis | I thought we decided in Raleigh that there would be no immediate plans to migrate. | 15:35 |
*** sdake has joined #openstack-cinder | 15:36 | |
geguileo | I can change the patch if we prefer to have all projects together | 15:36 |
jungleboyj | smcginnis: No immediate plans but I am also not encouraging people to not think in that direction. | 15:37 |
smcginnis | os-brick doesn't have its own bug tracking, it just uses Cinder's main launchpad. Personally I would rather stick to that. | 15:37 |
smcginnis | Since these are so interrelated. | 15:38 |
jungleboyj | smcginnis: We had talked about moving one of the projects as a starting point in the past. I.E. os-brick. | 15:38 |
geguileo | smcginnis: but os-brick has it's own tracking: https://bugs.launchpad.net/os-brick | 15:38 |
smcginnis | Oh, that's right! Oops. :) | 15:39 |
geguileo | :-) | 15:39 |
jungleboyj | geguileo: Yeah, I was thinking that was the project we had planned to try migrating first. | 15:41 |
geguileo | jungleboyj: so do we want to have cinderlib in storyboard, or not? | 15:41 |
geguileo | jungleboyj: is there anything we would need to do? like create the cinder group there? | 15:41 |
whoami-rajat | eharney xyang would appreciate if you could take a look again. https://review.openstack.org/#/c/627359/ . Thanks! | 15:42 |
jungleboyj | geguileo: That is a good question. I am not sure what the process is there to get a new project created in Storyboard. | 15:44 |
jbernard | kevko: can you comment on why the experimental py3 job is passing given the current get_fsid implementation? http://zuul.openstack.org/builds?job_name=devstack-plugin-ceph-tempest-py3 | 15:44 |
jungleboyj | I believe there is a process for requesting it out there. | 15:44 |
geguileo | jungleboyj: that patch would create the project automatically | 15:44 |
geguileo | jungleboyj: but I don't know if it would automatically create the "group" | 15:45 |
jbernard | kevko: it's reasonable to an observer to be confused in this case | 15:45 |
jungleboyj | Ok. Let me find out. | 15:45 |
*** lpetrut has quit IRC | 15:45 | |
geguileo | jungleboyj: thanks | 15:46 |
jungleboyj | No problem. As far as smcginnis concerns. I suppose we can discuss it further in our weekly meeting tomorrow. I think if os-brick is already in its own bug tracking, then this is fine to do. It would give a PoC space for us to implement the processes we want to use for our bug and feature tracking before doing it for all of Cinder. | 15:48 |
jungleboyj | It is also consistent with what was recommended we do in the past. | 15:49 |
whoami-rajat | jungleboyj smcginnis as per the ML for oslo [1], as there are similar patches in our repo's, do we also need these patches ? | 15:50 |
whoami-rajat | [1] http://lists.openstack.org/pipermail/openstack-discuss/2019-February/002869.html | 15:50 |
geguileo | jungleboyj: os-brick has it's tracking in lauchpad, not storyboard afaik | 15:50 |
kevko | jbernard: i was not studying tempest tests, but true is ..that it is working with current implementation, but not working cinder-glance acceleration .. and maybe tempest test is just testing if instance was launched ...not testing if instance's volume is copy-on-write in ceph , OR downloaded from ceph -> create_volume -> upload volume to ceph ( this is bad ) , and maybe tempest environment dont' have show_image_direct_url = tru | 15:50 |
kevko | e in glance-api.conf | 15:50 |
kevko | jbernard: this bug is quite tricky :/ | 15:50 |
smcginnis | whoami-rajat: Maybe, but there's also this I am trying to push - https://review.openstack.org/#/c/637866/ | 15:50 |
smcginnis | Not that adding specific py37 jobs wouldn't be fine at least until that effort goes through. | 15:51 |
*** sdake has quit IRC | 15:52 | |
openstackgerrit | Chuck Short proposed openstack/cinder master: Remove 'mkfs' from rootwrap https://review.openstack.org/637919 | 15:54 |
whoami-rajat | smcginnis: so IIUC the purpose of using this template will enable consistency between gate jobs in all projects? | 15:54 |
jungleboyj | geguileo: Right, but it has its own bug tracking. Not just part of Cinder. | 15:54 |
geguileo | jungleboyj: yes, that is correct | 15:55 |
smcginnis | whoami-rajat: Yep. And it would be a job template that would be set in each projects in-repo zuul config so as we branch stable branches, those branches would continue to run the expected versions for that release. | 15:55 |
smcginnis | And new release cycles would pick up the current expected versions. | 15:55 |
whoami-rajat | smcginnis: that sounds great. one ques, why is the py37 job marked non-voting for stein? | 15:56 |
smcginnis | whoami-rajat: Py37 isn't required in stein, but it will be in train. So it's kind of a forward-looking job to make sure there are no surprises on the switchover. | 15:56 |
jungleboyj | Makes sense to me. | 15:58 |
*** sdake has joined #openstack-cinder | 15:59 | |
*** e0ne has quit IRC | 16:00 | |
*** _alastor_ has joined #openstack-cinder | 16:00 | |
whoami-rajat | smcginnis: makes sense. as we're starting with stein, so we will be replacing all the py37 jobs with this template eventually even if we merge them ? | 16:00 |
*** ianychoi has quit IRC | 16:00 | |
smcginnis | whoami-rajat: Yeah, I think that's what we would probably want to do. | 16:01 |
*** sdake has quit IRC | 16:03 | |
*** e0ne has joined #openstack-cinder | 16:03 | |
whoami-rajat | smcginnis: ok, thanks for the info Sean. | 16:05 |
smcginnis | No problem, thanks for taking a look at the job template patch. | 16:05 |
whoami-rajat | smcginnis: np :) | 16:06 |
*** sipior has quit IRC | 16:07 | |
*** markvoelker has joined #openstack-cinder | 16:11 | |
*** ianychoi has joined #openstack-cinder | 16:12 | |
whoami-rajat | smcginnis: Could you please take a look at the behavior of functional backup test on gate. It was running fine in my local env (ceph) but failing on gate(swift) . Since there didn't exist any backup functional tests previously in OSC, i had doubts regarding whether including them is valid or not. | 16:27 |
whoami-rajat | https://review.openstack.org/#/c/624860/ | 16:27 |
smcginnis | whoami-rajat: Sure, I'll try to get to it a little later. Got it open in a tab for now. | 16:29 |
whoami-rajat | smcginnis: sure, whenever you get time. Thanks. | 16:30 |
openstackgerrit | Lee Yarwood proposed openstack/cinder master: docs: Add os-migrate_volume_completion api-ref https://review.openstack.org/637847 | 16:41 |
*** markvoelker has quit IRC | 16:44 | |
openstackgerrit | Merged openstack/cinder master: REST connector for Brocade zone driver https://review.openstack.org/617100 | 16:44 |
*** sipior has joined #openstack-cinder | 16:46 | |
openstackgerrit | Merged openstack/cinder master: Limit RBD discard to 32 bit chunks https://review.openstack.org/636818 | 16:50 |
whoami-rajat | anyone having any idea why this passed Zuul with pep8 issues ? https://review.openstack.org/#/c/637847/2 ? | 16:52 |
*** Florian has joined #openstack-cinder | 16:53 | |
*** pcaruana has quit IRC | 16:57 | |
smcginnis | This .inc files aren't included in any linter checks. | 16:58 |
smcginnis | s/This/Those/ | 16:58 |
enriquetaso | jungleboyj, smcginnis hey! I wonder if you can check this patch: https://review.openstack.org/#/c/636433 . It's a simple backport to Rocky :D | 17:00 |
smcginnis | Done | 17:02 |
whoami-rajat | smcginnis: ok. so could you suggest the possible workaround for this? | 17:04 |
enriquetaso | Thanks!! smcginnis jungleboyj | 17:05 |
smcginnis | whoami-rajat: Hmm, I tried testing directly on an inc file with extra spaces and it still didn't flag it. | 17:07 |
smcginnis | Looks like doc8 refuses to scan inc files. | 17:09 |
openstackgerrit | Sofia Enriquez proposed openstack/cinder stable/queens: Fix for HPE MSA 2050 login failures https://review.openstack.org/637941 | 17:09 |
smcginnis | $ .tox/docs/bin/doc8 -e inc api-ref/source/v2/volumes-v2-volumes-actions.inc # <- modified to have errors | 17:09 |
smcginnis | Total files scanned = 0 | 17:10 |
whoami-rajat | hmm. is it only the issue with new .inc files added or its not detected in existing .inc files too? | 17:10 |
smcginnis | Existing too. | 17:10 |
smcginnis | Looks like doc8 just doesn't support scanning inc files. | 17:10 |
whoami-rajat | meaning our api-ref could be filled with lots of pep8 issues. but do other projects catch it or its being ignored and unnoticed ever since. | 17:14 |
smcginnis | Looks like I found how to get it to recognize things correctly. We have a lot of errors in the inc files. | 17:22 |
whoami-rajat | i used intentional spaces in nova '.inc' files too, then ran tox -edocs (don't know if its the right one to find these issues), it ran successfully. | 17:26 |
*** lpetrut has joined #openstack-cinder | 17:27 | |
*** ociuhandu_ has joined #openstack-cinder | 17:33 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Include .inc files in doc8 linting https://review.openstack.org/637949 | 17:34 |
*** e0ne has quit IRC | 17:35 | |
*** ociuhandu has quit IRC | 17:36 | |
*** eharney has quit IRC | 17:37 | |
*** ociuhandu_ has quit IRC | 17:38 | |
*** psachin has quit IRC | 17:39 | |
*** lpetrut has quit IRC | 17:40 | |
*** markvoelker has joined #openstack-cinder | 17:41 | |
*** mma has quit IRC | 17:42 | |
*** ociuhandu has joined #openstack-cinder | 17:48 | |
*** ociuhandu has quit IRC | 17:52 | |
*** davidsha has quit IRC | 17:57 | |
openstackgerrit | Lee Yarwood proposed openstack/cinder master: docs: Add os-migrate_volume_completion api-ref https://review.openstack.org/637847 | 17:58 |
*** ociuhandu has joined #openstack-cinder | 18:04 | |
*** ociuhandu has quit IRC | 18:08 | |
*** markvoelker has quit IRC | 18:14 | |
openstackgerrit | Lee Yarwood proposed openstack/cinder master: docs: Add os-migrate_volume_completion api-ref https://review.openstack.org/637847 | 18:20 |
*** ociuhandu has joined #openstack-cinder | 18:30 | |
*** mvkr has quit IRC | 18:36 | |
*** lpetrut has joined #openstack-cinder | 18:39 | |
*** e0ne has joined #openstack-cinder | 18:41 | |
*** e0ne has quit IRC | 18:42 | |
openstackgerrit | Merged openstack/cinder master: Remove drbdadm from filters https://review.openstack.org/611694 | 18:48 |
*** e0ne has joined #openstack-cinder | 18:54 | |
*** mvkr has joined #openstack-cinder | 19:07 | |
*** markvoelker has joined #openstack-cinder | 19:11 | |
openstackgerrit | Chuck Short proposed openstack/cinder master: Remove 'ploop' from rootwrap https://review.openstack.org/637864 | 19:15 |
*** lpetrut has quit IRC | 19:18 | |
openstackgerrit | Merged openstack/cinder stable/rocky: Fix for HPE MSA 2050 login failures https://review.openstack.org/636433 | 19:19 |
*** eharney has joined #openstack-cinder | 19:34 | |
*** sdake has joined #openstack-cinder | 19:34 | |
*** jmlowe has joined #openstack-cinder | 19:38 | |
*** e0ne has quit IRC | 19:39 | |
*** e0ne has joined #openstack-cinder | 19:39 | |
*** markvoelker has quit IRC | 19:44 | |
*** e0ne has quit IRC | 19:45 | |
*** sdake has quit IRC | 19:52 | |
*** sdake has joined #openstack-cinder | 19:55 | |
*** sdake has quit IRC | 19:58 | |
*** thgcorrea has quit IRC | 19:59 | |
erlon | smcginnis, hey, can you help me with a python/python3 question? | 20:04 |
smcginnis | erlon: Just going into a meeting, but I can try. What's up? | 20:04 |
erlon | Im trying to settp our CI to run python3, but the slaves still are Ubuntu 16.04 | 20:04 |
erlon | I manage to install devstack with python3, but when I try to run the cinder-tempest-pluginsm I get the following error | 20:05 |
erlon | https://gist.github.com/sombrafam/c1e3965c2ae2f9318c3d16e9aa11ff51#file-gistfile1-txt-L34 | 20:05 |
erlon | I ping you tomorrow them | 20:08 |
smcginnis | Sorry, will just take me a few minutes... | 20:08 |
*** openstackgerrit has quit IRC | 20:09 | |
*** luizbag has quit IRC | 20:11 | |
*** Florian has quit IRC | 20:12 | |
*** efried has quit IRC | 20:29 | |
*** e0ne has joined #openstack-cinder | 20:37 | |
*** markvoelker has joined #openstack-cinder | 20:41 | |
smcginnis | erlon: It looks like the plugin package is not being installed. | 20:45 |
smcginnis | It can't just be present on the filesystem under /opt/stack, it actually needs to be installed into the python environment via something like "pip install /opt/stack/cinder-tempest-plugin" or "cd /opt/stack/cinder-tempest-plugin && python setup.py install" | 20:46 |
erlon | smcginnis, found the problem. The problem was only in this specific test. It was importing a local file 'import manager as scn_manager'. I think that python3 (or tox3) does not support those kind of imports. It worked with 'import cinder_tempest_plugin.scenario.manager as scn_manager' | 20:51 |
smcginnis | Ah! I wonder if that was a leftover from when the tests were in the cinder tree or something. | 20:52 |
smcginnis | That definitely should be fixed like you say. | 20:52 |
smcginnis | Good catch | 20:52 |
*** erlon has quit IRC | 20:55 | |
*** jmlowe has quit IRC | 20:57 | |
*** eharney has quit IRC | 21:03 | |
*** markvoelker has quit IRC | 21:13 | |
*** whoami-rajat has quit IRC | 21:21 | |
*** e0ne has quit IRC | 21:28 | |
*** sdake has joined #openstack-cinder | 21:59 | |
*** enriquetaso has quit IRC | 22:01 | |
*** dave-mccowan has quit IRC | 22:07 | |
*** carlos_silva has quit IRC | 22:08 | |
*** sdake has quit IRC | 22:09 | |
*** markvoelker has joined #openstack-cinder | 22:10 | |
*** rcernin has joined #openstack-cinder | 22:26 | |
*** openstackgerrit has joined #openstack-cinder | 22:31 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove DataCore volume drivers https://review.openstack.org/638029 | 22:31 |
*** markvoelker has quit IRC | 22:44 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Doc8: Stop skipping D001: Line too long https://review.openstack.org/638037 | 22:53 |
*** tkajinam has joined #openstack-cinder | 22:55 | |
*** eharney has joined #openstack-cinder | 23:26 | |
*** ociuhandu has quit IRC | 23:30 | |
*** ociuhandu_ has joined #openstack-cinder | 23:30 | |
*** imacdonn has joined #openstack-cinder | 23:32 | |
*** imacdonn_ has joined #openstack-cinder | 23:32 | |
*** imacdonn_ has quit IRC | 23:32 | |
*** jmlowe has joined #openstack-cinder | 23:40 | |
*** markvoelker has joined #openstack-cinder | 23:41 | |
*** mriedem has quit IRC | 23:41 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!