*** browne has quit IRC | 00:02 | |
*** dtardivel has quit IRC | 00:07 | |
*** mriedem has joined #openstack-oslo | 00:12 | |
*** tonytan4ever has joined #openstack-oslo | 00:24 | |
*** jamielennox is now known as jamielennox|away | 00:29 | |
*** tonytan4ever has quit IRC | 00:30 | |
*** jamielennox|away is now known as jamielennox | 00:33 | |
openstackgerrit | Anh Tran proposed openstack/oslo.messaging: Replace 'the' with 'to' in docstring https://review.openstack.org/378149 | 00:51 |
---|---|---|
openstackgerrit | Merged openstack/oslo.messaging: Remove default=None when set value in Config https://review.openstack.org/377213 | 01:11 |
*** mtanino has quit IRC | 01:16 | |
*** mriedem has quit IRC | 01:29 | |
*** ekcs has quit IRC | 01:37 | |
*** jamielennox is now known as jamielennox|away | 01:42 | |
*** jeckersb has quit IRC | 01:59 | |
*** jeckersb has joined #openstack-oslo | 01:59 | |
*** salv-orlando has joined #openstack-oslo | 02:01 | |
*** salv-orlando has quit IRC | 02:05 | |
*** jeckersb is now known as jeckersb_gone | 02:12 | |
*** jeckersb_gone is now known as jeckersb | 02:15 | |
*** yamamoto_ has joined #openstack-oslo | 02:59 | |
*** david-lyle has quit IRC | 03:04 | |
*** jamielennox|away is now known as jamielennox | 03:11 | |
*** links has joined #openstack-oslo | 03:33 | |
gerhardqux | harlowja: it would include context such as request-id and instance-id | 03:36 |
*** syjulian has quit IRC | 03:46 | |
*** hoonetorg has quit IRC | 03:47 | |
*** pcaruana has quit IRC | 03:47 | |
*** xek__ has quit IRC | 03:47 | |
*** d0ugal has quit IRC | 03:48 | |
*** eeiden has quit IRC | 03:48 | |
*** Kiall has quit IRC | 03:49 | |
*** Kiall has joined #openstack-oslo | 03:50 | |
*** eeiden has joined #openstack-oslo | 03:52 | |
*** hoonetorg has joined #openstack-oslo | 03:53 | |
*** syjulian has joined #openstack-oslo | 03:54 | |
*** d0ugal has joined #openstack-oslo | 03:57 | |
*** pcaruana has joined #openstack-oslo | 03:59 | |
*** lifeless has quit IRC | 04:02 | |
*** salv-orlando has joined #openstack-oslo | 04:03 | |
*** lifeless has joined #openstack-oslo | 04:05 | |
*** gcb has quit IRC | 04:06 | |
*** salv-orlando has quit IRC | 04:07 | |
*** gcb has joined #openstack-oslo | 04:08 | |
*** dkehn_ has quit IRC | 04:12 | |
gerhardqux | harlowja: ehhh, I think you're right | 04:15 |
gerhardqux | not sure how I missed that | 04:16 |
*** Rockyg has quit IRC | 04:19 | |
*** salv-orlando has joined #openstack-oslo | 04:42 | |
*** salv-orlando has quit IRC | 04:42 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/373775 | 04:42 |
*** salv-orlando has joined #openstack-oslo | 04:43 | |
*** amotoki has joined #openstack-oslo | 04:57 | |
*** amotoki has quit IRC | 05:02 | |
*** yamamoto_ has quit IRC | 05:04 | |
*** yamahata has joined #openstack-oslo | 05:05 | |
*** ozamiatin has joined #openstack-oslo | 05:13 | |
*** amotoki has joined #openstack-oslo | 05:17 | |
*** swatson__ has quit IRC | 05:18 | |
*** nkrinner_afk is now known as nkrinner | 05:27 | |
*** e0ne has joined #openstack-oslo | 05:41 | |
*** yamamoto_ has joined #openstack-oslo | 05:46 | |
*** xek has joined #openstack-oslo | 05:59 | |
*** ozamiatin has quit IRC | 06:05 | |
*** rcernin has joined #openstack-oslo | 06:22 | |
*** lpetrut has joined #openstack-oslo | 06:37 | |
*** dukhlov has joined #openstack-oslo | 06:43 | |
*** mhickey has joined #openstack-oslo | 06:48 | |
*** dtardivel has joined #openstack-oslo | 06:53 | |
openstackgerrit | Merged openstack/oslo.serialization: Updated from global requirements https://review.openstack.org/371119 | 06:58 |
openstackgerrit | Merged openstack/oslo.db: Updated from global requirements https://review.openstack.org/377533 | 07:01 |
*** lpetrut has quit IRC | 07:31 | |
*** salv-orlando has quit IRC | 07:37 | |
openstackgerrit | Merged openstack/oslo.messaging: [zmq] Remove unused methods from executors https://review.openstack.org/374964 | 07:44 |
*** eliqiao has quit IRC | 07:46 | |
*** salv-orlando has joined #openstack-oslo | 07:50 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-oslo | 08:00 | |
*** yassine has joined #openstack-oslo | 08:11 | |
*** yassine is now known as Guest42059 | 08:11 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/oslo.utils: Add method is_valid_ipv6_cidr https://review.openstack.org/376206 | 08:13 |
*** Guest42059 has quit IRC | 08:14 | |
*** yassine_ has joined #openstack-oslo | 08:14 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/oslo.utils: Add method is_valid_boolstr https://review.openstack.org/379261 | 08:15 |
*** cdent has joined #openstack-oslo | 08:19 | |
*** yamahata has quit IRC | 08:49 | |
*** jpena|off is now known as jpena | 08:57 | |
*** ihrachys has joined #openstack-oslo | 09:01 | |
*** ozamiatin has joined #openstack-oslo | 09:06 | |
*** zz_dimtruck has quit IRC | 09:10 | |
*** wendar has quit IRC | 09:10 | |
*** sambetts|afk is now known as sambetts | 09:18 | |
*** mhickey has quit IRC | 09:26 | |
openstackgerrit | Merged openstack/oslo.policy: doc: Add introduction to index page https://review.openstack.org/378490 | 09:33 |
*** lpetrut has joined #openstack-oslo | 09:34 | |
*** eliqiao has joined #openstack-oslo | 09:35 | |
*** code-R has joined #openstack-oslo | 09:39 | |
*** code-R_ has joined #openstack-oslo | 09:41 | |
*** ozamiatin_ has joined #openstack-oslo | 09:41 | |
openstackgerrit | Merged openstack/automaton: Updated from global requirements https://review.openstack.org/377423 | 09:43 |
*** code-R has quit IRC | 09:44 | |
*** code-R_ has quit IRC | 09:54 | |
*** code-R has joined #openstack-oslo | 09:55 | |
openstackgerrit | Merged openstack/oslo.messaging: Set the valid choices for the rabbit login methods https://review.openstack.org/372810 | 09:55 |
*** amotoki has quit IRC | 10:06 | |
*** dukhlov has quit IRC | 10:09 | |
*** dukhlov has joined #openstack-oslo | 10:10 | |
*** e0ne has quit IRC | 10:16 | |
*** ozamiatin_ has quit IRC | 10:19 | |
*** boden has joined #openstack-oslo | 10:34 | |
*** nkrinner has quit IRC | 10:36 | |
*** ihrachys has quit IRC | 10:37 | |
*** yamamoto_ has quit IRC | 10:40 | |
*** salv-orlando has quit IRC | 10:40 | |
*** salv-orlando has joined #openstack-oslo | 10:41 | |
gcb | dhellmann, I rewrite commit message in https://review.openstack.org/#/c/375871/, please revisit it when you're free . | 10:41 |
*** nkrinner has joined #openstack-oslo | 10:42 | |
*** nkrinner has quit IRC | 10:48 | |
*** nkrinner has joined #openstack-oslo | 10:48 | |
lpetrut | gcb: Hi | 10:49 |
gcb | hi | 10:49 |
lpetrut | gcb: we have an issue with the process resource limiting feature exposed by the processutils.execute function: it's not supported on Windows :) The fact that Cinder started using this flag when calling qemu-img broke our Cinder drivers. So, my proposal consists in ignoring this argument on Windows, logging a meaningful message for now: https://review.openstack.org/#/c/376622/ | 10:49 |
lpetrut | gcb: could you please take a look whenever you have time? | 10:50 |
gcb | lpetrut, will check that :-) | 10:50 |
*** salv-orlando has quit IRC | 10:54 | |
lpetrut | gcb: thank you | 10:55 |
openstackgerrit | Merged openstack/oslo.log: Updated from global requirements https://review.openstack.org/378874 | 11:00 |
openstackgerrit | Merged openstack/oslo.config: Updated from global requirements https://review.openstack.org/378873 | 11:00 |
openstackgerrit | Merged openstack/oslo.utils: Updated from global requirements https://review.openstack.org/378877 | 11:01 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/oslo.utils: Add method is_valid_boolstr https://review.openstack.org/379261 | 11:18 |
*** shardy has joined #openstack-oslo | 11:18 | |
*** yamamoto has joined #openstack-oslo | 11:18 | |
*** yamamoto_ has joined #openstack-oslo | 11:18 | |
*** yamamoto has quit IRC | 11:18 | |
*** openstackgerrit has quit IRC | 11:19 | |
*** openstackgerrit has joined #openstack-oslo | 11:19 | |
*** rtheis has joined #openstack-oslo | 11:21 | |
*** nkrinner has quit IRC | 11:26 | |
*** nkrinner has joined #openstack-oslo | 11:31 | |
*** lucas-afk is now known as lucasagomes | 11:33 | |
*** nkrinner has quit IRC | 11:37 | |
*** jpena is now known as jpena|lunch | 11:43 | |
*** nkrinner has joined #openstack-oslo | 11:49 | |
*** rohanion has joined #openstack-oslo | 11:51 | |
*** ozamiatin_ has joined #openstack-oslo | 11:59 | |
*** takedakn has joined #openstack-oslo | 12:02 | |
*** kgiusti has joined #openstack-oslo | 12:11 | |
*** kgiusti has quit IRC | 12:15 | |
*** edmondsw has joined #openstack-oslo | 12:15 | |
*** ihrachys has joined #openstack-oslo | 12:25 | |
*** takedakn1 has joined #openstack-oslo | 12:25 | |
*** salv-orlando has joined #openstack-oslo | 12:25 | |
*** takedakn has quit IRC | 12:26 | |
*** takedakn1 is now known as takedakn | 12:26 | |
*** salv-orlando has quit IRC | 12:30 | |
*** kgiusti has joined #openstack-oslo | 12:32 | |
*** mriedem has joined #openstack-oslo | 12:33 | |
*** gcb has quit IRC | 12:41 | |
*** shardy has quit IRC | 12:48 | |
*** e0ne has joined #openstack-oslo | 12:50 | |
*** yamamoto_ has quit IRC | 12:54 | |
*** david-lyle has joined #openstack-oslo | 12:57 | |
*** takedakn has quit IRC | 12:57 | |
*** takedakn has joined #openstack-oslo | 12:58 | |
*** jpena|lunch is now known as jpena | 12:59 | |
*** ozamiatin_ has quit IRC | 13:01 | |
openstackgerrit | Lucian Petrut proposed openstack/oslo.concurrency: Ignore prlimit argument on Windows https://review.openstack.org/376622 | 13:07 |
*** takedakn has quit IRC | 13:13 | |
*** yamamoto has joined #openstack-oslo | 13:16 | |
*** links has quit IRC | 13:17 | |
*** salv-orlando has joined #openstack-oslo | 13:27 | |
*** mriedem has quit IRC | 13:28 | |
*** salv-orlando has quit IRC | 13:31 | |
*** mtanino has joined #openstack-oslo | 13:32 | |
*** takedakn has joined #openstack-oslo | 13:37 | |
*** takedakn has quit IRC | 13:50 | |
*** dkehn has joined #openstack-oslo | 13:54 | |
*** ozamiatin_ has joined #openstack-oslo | 13:54 | |
*** spzala has joined #openstack-oslo | 13:56 | |
*** yamamoto has quit IRC | 13:57 | |
*** dukhlov has quit IRC | 13:58 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: [zmq] Monitor connections/disconnections to/from proxies https://review.openstack.org/377657 | 13:58 |
*** rohanion has quit IRC | 14:00 | |
*** mriedem has joined #openstack-oslo | 14:02 | |
*** szaher_ has quit IRC | 14:04 | |
*** yamamoto has joined #openstack-oslo | 14:05 | |
*** szaher has joined #openstack-oslo | 14:05 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Routing table refactoring, dynamic direct connections https://review.openstack.org/372001 | 14:16 |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Routing table refactoring, dynamic direct connections https://review.openstack.org/372001 | 14:21 |
*** yamamoto has quit IRC | 14:22 | |
*** yamamoto has joined #openstack-oslo | 14:23 | |
*** yamamoto has quit IRC | 14:27 | |
*** yamamoto has joined #openstack-oslo | 14:27 | |
*** yamamoto has quit IRC | 14:27 | |
*** salv-orlando has joined #openstack-oslo | 14:29 | |
*** yamahata has joined #openstack-oslo | 14:33 | |
*** amotoki has joined #openstack-oslo | 14:38 | |
*** gordc has joined #openstack-oslo | 14:46 | |
*** amotoki has quit IRC | 14:48 | |
*** gcb has joined #openstack-oslo | 14:51 | |
gcb | bnemec, could you look at https://review.openstack.org/#/c/376206/ again ? thanks | 14:52 |
*** jamielennox is now known as jamielennox|away | 14:53 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/oslo.utils: Add method is_valid_boolstr https://review.openstack.org/379261 | 14:59 |
bnemec | gcb: Heh, I had it open yesterday to point out the typo and forgot to send the comment. I see you caught it in the meantime. :-) | 14:59 |
bnemec | Is stable/newton branched in oslo? | 14:59 |
gcb | for master branch | 14:59 |
bnemec | It looks like it, but since I'm a little out of the loop I didn't want to merge it until I was sure. | 14:59 |
gcb | sorry, yes we cut the stable/newton branch | 15:00 |
*** kgiusti has quit IRC | 15:00 | |
*** jamielennox|away is now known as jamielennox | 15:00 | |
gcb | this is for master branch and will be released after newton | 15:00 |
bnemec | Cool | 15:00 |
*** thumpba has joined #openstack-oslo | 15:01 | |
gcb | it's safe to merge more code in oslo now | 15:01 |
gcb | bnemec, thanks btw would you attend the next summit ? | 15:03 |
bnemec | gcb: Yep, I'll be there. | 15:03 |
gcb | bnemec, cool I'll be there too, will be glad to see you again :-) | 15:04 |
bnemec | gcb: Likewise. :-) | 15:05 |
*** thumpba has quit IRC | 15:11 | |
*** lucasagomes is now known as lucas-hungry | 15:12 | |
*** thumpba has joined #openstack-oslo | 15:12 | |
gcb | lxsli , please look at https://review.openstack.org/#/c/375871/ when you're free :-), thanks | 15:13 |
*** amotoki has joined #openstack-oslo | 15:14 | |
*** ozamiatin_ has quit IRC | 15:14 | |
lxsli | gcb: ok, I'd ignored this until now because I didn't understand the commit message tbh | 15:16 |
lxsli | I think I get it now but the obvious question is - if no one's using it, why don't we just delete it? | 15:17 |
gcb | lxsli, consuming projects will use this one (after my fix) and don't use specific one for each project | 15:18 |
lxsli | gcb: ah so there is code like this already in other projects? | 15:18 |
lxsli | gcb: also might be better to test with version as a string, if we pass a version like "2.0" it could get weird | 15:19 |
lxsli | "2_0" might be better? | 15:19 |
*** swatson__ has joined #openstack-oslo | 15:19 | |
lxsli | I don't know, maybe that'd be OK | 15:19 |
*** amotoki has quit IRC | 15:20 | |
gcb | please see Doug Hellmann's comments , I can add test for string '2_0' :-) | 15:20 |
gcb | current projects use their specific version ( implemented in each project), will replace them with oslo ones | 15:21 |
gcb | one | 15:21 |
lxsli | ok let me check those comments | 15:21 |
gcb | lxsli, thanks | 15:23 |
*** yamahata has quit IRC | 15:24 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/oslo.utils: Add method is_valid_boolstr https://review.openstack.org/379261 | 15:26 |
*** amotoki has joined #openstack-oslo | 15:27 | |
*** yamamoto has joined #openstack-oslo | 15:28 | |
*** amotoki has quit IRC | 15:28 | |
*** yamamoto has quit IRC | 15:33 | |
*** amotoki has joined #openstack-oslo | 15:34 | |
*** thumpba has quit IRC | 15:34 | |
*** amotoki has quit IRC | 15:36 | |
*** swatson__ has quit IRC | 15:43 | |
*** tonytan4ever has joined #openstack-oslo | 15:45 | |
*** openstackgerrit has quit IRC | 15:49 | |
*** openstackgerrit has joined #openstack-oslo | 15:50 | |
openstackgerrit | Lucian Petrut proposed openstack/oslo.concurrency: Ignore prlimit argument on Windows https://review.openstack.org/376622 | 15:56 |
*** yamahata has joined #openstack-oslo | 15:57 | |
openstackgerrit | Merged openstack/oslo.utils: Add method is_valid_ipv6_cidr https://review.openstack.org/376206 | 16:02 |
*** pcaruana has quit IRC | 16:03 | |
*** rcernin has quit IRC | 16:05 | |
gcb | harlowja, my boss, please look at a simple replacement in tooz https://review.openstack.org/#/c/375920/ :-) | 16:06 |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: [zmq] Fix send_cast in AckManager https://review.openstack.org/379562 | 16:11 |
*** cdent has quit IRC | 16:15 | |
*** lucas-hungry is now known as lucasagomes | 16:15 | |
*** nkrinner is now known as nkrinner_afk | 16:19 | |
*** gcb has quit IRC | 16:19 | |
*** lifeless has quit IRC | 16:25 | |
*** e0ne has quit IRC | 16:30 | |
*** gcb has joined #openstack-oslo | 16:31 | |
*** lifeless has joined #openstack-oslo | 16:33 | |
*** coreycb has joined #openstack-oslo | 16:34 | |
*** gcb has quit IRC | 16:50 | |
*** browne has joined #openstack-oslo | 16:54 | |
*** eezhova has quit IRC | 16:55 | |
*** amotoki has joined #openstack-oslo | 16:57 | |
*** amotoki has quit IRC | 17:03 | |
*** ihrachys has quit IRC | 17:05 | |
*** code-R has quit IRC | 17:07 | |
*** kgiusti has joined #openstack-oslo | 17:12 | |
openstackgerrit | Merged openstack/oslo.utils: Add method is_valid_boolstr https://review.openstack.org/379261 | 17:22 |
*** jpena is now known as jpena|off | 17:23 | |
coreycb | fungi, hi | 17:45 |
coreycb | fungi, I'm looking at this and trying to figure out if only oslo.config gets security support: https://governance.openstack.org/reference/projects/oslo.html#oslo-privsep | 17:46 |
coreycb | fungi, ie. 'vulnerability:managed' tag | 17:46 |
*** e0ne has joined #openstack-oslo | 17:48 | |
coreycb | fungi, the problem I'm running into is our distro security team is hesitant to support a package if upstream doesn't provide security support for it | 17:48 |
coreycb | oslo.privsep is the package I'm trying to get security support for | 17:49 |
fungi | coreycb: security "support" is on the individual project developer teams to provide, though if it's tagged as "vulnerability:managed" then our central vulnerability management team provides oversight and imposes a more strict/rigorous process for their disclosure and reporting | 17:50 |
fungi | in the end though, it still winds up being the developers on each project who produce the security fixes and backports | 17:50 |
coreycb | fungi, ok that clears it up, thanks | 17:51 |
fungi | coreycb: the vmt's process is documented at https://security.openstack.org/vmt-process.html and we encourage teams to follow it even for deliverables without vulnerability:managed | 17:52 |
fungi | and we also still consult with teams on issues the vmt isn't overseeing, as time permits | 17:53 |
coreycb | fungi, ok so in the end if a CVE is reported for say, oslo.privsep, you'd expect the oslo.privsep developers to handle that appropriately and consult with the VMT if needed | 17:54 |
fungi | coreycb: correct | 17:57 |
coreycb | fungi, it does seem like there should be an effort for security sensitive projects to be tagged 'vulnerability:managed', like oslo.privsep. but i imagine it's also a resource issue as everything is. | 17:57 |
fungi | though generally a cve wouldn't be reported to the oslo.privsep team so much as a suspected vulnerability would be reported and then they would request a cve assignment for it | 17:57 |
fungi | coreycb: sure, oslo.privsep is relatively new too, so i think its authors simply haven't sought that level of assistance out yet | 17:58 |
fungi | coreycb: and we're still in the last stages of hashing out a threat analysis process for deliverables that want direct vmt overisght. we used to personally review the source of each before taking it on, but as the project has scaled up that's not something a handful of already busy people can tackle so we're trying to collaborate on a way that projects can work together on analyzing themselves and | 17:59 |
fungi | each other | 18:00 |
coreycb | fungi, I completely understand that | 18:00 |
coreycb | fungi, thanks for the discussion | 18:01 |
fungi | the openstack security team (of which the vmt is only a small part) have embarked on a http://git.openstack.org/cgit/openstack/security-analysis/ repo where standardized analyses can be curated | 18:01 |
fungi | which will make it easier for the vmt to be directly involved in reported vulnerabilities for more deliverables in the future | 18:01 |
fungi | coreycb: my pleasure. i'm happy to answer any other related questions should they arise | 18:03 |
coreycb | fungi, appreciate it! | 18:03 |
openstackgerrit | Lucian Petrut proposed openstack/oslo.concurrency: Ignore prlimit argument on Windows https://review.openstack.org/376622 | 18:10 |
openstackgerrit | Merged openstack/tooz: Use method ensure_tree from oslo.utils https://review.openstack.org/375920 | 18:14 |
*** lpetrut has quit IRC | 18:21 | |
openstackgerrit | Merged openstack/oslo.messaging: Replace 'the' with 'to' in docstring https://review.openstack.org/378149 | 18:22 |
*** dtardivel has quit IRC | 18:37 | |
*** spzala has quit IRC | 18:40 | |
*** spzala has joined #openstack-oslo | 18:41 | |
*** mhickey has joined #openstack-oslo | 18:45 | |
*** spzala has quit IRC | 18:46 | |
*** ihrachys has joined #openstack-oslo | 18:51 | |
*** Rockyg has joined #openstack-oslo | 18:53 | |
*** code-R has joined #openstack-oslo | 18:53 | |
*** ihrachys has quit IRC | 18:55 | |
*** salv-orlando has quit IRC | 18:56 | |
*** code-R_ has joined #openstack-oslo | 18:56 | |
*** code-R has quit IRC | 18:59 | |
*** yamahata has quit IRC | 19:01 | |
*** cdent has joined #openstack-oslo | 19:02 | |
*** lpetrut has joined #openstack-oslo | 19:10 | |
*** salv-orlando has joined #openstack-oslo | 19:18 | |
*** ihrachys has joined #openstack-oslo | 19:31 | |
*** flaper87 has quit IRC | 19:43 | |
*** purplerbot has quit IRC | 19:43 | |
*** salv-orl_ has joined #openstack-oslo | 19:44 | |
*** purplerbot has joined #openstack-oslo | 19:44 | |
*** toabctl_ has joined #openstack-oslo | 19:46 | |
*** salv-orlando has quit IRC | 19:47 | |
*** code-R_ has quit IRC | 19:49 | |
*** toabctl has quit IRC | 19:50 | |
*** toabctl_ is now known as toabctl | 19:50 | |
*** lucasagomes is now known as lucas-afk | 19:54 | |
openstackgerrit | John L. Villalovos proposed openstack/oslo.serialization: Add a title to the API Documentation page https://review.openstack.org/379790 | 19:54 |
*** ihrachys has quit IRC | 19:56 | |
*** spzala_ has joined #openstack-oslo | 19:57 | |
*** lpetrut has quit IRC | 20:01 | |
*** spzala_ has quit IRC | 20:01 | |
*** spzala has joined #openstack-oslo | 20:03 | |
njohnston | Hi! Is this the right channel to ask questions about Stevedore? | 20:06 |
*** yassine_ has quit IRC | 20:06 | |
dhellmann | njohnston : yes | 20:06 |
dhellmann | what's up? | 20:06 |
*** code-R has joined #openstack-oslo | 20:07 | |
njohnston | dhellmann: Well, I am trying to write a neutron fullstack test that tests the neutron agent extension pattern, and I am getting this error: 2016-09-29 19:35:09.629 29805 WARNING stevedore.named [req-c4847289-ab40-4d67-a8a2-9d9b201974a1 - - - - -] Could not load neutron.tests.fullstack.dummy_l3_extension.DummyL3Extension | 20:07 |
dhellmann | njohnston : do you have a link to the review so I can look at the code? | 20:08 |
njohnston | I haven't been able to figure out *why* stevedore can't load that extension. Do you have any suggestions for figuring that out? | 20:08 |
njohnston | dhellmann: Sure thing https://review.openstack.org/345631 | 20:09 |
dhellmann | looking | 20:09 |
njohnston | It's super-WIP, so please excuse it's rough edges | 20:10 |
njohnston | and thanks very much, too :-) | 20:10 |
dhellmann | njohnston : it looks like the plugin you're trying to use isn't registered | 20:10 |
dhellmann | http://docs.openstack.org/developer/stevedore/tutorial/creating_plugins.html#registering-the-plugins | 20:10 |
dhellmann | though for a test plugin, you probably don't want to register it and you want to use a mock with a test manager instead | 20:10 |
dhellmann | http://docs.openstack.org/developer/stevedore/managers.html#stevedore.named.NamedExtensionManager.make_test_instance | 20:11 |
dhellmann | you can build a test instance of a manager that you populate in your test code, and then use mock.patch() to replace the function that builds the real manager with one that returns your test instance | 20:12 |
dhellmann | that works for unit-style tests, where everything is happening in the test process | 20:12 |
dhellmann | you should use the make_test_instance() method of the type of manager you're replacing, obviously | 20:12 |
dhellmann | njohnston : make sense? | 20:13 |
dhellmann | unfortunately I'm not finding an example of doing this in any documentation | 20:13 |
*** yamahata has joined #openstack-oslo | 20:13 | |
njohnston | I think so. This is somewhat complicated because this is a fullstack test, so I am trying to replicate a scenario where I have neutron server -> RPC -> neutron agent -> stevedore -> dummy agent extension... so I am not actually mocking anything in the process that calls stevedore | 20:15 |
dhellmann | oh, well, in that case you're probably going to have to register the extension properly | 20:15 |
njohnston | ok | 20:15 |
*** kgiusti has left #openstack-oslo | 20:15 | |
njohnston | dhellmann: Thanks very much, that really helps! | 20:16 |
dhellmann | njohnston : sure thing, let me know if you run into trouble with any of those docs so we can improve them | 20:17 |
*** cburgess_ has quit IRC | 20:36 | |
*** browne has quit IRC | 20:37 | |
*** edmondsw has quit IRC | 20:37 | |
*** tonytan4ever has quit IRC | 20:38 | |
*** cburgess has joined #openstack-oslo | 20:38 | |
*** spzala has quit IRC | 20:43 | |
*** fnordahl has quit IRC | 20:43 | |
*** fnordahl_ has joined #openstack-oslo | 20:43 | |
*** spzala has joined #openstack-oslo | 20:43 | |
*** melwitt has quit IRC | 20:44 | |
*** spzala has quit IRC | 20:48 | |
*** spzala has joined #openstack-oslo | 20:49 | |
*** melwitt has joined #openstack-oslo | 20:51 | |
*** code-R has quit IRC | 20:51 | |
*** melwitt is now known as Guest99228 | 20:52 | |
*** spzala has quit IRC | 20:54 | |
*** spzala has joined #openstack-oslo | 20:57 | |
*** jpena|off has quit IRC | 21:00 | |
*** amotoki has joined #openstack-oslo | 21:00 | |
*** jpena|off has joined #openstack-oslo | 21:02 | |
*** spzala has quit IRC | 21:02 | |
*** mhickey has quit IRC | 21:02 | |
*** e0ne has quit IRC | 21:05 | |
*** amotoki has quit IRC | 21:05 | |
*** spzala has joined #openstack-oslo | 21:07 | |
*** e0ne has joined #openstack-oslo | 21:08 | |
*** sabari has left #openstack-oslo | 21:20 | |
*** yassine_ has joined #openstack-oslo | 21:20 | |
*** yassine_ has quit IRC | 21:21 | |
*** rtheis has quit IRC | 21:22 | |
*** tonytan4ever has joined #openstack-oslo | 21:38 | |
*** tonytan4ever has quit IRC | 21:44 | |
*** e0ne has quit IRC | 21:45 | |
*** e0ne has joined #openstack-oslo | 21:45 | |
*** e0ne has quit IRC | 21:45 | |
*** e0ne has joined #openstack-oslo | 21:46 | |
*** yamahata has quit IRC | 21:46 | |
*** e0ne has quit IRC | 21:46 | |
*** e0ne has joined #openstack-oslo | 21:47 | |
*** e0ne has quit IRC | 21:47 | |
*** e0ne has joined #openstack-oslo | 21:47 | |
*** e0ne has quit IRC | 21:48 | |
*** e0ne has joined #openstack-oslo | 21:48 | |
*** e0ne has quit IRC | 21:49 | |
*** e0ne has joined #openstack-oslo | 21:49 | |
*** e0ne has quit IRC | 21:49 | |
*** cdent has quit IRC | 21:51 | |
*** browne has joined #openstack-oslo | 21:53 | |
*** swatson__ has joined #openstack-oslo | 22:08 | |
*** boden has quit IRC | 22:10 | |
*** spzala has quit IRC | 22:11 | |
harlowja | gerhardqux :) | 22:17 |
*** Guest99228 is now known as melwitt | 22:23 | |
*** tonytan4ever has joined #openstack-oslo | 22:27 | |
*** mriedem has quit IRC | 22:32 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/373775 | 22:37 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tooz: Updated from global requirements https://review.openstack.org/346782 | 22:40 |
*** Kiall has quit IRC | 22:41 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Routing table refactoring, dynamic direct connections https://review.openstack.org/372001 | 22:55 |
openstackgerrit | Merged openstack/oslo.serialization: Add a title to the API Documentation page https://review.openstack.org/379790 | 22:55 |
openstackgerrit | Merged openstack/oslo.serialization: modify the home-page info with the developer documentation https://review.openstack.org/372916 | 22:55 |
openstackgerrit | Merged openstack/debtcollector: Update homepage with developer documentation page https://review.openstack.org/375816 | 22:56 |
openstackgerrit | Merged openstack/futurist: Updated from global requirements https://review.openstack.org/377436 | 22:56 |
*** gordc has quit IRC | 23:02 | |
*** jamielennox is now known as jamielennox|away | 23:11 | |
openstackgerrit | Merged openstack/futurist: Update homepage with developer documentation page https://review.openstack.org/375820 | 23:21 |
*** tonytan4ever has quit IRC | 23:35 | |
*** browne has quit IRC | 23:52 | |
*** jamielennox|away is now known as jamielennox | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!