*** Hosam_ has quit IRC | 00:00 | |
*** Hosam has joined #openstack-nova | 00:01 | |
*** ijw has quit IRC | 00:03 | |
*** ayoung has quit IRC | 00:04 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova-specs: Send service token along with user tokens https://review.openstack.org/387711 | 00:04 |
---|---|---|
*** Hosam has quit IRC | 00:05 | |
*** gyee has quit IRC | 00:06 | |
melwitt | mriedem: would it do something undesirable to get a uuid for the default group too from neutron? | 00:06 |
mriedem | melwitt: we don't try to get the uuid for a default group for some reason | 00:07 |
mriedem | that might be a nova-net thing though... | 00:07 |
mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L415 | 00:07 |
*** rama_y has quit IRC | 00:07 | |
mriedem | do we not have a 'default' security group in the database for nova-net, or is that a thing for neutron where a GET on that won't work? i don't have a devstack up to check | 00:08 |
melwitt | heh, great comment | 00:08 |
melwitt | let me check | 00:08 |
melwitt | we do have a 'default' group in the database for nova-net | 00:09 |
mriedem | do you have a neutron devstack? | 00:09 |
melwitt | yes | 00:09 |
*** ijw has joined #openstack-nova | 00:09 | |
mriedem | i wonder if that's a thing that changes on the neutron side depending on whether or not you have port security enabled...i'm not really sure how the data model works there | 00:10 |
mriedem | armax: does neutron store a 'default' security group per tenant in the db? | 00:10 |
mriedem | armax: trying to figure out why nova doesn't try to do a GET on the 'default' secgroup here https://github.com/openstack/nova/blob/master/nova/compute/api.py#L415 | 00:10 |
melwitt | I'm trying to remember why we couldn't stop sending the 'default' secgroup to neutron if it ignores it anyway | 00:10 |
armax | mriedem: yes, each tenant gets his/her own default security group | 00:11 |
*** Sukhdev has joined #openstack-nova | 00:11 | |
dansmith | mriedem: I thought I was writing that neutron secgroup object patch :) | 00:12 |
mriedem | armax: so i should be able to do like a neutron secgroup-show default? | 00:12 |
armax | mriedem: there’s something weird about the default security group where it’s provisioned on the fyl | 00:12 |
armax | fly | 00:12 |
mriedem | armax: that's what i thought and maybe why we don't try to do a GET on it in the compute API | 00:13 |
dansmith | mriedem: either way, o.vo will allow storing subclasses, if you tell the field that subclasses are okay | 00:13 |
*** victorhe has joined #openstack-nova | 00:13 | |
mriedem | dansmith: yup i've gotten past that | 00:13 |
pumaranikar | johnthetubaguy, updated spec for service token. Please review when you join back. https://review.openstack.org/#/c/387711/ | 00:13 |
mriedem | now dealing with this uuid vs 'default' thing | 00:13 |
dansmith | okay | 00:13 |
mriedem | if we store id='default' that's a bit goofy | 00:13 |
mriedem | and it means the NeutronSecurityGroup.id field can't be a uuid field | 00:13 |
mriedem | but if we can't do a GET on 'default' in neutron api then we can't really get around that | 00:14 |
mriedem | well i'll try to do the least disruptive thing and see how it looks | 00:14 |
armax | mriedem: yes, and a default security group is also provisioned on the fly for instance when creating a port | 00:15 |
*** portdirect is now known as portdirect_away | 00:15 | |
*** victorhe has quit IRC | 00:17 | |
*** ijw has quit IRC | 00:19 | |
openstackgerrit | Jin Li proposed openstack/nova: Transform instance.soft_delete notifications https://review.openstack.org/383970 | 00:20 |
melwitt | dansmith: I think the instance list problem is because get_all only works for a single cell (it doesn't do a db connection switch to read from cell database) | 00:21 |
*** ar1el has quit IRC | 00:22 | |
dansmith | melwitt: right | 00:26 |
dansmith | melwitt: I've collapsed the urls locally to basically make CONF.database.connection equal to the first cell database_connection | 00:26 |
*** gmann has joined #openstack-nova | 00:27 | |
mriedem | methinks i found a bug | 00:27 |
dansmith | oh, sorry | 00:27 |
dansmith | melwitt: you mean the reason that server detail doesn't show the instance? | 00:27 |
mriedem | wtf https://github.com/openstack/nova/blob/master/nova/objects/security_group.py#L87 | 00:27 |
mriedem | dansmith: is that intentional? ^ | 00:27 |
dansmith | melwitt: since I have the urls the same locally I don't think that's it | 00:27 |
melwitt | dansmith: no, I was thinking the server detail does show it and it does a connection switch and the list doesn't show it and it doesn't do a switch | 00:28 |
dansmith | mriedem: ISTR that was done for some compat reason.. secgroups was super problematic, but agree it's not good | 00:28 |
dansmith | melwitt: no, that's not the problem I'm hitting | 00:28 |
mriedem | so i can't create the list with objects set on init, but i can do it after it's created :) | 00:28 |
mriedem | seems odd | 00:28 |
dansmith | melwitt: I don't get instances that are only buildrequests in the api database (i.e. state=scheduling) in the API list, but can show them | 00:29 |
melwitt | dansmith: oh, okay. I went and printed the build requests from the get_all and there were none so it made me think they'd been scheduled | 00:29 |
dansmith | melwitt: the instances aren't in the cell database yet because they haven't been scheduled | 00:29 |
*** ijw has joined #openstack-nova | 00:31 | |
*** yuntongjin has joined #openstack-nova | 00:31 | |
*** kaisers has quit IRC | 00:33 | |
*** jerrygb has joined #openstack-nova | 00:34 | |
*** kaisers has joined #openstack-nova | 00:34 | |
*** ar1el has joined #openstack-nova | 00:34 | |
*** hongbin has quit IRC | 00:34 | |
dansmith | melwitt: well, there appear to be no buildrequests in the database if I do a get_all on BuildRequestList | 00:38 |
*** catintheroof has quit IRC | 00:38 | |
*** jerrygb has quit IRC | 00:38 | |
dansmith | melwitt: but it's clearly not scheduled so I'm not sure where it's coming from :P | 00:38 |
melwitt | hmm | 00:38 |
*** catintheroof has joined #openstack-nova | 00:39 | |
dansmith | indeed | 00:40 |
*** victorhe has joined #openstack-nova | 00:40 | |
*** ijw has quit IRC | 00:41 | |
*** ijw has joined #openstack-nova | 00:41 | |
*** catintheroof has quit IRC | 00:43 | |
*** jwcroppe has quit IRC | 00:44 | |
dansmith | hmm, so it is making it to the instance create, but the state is wrong | 00:44 |
dansmith | and I don't see it in the db | 00:44 |
*** jwcroppe has joined #openstack-nova | 00:44 | |
*** jamielennox is now known as jamielennox|away | 00:44 | |
*** victorhe has quit IRC | 00:45 | |
*** ayoung has joined #openstack-nova | 00:45 | |
*** ijw has quit IRC | 00:45 | |
*** ijw has joined #openstack-nova | 00:47 | |
melwitt | dansmith: is it not still a separate database for the cell because of CellDatabase? | 00:48 |
dansmith | melwitt: no I've got everything collapsed to one | 00:48 |
melwitt | okay, so locally everything is using the same database fixture | 00:48 |
*** walharthi has joined #openstack-nova | 00:49 | |
melwitt | I was concerned about the default DatabaseFixture that comes with test.TestCase | 00:50 |
dansmith | yeah | 00:50 |
dansmith | I've also converted to NoDBTestCase for integrated helpers | 00:50 |
*** ijw has quit IRC | 00:50 | |
*** womygod has joined #openstack-nova | 00:53 | |
*** ducttape_ has joined #openstack-nova | 00:55 | |
*** ducttape_ has quit IRC | 00:55 | |
*** ducttape_ has joined #openstack-nova | 00:56 | |
*** ijw has joined #openstack-nova | 00:58 | |
*** mlavalle has quit IRC | 01:01 | |
*** ijw has quit IRC | 01:01 | |
*** gmann has quit IRC | 01:03 | |
*** gmann_ has joined #openstack-nova | 01:03 | |
*** ijw has joined #openstack-nova | 01:04 | |
*** walharthi has quit IRC | 01:04 | |
*** walharthi has joined #openstack-nova | 01:05 | |
*** ijw has quit IRC | 01:06 | |
*** hoangcx has joined #openstack-nova | 01:06 | |
*** yuntongjin has quit IRC | 01:07 | |
*** yuntongjin has joined #openstack-nova | 01:08 | |
*** ducttape_ has quit IRC | 01:12 | |
*** jwcroppe has quit IRC | 01:13 | |
*** ducttape_ has joined #openstack-nova | 01:14 | |
*** jwcroppe has joined #openstack-nova | 01:15 | |
*** edmondsw has quit IRC | 01:21 | |
*** smatzek has joined #openstack-nova | 01:22 | |
*** dave-mccowan has joined #openstack-nova | 01:24 | |
*** rama_y has joined #openstack-nova | 01:35 | |
*** ducttape_ has quit IRC | 01:36 | |
*** zhangyufei has joined #openstack-nova | 01:48 | |
*** ducttape_ has joined #openstack-nova | 01:50 | |
*** Apoorva has quit IRC | 01:57 | |
*** baoli has joined #openstack-nova | 02:00 | |
*** Apoorva has joined #openstack-nova | 02:00 | |
*** browne has quit IRC | 02:03 | |
*** Apoorva has quit IRC | 02:05 | |
*** gcb has joined #openstack-nova | 02:06 | |
*** ducttape_ has quit IRC | 02:10 | |
*** coreywright has quit IRC | 02:10 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add NeutronSecurityGroup object https://review.openstack.org/396436 | 02:12 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Return list of NeutronSecurityGroup objects from populate_security_groups https://review.openstack.org/396469 | 02:12 |
mriedem | ok dansmith, ^ | 02:12 |
mriedem | we can talk about that tomorrow | 02:12 |
mriedem | some things i'm not crazy about in there, but not sure we have better options | 02:12 |
*** claudiub has quit IRC | 02:13 | |
*** kaisers_ has quit IRC | 02:16 | |
*** trisq has joined #openstack-nova | 02:23 | |
*** Sukhdev has quit IRC | 02:24 | |
mriedem | BobBall: https://bugs.launchpad.net/nova/+bug/1640993 xenserver + neutron ci failed on a vif plugging timeout | 02:33 |
openstack | Launchpad bug 1640993 in OpenStack Compute (nova) "xenserver hits vif plugging timeout with neutron CI job" [Undecided,New] | 02:33 |
mriedem | looks like it's waiting for 2 events and times out on one of them not coming back | 02:33 |
*** jerrygb has joined #openstack-nova | 02:34 | |
*** jerrygb_ has joined #openstack-nova | 02:37 | |
*** victorhe has joined #openstack-nova | 02:38 | |
*** jerrygb has quit IRC | 02:39 | |
*** victorhe has quit IRC | 02:43 | |
*** coreywright has joined #openstack-nova | 02:48 | |
*** gmann_ has quit IRC | 02:50 | |
*** mriedem has quit IRC | 02:52 | |
*** unicell has quit IRC | 02:53 | |
*** jerrygb has joined #openstack-nova | 02:54 | |
*** jerrygb_ has quit IRC | 02:56 | |
*** jaypipes has joined #openstack-nova | 02:58 | |
*** jerrygb has quit IRC | 03:00 | |
*** browne has joined #openstack-nova | 03:00 | |
*** baoli has quit IRC | 03:05 | |
*** browne has quit IRC | 03:06 | |
*** markvoelker has quit IRC | 03:07 | |
*** victorhe has joined #openstack-nova | 03:07 | |
*** esberglu has joined #openstack-nova | 03:07 | |
*** prashkre_ has joined #openstack-nova | 03:07 | |
*** victorhe has quit IRC | 03:08 | |
*** victorhe has joined #openstack-nova | 03:09 | |
*** tbachman has quit IRC | 03:09 | |
*** smatzek has quit IRC | 03:09 | |
*** fabian4 has quit IRC | 03:10 | |
*** ducttape_ has joined #openstack-nova | 03:11 | |
*** walharthi has quit IRC | 03:11 | |
*** esberglu has quit IRC | 03:11 | |
*** diogogmt has quit IRC | 03:15 | |
*** ducttape_ has quit IRC | 03:16 | |
*** dave-mcnally has quit IRC | 03:16 | |
*** dikonoor has joined #openstack-nova | 03:20 | |
*** yuntongjin has quit IRC | 03:31 | |
*** harlowja has quit IRC | 03:33 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Remove unused code in nova/api/openstack/wsgi.py https://review.openstack.org/396485 | 03:36 |
*** ijw has joined #openstack-nova | 03:39 | |
*** manasm has joined #openstack-nova | 03:41 | |
*** ijw has quit IRC | 03:43 | |
*** tojuvone has quit IRC | 03:44 | |
*** vladikr has quit IRC | 03:49 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Remove unused code in nova/api/openstack/wsgi.py https://review.openstack.org/396485 | 03:53 |
*** gmann has joined #openstack-nova | 03:58 | |
*** Apoorva has joined #openstack-nova | 04:01 | |
*** namnh has joined #openstack-nova | 04:04 | |
*** manasm has quit IRC | 04:05 | |
*** Apoorva has quit IRC | 04:06 | |
*** kevinz has quit IRC | 04:06 | |
*** kevinz has joined #openstack-nova | 04:09 | |
*** sudipto has joined #openstack-nova | 04:10 | |
*** sudipto_ has joined #openstack-nova | 04:10 | |
*** ducttape_ has joined #openstack-nova | 04:12 | |
*** jwcroppe has quit IRC | 04:13 | |
*** jwcroppe has joined #openstack-nova | 04:14 | |
*** tojuvone has joined #openstack-nova | 04:16 | |
*** GB21 has joined #openstack-nova | 04:16 | |
*** tojuvone_ has joined #openstack-nova | 04:16 | |
*** tojuvone has quit IRC | 04:17 | |
*** tojuvone_ has quit IRC | 04:17 | |
*** ducttape_ has quit IRC | 04:17 | |
*** jwcroppe has quit IRC | 04:18 | |
*** diogogmt has joined #openstack-nova | 04:18 | |
*** jerrygb has joined #openstack-nova | 04:20 | |
*** markvoelker has joined #openstack-nova | 04:21 | |
*** victorhe has quit IRC | 04:21 | |
*** jerrygb has quit IRC | 04:23 | |
*** nicolasbock has quit IRC | 04:23 | |
*** jerrygb has joined #openstack-nova | 04:23 | |
*** prashkre_ has quit IRC | 04:23 | |
*** jerrygb has quit IRC | 04:28 | |
*** ducttape_ has joined #openstack-nova | 04:28 | |
*** tlian has quit IRC | 04:29 | |
*** jwcroppe has joined #openstack-nova | 04:36 | |
*** harlowja has joined #openstack-nova | 04:38 | |
*** prateek has joined #openstack-nova | 04:40 | |
*** ducttape_ has quit IRC | 04:46 | |
*** manasm has joined #openstack-nova | 04:47 | |
*** phschwartz has quit IRC | 04:47 | |
*** fragatin_ has joined #openstack-nova | 04:53 | |
*** esberglu has joined #openstack-nova | 04:56 | |
*** fragatina has quit IRC | 04:57 | |
*** fragatin_ has quit IRC | 04:57 | |
*** Sukhdev has joined #openstack-nova | 04:58 | |
*** dave-mccowan has quit IRC | 04:59 | |
*** fragatina has joined #openstack-nova | 05:00 | |
*** esberglu has quit IRC | 05:00 | |
*** jerrygb has joined #openstack-nova | 05:01 | |
*** fragatina has quit IRC | 05:01 | |
*** jerrygb has quit IRC | 05:02 | |
*** sree has joined #openstack-nova | 05:10 | |
*** baoli has joined #openstack-nova | 05:18 | |
*** yamahata has quit IRC | 05:21 | |
*** baoli has quit IRC | 05:22 | |
*** jklare has quit IRC | 05:28 | |
*** jklare has joined #openstack-nova | 05:28 | |
*** khamtamtun has joined #openstack-nova | 05:30 | |
*** yuntongjin has joined #openstack-nova | 05:30 | |
*** Kevin_Zheng has quit IRC | 05:31 | |
*** GB21 has quit IRC | 05:31 | |
*** khamtamtun has quit IRC | 05:32 | |
*** GB21 has joined #openstack-nova | 05:33 | |
*** Kevin_Zheng has joined #openstack-nova | 05:34 | |
*** ratailor has joined #openstack-nova | 05:36 | |
*** fragatina has joined #openstack-nova | 05:37 | |
*** GB21 has quit IRC | 05:41 | |
*** armax has quit IRC | 05:43 | |
*** Apoorva has joined #openstack-nova | 05:43 | |
*** coolsvap has joined #openstack-nova | 05:45 | |
*** preethipy has joined #openstack-nova | 05:46 | |
*** ducttape_ has joined #openstack-nova | 05:46 | |
*** thorst_ has quit IRC | 05:49 | |
*** thorst_ has joined #openstack-nova | 05:50 | |
*** ducttape_ has quit IRC | 05:51 | |
*** harlowja has quit IRC | 05:51 | |
*** harlowja has joined #openstack-nova | 05:52 | |
*** pbandark has joined #openstack-nova | 05:56 | |
*** thorst_ has quit IRC | 05:59 | |
*** hshiina has joined #openstack-nova | 06:04 | |
openstackgerrit | Alex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters https://review.openstack.org/389003 | 06:05 |
*** sandanar has joined #openstack-nova | 06:06 | |
*** josecastroleon has quit IRC | 06:09 | |
*** gmann has quit IRC | 06:10 | |
*** Apoorva has quit IRC | 06:18 | |
*** kaisers_ has joined #openstack-nova | 06:21 | |
*** amotoki has quit IRC | 06:24 | |
*** amotoki has joined #openstack-nova | 06:24 | |
*** prashkre has joined #openstack-nova | 06:25 | |
*** rama_y has quit IRC | 06:26 | |
*** sridharg has joined #openstack-nova | 06:26 | |
*** lpetrut has joined #openstack-nova | 06:27 | |
*** jaosorior has joined #openstack-nova | 06:32 | |
*** gmann has joined #openstack-nova | 06:34 | |
*** Jeffrey4l has quit IRC | 06:35 | |
*** paul-carlton2 has joined #openstack-nova | 06:37 | |
*** SayyedMohammad has joined #openstack-nova | 06:43 | |
*** SayyedMohammad has left #openstack-nova | 06:44 | |
*** esberglu has joined #openstack-nova | 06:45 | |
*** SayyedMohammad has joined #openstack-nova | 06:47 | |
*** Jeffrey4l has joined #openstack-nova | 06:48 | |
*** esberglu has quit IRC | 06:49 | |
*** josecastroleon has joined #openstack-nova | 06:50 | |
openstackgerrit | int32bit proposed openstack/nova: Add lock actions to action-list https://review.openstack.org/396510 | 06:56 |
*** yamahata has joined #openstack-nova | 06:59 | |
*** Sukhdev has quit IRC | 07:01 | |
*** tesseract has joined #openstack-nova | 07:03 | |
*** tesseract is now known as Guest7605 | 07:03 | |
*** takashin has quit IRC | 07:07 | |
*** oanson has joined #openstack-nova | 07:09 | |
*** andreas_s has joined #openstack-nova | 07:10 | |
*** rcernin has joined #openstack-nova | 07:11 | |
*** jwcroppe has quit IRC | 07:11 | |
*** jwcroppe has joined #openstack-nova | 07:12 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add microversion cap information https://review.openstack.org/396514 | 07:12 |
*** belmoreira has joined #openstack-nova | 07:12 | |
*** claudiub has joined #openstack-nova | 07:13 | |
*** lpetrut has quit IRC | 07:15 | |
*** jwcroppe has quit IRC | 07:16 | |
*** markus_z has joined #openstack-nova | 07:16 | |
*** coolsvap has left #openstack-nova | 07:19 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add microversion cap information https://review.openstack.org/396514 | 07:21 |
*** lpetrut has joined #openstack-nova | 07:23 | |
markus_z | johnthetubaguy: bauzas: https://review.openstack.org/#/c/395067/ would solve an issue in our (not yet public) s390x CI, maybe you could take a look. | 07:30 |
*** jaosorior has quit IRC | 07:31 | |
*** preethipy has quit IRC | 07:35 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add microversion cap information https://review.openstack.org/396514 | 07:39 |
*** lpetrut has quit IRC | 07:40 | |
*** bhagyashris has quit IRC | 07:41 | |
*** manasm has quit IRC | 07:43 | |
*** ducttape_ has joined #openstack-nova | 07:48 | |
*** amotoki has quit IRC | 07:49 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add microversion cap information https://review.openstack.org/396514 | 07:49 |
*** ducttape_ has quit IRC | 07:53 | |
*** RuiChen_ has joined #openstack-nova | 07:55 | |
*** Kevin_Zheng has quit IRC | 07:56 | |
*** RuiChen has quit IRC | 07:56 | |
*** RuiChen_ is now known as RuiChen | 07:56 | |
*** andreas_s has quit IRC | 07:56 | |
*** alexchadin has joined #openstack-nova | 07:57 | |
*** Kevin_Zheng has joined #openstack-nova | 07:59 | |
*** thorst_ has joined #openstack-nova | 08:01 | |
*** avolkov has joined #openstack-nova | 08:02 | |
*** jaosorior has joined #openstack-nova | 08:02 | |
*** harlowja has quit IRC | 08:03 | |
*** alexchadin has quit IRC | 08:05 | |
*** alexchadin has joined #openstack-nova | 08:06 | |
*** markus_z has quit IRC | 08:08 | |
*** alexchadin has quit IRC | 08:08 | |
*** alexchadin has joined #openstack-nova | 08:08 | |
*** thorst_ has quit IRC | 08:08 | |
openstackgerrit | Bin Zhou proposed openstack/nova: Modify mistake of scsi adapter type class https://review.openstack.org/396529 | 08:10 |
*** dimtruck is now known as zz_dimtruck | 08:11 | |
*** moshele has joined #openstack-nova | 08:22 | |
*** markus_z has joined #openstack-nova | 08:22 | |
moshele | jaypipes: can you review my comments on the User-controlled SR-IOV ports allocation https://review.openstack.org/#/c/182242/ | 08:23 |
*** unicell has joined #openstack-nova | 08:26 | |
*** unicell has quit IRC | 08:30 | |
*** unicell has joined #openstack-nova | 08:32 | |
*** alexchadin has quit IRC | 08:32 | |
*** al_indigo has quit IRC | 08:33 | |
*** alexchadin has joined #openstack-nova | 08:33 | |
*** al_indigo has joined #openstack-nova | 08:33 | |
*** esberglu has joined #openstack-nova | 08:34 | |
*** alexchad_ has joined #openstack-nova | 08:36 | |
*** alexchadin has quit IRC | 08:36 | |
*** unicell has quit IRC | 08:37 | |
*** esberglu has quit IRC | 08:38 | |
*** wuhg has joined #openstack-nova | 08:39 | |
*** josecastroleon has quit IRC | 08:40 | |
*** josecastroleon has joined #openstack-nova | 08:41 | |
*** unicell has joined #openstack-nova | 08:44 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Transform instance.resize_revert notifications https://review.openstack.org/396544 | 08:47 |
*** ducttape_ has joined #openstack-nova | 08:49 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP]Transform instance.resize_revert notifications https://review.openstack.org/396544 | 08:49 |
*** ralonsoh has joined #openstack-nova | 08:53 | |
tdurakov | markus_z: hi, missed your message yesterday, l-m job is voting and looks o | 08:53 |
tdurakov | s/o/ok gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial | 08:54 |
*** ducttape_ has quit IRC | 08:54 | |
*** lpetrut has joined #openstack-nova | 08:58 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-nova | 09:01 | |
*** lpetrut has quit IRC | 09:02 | |
*** ar1el has quit IRC | 09:04 | |
openstackgerrit | Hussain Chachuliya proposed openstack/nova: Replace uuid4() with uuidsentinel https://review.openstack.org/377661 | 09:04 |
*** danpb has joined #openstack-nova | 09:06 | |
*** thorst_ has joined #openstack-nova | 09:07 | |
*** adreznec has quit IRC | 09:09 | |
*** bnemec has quit IRC | 09:09 | |
*** preethipy has joined #openstack-nova | 09:10 | |
*** ihrachys has joined #openstack-nova | 09:10 | |
*** adreznec has joined #openstack-nova | 09:11 | |
*** brault has joined #openstack-nova | 09:11 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP]Transform instance.resize_revert notifications https://review.openstack.org/396544 | 09:11 |
*** bnemec has joined #openstack-nova | 09:12 | |
*** thorst_ has quit IRC | 09:14 | |
*** shihanzhang has quit IRC | 09:15 | |
*** moshele has quit IRC | 09:15 | |
*** ar1el has joined #openstack-nova | 09:16 | |
*** claudiub|2 has joined #openstack-nova | 09:16 | |
*** brault has quit IRC | 09:16 | |
*** kylek3h has quit IRC | 09:16 | |
*** claudiub has quit IRC | 09:18 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP]Transform instance.rebuild.error notification https://review.openstack.org/396552 | 09:19 |
*** lucas-afk is now known as lucasagomes | 09:22 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Add 'preallocate_images' to 'virt' group https://review.openstack.org/392567 | 09:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Move virt options to a group https://review.openstack.org/392566 | 09:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Detail the 'injected_network_template' opt https://review.openstack.org/392565 | 09:22 |
openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Standardize formatting of virt https://review.openstack.org/392564 | 09:22 |
*** kylek3h has joined #openstack-nova | 09:23 | |
*** gmann has quit IRC | 09:23 | |
*** ihrachys has quit IRC | 09:24 | |
*** derekh has joined #openstack-nova | 09:25 | |
*** zenoway has joined #openstack-nova | 09:28 | |
*** derekh has quit IRC | 09:29 | |
*** amotoki has joined #openstack-nova | 09:32 | |
*** andymaier has joined #openstack-nova | 09:33 | |
*** esberglu has joined #openstack-nova | 09:34 | |
openstackgerrit | int32bit proposed openstack/nova: Raise exception if delete disabled terminate server https://review.openstack.org/396017 | 09:37 |
*** manasm has joined #openstack-nova | 09:38 | |
*** esberglu has quit IRC | 09:38 | |
*** amotoki has quit IRC | 09:38 | |
*** andymaier has quit IRC | 09:44 | |
*** derekh has joined #openstack-nova | 09:47 | |
*** openstackgerrit has quit IRC | 09:48 | |
*** openstackgerrit has joined #openstack-nova | 09:49 | |
*** slaweq_ has quit IRC | 09:52 | |
*** slaweq_ has joined #openstack-nova | 09:53 | |
*** ihrachys has joined #openstack-nova | 09:53 | |
*** lpetrut has joined #openstack-nova | 10:01 | |
*** slaweq_ has quit IRC | 10:08 | |
*** thorst_ has joined #openstack-nova | 10:12 | |
pkholkin | johnthetubaguy: bauzas: hello guys! waiting waiting for you in my spec) https://review.openstack.org/#/c/381912/, thanks! | 10:15 |
*** zhangyufei has quit IRC | 10:18 | |
*** thorst_ has quit IRC | 10:18 | |
*** manasm has quit IRC | 10:20 | |
*** alexchad_ has quit IRC | 10:21 | |
*** alexchadin has joined #openstack-nova | 10:22 | |
*** womygod has quit IRC | 10:23 | |
*** alexchad_ has joined #openstack-nova | 10:26 | |
*** markus_z has quit IRC | 10:26 | |
*** alexchadin has quit IRC | 10:26 | |
*** markus_z has joined #openstack-nova | 10:27 | |
*** esberglu has joined #openstack-nova | 10:28 | |
*** oanson has quit IRC | 10:29 | |
johnthetubaguy | pkholkin: sorry, I keep not getting around to re-reading that | 10:30 |
johnthetubaguy | pkholkin: the usage of the filter isn't clear enough for me | 10:30 |
johnthetubaguy | pkholkin: does my new comment make more sense than the last one? | 10:30 |
*** gmann has joined #openstack-nova | 10:31 | |
*** yuntongjin has quit IRC | 10:32 | |
*** portdirect_away is now known as portdirect | 10:32 | |
markus_z | tdurakov: I was looking at the wrong thing :/ sorry | 10:32 |
*** esberglu has quit IRC | 10:33 | |
*** sambetts|afk is now known as sambetts | 10:33 | |
*** slaweq_ has joined #openstack-nova | 10:33 | |
sfinucan | johnthetubaguy, markus_z: There's a pretty significant backlog of centralize-config-options patches built up. Could ye put it on the list to take a peek at them over the next week or two? | 10:35 |
danpb | sfinucan: got a topic link handy ? | 10:37 |
markus_z | sfinucan: Yes, I'll do a review day on Wednesday for that, would that help? (apologies for not having that on my radar) | 10:37 |
pkholkin | johnthetubaguy: will look soon | 10:38 |
*** yamahata has quit IRC | 10:38 | |
sfinucan | danpb: Sure https://review.openstack.org/#/q/topic:bp/centralize-config-options-ocata | 10:38 |
johnthetubaguy | sfinucan: yeah, its on my list, but current spec distracted and internal planning work distracted | 10:38 |
johnthetubaguy | currently | 10:39 |
*** stvnoyes1 has quit IRC | 10:40 | |
sfinucan | markus_z, johnthetubaguy: That's fine :) Just making sure they're not stuck in potential merge-conflict limbo forever :) | 10:40 |
*** stvnoyes has joined #openstack-nova | 10:41 | |
johnthetubaguy | sfinucan: annoyingly they are basically at the top of my review list, I just haven't got there :'( | 10:42 |
*** pbandark has quit IRC | 10:42 | |
*** manasm has joined #openstack-nova | 10:44 | |
*** cdent has joined #openstack-nova | 10:49 | |
*** wuhg has quit IRC | 10:51 | |
*** portdirect has quit IRC | 10:53 | |
*** owalsh has joined #openstack-nova | 11:00 | |
*** rfolco has joined #openstack-nova | 11:04 | |
*** sree has quit IRC | 11:06 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Strict isolation of group of hosts for image and flavor https://review.openstack.org/381912 | 11:08 |
*** sree has joined #openstack-nova | 11:09 | |
*** alexchad_ has quit IRC | 11:10 | |
*** alexchadin has joined #openstack-nova | 11:10 | |
*** hoangcx has quit IRC | 11:11 | |
*** sree has quit IRC | 11:14 | |
*** alexchadin has quit IRC | 11:15 | |
*** namnh has quit IRC | 11:18 | |
*** tbachman has joined #openstack-nova | 11:19 | |
*** thorst_ has joined #openstack-nova | 11:19 | |
*** thorst_ has quit IRC | 11:24 | |
openstackgerrit | zhaolihui proposed openstack/nova: Fix to rebuild vm not with the specified image https://review.openstack.org/395908 | 11:27 |
*** esberglu has joined #openstack-nova | 11:31 | |
*** GB21 has joined #openstack-nova | 11:35 | |
*** esberglu has quit IRC | 11:36 | |
*** nicolasbock has joined #openstack-nova | 11:37 | |
*** matrohon has joined #openstack-nova | 11:38 | |
*** preethi has joined #openstack-nova | 11:41 | |
openstackgerrit | Alex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters https://review.openstack.org/389003 | 11:42 |
*** jwcroppe has joined #openstack-nova | 11:42 | |
*** preethipy has quit IRC | 11:43 | |
*** alexchadin has joined #openstack-nova | 11:46 | |
*** jwcroppe has quit IRC | 11:47 | |
*** josecastroleon has quit IRC | 11:50 | |
*** trisq has quit IRC | 11:50 | |
*** manasm_ has joined #openstack-nova | 11:51 | |
*** smatzek has joined #openstack-nova | 11:52 | |
*** sandanar has quit IRC | 11:53 | |
*** manasm has quit IRC | 11:54 | |
*** manasm_ is now known as manasm | 11:54 | |
johnthetubaguy | pkholkin: would be good to catch up around that spec when you get chance | 11:54 |
johnthetubaguy | bauzas: any chance for you to look at my comments on: https://review.openstack.org/#/c/381912 I am wondering what you think? | 11:55 |
*** gszasz has joined #openstack-nova | 11:56 | |
openstackgerrit | Jabbar Ali proposed openstack/python-novaclient: Removing version error from command nova keypair-add help https://review.openstack.org/396312 | 11:57 |
pkholkin | johnthetubaguy: thanks for updating spec, I like you explanations there | 11:57 |
pkholkin | so <empty> and <no> differs when isolation is True or False | 11:58 |
johnthetubaguy | pkholkin: yeah, or if a host is not in an aggregate at all (the default) | 11:58 |
*** matrohon has quit IRC | 11:59 | |
pkholkin | johnthetubaguy: yes, agree | 11:59 |
pkholkin | now it looks more clear | 11:59 |
johnthetubaguy | I just wonder if we need configuration for that case, so you don't need to put the hosts into an aggregate, that might be overkill I guess | 12:00 |
*** clenimar has joined #openstack-nova | 12:00 | |
-openstackstatus- NOTICE: Our OpenStack CI systems are stuck and no new jobs are submitted. Please do not recheck - and do not approve changes until this is fixed. | 12:00 | |
pkholkin | we don't need configuration and we don't need to put hosts to aggregates just to boot smth | 12:02 |
pkholkin | default values help | 12:03 |
johnthetubaguy | sorry, confused, which default values? | 12:03 |
*** haplo37 has quit IRC | 12:05 | |
pkholkin | I saw your comment, I think we don't need anything | 12:05 |
openstackgerrit | Jabbar Ali proposed openstack/python-novaclient: Removing version error from command nova keypair-add help https://review.openstack.org/396312 | 12:05 |
pkholkin | if we mark isolation=True it must be <empty> column, otherwise it is <no> | 12:05 |
johnthetubaguy | my worry is people will want all the hosts that are not in aggregates to default to behaving like the <empty> column | 12:07 |
*** maticue has joined #openstack-nova | 12:07 | |
johnthetubaguy | I mean its not *required* as you can just add all hosts into an aggregate and mark isolation=True, its just that seems error prone | 12:08 |
johnthetubaguy | as you add capacity, etc | 12:08 |
*** lucasagomes is now known as lucas-hungry | 12:10 | |
*** pbandark has joined #openstack-nova | 12:10 | |
*** pbandark is now known as pbandark`brb | 12:10 | |
pkholkin | if we add some config option it will make more questions | 12:12 |
*** bhagyashris has joined #openstack-nova | 12:13 | |
pkholkin | I don't like to have global option and 'per-aggregate' options | 12:14 |
*** haplo37 has joined #openstack-nova | 12:15 | |
pkholkin | so, yes, operators should create some aggregate with isolation=True and add hosts to it | 12:15 |
pkholkin | I think it is not a problem | 12:15 |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Strict isolation of group of hosts for image and flavor https://review.openstack.org/381912 | 12:16 |
*** bvanhav_ has joined #openstack-nova | 12:16 | |
johnthetubaguy | pkholkin: so I added my worry as an alternative we could add in the future, if it turns out to be needed | 12:17 |
pkholkin | johnthetubaguy: looking, thank you! | 12:18 |
johnthetubaguy | pkholkin: With my cloud operator hat on, I think not defaulting to false would stop me be able to use this filter, but I could be thinking about that all wrong, and this is easy to add later, so I don't want to block the rest of your good work on that detail | 12:18 |
*** GB21 has quit IRC | 12:19 | |
*** thorst has joined #openstack-nova | 12:20 | |
*** ratailor has quit IRC | 12:27 | |
*** catintheroof has joined #openstack-nova | 12:27 | |
*** thorst has quit IRC | 12:29 | |
*** catintheroof has quit IRC | 12:31 | |
pkholkin | johnthetubaguy: ok, thanks a lot for support! | 12:32 |
openstackgerrit | jichenjc proposed openstack/nova: WIP: remove keypair server extension https://review.openstack.org/396611 | 12:33 |
*** josecastroleon has joined #openstack-nova | 12:35 | |
*** catintheroof has joined #openstack-nova | 12:40 | |
openstackgerrit | Vasyl Saienko proposed openstack/nova-specs: Proposal for ironic static portgroups support https://review.openstack.org/387534 | 12:40 |
openstackgerrit | jichenjc proposed openstack/nova: WIP: remove keypair server extension https://review.openstack.org/396611 | 12:42 |
*** prateek has quit IRC | 12:43 | |
*** jjung has joined #openstack-nova | 12:44 | |
*** markvoelker has quit IRC | 12:44 | |
*** gcb has quit IRC | 12:45 | |
*** preethi has quit IRC | 12:46 | |
*** jaosorior has quit IRC | 12:47 | |
*** brault has joined #openstack-nova | 12:48 | |
*** xavior1777 has joined #openstack-nova | 12:50 | |
*** hshiina has quit IRC | 12:52 | |
*** brault has quit IRC | 12:52 | |
openstackgerrit | Pavel Kholkin proposed openstack/nova-specs: Strict isolation of group of hosts for image and flavor https://review.openstack.org/381912 | 12:55 |
pkholkin | johnthetubaguy: I've fixed some typos and add one sentence after Timofey's comment | 12:56 |
pkholkin | please mark the patch once again, thanks! | 12:56 |
-openstackstatus- NOTICE: Our OpenStack CI system is coming back online again. Thanks for your patience. | 12:59 | |
*** ar1el has quit IRC | 12:59 | |
openstackgerrit | Vasyl Saienko proposed openstack/nova-specs: Proposal for ironic static portgroups support https://review.openstack.org/387534 | 13:01 |
*** thorst has joined #openstack-nova | 13:01 | |
*** thorst_ has joined #openstack-nova | 13:03 | |
bhagyashris | cdent: Hi, I am trying to test set_ and get_ aggregates on the ResourceProvider apis through curl command by applying the patch on my machine https://review.openstack.org/#/c/357993/ , but i am getting 404 Resource NotFound error. can you please guide me to run it successfully. | 13:05 |
*** thorst has quit IRC | 13:05 | |
pkholkin | johnthetubaguy: thanks again! | 13:07 |
*** ducttape_ has joined #openstack-nova | 13:08 | |
*** oanson has joined #openstack-nova | 13:08 | |
*** ar1el has joined #openstack-nova | 13:11 | |
*** gmann has quit IRC | 13:12 | |
*** thorst_ is now known as thorst | 13:12 | |
*** lamt has joined #openstack-nova | 13:16 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild notification https://review.openstack.org/384922 | 13:16 |
openstackgerrit | Alex Szarka proposed openstack/nova: [WIP]Transform error notification extend instance.rebuild https://review.openstack.org/396621 | 13:16 |
*** edmondsw has joined #openstack-nova | 13:16 | |
*** mkoderer___ is now known as mkoderer__ | 13:17 | |
*** alexchadin has quit IRC | 13:18 | |
*** dave-mccowan has joined #openstack-nova | 13:18 | |
*** Hosam has joined #openstack-nova | 13:18 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild notification https://review.openstack.org/384922 | 13:19 |
*** manasm has quit IRC | 13:19 | |
openstackgerrit | jichenjc proposed openstack/nova: WIP: no extension for multiple create https://review.openstack.org/396622 | 13:19 |
*** slaweq_ has quit IRC | 13:26 | |
*** slaweq_ has joined #openstack-nova | 13:26 | |
*** baoli has joined #openstack-nova | 13:28 | |
*** raildo has joined #openstack-nova | 13:28 | |
*** lucas-hungry is now known as lucasagomes | 13:29 | |
*** jwcroppe has joined #openstack-nova | 13:29 | |
*** jwcroppe has quit IRC | 13:29 | |
*** jwcroppe has joined #openstack-nova | 13:30 | |
*** annegentle has quit IRC | 13:32 | |
*** Hosam has quit IRC | 13:32 | |
*** Hosam has joined #openstack-nova | 13:32 | |
*** jwcroppe has quit IRC | 13:34 | |
*** jheroux has joined #openstack-nova | 13:35 | |
*** markvoelker has joined #openstack-nova | 13:35 | |
*** baoli_ has joined #openstack-nova | 13:35 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild.error notification https://review.openstack.org/396621 | 13:36 |
*** Hosam has quit IRC | 13:36 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild.error notification https://review.openstack.org/396621 | 13:37 |
*** claudiub has joined #openstack-nova | 13:38 | |
*** baoli has quit IRC | 13:38 | |
*** ducttape_ has quit IRC | 13:40 | |
*** claudiub|2 has quit IRC | 13:40 | |
*** xyang1 has joined #openstack-nova | 13:43 | |
*** cdent has quit IRC | 13:43 | |
*** jjung has quit IRC | 13:48 | |
*** alexchadin has joined #openstack-nova | 13:48 | |
*** slaweq_ has quit IRC | 13:48 | |
*** slaweq_ has joined #openstack-nova | 13:49 | |
*** tlian has joined #openstack-nova | 13:51 | |
*** alexchadin has quit IRC | 13:53 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild.error notification https://review.openstack.org/396621 | 13:53 |
*** xavior1777 has quit IRC | 14:00 | |
*** bvanhav_ has quit IRC | 14:00 | |
*** edleafe is now known as figleaf | 14:01 | |
alex_xu | jaypipes: hi, hope to get you help on this spec https://review.openstack.org/#/c/393205/, is it worth to add db index for all the columns which are allowed as filter/sort key? | 14:01 |
alex_xu | jaypipes: or is it ok without db index for admin-only filter/sort key? | 14:02 |
*** catinthe_ has joined #openstack-nova | 14:05 | |
*** tonytan4ever has joined #openstack-nova | 14:05 | |
jaypipes | alex_xu: hi. was out with the flu all week. trying to catch up on reviews today. will definitely try to get to that one. | 14:05 |
jaypipes | alex_xu: generally, yes, it's a very good idea to have indexes on anything that is sortable. | 14:05 |
*** mriedem has joined #openstack-nova | 14:07 | |
alex_xu | jaypipes: got it, thanks! | 14:07 |
*** catintheroof has quit IRC | 14:08 | |
*** tblakes has joined #openstack-nova | 14:11 | |
*** manasm has joined #openstack-nova | 14:11 | |
*** Dinesh_Bhor has quit IRC | 14:13 | |
*** tbachman has quit IRC | 14:14 | |
bhagyashris | jaypipes: Hi, I am trying to test [placement] REST API for resource classes curl command by applying the patch https://review.openstack.org/#/c/391959 on my machine , but i am getting 404 Resource NotFound error. can you please guide me to run it successfully. | 14:14 |
*** esberglu has joined #openstack-nova | 14:14 | |
*** haplo37_ has quit IRC | 14:16 | |
*** jorjoro has joined #openstack-nova | 14:16 | |
*** eharney has joined #openstack-nova | 14:17 | |
*** haplo37_ has joined #openstack-nova | 14:19 | |
*** bvanhav_ has joined #openstack-nova | 14:20 | |
*** smatzek has quit IRC | 14:20 | |
*** jjung has joined #openstack-nova | 14:22 | |
*** rama_y has joined #openstack-nova | 14:24 | |
mdbooth | mriedem: So, mea culpa, I'm throwing this over the wall and running away all next week: https://blueprints.launchpad.net/nova/+spec/libvirt-imagebackend-refactor | 14:26 |
openstackgerrit | jichenjc proposed openstack/nova: Add context param to confirm_migration virt call https://review.openstack.org/383699 | 14:26 |
*** victorhe has joined #openstack-nova | 14:26 | |
mdbooth | The bp covers only the refactor, not any of the new functionality | 14:26 |
mdbooth | As we put that out of scope for Ocata anyway | 14:27 |
mriedem | mdbooth: we already had an ocata bp for that | 14:27 |
*** eharney has quit IRC | 14:27 | |
mriedem | https://blueprints.launchpad.net/nova/+spec/libvirt-instance-storage-ocata | 14:27 |
mdbooth | Bleargh | 14:27 |
*** dikonoor has quit IRC | 14:28 | |
mriedem | if you want to use the new bp and supersede https://blueprints.launchpad.net/nova/+spec/libvirt-instance-storage-ocata then that's fine | 14:28 |
mdbooth | Well I guess it's whatever's most useful to you | 14:28 |
mdbooth | I already know what it's about :) | 14:28 |
*** jwcroppe has joined #openstack-nova | 14:28 | |
*** jjung has quit IRC | 14:28 | |
*** victorhe has quit IRC | 14:28 | |
mdbooth | mriedem: I don't mind. I haven't pushed the series which references that bp, yet. | 14:29 |
mriedem | yeah we'll just go with https://blueprints.launchpad.net/nova/+spec/libvirt-imagebackend-refactor since we don't know what we're doing with the storage part yet | 14:29 |
mriedem | i'll supersede the other one | 14:29 |
mdbooth | mriedem: Ok, thanks. | 14:29 |
*** victorhe has joined #openstack-nova | 14:29 | |
mdbooth | I'm going to attempt to push the part of the series I've rebased automatically over the weekend, btw. | 14:29 |
mdbooth | To avoid unnecessary CI load | 14:30 |
mdbooth | So if it doesn't turn up at all, it's because my job failed | 14:30 |
* mdbooth might manually push the first few | 14:30 | |
*** eharney has joined #openstack-nova | 14:31 | |
mdbooth | Although it's not that long right now: only about 9 patches I think | 14:31 |
*** kuzko has quit IRC | 14:31 | |
mriedem | ok | 14:31 |
*** tbachman has joined #openstack-nova | 14:33 | |
*** victorhe has quit IRC | 14:34 | |
mdbooth | mriedem: Thanks | 14:34 |
*** sree has joined #openstack-nova | 14:34 | |
*** kuzko has joined #openstack-nova | 14:35 | |
*** ducttape_ has joined #openstack-nova | 14:36 | |
mdbooth | mriedem: Could you remove your procedural -2 on this one, btw: https://review.openstack.org/#/c/331115/ | 14:36 |
mriedem | done | 14:37 |
*** dansmith is now known as superdan | 14:37 | |
*** sree has quit IRC | 14:39 | |
mriedem | superdan: so, i got to thinking about this series on the way into work https://review.openstack.org/#/c/396469/ | 14:40 |
*** ducttape_ has quit IRC | 14:40 | |
mriedem | superdan: with nova-net, when we store the secgroups we're only getting the name | 14:40 |
mriedem | with ^ we'd be setting the id as the secgroup uuid (except the 'default' group) | 14:41 |
mriedem | so thinking about how this will be used, | 14:41 |
mriedem | when i pull the secgroups out of the reqspec, and i want to look up a secgroup, i have to determine if the name or id is set, and possibly what the type is | 14:41 |
mriedem | which is pretty gross | 14:41 |
mriedem | so i was thinking we could avoid the NeutronSecurityGroup new object if we just populate the list of secgroups for neutron by setting the name field too, like nova-net | 14:42 |
openstackgerrit | Julian proposed openstack/nova: Remove redundant VersionedObject Fields https://review.openstack.org/389840 | 14:42 |
superdan | you mean for rebuild? | 14:42 |
mriedem | superdan: for whatever uses it | 14:42 |
mriedem | setting secgroup.name = uuid isn't awesome either, | 14:42 |
mriedem | but i'm not sure it's worse, and it would be less new complexity from that series, | 14:42 |
mriedem | and on the side that uses it, you'd still need to know if you're using neutron or not | 14:43 |
superdan | mriedem: but a method on the object called get_ident() that does the logic of "if uuid, else name" would be easy, then you don't need to care | 14:43 |
superdan | name = uuid is not okay, IMHO | 14:43 |
mriedem | id = 'default' isn't great either :) | 14:43 |
mriedem | but i guess we could massage that | 14:43 |
mriedem | with a get_identifer() method | 14:43 |
artom | mdbooth, oy, enough with the sass ;) | 14:43 |
superdan | mriedem: yeah | 14:43 |
mdbooth | artom: I wasn't trying to be sassy :) | 14:43 |
mriedem | i didn't detect any sass | 14:44 |
mdbooth | artom: Well, maybe a little | 14:44 |
mdbooth | :P | 14:44 |
mdbooth | mriedem: I assume artom is referring to https://review.openstack.org/#/c/337158/ | 14:44 |
*** dhellmann has left #openstack-nova | 14:45 | |
mdbooth | artom: It's fine, I spent about 20 minutes rewriting the commit message, and I had to remind myself of the code flow. | 14:45 |
mdbooth | It was a useful exercise. | 14:45 |
mriedem | superdan: ok so to make sure we're on the same page, in https://review.openstack.org/#/c/396436/2 the NeutronSecurityGroup.id field is a uuid field | 14:45 |
artom | mdbooth, your essay will be evaluated on structure, quality of English, and creativity. | 14:45 |
mriedem | and can only be uuid | 14:45 |
mriedem | superdan: if the secgroup requested is 'default', then we set NeutronSecurityGroup.name | 14:46 |
mdbooth | artom: Prepare to be disappointed :) | 14:46 |
superdan | mriedem: yeah | 14:46 |
* mdbooth likes lists of things and short sentences. | 14:46 | |
mriedem | superdan: and then the get_identifier() method goes on NeutronSecurityGroup or the parent SecurityGroup class? | 14:46 |
superdan | mriedem: tbh, I wasn't going to do a separate object for this, and I'm not sure it really helps anything | 14:46 |
mriedem | i guess the parent would have the id field and know if it's a uuid or not | 14:47 |
mriedem | i was following what was done for NeutronFloatingIP | 14:47 |
superdan | mriedem: oh, I missed that you redefined id in the subclass.. not really sure that's okay actually | 14:47 |
mriedem | that's what we do for NeutronFloatingIP... | 14:47 |
*** krtaylor has joined #openstack-nova | 14:47 | |
superdan | mriedem: especially if you expect to put it into a field that expects subclasses | 14:47 |
mriedem | it's not a registered object, it's just for passing around | 14:47 |
*** sridharg has quit IRC | 14:48 | |
*** bvanhav_ is now known as bvanhav | 14:48 | |
superdan | mriedem: you can't ever pass it over rpc if it's not registered | 14:48 |
mriedem | right, i was wondering about that last night, because with NeutronFloatingIP it's just a convenience between the neutronv2 api code and the REST API | 14:48 |
superdan | yeah | 14:48 |
mriedem | and we don't put it into a FloatingIPList object, we just return it in a primitive list | 14:48 |
mriedem | ok so you're thinking add a uuid field to SecurityGroup proper? | 14:49 |
mriedem | i'm ok with that approach too | 14:49 |
mriedem | thought about it last night but didn't think of a get_identifier() helper | 14:49 |
superdan | mriedem: yeah so if we just added a uuid, we could potentially stuff it into the db for n-net if we were keeping it, but since we're not, just make sure it's ignored there | 14:49 |
superdan | mriedem: that would be my plan | 14:49 |
mriedem | yeah, like i said, n-net just sets the name field and neutron would set name or uuid field | 14:49 |
superdan | yeah | 14:50 |
mriedem | ok, i'll make that change | 14:50 |
*** david-lyle_ has joined #openstack-nova | 14:50 | |
superdan | I would set name and uuid for neutron, but understand why you might not want to | 14:50 |
superdan | and then prefer uuid always | 14:50 |
superdan | but anyway | 14:50 |
mriedem | well, we don't have both at that point | 14:50 |
mriedem | we could, but it's more change | 14:50 |
mriedem | in compute api i mean | 14:50 |
superdan | compute api would just use get_ident() everywhere | 14:51 |
superdan | but anyway, it's a nit | 14:51 |
mriedem | name for n-net is unique per tenant, but not for neutron - uuid is unique | 14:51 |
superdan | right, I know | 14:51 |
mriedem | i meant in _check_requested_security_groups | 14:51 |
mriedem | that's my bottom change | 14:51 |
mriedem | translates the list of name string sto uuids | 14:51 |
superdan | anyway, this is a tiny detail, not worth arguing over | 14:51 |
mriedem | i'll be in the cold cold ground before i submit to your name + uuid idea, smith! | 14:52 |
superdan | heh | 14:52 |
* superdan steals a shameless review stat | 14:57 | |
*** bhagyashris has quit IRC | 14:57 | |
mriedem | superdan: for SecurityGroup.save do we worry about popping 'uuid' out of updates in case someone mistakenly tries to save a secgroup object that has uuid set? | 14:58 |
*** kaisers_ has quit IRC | 14:58 | |
mriedem | that would blow up in the db api | 14:58 |
rfolco | mdbooth, I'm looking for an entry point to help with https://blueprints.launchpad.net/nova/+spec/libvirt-imagebackend-refactor, any pointers on where to start ? | 14:58 |
mriedem | so not sure we need to explicitly check for it | 14:58 |
*** kaisers_ has joined #openstack-nova | 14:59 | |
mdbooth | rfolco: You've caught me at the absolute worst time, unfortunately, as I'm about to have a meeting then leave for the day, and I'm off next week. | 14:59 |
superdan | mriedem: yeah I think you should | 14:59 |
superdan | mriedem: I mean, it's one line for safety right? | 14:59 |
mdbooth | rfolco: Is there any chance you could ping me again a week on Monday? | 14:59 |
mriedem | superdan: yeah | 15:00 |
mriedem | will do | 15:00 |
rfolco | mdbooth, works for me. Will study the code until there. | 15:00 |
rfolco | mdbooth, enjoy your week off, thanks. | 15:01 |
*** walharthi has joined #openstack-nova | 15:03 | |
*** tbachman_ has joined #openstack-nova | 15:04 | |
*** preethipy has joined #openstack-nova | 15:04 | |
*** pbandark`brb has quit IRC | 15:05 | |
*** tanvirt has joined #openstack-nova | 15:05 | |
*** jorjoro has left #openstack-nova | 15:05 | |
*** tbachman has quit IRC | 15:06 | |
*** rama_y has quit IRC | 15:06 | |
*** tbachman_ is now known as tbachman | 15:06 | |
*** rama_y has joined #openstack-nova | 15:06 | |
*** ducttape_ has joined #openstack-nova | 15:07 | |
*** mlavalle has joined #openstack-nova | 15:07 | |
*** cdent has joined #openstack-nova | 15:08 | |
cdent | did anyone respond to bhagyashris about needing to set the microversion? I've added a note to the review | 15:14 |
*** smatzek has joined #openstack-nova | 15:14 | |
mriedem | cdent: which review? | 15:15 |
cdent | mriedem: [t qk3] | 15:15 |
purplerbot | <bhagyashris> cdent: Hi, I am trying to test set_ and get_ aggregates on the ResourceProvider apis through curl command by applying the patch on my machine https://review.openstack.org/#/c/357993/ , but i am getting 404 Resource NotFound error. can you please guide me to run it successfully. [2016-11-11 13:05:40] [n qk3] | 15:15 |
mriedem | oh i see | 15:16 |
mriedem | they asked the same about getting resource classes | 15:16 |
cdent | yeah [t BCs] | 15:16 |
purplerbot | <bhagyashris> jaypipes: Hi, I am trying to test [placement] REST API for resource classes curl command by applying the patch https://review.openstack.org/#/c/391959 on my machine , but i am getting 404 Resource NotFound error. can you please guide me to run it successfully. [2016-11-11 14:14:15] [n BCs] | 15:16 |
*** sree has joined #openstack-nova | 15:16 | |
mriedem | we need some docs on the placement REST API | 15:16 |
mriedem | so we can redirect there | 15:16 |
cdent | both boost the microversion, so I'm guessing that wasn't clear obvious | 15:16 |
cdent | yes: that's in here: http://lists.openstack.org/pipermail/openstack-dev/2016-November/107171.html | 15:17 |
mriedem | probably not, especially since we don't have an external client doing that for you automatically like novaclient does | 15:17 |
cdent | "that's" == need for docs | 15:17 |
cdent | I actually think that until we are "done" with what could legitmately be called the first version of the placement api we should be microversioning | 15:18 |
cdent | we should wait for stability and call that 1.0 | 15:18 |
melwitt | jaypipes: at the summit you mentioned re-looking at the root_gb=0 RT patch of mine again. when you get a chance, could you let me know what your thoughts were on that? | 15:18 |
cdent | without aggregates and resource classes it isn't fully functional | 15:18 |
*** yamahata has joined #openstack-nova | 15:19 | |
cdent | s/we should be/we shouldn't be/ (sigh) | 15:20 |
*** mlavalle has quit IRC | 15:21 | |
openstackgerrit | Vasyl Saienko proposed openstack/nova-specs: Proposal for ironic static portgroups support https://review.openstack.org/387534 | 15:21 |
*** tbachman_ has joined #openstack-nova | 15:22 | |
*** mlavalle has joined #openstack-nova | 15:24 | |
*** tbachman has quit IRC | 15:25 | |
*** tbachman_ is now known as tbachman | 15:25 | |
*** manasm has quit IRC | 15:26 | |
*** mriedem1 has joined #openstack-nova | 15:26 | |
mriedem1 | cdent: not really sure but we are going to require that the placement service is stood up in neutron before going to ocata, | 15:26 |
mriedem1 | so it seemed prudent to have microversions for the client in ocata to know what it can and can't do | 15:27 |
*** liverpooler has joined #openstack-nova | 15:27 | |
*** mriedem has quit IRC | 15:27 | |
mriedem1 | s/neutron/newton/ | 15:27 |
* cdent nods | 15:27 | |
cdent | microversioning is all there and fine and working, so its fine | 15:28 |
mriedem1 | we should probably figure out the upgrade plan before making a decision on if we need to be using microversions in ocata, | 15:28 |
cdent | that's on that email summary too | 15:29 |
mriedem1 | i.e. do we require that placement (nova-api) is updated before computes | 15:29 |
cdent | (the need to figure it out, not a plan) | 15:29 |
*** claudiub|2 has joined #openstack-nova | 15:29 | |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: Add new hv_type for supporting nested hypervisors https://review.openstack.org/396670 | 15:29 |
mriedem1 | superdan: keep me honest, today upgrade is conductor, api and then computes? | 15:29 |
superdan | mriedem1: yar | 15:29 |
mriedem1 | if we have ^ then newton computes will be fine with ocata placement api | 15:30 |
mriedem1 | as long as the ocata placement api doesn't make backward incompatible changes to break newton computes | 15:30 |
mriedem1 | which is why we have microversions | 15:30 |
*** claudiub|3 has joined #openstack-nova | 15:30 | |
mriedem1 | newton computes will only request 1.0 | 15:30 |
*** mriedem1 is now known as mriedem | 15:30 | |
mriedem | but with cells v2 we're going to do conductors in all cells, then computes, and then api | 15:31 |
cdent | it's going to be so much fun | 15:31 |
superdan | mriedem: no | 15:31 |
mriedem | all conductors, then api and then all computes? | 15:32 |
superdan | mriedem: for cell in cells: conductor; compute; api | 15:32 |
*** takedakn has joined #openstack-nova | 15:32 | |
superdan | hmm, that wasn't as obvious as I wanted | 15:32 |
superdan | upgrade cells then upgrade api when they're all done | 15:32 |
superdan | within a cell, conductors first, if desired | 15:32 |
*** claudiub has quit IRC | 15:33 | |
*** preethipy has quit IRC | 15:33 | |
*** claudiub|2 has quit IRC | 15:33 | |
*** zzelle has joined #openstack-nova | 15:33 | |
mriedem | i think i meant the same thing | 15:34 |
superdan | okay sorry | 15:34 |
mriedem | either way api last | 15:34 |
superdan | yeah | 15:34 |
superdan | I want to avoid having to touch all cells in parallel, | 15:34 |
mriedem | so ocata RT can be requesting let's say placement 1.3 and placement might be newton so only has 1.0 | 15:34 |
*** tbachman has quit IRC | 15:34 | |
superdan | so "conductors in all cells, then computes" makes it sound like you upgrade the conductor in each cell, then you can move on and do computes in each cell, etc | 15:34 |
mriedem | cell by cell | 15:35 |
mriedem | within a cell, conductors and then computes | 15:35 |
mriedem | once all done, upgrade api | 15:35 |
mriedem | right? | 15:35 |
superdan | mriedem: yeah, but when placement is moved outside, we'll need to be legit with our api negotiation anyway, just like we should be with cinder and neutron | 15:35 |
superdan | mriedem: yes | 15:35 |
mriedem | superdan: right, that too, i want to treat placement like it's already outside of nova | 15:35 |
superdan | mriedem: right | 15:35 |
mriedem | so doing version discovery | 15:35 |
superdan | agreed | 15:35 |
cdent | speaking of the tracker using placement, superdan, have you seen : https://review.openstack.org/#/c/395971/ | 15:36 |
cdent | fix for max_unit handling | 15:36 |
*** marst has quit IRC | 15:38 | |
*** jaypipes is now known as leakypipes | 15:39 | |
leakypipes | melwitt: link for me pls? | 15:39 |
leakypipes | purplerbot: did you add "OpenStack-API-Version: placement 1.1" header? | 15:40 |
*** rama_y has quit IRC | 15:40 | |
melwitt | leakypipes: https://review.openstack.org/355091 | 15:40 |
leakypipes | melwitt: cheers | 15:40 |
*** rama_y has joined #openstack-nova | 15:40 | |
*** liverpooler has quit IRC | 15:40 | |
*** mriedem has quit IRC | 15:41 | |
*** zz_dimtruck is now known as dimtruck | 15:44 | |
superdan | cdent: I had not | 15:44 |
*** mriedem has joined #openstack-nova | 15:45 | |
superdan | cdent: also not sure that's the right thing, as I thought max_unit was supposed to be a multiplier | 15:45 |
cdent | superdan: the author of that basically pulled the necessary from leakypipes' original version | 15:45 |
superdan | hmm | 15:45 |
cdent | step_size is the multiplier | 15:45 |
*** fragatina has quit IRC | 15:45 | |
*** alexchadin has joined #openstack-nova | 15:45 | |
superdan | so max_unit is the largest single allocation allowed? | 15:45 |
cdent | yes | 15:46 |
superdan | so should that be memory - reserved ? | 15:46 |
*** fragatina has joined #openstack-nova | 15:46 | |
cdent | that's a question I was wondering too | 15:46 |
markus_z | melwitt / danpb: https://review.openstack.org/#/c/395067/ would resolve a bug which blocks the s390x CI. Would be nice if you could take a look. | 15:47 |
superdan | cdent: but you didn't comment? | 15:47 |
superdan | cdent: left a question, maybe leakypipes will hit that first | 15:47 |
*** marst has joined #openstack-nova | 15:47 | |
*** eharney has quit IRC | 15:47 | |
cdent | I didn't leave a comment because the original code: https://review.openstack.org/#/c/363583/1/nova/scheduler/client/report.py | 15:48 |
cdent | line 85 | 15:48 |
superdan | still deserves review, but we'll see what he says | 15:48 |
cdent | well, exactly, that's why I'm pointing it out to you | 15:49 |
cdent | my +1 is effectively "this is good enough to be in the ballpark, and I'd like jay and dan to see it" | 15:49 |
melwitt | markus_z: will take a look | 15:50 |
mriedem | markus_z: that's a beautiful change and i applaud the submitter | 15:50 |
*** walharthi has quit IRC | 15:51 | |
markus_z | mriedem: I printed and framed it already. Looks nice at my wall. | 15:51 |
superdan | cdent: noted | 15:51 |
cdent | roger over under done | 15:51 |
melwitt | :) | 15:51 |
*** slaweq_ has quit IRC | 15:52 | |
markus_z | mriedem: thanks a lot for solving that (I didn't say that before) | 15:52 |
*** hongbin has joined #openstack-nova | 15:52 | |
*** jaugustine has joined #openstack-nova | 15:53 | |
mriedem | np | 15:53 |
*** slaweq_ has joined #openstack-nova | 15:53 | |
leakypipes | superdan, cdent: trying to get through email backlog. will be reviewing in about 30 minutes (hopefully). sorry, still catching up after being in bed for the past 3 days. :( | 15:54 |
cdent | leakypipes: we have all the time in the world, really. No rush. You feeling better, or just too bored not to work? | 15:54 |
*** sree has quit IRC | 15:55 | |
*** rrecio_ has joined #openstack-nova | 15:56 | |
leakypipes | cdent: I'm cogent, which is more than I can say for the last couple days. | 15:57 |
openstackgerrit | Roman Podoliaka proposed openstack/nova: block_device: fix propagation of instance AZ to Cinder https://review.openstack.org/366724 | 15:57 |
*** smatzek has quit IRC | 15:57 | |
cdent | cogency is overrated | 15:57 |
*** alexchadin has quit IRC | 15:57 | |
*** zenoway has quit IRC | 15:57 | |
cdent | leakypipes: I assume you've grokked that I rebased your nested resource providers stack. Anything else like that I could or should help with? | 15:58 |
*** zenoway has joined #openstack-nova | 15:58 | |
*** smatzek has joined #openstack-nova | 15:58 | |
*** prashkre has quit IRC | 15:58 | |
*** eharney has joined #openstack-nova | 16:00 | |
*** sree has joined #openstack-nova | 16:01 | |
*** Zer0Byte__ has joined #openstack-nova | 16:01 | |
*** zenoway has quit IRC | 16:02 | |
leakypipes | cdent: no, thank you for that. I had it done locally but didn't push it before going into flu-bed. :( | 16:03 |
leakypipes | superdan, cdent: ok, reviewed the max_unit patch. | 16:03 |
*** annegentle has joined #openstack-nova | 16:04 | |
*** jaugustine has quit IRC | 16:04 | |
*** rcernin has quit IRC | 16:05 | |
*** sree has quit IRC | 16:05 | |
openstackgerrit | Sujitha proposed openstack/nova: Improve help text for network options. https://review.openstack.org/341203 | 16:05 |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: fix formatting in wsgi https://review.openstack.org/396374 | 16:06 |
*** Oku_OS is now known as Oku_OS-away | 16:06 | |
*** marst has quit IRC | 16:06 | |
*** marst has joined #openstack-nova | 16:06 | |
*** david-lyle_ has quit IRC | 16:07 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: return SecurityGroupList from populate_security_groups https://review.openstack.org/396469 | 16:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add uuid field to SecurityGroup object https://review.openstack.org/396436 | 16:07 |
*** rama_y has quit IRC | 16:07 | |
mriedem | alright superdan thar she blar | 16:08 |
superdan | ack | 16:08 |
mriedem | i'm going to be out for awhile | 16:08 |
leakypipes | CIAO | 16:11 |
leakypipes | yikes, sorry for yelling. | 16:11 |
*** scsnow has quit IRC | 16:11 | |
*** ijw has joined #openstack-nova | 16:13 | |
*** fragatina has quit IRC | 16:16 | |
*** jvgrant__ has quit IRC | 16:16 | |
*** jvgrant__ has joined #openstack-nova | 16:17 | |
*** efried1 has joined #openstack-nova | 16:20 | |
*** tonytan4ever has quit IRC | 16:20 | |
*** bnemec is now known as beekneemech | 16:22 | |
*** efried has quit IRC | 16:22 | |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Rewrite _test_finish_migration https://review.openstack.org/337159 | 16:22 |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Test disk creation in test_hard_reboot https://review.openstack.org/338993 | 16:22 |
openstackgerrit | Matthew Edmonds proposed openstack/nova: Separate CRUD policy for server_groups https://review.openstack.org/391113 | 16:23 |
*** gmann has joined #openstack-nova | 16:23 | |
*** armax has joined #openstack-nova | 16:23 | |
mriedem | cdent: ok put some thoughts in your rp update email | 16:24 |
mriedem | thanks for sending that | 16:24 |
cdent | cool | 16:24 |
* cdent likes thoughts | 16:24 | |
openstackgerrit | Matthew Edmonds proposed openstack/nova: Separate CRUD policy for server_groups https://review.openstack.org/391113 | 16:25 |
*** bvanhav has quit IRC | 16:26 | |
*** mriedem1 has joined #openstack-nova | 16:26 | |
*** mriedem1 has quit IRC | 16:26 | |
*** mriedem1 has joined #openstack-nova | 16:27 | |
*** mriedem1 is now known as mriedem_afk | 16:27 | |
*** gmann has quit IRC | 16:28 | |
*** annegentle has quit IRC | 16:28 | |
*** mriedem has quit IRC | 16:28 | |
*** annegentle has joined #openstack-nova | 16:29 | |
*** takedakn has quit IRC | 16:30 | |
leakypipes | alex_xu: reviewed. | 16:31 |
*** haplo37 has quit IRC | 16:31 | |
* cdent heads home | 16:32 | |
*** cdent has quit IRC | 16:32 | |
*** haplo37 has joined #openstack-nova | 16:32 | |
*** avolkov has quit IRC | 16:38 | |
edmondsw | mriedem, can you help get this cherrypick merged? https://review.openstack.org/#/c/396151/ | 16:39 |
edmondsw | that regression issue we fixed in master the other day | 16:39 |
*** browne has joined #openstack-nova | 16:41 | |
*** victorhe has joined #openstack-nova | 16:41 | |
*** Guest7605 has quit IRC | 16:43 | |
*** josecastroleon has quit IRC | 16:44 | |
*** slaweq_ has quit IRC | 16:46 | |
*** stewie925 has joined #openstack-nova | 16:47 | |
*** slaweq_ has joined #openstack-nova | 16:47 | |
*** armax has quit IRC | 16:49 | |
*** bvanhav has joined #openstack-nova | 16:50 | |
*** tbachman has joined #openstack-nova | 16:50 | |
*** belmoreira has quit IRC | 16:50 | |
*** armax has joined #openstack-nova | 16:51 | |
*** ralonsoh has quit IRC | 16:53 | |
*** david-lyle_ has joined #openstack-nova | 16:56 | |
*** walharthi has joined #openstack-nova | 16:57 | |
*** nic has joined #openstack-nova | 16:58 | |
*** david-lyle has quit IRC | 16:58 | |
*** preethipy has joined #openstack-nova | 16:59 | |
*** david-lyle_ has quit IRC | 17:01 | |
*** rama_y has joined #openstack-nova | 17:04 | |
openstackgerrit | int32bit proposed openstack/nova: Add lock actions to action-list https://review.openstack.org/396510 | 17:05 |
markus_z | :D that username | 17:06 |
*** tojuvone has joined #openstack-nova | 17:06 | |
*** tojuvone_ has joined #openstack-nova | 17:06 | |
*** walharthi has quit IRC | 17:07 | |
*** tojuvone_ has quit IRC | 17:07 | |
*** Swami has joined #openstack-nova | 17:08 | |
*** mdrabe has joined #openstack-nova | 17:11 | |
*** mrsoul has quit IRC | 17:12 | |
*** cdent has joined #openstack-nova | 17:12 | |
*** sambetts is now known as sambetts|afk | 17:16 | |
*** Apoorva has joined #openstack-nova | 17:22 | |
*** paul-carlton2 has quit IRC | 17:26 | |
leakypipes | melwitt: reviewed. | 17:27 |
*** unicell has quit IRC | 17:28 | |
melwitt | thanks leakypipes | 17:28 |
*** unicell has joined #openstack-nova | 17:28 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild.error notification https://review.openstack.org/396621 | 17:39 |
*** derekh has quit IRC | 17:42 | |
*** danpb has quit IRC | 17:42 | |
*** lucasagomes is now known as lucas-afk | 17:48 | |
*** smatzek has quit IRC | 17:51 | |
*** sudipto has quit IRC | 17:51 | |
*** sudipto_ has quit IRC | 17:51 | |
*** esberglu has quit IRC | 17:51 | |
*** smatzek has joined #openstack-nova | 17:52 | |
*** esberglu has joined #openstack-nova | 17:52 | |
*** nherciu has joined #openstack-nova | 17:53 | |
*** kaisers_ has quit IRC | 17:54 | |
*** yamahata has quit IRC | 17:55 | |
*** walharthi has joined #openstack-nova | 17:56 | |
*** bvanhav_ has joined #openstack-nova | 17:58 | |
*** david-lyle has joined #openstack-nova | 17:58 | |
*** bvanhav has quit IRC | 17:58 | |
*** alexchadin has joined #openstack-nova | 18:01 | |
*** david-lyle has quit IRC | 18:03 | |
mriedem_afk | edmondsw: yeah | 18:03 |
*** mriedem_afk is now known as mriedem | 18:03 | |
*** tbachman_ has joined #openstack-nova | 18:04 | |
openstackgerrit | jichenjc proposed openstack/python-novaclient: Add version pin for image related function https://review.openstack.org/392523 | 18:05 |
*** tbachman has quit IRC | 18:07 | |
*** tbachman_ is now known as tbachman | 18:07 | |
dims | feeling better leakypipes ? | 18:09 |
mriedem | cdent: i'll take a crack at starting some placement docs in tree this afternoon | 18:09 |
mriedem | nothing fancy but it'll be a start | 18:09 |
*** fragatina has joined #openstack-nova | 18:10 | |
*** rcernin has joined #openstack-nova | 18:10 | |
cdent | mriedem: great, having a structure to start attaching things to will overcome inertia nicely | 18:11 |
*** ihrachys has quit IRC | 18:12 | |
*** rcernin has quit IRC | 18:12 | |
*** brault has joined #openstack-nova | 18:12 | |
*** ijw has quit IRC | 18:15 | |
*** brault has quit IRC | 18:17 | |
*** lpetrut has quit IRC | 18:22 | |
*** tbachman has quit IRC | 18:23 | |
hongbin | mriedem: hi, i am working on openstack container project, i need your help for one thing. do you have a minute? | 18:24 |
*** preethipy has quit IRC | 18:27 | |
*** fragatina has quit IRC | 18:27 | |
*** unicell has quit IRC | 18:28 | |
*** fragatina has joined #openstack-nova | 18:28 | |
mriedem | hongbin: sure, i'll try | 18:29 |
*** david-lyle_ has joined #openstack-nova | 18:30 | |
hongbin | mriedem: thx. i have a patch for devstack, and they wanted someone from nova to review it before approving. i wanted to know if you could recommend a person to review it | 18:30 |
hongbin | mriedem: here is the patch: https://review.openstack.org/#/c/382063/ | 18:30 |
mriedem | this is to load a nova-docker virt driver or something? | 18:32 |
hongbin | yes, to load a virt driver that is similar to nova-docker | 18:32 |
mriedem | btw ianw asked for sdague probably because of his involvement with devstack more so than nova | 18:32 |
*** victorhe has quit IRC | 18:32 | |
hongbin | sdague is not online.... | 18:33 |
mriedem | hongbin: why can't you use something like https://github.com/openstack-dev/devstack/tree/master/lib/nova_plugins/hypervisor-docker then? | 18:33 |
*** smatzek_ has joined #openstack-nova | 18:33 | |
hongbin | the link is broken | 18:33 |
*** victorhe has joined #openstack-nova | 18:34 | |
mriedem | i mean a lib/nova_plugins/hypervisor-docker file | 18:34 |
mriedem | in the devstack plugin | 18:34 |
*** alexchadin has quit IRC | 18:34 | |
mriedem | why does the lib/nova_plugins dir need to be overridden? | 18:34 |
mriedem | if you're loading something up into nova | 18:34 |
hongbin | that means every chances we wanted need to go through devstack review process | 18:34 |
hongbin | which is undesirable..... | 18:34 |
*** yamahata has joined #openstack-nova | 18:35 | |
edmondsw | mriedem tx | 18:35 |
mriedem | i'm not following, you don't need to go through devstack if you have a devstack plugin | 18:35 |
mriedem | hongbin: see how the powervm driver does it http://git.openstack.org/cgit/openstack/nova-powervm/tree/devstack/plugin.sh | 18:35 |
hongbin | if i override the variable, i can have a out-of-tree plugin | 18:35 |
mriedem | hongbin: you don't need to override the variable to have an out of tree plugin | 18:35 |
hongbin | mriedem: ok, good to know. let me checkout the powervm plugin | 18:36 |
*** smatzek has quit IRC | 18:36 | |
*** david-lyle_ has quit IRC | 18:36 | |
*** david-lyle has joined #openstack-nova | 18:36 | |
*** markus_z has quit IRC | 18:37 | |
*** annegentle has quit IRC | 18:37 | |
*** annegentle has joined #openstack-nova | 18:39 | |
*** Sukhdev has joined #openstack-nova | 18:40 | |
mriedem | superdan: can you answer the riddle of 'these' in here? https://review.openstack.org/#/c/396436/3/nova/objects/security_group.py | 18:41 |
superdan | yes | 18:42 |
mriedem | ok i'll remove it | 18:42 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: return SecurityGroupList from populate_security_groups https://review.openstack.org/396469 | 18:43 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add uuid field to SecurityGroup object https://review.openstack.org/396436 | 18:43 |
*** rrecio_ has quit IRC | 18:43 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: actually populate list in populate_security_groups https://review.openstack.org/396469 | 18:44 |
*** gszasz has quit IRC | 18:44 | |
*** yamahata has quit IRC | 18:44 | |
*** josecastroleon has joined #openstack-nova | 18:45 | |
*** yamahata has joined #openstack-nova | 18:45 | |
*** josecastroleon has quit IRC | 18:46 | |
*** ijw has joined #openstack-nova | 18:47 | |
*** tonytan4ever has joined #openstack-nova | 18:54 | |
*** tonytan4ever has quit IRC | 18:54 | |
*** tonytan4ever has joined #openstack-nova | 18:54 | |
*** harlowja has joined #openstack-nova | 18:55 | |
*** catinthe_ has quit IRC | 18:56 | |
*** unicell has joined #openstack-nova | 18:56 | |
*** annegentle has quit IRC | 18:59 | |
*** annegentle has joined #openstack-nova | 19:01 | |
*** nherciu has quit IRC | 19:04 | |
openstackgerrit | Karen Bradshaw proposed openstack/nova: API Ref: update server_id params https://review.openstack.org/384664 | 19:05 |
*** ijw has quit IRC | 19:06 | |
*** pcaruana has quit IRC | 19:07 | |
*** cdent has quit IRC | 19:08 | |
*** belliott has joined #openstack-nova | 19:08 | |
*** jamesdenton has joined #openstack-nova | 19:14 | |
*** tbachman has joined #openstack-nova | 19:15 | |
openstackgerrit | int32bit proposed openstack/nova: Add lock actions to action-list https://review.openstack.org/396510 | 19:15 |
*** nherciu has joined #openstack-nova | 19:17 | |
*** tbachman_ has joined #openstack-nova | 19:20 | |
*** burt has joined #openstack-nova | 19:22 | |
*** tbachman has quit IRC | 19:22 | |
*** tbachman_ is now known as tbachman | 19:22 | |
*** catintheroof has joined #openstack-nova | 19:25 | |
*** catintheroof has quit IRC | 19:27 | |
*** tbachman_ has joined #openstack-nova | 19:28 | |
*** catintheroof has joined #openstack-nova | 19:29 | |
*** victorhe has quit IRC | 19:29 | |
*** tbachman has quit IRC | 19:31 | |
*** tbachman_ is now known as tbachman | 19:31 | |
*** ijw has joined #openstack-nova | 19:33 | |
*** nherciu has quit IRC | 19:33 | |
*** lpetrut has joined #openstack-nova | 19:36 | |
*** ijw has quit IRC | 19:37 | |
*** catintheroof has quit IRC | 19:42 | |
*** annegentle has quit IRC | 19:44 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Placement api: Add informative message to 404 response. https://review.openstack.org/392321 | 19:46 |
*** annegentle has joined #openstack-nova | 19:49 | |
*** zzelle has quit IRC | 19:51 | |
superdan | melwitt: is the db_connection in context supposed to make it over rpc? | 19:53 |
*** catintheroof has joined #openstack-nova | 19:55 | |
*** TravT has joined #openstack-nova | 19:56 | |
*** catintheroof has quit IRC | 19:59 | |
*** annegentle has quit IRC | 19:59 | |
*** annegentle has joined #openstack-nova | 20:00 | |
melwitt | superdan: no | 20:02 |
superdan | okay good, 'cause it ain't :) | 20:02 |
melwitt | heh. it contains sensitive info so we didn't want it to go over rpc if possible | 20:03 |
melwitt | and we thought it shouldn't be needed to go over rpc | 20:03 |
superdan | yeah | 20:03 |
kashyap | melwitt: mriedem: Following up from last evening, I just tested -- you don't need to have to `pip install` anything when you download a patch from `git-review -d $ID` into Nova's Git tree | 20:09 |
kashyap | DevStack works it out just fine. | 20:09 |
*** rrecio has joined #openstack-nova | 20:11 | |
*** cfriesen has quit IRC | 20:11 | |
melwitt | kashyap: that's good. did you figure out what was going wrong with whatever you were trying to do last time? | 20:12 |
*** walharthi has quit IRC | 20:12 | |
kashyap | melwitt: No, that franken VM has given me enough grief that I don't want to touch it with a 10 foot pole | 20:13 |
* kashyap is on flaky mobile data on the train; might drop off any moment | 20:13 | |
melwitt | haha, okay | 20:13 |
*** ijw has joined #openstack-nova | 20:14 | |
*** ijw has quit IRC | 20:19 | |
*** tblakes has quit IRC | 20:21 | |
*** rrecio_ has joined #openstack-nova | 20:23 | |
*** rrecio has quit IRC | 20:23 | |
*** Hosam has joined #openstack-nova | 20:23 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add the initial documentation for the placement API https://review.openstack.org/396761 | 20:27 |
mriedem | figleaf: leakypipes: superdan: ^ a start | 20:27 |
figleaf | mriedem: cool; will review soon | 20:28 |
*** thorst has quit IRC | 20:31 | |
leakypipes | mriedem: awesomesauce. will hop right on that. thanks much! | 20:31 |
*** thorst has joined #openstack-nova | 20:32 | |
leakypipes | how did nicolas bock do a +1 so quickly on that? :( | 20:32 |
leakypipes | must be a very fast reader. | 20:33 |
openstackgerrit | jichenjc proposed openstack/nova: WIP: no extension for multiple create https://review.openstack.org/396622 | 20:34 |
*** thorst has quit IRC | 20:36 | |
*** tblakes has joined #openstack-nova | 20:37 | |
*** clenimar has quit IRC | 20:37 | |
*** sayalilunkad has quit IRC | 20:38 | |
*** sayalilunkad has joined #openstack-nova | 20:39 | |
*** alexpilotti has joined #openstack-nova | 20:40 | |
*** bvanhav_ has quit IRC | 20:41 | |
*** smatzek_ has quit IRC | 20:41 | |
*** jwcroppe has quit IRC | 20:44 | |
*** jwcroppe has joined #openstack-nova | 20:45 | |
*** catintheroof has joined #openstack-nova | 20:45 | |
jroll | our portgroups thing is now ready for y'all's spec review jam :) https://review.openstack.org/#/c/387534/ | 20:45 |
*** jwcroppe has quit IRC | 20:49 | |
*** Sukhdev has quit IRC | 20:50 | |
openstackgerrit | Sujitha proposed openstack/nova: Improve help text for network options. https://review.openstack.org/341203 | 20:51 |
*** thorst has joined #openstack-nova | 20:54 | |
*** victorhe has joined #openstack-nova | 20:55 | |
*** raildo has quit IRC | 20:55 | |
*** ijw has joined #openstack-nova | 20:55 | |
*** lpetrut has quit IRC | 20:57 | |
*** thorst has quit IRC | 20:58 | |
*** Apoorva has quit IRC | 20:58 | |
*** haplo37 has quit IRC | 20:58 | |
superdan | melwitt: so, I've made progress such that I can run functional tests in a single thread | 20:59 |
superdan | melwitt: but not in parallel | 20:59 |
superdan | melwitt: we have to be able to configure the main db connection to be the cell because compute manager just expects that it is in its own cell, which is fine for real environments | 20:59 |
superdan | melwitt: but there are all kinds of crazy problems trying to do that for the tests | 20:59 |
superdan | melwitt: about to push up what I have with a big brain dump in the fixture patch | 21:00 |
*** rfolco has quit IRC | 21:00 | |
superdan | melwitt: I'm nearly at table-flip stage, so I'm going to push it up if you want to look and otherwise defer to monday | 21:00 |
*** ijw has quit IRC | 21:00 | |
*** TravT has quit IRC | 21:00 | |
openstackgerrit | Dan Smith proposed openstack/nova: Return build_requests instead of instances https://review.openstack.org/374267 | 21:01 |
openstackgerrit | Dan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method https://review.openstack.org/367557 | 21:01 |
openstackgerrit | Dan Smith proposed openstack/nova: Refactor two nearly useless secgroup tests https://review.openstack.org/393438 | 21:01 |
openstackgerrit | Dan Smith proposed openstack/nova: Make RPCFixture support multiple connections https://review.openstack.org/396417 | 21:01 |
openstackgerrit | Dan Smith proposed openstack/nova: WIP Add CellDatabase fixture https://review.openstack.org/392925 | 21:01 |
openstackgerrit | Dan Smith proposed openstack/nova: Store security groups in RequestSpec https://review.openstack.org/394656 | 21:01 |
openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 21:01 |
openstackgerrit | Dan Smith proposed openstack/nova: WIP Multicell support for instance listing https://review.openstack.org/396775 | 21:01 |
melwitt | superdan: okay, will take a look. thanks for the hard work | 21:01 |
superdan | well, wish it was farther | 21:02 |
superdan | I mean it works for real, and works in a single thread | 21:02 |
superdan | but.. | 21:02 |
melwitt | meh, I know what it's like trying to get our test fixtures to work in this way and ... it is very hard. so I think this is pretty good | 21:03 |
*** walharthi has joined #openstack-nova | 21:03 | |
*** maticue has quit IRC | 21:03 | |
*** thorst has joined #openstack-nova | 21:04 | |
*** jwcroppe has joined #openstack-nova | 21:07 | |
*** jwcroppe has quit IRC | 21:07 | |
*** jwcroppe has joined #openstack-nova | 21:07 | |
*** haplo37 has joined #openstack-nova | 21:07 | |
*** ducttape_ has quit IRC | 21:09 | |
mriedem | leakypipes: i'll update the first thing, but see if my reply still makes you want your update here https://review.openstack.org/#/c/396761/1/doc/source/placement.rst@60 | 21:14 |
leakypipes | mriedem: yeah, makes sense. it was more of a nit than anything else. | 21:20 |
*** jheroux has quit IRC | 21:22 | |
*** tanvirt has quit IRC | 21:26 | |
*** oanson has quit IRC | 21:27 | |
*** rama_y has quit IRC | 21:30 | |
figleaf | leakypipes: mriedem: I thought that the separate DB for placement was optional in Newton. Did we remove that? | 21:31 |
*** harlowja has quit IRC | 21:32 | |
mriedem | figleaf: yes was reverted | 21:32 |
figleaf | ah, ok. I thought that only the *requirement* was reverted | 21:32 |
*** Jeffrey4l has quit IRC | 21:33 | |
*** catinthe_ has joined #openstack-nova | 21:34 | |
*** catintheroof has quit IRC | 21:34 | |
jroll | mriedem: so this was either deprecated or removed in nova, forget which, did you want to keep the tempest tests around for stable branches? https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_baremetal_nodes.py | 21:38 |
jroll | this is the os-baremetal extension or whatever | 21:38 |
*** walharthi has quit IRC | 21:38 | |
*** rama_y has joined #openstack-nova | 21:40 | |
*** sacharya has joined #openstack-nova | 21:43 | |
*** sacharya has quit IRC | 21:43 | |
jroll | mriedem: now that I think about it, those probably never run, because we use a regex on our tempest runs that wouldn't match that | 21:44 |
jroll | and it only runs if ironic is enabled | 21:44 |
*** ducttape_ has joined #openstack-nova | 21:44 | |
*** brault has joined #openstack-nova | 21:48 | |
edmondsw | mriedem, another cherrypick to stable/newton: https://review.openstack.org/#/c/396782/ | 21:50 |
*** Apoorva has joined #openstack-nova | 21:52 | |
*** brault has quit IRC | 21:53 | |
*** eharney has quit IRC | 21:57 | |
mriedem | jroll: it's deprecated in nova microversion 2.36, we won't be removing that unless we ever have the minimum microversion at that level | 21:59 |
mriedem | which might never happen, but if it did would probably be a long ways out | 21:59 |
*** Sukhdev has joined #openstack-nova | 21:59 | |
mriedem | edmondsw: i'll look at that if superdan does | 21:59 |
*** browne has quit IRC | 22:01 | |
jroll | mriedem: ah right | 22:03 |
*** Sukhdev has quit IRC | 22:05 | |
*** dave-mccowan has quit IRC | 22:06 | |
*** tonytan4ever has quit IRC | 22:07 | |
*** victorhe has quit IRC | 22:08 | |
edmondsw | mriedem, tx | 22:09 |
*** gmann has joined #openstack-nova | 22:09 | |
*** edmondsw has quit IRC | 22:12 | |
openstackgerrit | jichenjc proposed openstack/nova: WIP: no extension for multiple create https://review.openstack.org/396622 | 22:13 |
*** Sukhdev has joined #openstack-nova | 22:14 | |
*** ijw has joined #openstack-nova | 22:14 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add the initial documentation for the placement API https://review.openstack.org/396761 | 22:21 |
mriedem | leakypipes: made that little s/will/may/ change for you ^ | 22:22 |
*** rama_y has quit IRC | 22:22 | |
mriedem | btw this is what it ends up looking like http://docs-draft.openstack.org/61/396761/1/check/gate-nova-docs-ubuntu-xenial/21829cc//doc/build/html/placement.html | 22:22 |
*** rama_y has joined #openstack-nova | 22:23 | |
leakypipes | mriedem: +2 again. | 22:26 |
mriedem | thanks | 22:26 |
mriedem | with that i'm going to call it a day | 22:26 |
*** mriedem has quit IRC | 22:27 | |
*** rama_y has quit IRC | 22:28 | |
*** jamesdenton has quit IRC | 22:28 | |
*** tblakes has quit IRC | 22:34 | |
*** esberglu has quit IRC | 22:41 | |
*** esberglu has joined #openstack-nova | 22:41 | |
*** esberglu has quit IRC | 22:46 | |
openstackgerrit | ijw-ubuntu proposed openstack/nova-specs: Add the information Nova requires for a port to the port itself https://review.openstack.org/390513 | 22:48 |
*** mariusv has quit IRC | 22:51 | |
*** mariusv has joined #openstack-nova | 22:51 | |
*** mariusv has quit IRC | 22:51 | |
*** mariusv has joined #openstack-nova | 22:51 | |
*** sree has joined #openstack-nova | 22:57 | |
*** gmann_ has joined #openstack-nova | 22:57 | |
*** mdrabe has quit IRC | 22:58 | |
*** gmann has quit IRC | 22:59 | |
*** sree has quit IRC | 23:01 | |
*** harlowja has joined #openstack-nova | 23:01 | |
*** annegentle has quit IRC | 23:04 | |
*** rama_y has joined #openstack-nova | 23:06 | |
*** ijw has quit IRC | 23:09 | |
stewie925 | hello, whats the difference between resize and resize_prep instance states? | 23:09 |
*** ijw has joined #openstack-nova | 23:12 | |
*** rama_y has quit IRC | 23:13 | |
*** walharthi has joined #openstack-nova | 23:15 | |
*** xyang1 has quit IRC | 23:15 | |
*** ijw has quit IRC | 23:16 | |
*** ijw has joined #openstack-nova | 23:16 | |
*** inara has quit IRC | 23:17 | |
*** inara has joined #openstack-nova | 23:18 | |
*** pradk has quit IRC | 23:19 | |
stewie925 | basically, am looking to test resize_prep notifications, but couldnt find anything online that shows how to do the request body for the resize_prep | 23:21 |
*** ijw has quit IRC | 23:22 | |
*** umbSublime_ has quit IRC | 23:24 | |
*** ducttape_ has quit IRC | 23:24 | |
*** baoli_ has quit IRC | 23:25 | |
*** rrecio_ has quit IRC | 23:26 | |
*** lamt has quit IRC | 23:29 | |
*** Jeffrey4l has joined #openstack-nova | 23:31 | |
*** unicell has quit IRC | 23:31 | |
*** rfolco has joined #openstack-nova | 23:33 | |
stewie925 | or I mean, what 'post server action' would result in a resize_prep statu | 23:35 |
stewie925 | status | 23:35 |
*** marst has quit IRC | 23:40 | |
openstackgerrit | Sujitha proposed openstack/nova: Change os-server-tags default policy https://review.openstack.org/396420 | 23:42 |
*** Sukhdev has quit IRC | 23:42 | |
*** victorhe has joined #openstack-nova | 23:43 | |
*** umbSublime_ has joined #openstack-nova | 23:43 | |
*** ducttape_ has joined #openstack-nova | 23:46 | |
*** ducttape_ has quit IRC | 23:47 | |
*** ducttape_ has joined #openstack-nova | 23:47 | |
*** nicolasbock has quit IRC | 23:47 | |
*** rfolco has quit IRC | 23:48 | |
*** umbSublime_ has quit IRC | 23:49 | |
*** umbSublime_ has joined #openstack-nova | 23:49 | |
*** marst has joined #openstack-nova | 23:50 | |
*** ijw has joined #openstack-nova | 23:50 | |
*** Hosam has quit IRC | 23:53 | |
*** Hosam has joined #openstack-nova | 23:54 | |
*** ijw has quit IRC | 23:55 | |
*** ducttape_ has quit IRC | 23:57 | |
*** catinthe_ has quit IRC | 23:58 | |
*** Hosam has quit IRC | 23:58 | |
*** unicell has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!