*** sticker has joined #openstack-manila | 00:25 | |
*** gouthamr has quit IRC | 00:31 | |
*** jiaopengju has joined #openstack-manila | 00:43 | |
*** junboli has joined #openstack-manila | 00:52 | |
*** gouthamr has joined #openstack-manila | 01:10 | |
*** Saaxy has joined #openstack-manila | 01:14 | |
*** ganso has quit IRC | 01:28 | |
*** tinwood has quit IRC | 01:29 | |
*** tinwood has joined #openstack-manila | 01:30 | |
*** Saaxy has quit IRC | 01:38 | |
openstackgerrit | junbo.li proposed openstack/manila master: Api ref contains incorrect parameters. https://review.openstack.org/507441 | 02:32 |
---|---|---|
*** vivsoni has joined #openstack-manila | 02:36 | |
*** vivsoni has quit IRC | 02:50 | |
*** vivsoni_ has joined #openstack-manila | 02:50 | |
*** YuYangWang has joined #openstack-manila | 03:25 | |
*** YuYangWang2 has joined #openstack-manila | 03:33 | |
*** eharney has quit IRC | 03:49 | |
*** eharney has joined #openstack-manila | 03:50 | |
* zhongjun bswartz: ganso: is there ANY valid reason to have ipv4/ipv6_support NOT match the values returned by get_configed_ip_version() if yes, then the code stays the way it is. I would say yes. | 03:53 | |
zhongjun | bswartz: ganso: is there ANY valid reason to have ipv4/ipv6_support NOT match the values returned by get_configed_ip_version() if yes, then the code stays the way it is. I would say yes. | 03:54 |
zhongjun | bswartz: Maybe the ipv6_support is a little different than other capabilities. In other capabilities, if it set to true, it means they can support it. But the ipv6 capabilities is not just up to driver capacity, but also up to the value from the configuration. | 03:56 |
zhongjun | If you set ipv6_support to true and only configure the ipv4 in the driver configuration options, it won’t really support ipv6. Maybe we should not tell the scheduler about the driver support the ipv6. | 03:56 |
*** YuYangWang has quit IRC | 03:59 | |
*** vivsoni_ has quit IRC | 04:04 | |
*** vivsoni has joined #openstack-manila | 04:04 | |
*** eharney has quit IRC | 04:09 | |
*** vivsoni has quit IRC | 04:12 | |
bswartz | zhongjun: we haven't really talked about config options related to ipv6 support -- my theory is that none are needed | 04:24 |
bswartz | if the driver supports IPv6, and the backend has it configured, then the driver should tell that to the scheduler | 04:24 |
bswartz | if some vendor wanted a special config option to override the value they could add one | 04:25 |
bswartz | but the point I was getting it is that there are only 2 things that matter: | 04:25 |
bswartz | 1) should the share manager send ipv6 access rules to the driver? this is controlled by the ipv6_implemented flag | 04:25 |
bswartz | 2) should the share manager report ipv6 (and ipv4) support to the scheduler? this should be controlled by the get_configured_ip_version() method's return values | 04:26 |
bswartz | there is no need for a 3rd piece of information from the driver | 04:26 |
openstackgerrit | Rajat Sharma proposed openstack/manila master: Remove setting of version/release from releasenotes https://review.openstack.org/520880 | 04:39 |
*** sticker has quit IRC | 04:46 | |
openstackgerrit | junbo.li proposed openstack/manila master: Api ref contains incorrect parameters. https://review.openstack.org/507441 | 05:28 |
*** vivsoni has joined #openstack-manila | 05:30 | |
*** gouthamr has quit IRC | 05:34 | |
*** YuYangWang2 has quit IRC | 05:58 | |
*** wes_dillingham has quit IRC | 06:04 | |
openstackgerrit | junbo.li proposed openstack/manila master: Add ssl support for manila API access https://review.openstack.org/519206 | 06:15 |
*** lpetrut_ has joined #openstack-manila | 06:19 | |
*** jprovazn has joined #openstack-manila | 06:24 | |
*** jprovazn has quit IRC | 06:25 | |
*** belmoreira has joined #openstack-manila | 06:43 | |
*** pcaruana has joined #openstack-manila | 07:04 | |
*** markstur has joined #openstack-manila | 07:07 | |
*** markstur has quit IRC | 07:11 | |
zhongjun | bswartz: What I was trying to say is the original design that the get_configured_ip_version() method only care about the IP version from config options, and we also have ipv4/ipv6_support in driver update data dict by driver itself. Then we will check those two value in driver.py. We designed like that because of we thought there are two different concepts. | 07:12 |
zhongjun | bswartz: We could combine those two values, and just get the ipv4/ipv6_support value from the get_configured_ip_version() method's return values in driver.py | 07:13 |
*** rraja has joined #openstack-manila | 07:20 | |
*** lpetrut_ has quit IRC | 07:31 | |
*** lpetrut_ has joined #openstack-manila | 07:39 | |
openstackgerrit | zhongjun proposed openstack/manila master: Do not merge https://review.openstack.org/520994 | 07:45 |
*** AlexeyAbashkin has joined #openstack-manila | 08:01 | |
*** lpetrut_ has quit IRC | 08:11 | |
*** markstur has joined #openstack-manila | 08:14 | |
*** markstur has quit IRC | 08:18 | |
*** jmlowe has quit IRC | 08:27 | |
*** ianychoi_ is now known as ianychoi | 08:27 | |
*** vivsoni has quit IRC | 08:36 | |
*** lpetrut_ has joined #openstack-manila | 08:43 | |
*** vivsoni has joined #openstack-manila | 08:45 | |
*** belmoreira has quit IRC | 08:47 | |
*** lpetrut_ has quit IRC | 08:50 | |
*** lpetrut_ has joined #openstack-manila | 08:50 | |
openstackgerrit | junbo.li proposed openstack/manila master: Add ssl support for manila API access https://review.openstack.org/519206 | 08:52 |
*** lpetrut_ has quit IRC | 08:57 | |
zhongjun | tbarron: ping | 09:07 |
zhongjun | tbarron: It looks like our manila-tempest-minimal-dsvm-lvm-centos-7 gate always got error recently http://logs.openstack.org/94/520994/1/check/manila-tempest-minimal-dsvm-lvm-centos-7/ec413a7/job-output.txt.gz#_2017-11-17_08_22_13_050867 | 09:09 |
zhongjun | tbarron: Cloud you take a look | 09:09 |
*** a-pugachev has joined #openstack-manila | 09:18 | |
*** junboli has quit IRC | 09:36 | |
*** ganso has joined #openstack-manila | 09:53 | |
*** markstur has joined #openstack-manila | 10:14 | |
*** jiaopengju has quit IRC | 10:14 | |
*** markstur has quit IRC | 10:19 | |
*** lpetrut_ has joined #openstack-manila | 10:30 | |
openstackgerrit | Dongdong Pei proposed openstack/manila master: Remove unused functions from api/extensions.py https://review.openstack.org/521059 | 10:48 |
*** lpetrut_ has quit IRC | 11:14 | |
*** lpetrut has joined #openstack-manila | 11:15 | |
*** carthaca_ has joined #openstack-manila | 11:34 | |
*** dgonzalez_ has joined #openstack-manila | 11:34 | |
*** dgonzalez_ has quit IRC | 11:35 | |
*** carthaca_ has quit IRC | 11:35 | |
*** gcb has quit IRC | 12:47 | |
openstackgerrit | Merged openstack/manila master: Api ref contains incorrect parameters. https://review.openstack.org/507441 | 12:51 |
tbarron | zhongjun: yes, centos 7 jobs are failing. I brought the issue up with #openstack-infra last night and ianw has proposed this fix: https://review.openstack.org/#/c/520837 | 12:58 |
tbarron | zhongjun: I just asked for reviews in #openstack-infra explaining that our gate is blocked | 13:03 |
*** wes_dillingham has joined #openstack-manila | 13:23 | |
zhongjun | tbarron: Good! Thank you! | 13:58 |
tbarron | zhongjun: thank you for noticing, I was going to mention it in yesterday's meeting but got distracted. | 14:00 |
*** dustins has joined #openstack-manila | 14:04 | |
*** gouthamr has joined #openstack-manila | 14:10 | |
*** netapp has joined #openstack-manila | 14:34 | |
openstackgerrit | junbo.li proposed openstack/manila master: Add ssl support for manila API access https://review.openstack.org/519206 | 14:53 |
*** eharney has joined #openstack-manila | 15:08 | |
openstackgerrit | junbo.li proposed openstack/python-manilaclient master: Switch to use stestr instead of testr https://review.openstack.org/518681 | 15:19 |
*** markstur has joined #openstack-manila | 15:45 | |
*** rraja has quit IRC | 15:55 | |
*** AlexeyAbashkin has quit IRC | 16:05 | |
openstackgerrit | Amit Oren proposed openstack/manila master: INFINIDAT add Manila driver https://review.openstack.org/515432 | 17:34 |
*** a-pugachev has quit IRC | 17:40 | |
*** sapcc-bot2 has quit IRC | 18:12 | |
*** sapcc-bot has joined #openstack-manila | 18:12 | |
*** dustins has quit IRC | 18:23 | |
openstackgerrit | Tom Barron proposed openstack/manila-specs master: Remove unit test targets https://review.openstack.org/521156 | 18:41 |
*** dustins has joined #openstack-manila | 19:09 | |
*** lpetrut has quit IRC | 19:09 | |
tbarron | devstack review https://review.openstack.org/520837 has merged so centos7 jobs should pass again | 19:10 |
tbarron | if your reviews were failing on the lvm job, recheck :D | 19:10 |
*** rraja has joined #openstack-manila | 19:18 | |
*** gouthamr has quit IRC | 19:35 | |
zigo | Hi there! When building the manila-ui package for Pike in Debian, I get this: http://paste.openstack.org/show/626655/ | 19:40 |
zigo | What's going on, and can someone help? | 19:40 |
dustins | zigo: That's an odd one, I'm guessing that you're trying to run the unit tests for the Manila UI plugin? | 19:46 |
bswartz | zigo: there have been some recent changes to requirements -- it's possible that manila-ui was affected | 19:50 |
bswartz | I wouldn't be surprised if there's an implicit requirement that needs to be made explicit | 19:50 |
zigo | dustins: Yes, when building the manila-ui package for Debian. | 19:51 |
dustins | bswartz: Probably, and I don't know when the manila-ui unit tests were last updated with those requirements | 19:52 |
zigo | bswartz: Well, it needs at least python-cffi that wasn't declared. | 19:52 |
bswartz | I notice that the stack trace you posted has no actual manila-ui modules | 19:53 |
bswartz | so it seems likely there's a test infrastructure bug | 19:53 |
zigo | bswartz: The stack trace is *not* in "infra", it's when doing gbp-buildpackage (ie: building the Debian package). | 20:03 |
*** lpetrut has joined #openstack-manila | 20:06 | |
*** vivsoni has quit IRC | 20:09 | |
*** vivsoni has joined #openstack-manila | 20:09 | |
zigo | bswartz: dustins: Oh, also, that's manila-ui 2.10.2, the Pike release... | 20:12 |
zigo | So no requirements recent change. | 20:13 |
dustins | zigo: What bswartz said still raises a good point, that it looks like it's not getting to the plugin code itself | 20:19 |
bswartz | zigo: the UI plugin is unfortunately the part of Manila I know least about | 20:28 |
bswartz | the testing of it I know almost nothing about | 20:28 |
bswartz | we generally follow the pattern of other horizon plugins | 20:29 |
dustins | I've done some testing of it, but only after it's been installed and working :( | 20:29 |
bswartz | so if we're missing something, it's possible we can figure it out by looking at the differences between manila-ui and other horizon plugins | 20:29 |
dustins | bswartz: Correct me if I'm wrong, but do we run the manila-ui unit tests in our gates? | 20:29 |
zigo | :/ | 20:29 |
bswartz | dustins: manila-ui has it's own gate tests | 20:30 |
bswartz | I checked, and the rest that zigo referred to is in fact running in our gate and passes as recently as yesterday | 20:30 |
bswartz | s/rest/test/ | 20:30 |
zigo | I hated the decision from the Horizon team to not take care of all projects. Most project contributors know nearly nothing about Django and such, and I'm not surprised by what you're saying. | 20:30 |
dustins | Yeah... | 20:31 |
zigo | I will ask in the Horizon channel, let's see if they can help, they are generally very nice and helpful. | 20:31 |
dustins | zigo: Sorry we weren't of more help, let us know if you find anything | 20:31 |
dustins | We should know more and be able to help in scenarios like this | 20:31 |
zigo | dustins: Don't worry, I understand it's not easy to know everything, and Django is one nasty beast. | 20:32 |
dustins | zigo: So I've heard, but it's not untamable either ;) | 20:33 |
*** a-pugachev has joined #openstack-manila | 20:36 | |
*** markstur has quit IRC | 20:42 | |
*** rraja has quit IRC | 20:43 | |
*** eharney has quit IRC | 20:51 | |
*** jmlowe has joined #openstack-manila | 20:55 | |
*** jmlowe has quit IRC | 21:01 | |
*** wes_dillingham has quit IRC | 21:03 | |
*** vivsoni has quit IRC | 21:13 | |
*** vivsoni has joined #openstack-manila | 21:14 | |
*** AlexeyAbashkin has joined #openstack-manila | 21:19 | |
*** ganso has quit IRC | 21:20 | |
*** AlexeyAbashkin has quit IRC | 21:24 | |
dustins | zhongjun: I put together some bugs to crush for the bug squash next week: https://etherpad.openstack.org/p/manila-bug-triage-pad | 21:29 |
dustins | Hope this helps out :D | 21:29 |
*** eharney has joined #openstack-manila | 21:42 | |
*** a-pugachev has quit IRC | 21:51 | |
*** dustins has quit IRC | 22:03 | |
*** wes_dillingham has joined #openstack-manila | 22:23 | |
*** hoonetorg has quit IRC | 22:28 | |
*** wes_dillingham has quit IRC | 22:32 | |
*** lpetrut has quit IRC | 22:57 | |
*** wes_dillingham has joined #openstack-manila | 23:01 | |
*** gouthamr has joined #openstack-manila | 23:33 | |
*** gouthamr_ has joined #openstack-manila | 23:36 | |
*** gouthamr has quit IRC | 23:38 | |
*** pcaruana has quit IRC | 23:54 | |
*** wes_dillingham has quit IRC | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!