| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-28 | |||
| 14:32:25 | dansmith | *meant | |
| 14:32:45 | sdague | https://github.com/openstack-dev/pbr/blob/823165d068a77642db9e907e465db91f841ab424/pbr/packaging.py#L276-L330 | |
| 14:32:53 | sdague | ah, no, sorry | |
| 14:33:28 | sdague | it's just this race about how in some mod_wsgi daemon configs you are running in shared memory space | |
| 14:33:51 | sdague | and oslo.config parsing doesn't expect multiple concurrent parses | |
| 14:34:04 | sdague | so it errors out some of the workers if they hit at exactly the same time | |
| 14:34:13 | sdague | iirc | |
| 14:34:48 | sdague | if it's not quite that problem, it's a very related one with similar symptoms | |
| 14:35:19 | sdague | and only calling init once addresses it | |
| 14:36:08 | sdague | ok, https://review.openstack.org/#/c/457715 is rebased, (the real new wsgi direction) | |
| 14:36:21 | sdague | if it passes tests, will get it merged | |
| 14:38:46 | dansmith | cool | |
| 14:39:04 | sdague | dansmith: honestly, you could stack a delete of those files as a depends-on for that | |
| 14:39:12 | sdague | then we can not even bother with the warning message | |
| 14:39:42 | dansmith | sdague: come on, I want to land my warning message | |
| 14:39:55 | sdague | the real issue that ends up happening with those old wsgi.py scripts is that people manually copy them to /var/www | |
| 14:40:04 | sdague | so they never get updates | |
| 14:42:59 | jaypipes | sdague: wait... deployers manually copy files without looking at them? no! | |
| 14:45:02 | sdague | jaypipes: well... more importantly they did that in liberty | |
| 14:45:08 | sdague | and then never again | |
| 14:45:16 | sdague | so any fixes we have ... never get to them | |
| 14:45:22 | sdague | because it was outside of the python install path | |
| 14:45:38 | jaypipes | sdague: I was being facetious :) | |
| 14:45:40 | sdague | :) | |
| 14:45:48 | sdague | that's fine, I was being specific :) | |
| 14:47:11 | openstackgerrit | Lucian Petrut proposed openstack/nova master: Raise MigrationPreCheckError if instance location is unavailable https://review.openstack.org/469779 | |
| 14:50:59 | sdague | you forgot the sudo | |
| 15:02:11 | sfinucan | sdague: It's https://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html | |
| 15:02:46 | sfinucan | So I guess I should use 'os-manuals-migration', even though the spec says to use 'doc-rework' as the topic | |
| 15:03:57 | sdague | sfinucan: ok, but that's not the blueprint | |
| 15:04:16 | sdague | the blueprint: .... only works if that is the short name of the blueprint | |
| 15:04:28 | avanko_ | i'm looking for somebody who can help to solve issue with Openstack-Neutron. our team is processing cluster stress test on OPNFV FDS, OStack-ODL-FDSio-HA, when running the test, we have a blocker, at some stage, the nova stops adding ports to VM | |
| 15:05:16 | avanko_ | is there anybody to help us with this? please contact me on avanko@cisco.com, so we can set up a brief call and go through the discovered issue in more detail. currently we are blocked by this issue | |
| 15:09:30 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_remote_consoles.py https://review.openstack.org/478423 | |
| 15:20:21 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 15:26:12 | cfriesen | someone just pointed out that after you boot up an instance with neutron ports it's possible to delete the ports in neutron. Does this seem odd to anyone else? | |
| 15:41:06 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671 | |
| 15:54:21 | sdague | sfinucan / johnthetubaguy - on the config guide side, how are we feeling about adding all the pass through keystone auth options? | |
| 15:54:22 | sdague | https://review.openstack.org/#/c/358701/ | |
| 15:54:30 | openstack | Launchpad bug 1615666 in OpenStack Compute (nova) "Nova config misses [cinder] insecure option" [Undecided,New] | |
| 15:54:30 | sdague | https://bugs.launchpad.net/nova/+bug/1615666 | |
| 16:02:34 | edleafe | jaypipes: figured out the problem with the failing functional test in https://review.openstack.org/#/c/475448/ | |
| 16:03:06 | edleafe | jaypipes: the DB fixture is wiping the database, but the internal _TRAITS_SYNCED flag is still True | |
| 16:03:23 | edleafe | jaypipes: So in subsequent tests, none of the standard traits are there | |
| 16:03:24 | jaypipes | edleafe: ah!. | |
| 16:03:29 | jaypipes | edleafe: great catch. | |
| 16:03:42 | jaypipes | edleafe: thank you! | |
| 16:03:43 | edleafe | I fixed it by adding a reset method to the Trait objects | |
| 16:03:48 | edleafe | object | |
| 16:03:59 | edleafe | and calling it from the db fixture | |
| 16:04:03 | edleafe | Does that sound OK? | |
| 16:04:06 | jaypipes | edleafe: ++ | |
| 16:04:15 | edleafe | cool, will push in a few | |
| 16:04:19 | jaypipes | edleafe: ty sir! | |
| 16:04:42 | jaypipes | edleafe: if you feel like getting those parens style issues I missed that efried found, that would be cool too. | |
| 16:05:24 | edleafe | jaypipes: sure, why not? | |
| 16:05:33 | jaypipes | thanks! | |
| 16:05:37 | edleafe | jaypipes: anything to clean up that ugly style :;P | |
| 16:05:41 | jaypipes | :) | |
| 16:16:36 | openstackgerrit | Sean Dague proposed openstack/nova master: Clarify that vlan feature means nova-network support https://review.openstack.org/478551 | |
| 16:23:52 | cfriesen | I'd welcome some more reviews of https://review.openstack.org/#/c/461456 It appears that Sahid and I are at a bit of an impasse as to what the semantics of "hw:cpu_realtime_mask" should be. I think it's confusing to have an implied "all-cpus-are-realtime" and that it should be treated as indicating which vCPUs are realtime rather than an exclusion mask indicating which vCPUs are not realtime | |
| 16:56:24 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add a reset for traits DB sync https://review.openstack.org/478564 | |
| 16:56:41 | edleafe | jaypipes: ^^ Made it a separate patch | |
| 16:57:25 | edleafe | jaypipes: I can rebase the series on that, or if you think it can merge fast, you can rebase the series on master later | |
| 17:01:21 | jaypipes | edleafe: ah, cool. | |
| 17:02:34 | jaypipes | edleafe: +2 from me. | |
| 17:02:42 | jaypipes | bauzas: feel free to fast-track above patch from edleafe | |
| 17:03:41 | jaypipes | in other news, someone just needs to boot the damn EMC ScaleIO CI system. it's completely unreliable and just noise. | |
| 17:04:40 | edleafe | jaypipes: bauzas is on PTO today | |
| 17:05:02 | jaypipes | alex_xu, melwitt: see edleafe's patch above. simple and sweet and would be good to merge. | |
| 17:05:11 | jaypipes | dansmith: ^^ you too. | |
| 17:05:18 | dansmith | yeah I'm looking | |
| 17:05:25 | jaypipes | danke | |
| 17:05:25 | dansmith | it seems like a weird way to accomplish that | |
| 17:05:41 | dansmith | being in the database fixture I mean | |
| 17:05:46 | jaypipes | dansmith: you mean as opposed to just resetting the _TRAITS_SYNC variable? | |
| 17:05:54 | dansmith | we do all the rest of these global resets in the test base | |
| 17:06:07 | dansmith | lemme get you a link | |
| 17:06:32 | dansmith | https://github.com/openstack/nova/blob/master/nova/test.py#L259 | |
| 17:06:33 | dansmith | for example | |
| 17:06:47 | dansmith | and https://github.com/openstack/nova/blob/master/nova/test.py#L295 | |
| 17:06:48 | dansmith | and https://github.com/openstack/nova/blob/master/nova/test.py#L306 | |
| 17:07:02 | dansmith | and https://github.com/openstack/nova/blob/master/nova/test.py#L267 | |
| 17:07:08 | openstackgerrit | Ed Leafe proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448 | |
| 17:07:21 | dansmith | and https://github.com/openstack/nova/blob/master/nova/test.py#L266 | |
| 17:08:08 | jaypipes | dansmith: so in short, you'd prefer to call that in the base test setup, yes? | |
| 17:08:53 | dansmith | jaypipes: that and I don't think it should be an object method | |
| 17:09:02 | melwitt | I also don't like the arbitrary putting it on the DatabaseFixture. it's not related to that | |
| 17:09:17 | edleafe | dansmith: so maybe around line 276 in that file? | |
| 17:09:22 | dansmith | jaypipes: we have a bunch of other things in the test base that just import a module and nuke the module-level variable | |
| 17:09:57 | dansmith | edleafe: not in that if, no.. just put it down like 296 or so | |
| 17:10:05 | dansmith | no need to make it conditional, just always nuke | |
| 17:11:40 | edleafe | dansmith: ok, so no worries about it being _PRIVATE? I see the others are referenced directly | |
| 17:11:50 | dansmith | edleafe: no | |
| 17:11:54 | edleafe | k | |
| 17:12:12 | dansmith | all the tests do all day is butt into the private business of other stuff.. :) | |
| 17:18:28 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add a reset for traits DB sync https://review.openstack.org/478564 | |
| 17:18:36 | edleafe | dansmith: jaypipes: amended^^ | |
| 17:20:19 | jaypipes | edleafe: approved. | |
| 17:42:29 | openstackgerrit | melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700 | |