*** ijw has joined #openstack-nova | 00:00 | |
*** igordcard_ has quit IRC | 00:01 | |
*** ssurana1 has joined #openstack-nova | 00:02 | |
*** haomai___ has joined #openstack-nova | 00:02 | |
*** tjones2 has quit IRC | 00:02 | |
*** tjones1 has joined #openstack-nova | 00:02 | |
*** haomaiwa_ has quit IRC | 00:03 | |
*** ozialien has quit IRC | 00:04 | |
*** ssurana has quit IRC | 00:04 | |
*** Nikolay_St has quit IRC | 00:08 | |
*** armax has joined #openstack-nova | 00:09 | |
*** takedakn has joined #openstack-nova | 00:11 | |
*** ijw has quit IRC | 00:13 | |
*** ijw has joined #openstack-nova | 00:14 | |
*** takedakn has quit IRC | 00:16 | |
*** Nikolay_St has joined #openstack-nova | 00:21 | |
*** mtanino has quit IRC | 00:22 | |
*** otter768 has joined #openstack-nova | 00:23 | |
*** sdake has quit IRC | 00:24 | |
*** isd has quit IRC | 00:26 | |
*** yingjun has joined #openstack-nova | 00:28 | |
*** ferest has quit IRC | 00:30 | |
*** baoli has joined #openstack-nova | 00:32 | |
*** haomai___ has quit IRC | 00:34 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/python-novaclient: cleanup openstack-common.conf and sync updated files https://review.openstack.org/189119 | 00:38 |
---|---|---|
*** tpeoples has joined #openstack-nova | 00:38 | |
*** ctrath has quit IRC | 00:38 | |
*** jasondotstar has quit IRC | 00:42 | |
*** oro has quit IRC | 00:42 | |
*** tjones1 has quit IRC | 00:44 | |
tonyb | Do we actually support python-novaclient windows? (well anything otehr than MacOS and Linux really) | 00:45 |
*** tjones1 has joined #openstack-nova | 00:47 | |
*** agireud has quit IRC | 00:47 | |
*** ssurana1 has quit IRC | 00:47 | |
*** sdake has joined #openstack-nova | 00:48 | |
openstackgerrit | wangxiyuan proposed openstack/nova: Fix wrong check when use image in local https://review.openstack.org/186976 | 00:50 |
*** mdrabe has quit IRC | 00:50 | |
*** electrocucaracha has quit IRC | 00:52 | |
openstackgerrit | Li Yingjun proposed openstack/nova: Raise an exception when adding an already associated security group https://review.openstack.org/160689 | 00:54 |
*** otter768 has quit IRC | 00:54 | |
*** tjones1 has quit IRC | 00:55 | |
*** sdake_ has joined #openstack-nova | 00:58 | |
*** sdake has quit IRC | 01:01 | |
*** tobe has joined #openstack-nova | 01:05 | |
*** zz_jgrimm is now known as jgrimm | 01:06 | |
*** jgrimm is now known as zz_jgrimm | 01:07 | |
*** diogogmt has quit IRC | 01:07 | |
*** penick has quit IRC | 01:08 | |
*** ijw has quit IRC | 01:08 | |
*** apuimedo has quit IRC | 01:09 | |
*** ijw__ has joined #openstack-nova | 01:09 | |
*** unicell has quit IRC | 01:09 | |
*** ijw__ has quit IRC | 01:10 | |
*** _liusheng has joined #openstack-nova | 01:10 | |
*** ijw__ has joined #openstack-nova | 01:10 | |
*** dsanders has quit IRC | 01:11 | |
*** ijw__ has quit IRC | 01:11 | |
*** _liusheng has quit IRC | 01:11 | |
*** liusheng_ has joined #openstack-nova | 01:12 | |
*** zxen has quit IRC | 01:12 | |
*** patrickeast has joined #openstack-nova | 01:12 | |
*** SourabhP has quit IRC | 01:12 | |
*** patrickeast has left #openstack-nova | 01:19 | |
openstackgerrit | wangxiyuan proposed openstack/nova: Fix bdm image_meta when boot from volume https://review.openstack.org/186247 | 01:20 |
*** bradjones has quit IRC | 01:21 | |
*** patrickeast has joined #openstack-nova | 01:21 | |
*** baoli has quit IRC | 01:24 | |
*** claudiub has joined #openstack-nova | 01:26 | |
*** bradjones has joined #openstack-nova | 01:29 | |
*** bradjones has quit IRC | 01:29 | |
*** bradjones has joined #openstack-nova | 01:29 | |
*** Guest68549 is now known as dan | 01:30 | |
*** patrickeast has quit IRC | 01:32 | |
*** mtecer has quit IRC | 01:34 | |
*** kiran-r has joined #openstack-nova | 01:37 | |
tonyb | dims: Thanks for the support on: https://review.openstack.org/190472 | 01:43 |
*** ociuhandu has quit IRC | 01:46 | |
*** diogogmt has joined #openstack-nova | 01:49 | |
*** pixelbeat has joined #openstack-nova | 01:49 | |
*** ssurana has joined #openstack-nova | 01:49 | |
*** gyee is now known as operator99 | 01:49 | |
*** baoli has joined #openstack-nova | 01:49 | |
*** pixelb has quit IRC | 01:49 | |
*** nic has quit IRC | 01:55 | |
*** ssurana has quit IRC | 01:57 | |
*** iamjarvo has joined #openstack-nova | 01:57 | |
*** baoli has quit IRC | 01:59 | |
*** mdrabe has joined #openstack-nova | 02:00 | |
dims | tonyb: my pleasure | 02:02 |
*** vilobhmm has joined #openstack-nova | 02:02 | |
*** dims has quit IRC | 02:04 | |
*** yingjun has quit IRC | 02:05 | |
*** vilobhmm has quit IRC | 02:07 | |
*** ypraveen has quit IRC | 02:07 | |
*** dims has joined #openstack-nova | 02:08 | |
*** jasondotstar has joined #openstack-nova | 02:09 | |
*** jasondotstar has quit IRC | 02:09 | |
*** jwcroppe has quit IRC | 02:10 | |
*** jasondotstar has joined #openstack-nova | 02:10 | |
*** kiran-r has quit IRC | 02:11 | |
*** Marga_ has quit IRC | 02:14 | |
*** Marga_ has joined #openstack-nova | 02:16 | |
*** Marga_ has quit IRC | 02:16 | |
*** Marga_ has joined #openstack-nova | 02:17 | |
*** dims has quit IRC | 02:19 | |
openstackgerrit | wangxiyuan proposed openstack/nova: Fix bdm image_meta when boot from volume https://review.openstack.org/186247 | 02:21 |
*** mdrabe has quit IRC | 02:23 | |
*** jasondotstar has quit IRC | 02:24 | |
*** claudiub has quit IRC | 02:29 | |
*** changbl has joined #openstack-nova | 02:32 | |
*** Marga_ has quit IRC | 02:33 | |
*** Marga_ has joined #openstack-nova | 02:34 | |
*** Marga_ has quit IRC | 02:37 | |
*** haomaiwang has joined #openstack-nova | 02:38 | |
*** liusheng_ has quit IRC | 02:48 | |
*** liusheng has joined #openstack-nova | 02:48 | |
*** dsanders has joined #openstack-nova | 02:50 | |
openstackgerrit | heijlong proposed openstack/nova-specs: Add uuid field to security-groups for server show https://review.openstack.org/184201 | 02:50 |
*** baoli has joined #openstack-nova | 02:51 | |
*** tjones1 has joined #openstack-nova | 02:54 | |
*** otter768 has joined #openstack-nova | 02:55 | |
*** pixelbeat has quit IRC | 02:55 | |
*** dpaterson has joined #openstack-nova | 02:55 | |
*** otter768 has quit IRC | 02:59 | |
*** sdake_ is now known as sdake | 03:00 | |
*** varunarya has joined #openstack-nova | 03:03 | |
*** kiran-r has joined #openstack-nova | 03:05 | |
*** unicell has joined #openstack-nova | 03:06 | |
*** Marga_ has joined #openstack-nova | 03:09 | |
*** marzif has joined #openstack-nova | 03:09 | |
*** liusheng has quit IRC | 03:09 | |
*** liusheng has joined #openstack-nova | 03:10 | |
*** Marga_ has quit IRC | 03:14 | |
*** prashup has quit IRC | 03:18 | |
*** varunarya_ has joined #openstack-nova | 03:19 | |
*** varunarya has quit IRC | 03:21 | |
*** raildo has quit IRC | 03:21 | |
*** samueldmq has quit IRC | 03:21 | |
*** htruta has quit IRC | 03:22 | |
*** afaranha has quit IRC | 03:22 | |
*** ericksonsantos has quit IRC | 03:22 | |
*** tellesnobrega has quit IRC | 03:22 | |
*** baoli has quit IRC | 03:24 | |
*** iamjarvo has quit IRC | 03:26 | |
*** tpeoples has quit IRC | 03:26 | |
*** iamjarvo has joined #openstack-nova | 03:29 | |
*** baoli has joined #openstack-nova | 03:29 | |
*** markvoelker has quit IRC | 03:31 | |
*** coolsvap|afk is now known as coolsvap | 03:34 | |
*** ijw has joined #openstack-nova | 03:37 | |
*** tjones1 has quit IRC | 03:39 | |
*** baoli has quit IRC | 03:40 | |
*** aswadr has joined #openstack-nova | 03:40 | |
*** Marga_ has joined #openstack-nova | 03:43 | |
*** tjones1 has joined #openstack-nova | 03:44 | |
*** Marga__ has joined #openstack-nova | 03:45 | |
*** rajesht has joined #openstack-nova | 03:47 | |
*** Marga_ has quit IRC | 03:47 | |
*** Marga_ has joined #openstack-nova | 03:48 | |
*** Marga__ has quit IRC | 03:49 | |
*** vilobhmm has joined #openstack-nova | 03:53 | |
*** ildikov has quit IRC | 03:58 | |
*** iamjarvo has quit IRC | 04:00 | |
*** prashup has joined #openstack-nova | 04:01 | |
*** varunarya_ has quit IRC | 04:02 | |
*** ozialien has joined #openstack-nova | 04:03 | |
*** tjones1 has quit IRC | 04:03 | |
*** kiran-r has quit IRC | 04:06 | |
openstackgerrit | Diana Clarke proposed openstack/nova: Add unit tests for the exact filters https://review.openstack.org/192514 | 04:07 |
*** puniuee has joined #openstack-nova | 04:08 | |
*** rook has quit IRC | 04:09 | |
*** unicell has quit IRC | 04:10 | |
*** rushiagr_away is now known as rushiagr | 04:10 | |
lifeless | nova / IBM peoples | 04:12 |
lifeless | is ORD the right airport for the midcycle ? | 04:12 |
*** marzif has quit IRC | 04:13 | |
lifeless | no | 04:13 |
lifeless | RST?? | 04:13 |
*** dpaterson has quit IRC | 04:15 | |
StevenK | lifeless: RST is in Minnesota, not the other Rochester | 04:15 |
StevenK | ROC is the Rochester in NY | 04:15 |
*** patrickeast has joined #openstack-nova | 04:15 | |
puniuee | Hello all, I need some help in figuring out why some of my CellsComputeAPITestCase are failing | 04:16 |
*** ijw_ has quit IRC | 04:19 | |
*** ijw_ has joined #openstack-nova | 04:19 | |
lifeless | StevenK: https://wiki.openstack.org/wiki/Sprints/NovaLibertySprint | 04:22 |
*** ijw_ has quit IRC | 04:22 | |
lifeless | StevenK: says MN | 04:22 |
lifeless | and RST | 04:22 |
lifeless | / MSP | 04:22 |
*** achanda has joined #openstack-nova | 04:23 | |
*** patrickeast has quit IRC | 04:23 | |
StevenK | lifeless: Blah, I keep forgetting that the US likes reusing city names | 04:24 |
*** puniuee has quit IRC | 04:26 | |
*** josecastroleon has quit IRC | 04:31 | |
*** beagles has quit IRC | 04:31 | |
*** achanda has quit IRC | 04:32 | |
*** markvoelker has joined #openstack-nova | 04:32 | |
*** josecastroleon has joined #openstack-nova | 04:32 | |
*** markvoelker has quit IRC | 04:37 | |
*** varunarya_ has joined #openstack-nova | 04:38 | |
*** ijw_ has joined #openstack-nova | 04:40 | |
*** puniuee has joined #openstack-nova | 04:43 | |
*** ijw has quit IRC | 04:44 | |
tonyb | lifeless: No MSP | 04:46 |
tonyb | lifeless: https://wiki.openstack.org/wiki/Sprints/NovaLibertySprint | 04:46 |
tonyb | And now I read the whole scrollback | 04:47 |
openstackgerrit | Fei Long Wang proposed openstack/nova: Refactor image's code to support both versions https://review.openstack.org/144875 | 04:47 |
*** b3nt_pin has joined #openstack-nova | 04:47 | |
*** b3nt_pin is now known as Guest71075 | 04:48 | |
*** ijw has joined #openstack-nova | 04:50 | |
*** fawadkhaliq has joined #openstack-nova | 04:55 | |
*** ijw has quit IRC | 04:55 | |
*** otter768 has joined #openstack-nova | 04:56 | |
*** tojuvone has joined #openstack-nova | 04:58 | |
*** irenab has joined #openstack-nova | 04:58 | |
*** otter768 has quit IRC | 05:00 | |
*** ozialien has quit IRC | 05:01 | |
*** tojuvone has quit IRC | 05:02 | |
*** achanda has joined #openstack-nova | 05:02 | |
*** whenry has quit IRC | 05:03 | |
*** fifieldt has joined #openstack-nova | 05:10 | |
*** IanGovett has joined #openstack-nova | 05:12 | |
*** nihilifer has joined #openstack-nova | 05:12 | |
*** ypraveen has joined #openstack-nova | 05:13 | |
*** haomaiwang has quit IRC | 05:14 | |
*** Kennan has quit IRC | 05:14 | |
*** ypraveen has quit IRC | 05:16 | |
*** dsanders has quit IRC | 05:16 | |
*** vishwana_ has joined #openstack-nova | 05:16 | |
*** browne has quit IRC | 05:16 | |
*** Maike has joined #openstack-nova | 05:18 | |
*** vishwanathj has quit IRC | 05:19 | |
*** Kennan has joined #openstack-nova | 05:20 | |
*** ypraveen has joined #openstack-nova | 05:22 | |
lifeless | StevenK: *cough newcastle* | 05:23 |
lifeless | StevenK: *cough perth* | 05:24 |
lifeless | StevenK: *cough... need I continue ? | 05:24 |
*** Nikolay_St has quit IRC | 05:26 | |
*** nkrinner has joined #openstack-nova | 05:31 | |
*** achanda has quit IRC | 05:31 | |
*** lpetrut has joined #openstack-nova | 05:33 | |
*** Guest71075 has quit IRC | 05:34 | |
*** b3nt_pin_ has joined #openstack-nova | 05:37 | |
openstackgerrit | Dheeraj Gupta proposed openstack/nova: Add HostMapping object https://review.openstack.org/192098 | 05:38 |
*** tobe has quit IRC | 05:39 | |
*** arnaud___ has joined #openstack-nova | 05:41 | |
*** sudipto has joined #openstack-nova | 05:43 | |
*** belmoreira has joined #openstack-nova | 05:44 | |
*** oomichi has quit IRC | 05:45 | |
*** doron_afk has joined #openstack-nova | 05:45 | |
*** b3nt_pin_ is now known as b3nt_pin | 05:48 | |
*** ijw has joined #openstack-nova | 05:48 | |
*** slong has quit IRC | 05:49 | |
*** b3nt_pin is now known as Guest94194 | 05:49 | |
*** ijw has quit IRC | 05:50 | |
*** tojuvone has joined #openstack-nova | 05:51 | |
*** deepthi has joined #openstack-nova | 05:53 | |
*** IanGovett has quit IRC | 05:55 | |
*** markvoelker has joined #openstack-nova | 05:55 | |
*** ypraveen has quit IRC | 05:56 | |
*** ypraveen has joined #openstack-nova | 05:58 | |
*** haomaiwa_ has joined #openstack-nova | 05:59 | |
*** markvoelker has quit IRC | 06:00 | |
*** Mike_D_laptop has joined #openstack-nova | 06:01 | |
openstackgerrit | garyk proposed openstack/nova: VMware: use min supported VC version in fake driver https://review.openstack.org/192524 | 06:01 |
*** kiran-r has joined #openstack-nova | 06:02 | |
*** Mike_D_laptop has quit IRC | 06:02 | |
*** browne has joined #openstack-nova | 06:03 | |
*** garyk has quit IRC | 06:03 | |
*** IanGovett has joined #openstack-nova | 06:05 | |
*** destiny has joined #openstack-nova | 06:06 | |
*** doron_afk is now known as doron_ | 06:07 | |
*** IanGovett has quit IRC | 06:09 | |
*** moshele has joined #openstack-nova | 06:13 | |
*** browne has quit IRC | 06:14 | |
*** radek_ has joined #openstack-nova | 06:15 | |
*** abhishekk has joined #openstack-nova | 06:15 | |
*** doron_ is now known as doron_afk | 06:15 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/190034 | 06:15 |
*** puniuee has quit IRC | 06:16 | |
*** fawadkhaliq has quit IRC | 06:17 | |
*** rajesht has quit IRC | 06:18 | |
*** exploreshaifali has joined #openstack-nova | 06:19 | |
*** ypraveen has quit IRC | 06:19 | |
*** tobe has joined #openstack-nova | 06:20 | |
*** unicell has joined #openstack-nova | 06:21 | |
*** rajesht has joined #openstack-nova | 06:22 | |
*** dulek has joined #openstack-nova | 06:22 | |
*** Marga_ has quit IRC | 06:23 | |
*** afazekas has joined #openstack-nova | 06:23 | |
*** dulek has quit IRC | 06:25 | |
*** park_ has joined #openstack-nova | 06:26 | |
abhishekk | hi all, is there any meeting for cinder nova volume discussion? | 06:26 |
*** mpaolino has joined #openstack-nova | 06:27 | |
*** browne has joined #openstack-nova | 06:32 | |
*** puniuee has joined #openstack-nova | 06:33 | |
*** ianbrown has quit IRC | 06:39 | |
*** salv-orl_ has joined #openstack-nova | 06:41 | |
*** boris-42 has quit IRC | 06:42 | |
*** salv-orlando has quit IRC | 06:44 | |
*** sahid has joined #openstack-nova | 06:46 | |
*** garyk has joined #openstack-nova | 06:46 | |
*** dulek has joined #openstack-nova | 06:49 | |
*** browne has quit IRC | 06:49 | |
*** Longgeek has joined #openstack-nova | 06:49 | |
openstackgerrit | Ankit Agrawal proposed openstack/nova: Test patch race condition in delete (do not merge) https://review.openstack.org/191732 | 06:50 |
*** lpetrut has quit IRC | 06:50 | |
*** moshele has quit IRC | 06:51 | |
*** arnaud___ has quit IRC | 06:52 | |
*** Longgeek_ has joined #openstack-nova | 06:53 | |
*** romainh has joined #openstack-nova | 06:56 | |
*** otter768 has joined #openstack-nova | 06:56 | |
*** Longgeek has quit IRC | 06:57 | |
*** vilobhmm has quit IRC | 06:58 | |
*** moshele has joined #openstack-nova | 07:01 | |
*** otter768 has quit IRC | 07:01 | |
*** armax has quit IRC | 07:03 | |
*** pilgrimstack has joined #openstack-nova | 07:04 | |
*** pilgrimstack has joined #openstack-nova | 07:04 | |
*** pilgrimstack has left #openstack-nova | 07:04 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova: Validate int using utils.validate_integer method https://review.openstack.org/192535 | 07:15 |
*** sdake has quit IRC | 07:20 | |
*** puniuee has quit IRC | 07:21 | |
*** Longgeek_ has quit IRC | 07:26 | |
*** alex_klimov has joined #openstack-nova | 07:26 | |
*** matrohon has joined #openstack-nova | 07:28 | |
*** haypo has joined #openstack-nova | 07:28 | |
*** ijw has joined #openstack-nova | 07:30 | |
*** park_ has quit IRC | 07:34 | |
*** cfriesen has quit IRC | 07:34 | |
*** unicell has quit IRC | 07:35 | |
*** unicell has joined #openstack-nova | 07:35 | |
*** puniuee has joined #openstack-nova | 07:38 | |
*** doude has joined #openstack-nova | 07:39 | |
*** ociuhandu has joined #openstack-nova | 07:42 | |
*** jlanoux has joined #openstack-nova | 07:43 | |
*** ildikov has joined #openstack-nova | 07:44 | |
*** markvoelker has joined #openstack-nova | 07:44 | |
*** ociuhandu has quit IRC | 07:45 | |
*** doude has quit IRC | 07:46 | |
*** doude has joined #openstack-nova | 07:47 | |
*** Longgeek has joined #openstack-nova | 07:47 | |
*** park_ has joined #openstack-nova | 07:47 | |
*** Longgeek has quit IRC | 07:48 | |
*** Longgeek has joined #openstack-nova | 07:48 | |
*** markvoelker has quit IRC | 07:49 | |
*** Nikolay_St has joined #openstack-nova | 07:50 | |
*** varunarya_ has quit IRC | 07:52 | |
*** jyuso1 has joined #openstack-nova | 07:53 | |
*** markus_z has joined #openstack-nova | 07:56 | |
*** lucasagomes has joined #openstack-nova | 07:59 | |
*** ZZelle has quit IRC | 08:02 | |
*** ZZelle has joined #openstack-nova | 08:03 | |
*** ociuhandu has joined #openstack-nova | 08:04 | |
*** apuimedo has joined #openstack-nova | 08:07 | |
openstackgerrit | Bartosz Fic proposed openstack/nova: Live migration failure in API doesn't set vm_state to Error https://review.openstack.org/168916 | 08:08 |
*** park_ has quit IRC | 08:08 | |
*** jcoufal has joined #openstack-nova | 08:08 | |
*** jcoufal has quit IRC | 08:09 | |
*** jcoufal has joined #openstack-nova | 08:10 | |
*** derekh has joined #openstack-nova | 08:12 | |
*** doron_afk has quit IRC | 08:13 | |
*** dsanders has joined #openstack-nova | 08:15 | |
*** dsanders has quit IRC | 08:16 | |
*** aswadr has quit IRC | 08:18 | |
*** Nikolay_St has quit IRC | 08:18 | |
*** aswadr has joined #openstack-nova | 08:19 | |
*** Nikolay_St has joined #openstack-nova | 08:20 | |
openstackgerrit | Alex Xu proposed openstack/nova-specs: Allow specifying cpu model/features in flavor extra-specs or image metadata https://review.openstack.org/168982 | 08:20 |
park_heijlong | hello johnthetubaguy, would you please help to take a review about https://review.openstack.org/#/c/184201/....I updated as suggestion...thank you ~ | 08:21 |
*** park_ has joined #openstack-nova | 08:21 | |
*** asrangne has joined #openstack-nova | 08:21 | |
johnthetubaguy | park_heijlong: just looking at your questions on there | 08:21 |
johnthetubaguy | park_heijlong: I am saying we generate a uuid for all the nova security groups, and use the neutron ones like we do today | 08:22 |
johnthetubaguy | park_heijlong: its possibly I am totally miss understanding the relationship there | 08:23 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: Nova part of volume snapshot improvements https://review.openstack.org/172813 | 08:23 |
park_heijlong | johnthetubaguy, yep, my latest version made the correction.. | 08:23 |
*** pixelbeat has joined #openstack-nova | 08:24 | |
park_heijlong | johnthetubaguy: the latest made the correction as your suggestion. | 08:24 |
*** yassine has joined #openstack-nova | 08:24 | |
*** aswadr has quit IRC | 08:24 | |
johnthetubaguy | park_heijlong: OK, thank you | 08:25 |
bauzas | morning Nova | 08:25 |
park_heijlong | johnthetubaguy, really thanks for review :) feel free to leave comments | 08:25 |
johnthetubaguy | park_heijlong: no problems, I am really happy to see someone fixing up this API :) | 08:25 |
*** doron_afk has joined #openstack-nova | 08:27 | |
*** varunarya has joined #openstack-nova | 08:30 | |
*** puniuee has quit IRC | 08:31 | |
johnthetubaguy | park_heijlong: so I think there is one extra bit, we need a note about all the add security group API, etc, that will need to now accept uuids instead of name or id? | 08:32 |
*** rotbeard has joined #openstack-nova | 08:32 | |
* johnthetubaguy is having to drop online while he drives over to talk to some Xen folks | 08:33 | |
*** belmoreira has quit IRC | 08:33 | |
johnthetubaguy | BobBall: see you in a bit! | 08:33 |
BobBall | Indeed! | 08:33 |
*** belmoreira has joined #openstack-nova | 08:33 | |
park_heijlong | johnthetubaguy: make sense, will update | 08:34 |
*** varunarya has quit IRC | 08:37 | |
*** dguitarbite has quit IRC | 08:38 | |
*** dguitarbite has joined #openstack-nova | 08:38 | |
pczesno | moshele: Hi | 08:41 |
*** mdbooth has quit IRC | 08:42 | |
*** mdbooth has joined #openstack-nova | 08:45 | |
*** ociuhandu has quit IRC | 08:52 | |
*** coolsvap is now known as coolsvap|afk | 08:52 | |
garyk | anyone had the latest devstack and nova blow up on them - please see http://paste.openstack.org/show/297829/ | 08:56 |
*** ociuhandu has joined #openstack-nova | 08:57 | |
kashyap | garyk: I just did a `make unstack` away on an F22 machine. Will let you know if my next run goes well. | 08:57 |
*** otter768 has joined #openstack-nova | 08:57 | |
garyk | kashyap: thanks | 08:58 |
*** tobe has quit IRC | 08:58 | |
garyk | i am not sure if it is my env or the code … gut feeling is that it is the former | 08:58 |
*** tobe has joined #openstack-nova | 08:59 | |
kashyap | garyk: Your gut feeling is right, it's always DevStack :-) | 09:00 |
*** pask81 has joined #openstack-nova | 09:01 | |
*** tobe_ has joined #openstack-nova | 09:01 | |
*** tobe_ has quit IRC | 09:01 | |
*** danpb has joined #openstack-nova | 09:02 | |
*** otter768 has quit IRC | 09:02 | |
*** coolsvap|afk is now known as coolsvap | 09:03 | |
*** tobe has quit IRC | 09:03 | |
*** gszasz has joined #openstack-nova | 09:04 | |
*** dguitarbite has quit IRC | 09:07 | |
*** dguitarbite has joined #openstack-nova | 09:08 | |
*** park_ has quit IRC | 09:09 | |
*** varunarya has joined #openstack-nova | 09:09 | |
*** dguitarbite has quit IRC | 09:10 | |
openstackgerrit | Stefan Amann proposed openstack/nova: Support host type specific block volume attachment https://review.openstack.org/192556 | 09:14 |
*** igordcard_ has joined #openstack-nova | 09:17 | |
snikitin | Guys, I have a question. Do we have any freeze period for nova specs? | 09:17 |
*** ndipanov has joined #openstack-nova | 09:21 | |
*** tobe has joined #openstack-nova | 09:23 | |
garyk | kashyap: seems to have been something in my env… thanks | 09:28 |
*** varunarya has quit IRC | 09:30 | |
*** markvoelker has joined #openstack-nova | 09:33 | |
*** markvoelker has quit IRC | 09:37 | |
*** jyuso1 has quit IRC | 09:40 | |
*** haomaiwa_ has quit IRC | 09:41 | |
*** pask811 has joined #openstack-nova | 09:43 | |
*** pask81 has quit IRC | 09:46 | |
*** pask811 has quit IRC | 09:48 | |
*** mpaolino has quit IRC | 09:49 | |
*** arnaud___ has joined #openstack-nova | 09:53 | |
*** Marga_ has joined #openstack-nova | 09:54 | |
*** mpaolino has joined #openstack-nova | 09:56 | |
*** tdurakov has quit IRC | 09:56 | |
*** arnaud___ has quit IRC | 09:57 | |
kaisers | Hi Nova! Can somebody please give me a hint for debugging a DHCP issue in a devstack/nova-network setup: Guests do send DHCP requests, dnsmasq receives the requests but the answers seem to be lost. Happens with cirros & Ubuntu 14.04 guest OS. Hints on where to look for reasons? | 09:58 |
*** tdurakov has joined #openstack-nova | 09:58 | |
*** romainh has quit IRC | 09:59 | |
kashyap | garyk: Yes, likewise here too. | 09:59 |
kaisers | Settings are using FlatDHCPManager | 09:59 |
kaisers | Oh, whoopsie, this channel was only for dev, not help req, right? If yes: my apologies! | 10:01 |
openstackgerrit | PrashantUpadhyay proposed openstack/nova: adding check to see if volume size specifed by user is greater than the size of OS-image specified by user https://review.openstack.org/192571 | 10:05 |
*** dguitarbite has joined #openstack-nova | 10:06 | |
*** oro has joined #openstack-nova | 10:07 | |
*** ianbrown has joined #openstack-nova | 10:07 | |
openstackgerrit | PrashantUpadhyay proposed openstack/nova: Adding check to see if volume size specifed by user is greater than the size of OS-image specified by user https://review.openstack.org/192571 | 10:07 |
openstackgerrit | PrashantUpadhyay proposed openstack/nova: Adding check to see if volume size specifed by user is greater than the size of OS-image specified by user https://review.openstack.org/192571 | 10:09 |
*** jmccrory has quit IRC | 10:10 | |
*** lpetrut has joined #openstack-nova | 10:10 | |
*** lpetrut has quit IRC | 10:10 | |
*** varunarya has joined #openstack-nova | 10:10 | |
openstackgerrit | PrashantUpadhyay proposed openstack/nova: Adding check to see if volume size specifed by user is greater than the size of OS-image https://review.openstack.org/192571 | 10:11 |
*** davidlenwell has quit IRC | 10:12 | |
*** davidlenwell has joined #openstack-nova | 10:12 | |
*** jmccrory has joined #openstack-nova | 10:12 | |
*** alex_klimov has quit IRC | 10:13 | |
openstackgerrit | PrashantUpadhyay proposed openstack/nova: check for volume size https://review.openstack.org/192571 | 10:13 |
*** oro has quit IRC | 10:14 | |
*** varunarya has quit IRC | 10:16 | |
*** dims has joined #openstack-nova | 10:17 | |
*** tobe has quit IRC | 10:17 | |
*** dims has quit IRC | 10:17 | |
*** dims has joined #openstack-nova | 10:18 | |
*** ianbrown has quit IRC | 10:19 | |
openstackgerrit | PrashantUpadhyay proposed openstack/nova: check for volume size https://review.openstack.org/192571 | 10:20 |
*** lpetrut has joined #openstack-nova | 10:21 | |
*** ianbrown has joined #openstack-nova | 10:26 | |
*** haomaiwang has joined #openstack-nova | 10:28 | |
*** claudiub has joined #openstack-nova | 10:31 | |
openstackgerrit | Chris Gacsal proposed openstack/nova: Allow multiple volume attachments https://review.openstack.org/153033 | 10:31 |
*** oomichi has joined #openstack-nova | 10:32 | |
*** kiran-r has quit IRC | 10:39 | |
*** vladikr has quit IRC | 10:41 | |
*** jasondotstar has joined #openstack-nova | 10:41 | |
*** varunarya has joined #openstack-nova | 10:45 | |
*** mwagner_afk has quit IRC | 10:48 | |
*** alex_klimov has joined #openstack-nova | 10:49 | |
*** aix has joined #openstack-nova | 10:49 | |
openstackgerrit | Andrzej Kuriata proposed openstack/nova-specs: Introduce Normalized Compute Units https://review.openstack.org/192609 | 10:52 |
*** chaochin has quit IRC | 10:53 | |
garyk | danpb: i have tried to address your comment with the VC API link. | 10:54 |
*** otter768 has joined #openstack-nova | 10:58 | |
*** Marga_ has quit IRC | 11:02 | |
*** otter768 has quit IRC | 11:03 | |
*** sahid has quit IRC | 11:05 | |
*** sahid has joined #openstack-nova | 11:06 | |
*** jasondotstar has quit IRC | 11:06 | |
danpb | garyk: ok, i'm afraid I think we might want to go the blueprint+spec route for this first | 11:08 |
danpb | garyk: i didn't -2 it though - wait to see whether other nova cores agree or not | 11:09 |
*** Marga_ has joined #openstack-nova | 11:09 | |
garyk | danpb: ok, thanks. i already have a BP for it - https://blueprints.launchpad.net/nova/+spec/vmware-limits | 11:10 |
garyk | the CPU limits were approved in J and are used in production today. | 11:11 |
garyk | they are supported by flavors | 11:11 |
garyk | do you still think that I need a spec? | 11:11 |
garyk | i think tat we are exposing VC functionality like any other driver is exposing its ability to do resource allocations | 11:12 |
danpb | usage against flavours is ok, as the admin is in control of that and they understand their hardware | 11:12 |
danpb | i think usage against images though introduces the new problems i menton there | 11:12 |
danpb | because you're pushing decisions to the tenant user who is not supposed to know about hardware | 11:13 |
danpb | so i'm not sure how they'd meaningfully decide on cpu limits expressed in this manner | 11:13 |
garyk | ok, understood. | 11:13 |
*** varunarya has quit IRC | 11:13 | |
danpb | there's also the permissions issue that the cloud admin may not want to allow control over this | 11:13 |
danpb | so we need to at least decide policy for usage of image vs flavour | 11:14 |
danpb | to ensure the cloud admin can prevent usage on the image if desired | 11:14 |
openstackgerrit | Peter Penchev proposed openstack/nova-specs: Resubmit the StorPool libvirt attachment driver. https://review.openstack.org/192613 | 11:14 |
danpb | for numa we simply said flavour always takes priority if set | 11:14 |
danpb | so cloud admin can block usage on flavor by simply setting numa policy on the image | 11:14 |
garyk | the permissions are a good point. i will go to look at glance and see how these are enforced. | 11:14 |
danpb | but i'm not sure it is so easy to achieve that in this case | 11:15 |
garyk | i am not sure why a tenant would set meta properties as this is information that is is related to the infra and the code | 11:15 |
danpb | as the cloud admin may wish to just have the default behaviour without setting anything on the flavor | 11:15 |
alex_xu | danpb: hi, I submitted initial version for introduce api for cpu model https://review.openstack.org/168982 . would you like take a look at whether the direction is right, then I'm going to fill up more detail. thanks in advance! | 11:16 |
*** oro has joined #openstack-nova | 11:16 | |
*** tojuvone has quit IRC | 11:16 | |
garyk | danpb: so why would we enable the user to set numa cpu's nodes etc. that should only be available to the admin | 11:17 |
danpb | there are use cases where performance critical applications have a need for very specific numa topologies | 11:18 |
garyk | that is the same case here | 11:18 |
*** romainh has joined #openstack-nova | 11:18 | |
danpb | so to avoid an explosion in the number of flavours, its possible to set it on the image | 11:19 |
garyk | someone may need X resource for a specific image | 11:19 |
danpb | but any settings on the image are only honoured, if the flavour does not have a specific topology set | 11:19 |
danpb | that wa the cloud admin can prevent tenant user from overriding stuff if they desire | 11:19 |
garyk | but the user can exploit that if she.he knew how to go about that | 11:19 |
garyk | danpb: the glance metadata can be controlled by the policy.json file | 11:21 |
*** markvoelker has joined #openstack-nova | 11:22 | |
*** coolsvap is now known as coolsvap|away | 11:22 | |
*** ianbrown has quit IRC | 11:22 | |
*** deepthi has quit IRC | 11:23 | |
*** rushiagr is now known as rushiagr_away | 11:24 | |
openstackgerrit | Peter Penchev proposed openstack/nova: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733 | 11:26 |
*** markvoelker has quit IRC | 11:27 | |
openstackgerrit | ShaoHe Feng proposed openstack/nova-specs: Attach/detach SR-IOV interface https://review.openstack.org/139910 | 11:31 |
*** shaohe_robot has joined #openstack-nova | 11:32 | |
*** tellesnobrega has joined #openstack-nova | 11:33 | |
openstackgerrit | Petrut Lucian proposed openstack/nova: Hyper-V: Fix serial port issue on Windows Threshold https://review.openstack.org/192617 | 11:34 |
*** samueldmq has joined #openstack-nova | 11:34 | |
*** ericksonsantos has joined #openstack-nova | 11:36 | |
openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V imagechace cleanup https://review.openstack.org/192618 | 11:36 |
*** mpaolino has quit IRC | 11:38 | |
*** kiran-r has joined #openstack-nova | 11:38 | |
openstackgerrit | ShaoHe Feng proposed openstack/nova-specs: enable pci api https://review.openstack.org/192622 | 11:40 |
*** tellesnobrega has quit IRC | 11:40 | |
*** ericksonsantos has quit IRC | 11:41 | |
*** sdake has joined #openstack-nova | 11:41 | |
*** samueldmq has quit IRC | 11:41 | |
*** oomichi has quit IRC | 11:42 | |
*** tellesnobrega has joined #openstack-nova | 11:43 | |
*** prashup has quit IRC | 11:44 | |
openstackgerrit | Petrut Lucian proposed openstack/nova: Hyper-V: Fix serial port issue on Windows Threshold https://review.openstack.org/192617 | 11:46 |
*** thangp has joined #openstack-nova | 11:46 | |
*** lucasagomes is now known as lucas-hungry | 11:47 | |
*** prashup has joined #openstack-nova | 11:48 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: virt: convert hardware module to use nova.objects.ImageMeta https://review.openstack.org/188789 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: test: add MatchType helper class as equivalent of mox.IsA https://review.openstack.org/190144 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 11:48 |
openstackgerrit | Daniel Berrange proposed openstack/nova: utils: ignore block device mapping in system metadata https://review.openstack.org/192627 | 11:48 |
openstackgerrit | Diana Clarke proposed openstack/nova: Add unit tests for the exact filters https://review.openstack.org/192514 | 11:49 |
*** dprince has joined #openstack-nova | 11:53 | |
sudipto | bauzas, Hi, I am doing a BP implementation to remove the scheduled_at from the instances table. Can you give any pointers on what might be required from the nova objects per say? I don't see any code in Nova currently working on that field - does it mean - that we still have to care about the objects? | 11:54 |
garyk | danpb: i understand the issues with the image meta and need to figure out how we can address that. would it be possible to at least get the BP for the flavor stuff approved. | 11:55 |
sudipto | sudipto, rather - does that mean - we can do away with the object changes? Also - since this field was meant/updated from the scheduler, do we need to bother about the conductor? | 11:55 |
danpb | garyk: oh is it not approved - i thught you'd already done the flavor stuff | 11:56 |
garyk | danpb: the flavors for cpu was added in juno. now we are adding vif, disk and memeory | 11:57 |
*** dulek_ has joined #openstack-nova | 11:57 | |
sudipto | dansmith, if you could also give your thoughts on my question above, that'd really help... | 11:57 |
*** samueldmq has joined #openstack-nova | 12:01 | |
*** dulek has quit IRC | 12:01 | |
danpb | garyk: if its just a blueprint no spec for that, then i believe johnthetubaguy has to toggle the approved flag | 12:01 |
garyk | danpb: ok, thanks. i'll reach out to him | 12:02 |
garyk | johnthetubaguy: can you please take a look at https://blueprints.launchpad.net/nova/+spec/vmware-limits | 12:03 |
garyk | it would be nice if we could get this approved to help get the patches for the featre in. | 12:03 |
*** sdake has quit IRC | 12:04 | |
*** Marga_ has quit IRC | 12:04 | |
*** markvoelker has joined #openstack-nova | 12:05 | |
*** htruta has joined #openstack-nova | 12:06 | |
bauzas | sudipto: I don't understand your question, you mean how to remove scheduled_at from the object, right ? | 12:10 |
sudipto | bauzas, yeah sort of. | 12:10 |
*** puniuee has joined #openstack-nova | 12:10 | |
bauzas | sudipto: ok, then it's not possible | 12:10 |
sudipto | bauzas, yeah - i mean it probably will exist there but we will ensure it's not written to the db? | 12:11 |
bauzas | sudipto: I mean, since the object needs to provide a backwards compat, you have to keep scheduled_at for the Instance object | 12:11 |
bauzas | sudipto: so, the field is nullable atm, right ? | 12:11 |
*** rushiagr_away is now known as rushiagr | 12:11 | |
sudipto | bauzas, yeah :) | 12:11 |
*** jlvillal has quit IRC | 12:11 | |
sudipto | bauzas, actually i see it initialized to NULL itself. | 12:11 |
bauzas | sudipto: okay, then you don't need to remove it, just leave a comment saying that it would be removed for a new major version for the Instance object | 12:12 |
bauzas | sudipto: but you can remove it from the DB | 12:12 |
bauzas | sudipto: and also remove it from the API, if it's given | 12:12 |
bauzas | sudipto: meaning that you won't have a scheduled_at field for the API and DB, but leave it for the object | 12:13 |
bauzas | sudipto: until we are sure that all the deployments are not using it, then we can remove it from the object | 12:13 |
sudipto | bauzas, ok - do i need to handle a case - like if the instance object has scheduled_at then we make it NULL etc? | 12:13 |
*** ferest has joined #openstack-nova | 12:14 | |
sudipto | bauzas, i was thinking that's not needed, since we are initializing it to NULL atm... | 12:14 |
*** moshele has quit IRC | 12:14 | |
bauzas | sudipto: the field is possibly updated by any object method, right ? | 12:15 |
bauzas | sudipto: so you perhaps need to work on leaving it unset | 12:15 |
bauzas | I mean for the object | 12:16 |
*** Marga_ has joined #openstack-nova | 12:16 | |
*** Guest94194 is now known as b3nt_pin | 12:16 | |
*** b3nt_pin is now known as beagles | 12:16 | |
bauzas | remove it from the DB, make sure the field is left unset for the Instance object, and then remove it from the API | 12:16 |
bauzas | sudipto: do you have a BP for that work ? | 12:16 |
bauzas | sudipto: even a spec if you need a REST API change N | 12:16 |
bauzas | ? | 12:16 |
sudipto | bauzas, don't see it being updated by any object method. | 12:16 |
sudipto | bauzas, yeah - approved one here: https://blueprints.launchpad.net/nova/+spec/cleanup-scheduled-at | 12:17 |
sudipto | bauzas, since none of the methods are updating it, i am guessing the unset requirement is automatically met? | 12:18 |
bauzas | sudipto: then yet | 12:20 |
bauzas | yeah | 12:20 |
bauzas | sudipto: possibly then that you just have to remove it from the API | 12:20 |
*** VW has joined #openstack-nova | 12:20 | |
bauzas | sudipto: ok, your spec is not mentioning a change in the REST API | 12:22 |
*** aysyanne has joined #openstack-nova | 12:22 | |
bauzas | sudipto: is the field exposed by the API? | 12:22 |
*** lpetrut has quit IRC | 12:22 | |
sudipto | bauzas, great! Yeah - there isn't a REST API change needed IMHO... | 12:22 |
*** lpetrut has joined #openstack-nova | 12:22 | |
sudipto | bauzas, no afaik - on code grep. | 12:22 |
bauzas | sudipto: re: your question about objects | 12:22 |
bauzas | sudipto: so, what you need is to make sure that you have a compatibility section for the object saying if scheduled_at is provided by the DB or not | 12:23 |
*** oro has quit IRC | 12:23 | |
*** doron_afk is now known as doron_ | 12:23 | |
bauzas | sudipto: since the object can be calling remotely the DB, you can't make sure that both DB and object are in sync | 12:23 |
*** jlvillal has joined #openstack-nova | 12:23 | |
bauzas | sudipto: but that's basically it | 12:23 |
*** ferest has quit IRC | 12:24 | |
sudipto | bauzas, yeah so i guess what you are saying is - if the object contains scheduled_at then do xyz (that is probably make it NULL)? | 12:24 |
bauzas | sudipto: ie. given the version you'll bump, you'll provide the scheduled_at field from the previous versions, or not if that's the new versions | 12:24 |
*** mwagner_afk has joined #openstack-nova | 12:24 | |
bauzas | sudipto: I'm just saying that if the field is set, you have to assume different behaviours depending on if the DB is including that field or not | 12:25 |
*** Marga_ has quit IRC | 12:25 | |
*** raildo has joined #openstack-nova | 12:26 | |
*** ociuhandu has quit IRC | 12:26 | |
sudipto | bauzas, yeah and by setting means - i understood - any object method working over it? which is not the case. Infact, there isn't anything that's setting or working on that field in the code base i greped on. | 12:26 |
bauzas | sudipto: the question I have is, any method is using scheduled_at or not ? | 12:26 |
*** tojuvone has joined #openstack-nova | 12:27 | |
bauzas | sudipto: do you know the last time when that field has been set ? | 12:27 |
*** exploreshaifali has quit IRC | 12:27 | |
bauzas | sudipto: oh you said that https://review.openstack.org/#/c/143725/ | 12:27 |
sudipto | bauzas, Afaik, no - this is not being set by any method. The reference, where it was removed is here: https://review.openstack.org/#/c/143725/ | 12:28 |
sudipto | bauzas, yeah | 12:28 |
bauzas | sudipto: ok, so previously that commit, it was set, righr ? | 12:29 |
sudipto | bauzas, yeah previous to that commit it was being set. | 12:29 |
sudipto | bauzas, which should be around Juno. | 12:31 |
*** kylek3h has joined #openstack-nova | 12:31 | |
*** ivasev has joined #openstack-nova | 12:33 | |
*** dane-fichter has joined #openstack-nova | 12:33 | |
bauzas | sudipto: okay, so if the object field is unset, you probably have to find a way to set it, if some old nodes waiting for it ? | 12:34 |
bauzas | (open question) | 12:34 |
*** dulek_ is now known as dulek | 12:34 | |
sudipto | bauzas, Isn't that something which should be dealt with even without this BP? | 12:35 |
*** oro has joined #openstack-nova | 12:36 | |
*** dims has quit IRC | 12:36 | |
*** dims has joined #openstack-nova | 12:36 | |
*** kodoku has joined #openstack-nova | 12:39 | |
*** dulek has quit IRC | 12:39 | |
bauzas | sudipto: honestly, I dunno, I didn't code-review that stuff | 12:39 |
*** kodoku has left #openstack-nova | 12:40 | |
*** shz has joined #openstack-nova | 12:40 | |
*** dulek has joined #openstack-nova | 12:40 | |
sudipto | bauzas, ok - let me just get a patchset out - and then ask people for their review comments. | 12:40 |
bauzas | fair | 12:40 |
*** shihanzhang has quit IRC | 12:43 | |
*** kylek3h has quit IRC | 12:44 | |
openstackgerrit | Li Yingjun proposed openstack/nova: Fixes TypeError when libvirt version is BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/192643 | 12:45 |
*** puniuee has quit IRC | 12:45 | |
*** lucas-hungry is now known as lucasagomes | 12:45 | |
*** jlanoux has quit IRC | 12:46 | |
*** exploreshaifali has joined #openstack-nova | 12:46 | |
*** moshele has joined #openstack-nova | 12:48 | |
*** VW has quit IRC | 12:49 | |
*** doron_ is now known as doron_afk | 12:49 | |
*** prashup has quit IRC | 12:50 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova-specs: Allow list all instances for non-admin based on policy https://review.openstack.org/191151 | 12:53 |
*** VW has joined #openstack-nova | 12:54 | |
dims | johnthetubaguy: filed a spec in support of a bug that seems to need a microversion. please take a look when you get a chance ^ | 12:54 |
*** baoli_ has joined #openstack-nova | 12:55 | |
*** eharney has joined #openstack-nova | 12:55 | |
*** moshele has quit IRC | 12:56 | |
*** atuvenie has joined #openstack-nova | 12:56 | |
*** IanGovett has joined #openstack-nova | 12:56 | |
*** moshele has joined #openstack-nova | 12:56 | |
*** ddutta has joined #openstack-nova | 12:57 | |
*** edmondsw has joined #openstack-nova | 12:57 | |
*** baoli_ has quit IRC | 12:58 | |
*** baoli has joined #openstack-nova | 12:58 | |
*** otter768 has joined #openstack-nova | 12:59 | |
*** vladikr has joined #openstack-nova | 13:02 | |
*** ddutta has quit IRC | 13:02 | |
*** mjturek1 has joined #openstack-nova | 13:02 | |
openstackgerrit | Nikola Dipanov proposed openstack/nova: Add documentation for block device mapping https://review.openstack.org/192651 | 13:03 |
*** otter768 has quit IRC | 13:03 | |
ndipanov | johnthetubaguy, sdague ^^ some docs | 13:04 |
sdague | ndipanov: \o/ more docs always good | 13:04 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: Add documentation for block device mapping https://review.openstack.org/192651 | 13:05 |
ndipanov | yeah I was embarassed enough about never writing those that I decided to write them :) | 13:05 |
*** mdrabe has joined #openstack-nova | 13:06 | |
*** sudipto has quit IRC | 13:06 | |
johnthetubaguy | ndipanov: thats awesome, thank you | 13:07 |
*** diogogmt has quit IRC | 13:07 | |
johnthetubaguy | ndipanov: can I add that patch into the blueprint I am working on? | 13:07 |
johnthetubaguy | ndipanov: this one: https://blueprints.launchpad.net/nova/+spec/devref-refresh-liberty | 13:07 |
openstackgerrit | John Garbutt proposed openstack/nova: Add documentation for block device mapping https://review.openstack.org/192651 | 13:08 |
ndipanov | sure - but I left it as a last patch on branch for a reason if that's ok | 13:08 |
* johnthetubaguy stops hiding now he is back home from visiting the Xen peoples | 13:08 | |
johnthetubaguy | garyk: I here there is a blueprint that needs approving? | 13:09 |
*** VW has quit IRC | 13:09 | |
johnthetubaguy | s/here/hear/ | 13:09 |
johnthetubaguy | ndipanov: I am totally cool with that | 13:09 |
kashyap | ndipanov: Nice work on block dev docs! Bookmarked to read :-) | 13:10 |
dane-fichter | johnthetubaguy: is nova moving to using python-glanceclient anytime soon? | 13:10 |
openstackgerrit | Jim Rollenhagen proposed openstack/nova: Ironic: block storage support https://review.openstack.org/184652 | 13:11 |
johnthetubaguy | dane-fichter: so we still have to work out whats the right thing for those, its a little taffled up in the openstack SDK discussions | 13:12 |
*** sahid has quit IRC | 13:12 | |
johnthetubaguy | dane-fichter: I think the real issue is having something a bit like brick we are getting from cinder for doing things with glance | 13:12 |
bauzas | johnthetubaguy: ndipanov: if you please, let me advert some spec I really want to get it approved for Liberty since I'm targeting to have it priotized https://review.openstack.org/#/c/179224/9/specs/liberty/approved/check-destination-on-migrations.rst,cm | 13:12 |
*** dpaterson has joined #openstack-nova | 13:13 | |
ndipanov | bauzas, ok will look but coffee first! | 13:13 |
bauzas | johnthetubaguy: ndipanov: took the last comments from you, guys | 13:13 |
garyk | johnthetubaguy: hi, yes, if possible could we please get https://blueprints.launchpad.net/nova/+spec/vmware-limits approved | 13:13 |
johnthetubaguy | bauzas: so I think we had users asking for that, I was just a bit worried about the use case bit last time I looked, not had chance to revisit that yet | 13:13 |
bauzas | ndipanov: coffee is definitely a good option | 13:13 |
garyk | it is exposing specifc driver functionality via flavors | 13:13 |
bauzas | johnthetubaguy: yeah, we have customers waiting for it | 13:13 |
*** jamielennox is now known as jamielennox|away | 13:13 | |
garyk | via image properties requieres discussing and i need to post a spec and another bp | 13:13 |
*** jecarey has joined #openstack-nova | 13:13 | |
*** ociuhandu has joined #openstack-nova | 13:13 | |
bauzas | johnthetubaguy: and that's also caring about fixing how migrations are done to be consistent with regular boots | 13:14 |
*** amotoki_ has joined #openstack-nova | 13:14 | |
*** nihilifer has quit IRC | 13:14 | |
bauzas | not the story with lots of 'if' branches to explain to ops why the request didn't fail or failed because they specified a dest host or not | 13:14 |
johnthetubaguy | garyk: is this all feature parity stuff, or are some of these new? | 13:14 |
*** jog0 has joined #openstack-nova | 13:15 | |
johnthetubaguy | garyk: we have done a spec for this one, but its was a really small one: http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/hyperv-storage-qos.html | 13:15 |
*** tojuvone has quit IRC | 13:15 | |
johnthetubaguy | garyk: if its just a "me too" blueprint, we can approve that at the next nova meeting (I added it to the agenda to make sure I don't forget) | 13:15 |
garyk | johnthetubaguy: ok, thanks | 13:16 |
johnthetubaguy | garyk: actually general question, do you folks, as a vmware subteam I mean, have any other blueprint "paperwork" thats needed for liberty at this point | 13:16 |
johnthetubaguy | garyk: do let me know soon, and we should still be able to get it sorted in time, at this point | 13:17 |
johnthetubaguy | garyk: I think the blueprint is just missing a few detail, btw, like the names of the image properties / flavor extra specs that are required to activate the code | 13:17 |
garyk | johnthetubaguy: no, just this one and the console support (but that spec has already been approved) | 13:17 |
johnthetubaguy | garyk: ah, thats good, so we are close | 13:17 |
dims | johnthetubaguy: not sure if you saw my request above (https://review.openstack.org/191151) in support of a bug fix | 13:18 |
garyk | johnthetubaguy: for the image properties dan want me to draft a seperate spec. i will do that tomorrow | 13:18 |
johnthetubaguy | garyk: so if you could submit a brief spec for that vmware-limits blueprint, I think thats the easiest way | 13:18 |
johnthetubaguy | dims: sorry, znc missed that some how, we have been falling out | 13:18 |
garyk | johnthetubaguy: ok, sure will do | 13:18 |
johnthetubaguy | garyk: awesome, thank you | 13:18 |
dims | thanks just queueing it up on your list :) | 13:18 |
*** mpaolino has joined #openstack-nova | 13:18 | |
*** kylek3h has joined #openstack-nova | 13:19 | |
johnthetubaguy | dims: oh thats a good one to get fixed, not totally sure that actually need a spec, but probably easier to do it as a spec | 13:19 |
johnthetubaguy | dims: its just a policy change right? | 13:19 |
dane-fichter | johnthetubaguy: can i ask you a couple questions about your comments on the image signing / encryption spec? | 13:20 |
johnthetubaguy | dane-fichter: yes, sure | 13:20 |
*** jamielennox|away is now known as jamielennox | 13:21 | |
dims | johnthetubaguy: check the policy and if policy is ok then allow them to get the data they need | 13:21 |
johnthetubaguy | dims: ignore me, I understand why this needs a spec now, thanks | 13:21 |
dims | needed a microversion | 13:21 |
johnthetubaguy | dims: yeah, sounds good... slight idea though, I will add it on the spec | 13:21 |
*** xyang1 has joined #openstack-nova | 13:23 | |
dims | ack thanks | 13:23 |
*** lucasagomes is now known as lucas-brb | 13:23 | |
*** ddutta has joined #openstack-nova | 13:25 | |
*** IanGovett has quit IRC | 13:25 | |
alex_xu | dansmith: hi, I submitted initial version for new api of cpu model https://review.openstack.org/168982, hope you give suggestion if this is on the right direction when you have time, then I will fill up more detail. thanks! | 13:26 |
openstackgerrit | Sylvain Bauza proposed openstack/nova-specs: Proposes to check destination on migrations https://review.openstack.org/179224 | 13:27 |
*** artom has joined #openstack-nova | 13:29 | |
BobBall | bauzas: FYI I'm looking at redeploying to pick up a new zuul in case there is another fix. A redeploy likely won't be ready for a few days as I also need to switch accounts (to avoid redploying twice in quick succession) | 13:33 |
*** jamielennox is now known as jamielennox|away | 13:33 | |
bauzas | BobBall: no worries, my finger likes copy/pasting "xen: recheck" ;) | 13:33 |
bauzas | BobBall: thanks for the update and the great investigation job you did | 13:33 |
BobBall | Ah good. Well, maybe I'll leave it as it is then. I wouldn't want to deprive your fingers of such joy ;) | 13:33 |
bauzas | BobBall: eh :) | 13:34 |
ndipanov | kashyap, do give feedback if something is unclear or missing on that doc pls! | 13:34 |
bauzas | BobBall: seriously surprised to see I'm the only one yelling at it | 13:34 |
bauzas | BobBall: probably because I'm French, I assume | 13:34 |
kashyap | ndipanov: Will do this week. Testing the selective block dev migration patches from libvirt git now. | 13:35 |
BobBall | I assume it's something particular about your patchset that's triggering it | 13:35 |
bauzas | BobBall: my git-review client, maybe ? | 13:35 |
bauzas | BobBall: because my branch is just a regular git branch with a certain number of commits :) | 13:35 |
BobBall | Or a strange interaction between the patches you've got and the current state of the repo? | 13:35 |
BobBall | Have you rebased on master lately? | 13:36 |
bauzas | BobBall: doing that very often | 13:36 |
*** erlon has joined #openstack-nova | 13:36 | |
bauzas | BobBall: I'm working on the nova objects, and those stuff don't let you being late on your branch | 13:36 |
BobBall | Or perhaps the number of changes - if you change the same files several times, maybe there is something that exposes. Dunno. | 13:36 |
bauzas | BobBall: or perhaps I'm the only guy who sends more than one commit, meaning I'm the only guy who works here ? maybe | 13:37 |
BobBall | hehe | 13:37 |
*** jskinner has joined #openstack-nova | 13:37 | |
BobBall | True. I try to avoid working whenever I can. | 13:37 |
* BobBall looks around to check that his boss hasn't joined IRC. | 13:37 | |
bauzas | we all do, that's why Frenchies have decent vacations | 13:37 |
* BobBall should move to France. | 13:38 | |
bauzas | think moving in August or May, you should enjoy | 13:38 |
*** VW has joined #openstack-nova | 13:39 | |
bauzas | BobBall: anyway, thanks for your support, greatly appreciated :) | 13:39 |
BobBall | Anytime | 13:40 |
openstackgerrit | Chris Gacsal proposed openstack/nova: REST API changes to support multiple volume attach https://review.openstack.org/153038 | 13:40 |
*** burt has joined #openstack-nova | 13:41 | |
*** lucas-brb is now known as lucasagomes | 13:42 | |
*** ctrath has joined #openstack-nova | 13:42 | |
*** jamielennox|away is now known as jamielennox | 13:42 | |
*** mriedem_away is now known as mriedem | 13:42 | |
*** moshele has quit IRC | 13:44 | |
*** kbyrne has quit IRC | 13:46 | |
*** Longgeek has quit IRC | 13:48 | |
*** aswadr has joined #openstack-nova | 13:49 | |
*** Longgeek has joined #openstack-nova | 13:49 | |
mriedem | garyk: https://review.openstack.org/#/c/169732/ looks ok to me now if you want to get it rebased | 13:50 |
garyk | mriedem: thanks! i'll rebase | 13:51 |
*** asrangne has quit IRC | 13:52 | |
openstackgerrit | garyk proposed openstack/nova-specs: Propose VMware limits, reservation and shares https://review.openstack.org/192675 | 13:52 |
*** alex_klimov has quit IRC | 13:54 | |
*** prashup has joined #openstack-nova | 13:55 | |
*** moshele has joined #openstack-nova | 13:56 | |
garyk | johnthetubaguy: ^ | 13:57 |
openstackgerrit | garyk proposed openstack/nova: VMware: detach cinder volume when instance destroyed https://review.openstack.org/169732 | 13:58 |
garyk | mriedem: ^ rebased. thanks | 13:59 |
openstackgerrit | garyk proposed openstack/nova: VMware: remove redundant check for block devices https://review.openstack.org/170641 | 14:00 |
*** cfriesen has joined #openstack-nova | 14:01 | |
mriedem | BobBall: i see that xenproject ci voted +1 on this immediately https://review.openstack.org/#/c/169732/ - is that due to filtering the change? i.e. since it's not xen or common code it's ignored by xenproject ci? | 14:03 |
BobBall | correct mriedem | 14:03 |
* BobBall checks on the status of the zuul change to stop that | 14:03 | |
mriedem | BobBall: is there any way from it not giving the +1? | 14:03 |
openstackgerrit | garyk proposed openstack/nova: VMware: Use actual VM state instead of using the instance vm_state https://review.openstack.org/190627 | 14:03 |
mriedem | ah ok | 14:03 |
dansmith | ah, I was wondering the same yesterday | 14:03 |
dansmith | it's very misleading | 14:04 |
BobBall | mriedem / dansmith: https://review.openstack.org/#/c/188383/ | 14:04 |
dansmith | hmm, seems like there should be another way to prevent that, but IANA3PCIA | 14:05 |
*** ijuwang has quit IRC | 14:06 | |
BobBall | The only option we have ATM is to run a job - whether it is the real job or a noop. "A result" will always be reported to a project that you're running tests for | 14:06 |
openstackgerrit | garyk proposed openstack/nova: VMware: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189085 | 14:06 |
*** tonytan4ever has joined #openstack-nova | 14:06 | |
BobBall | The issue here is we use skipif to not run on changes that can't affect our CI - I guess mostly to reduce load | 14:07 |
BobBall | So from there there are no jobs :) | 14:07 |
dansmith | BobBall: you don't have to explain, it just seems like you should be able to do frontend filtering, but I guess you're trying to use the (newer) skipif bits and that's buggy, right? | 14:07 |
BobBall | frontend filtering? | 14:07 |
BobBall | I'm happy to filter any-which-way that makes sense. I just assumed that skipif was the best way to do it | 14:07 |
*** ijuwang has joined #openstack-nova | 14:07 | |
dansmith | like, never letting gerrit events get to you that don't apply | 14:07 |
dansmith | BobBall: I'm sure it is | 14:08 |
BobBall | Ah... Unfortunately the gerrit stream doesn't contain any information on the changes in the patchset. Only zuul gets to see them when it pulls the change | 14:08 |
dansmith | hmm, okay | 14:09 |
dansmith | before skipif, | 14:09 |
BobBall | Would you prefer that the CI had a "I'm not testing this change" no op job? | 14:09 |
dansmith | people were not running things against changes that didn't apply, | 14:09 |
BobBall | e.g. "dummy-not-testing"? | 14:09 |
dansmith | but maybe they weren't using zuul | 14:09 |
dansmith | BobBall: no, this fix is the right fix I'm sure, no need to change course, I'm just expressing confusion not proposing a change | 14:09 |
BobBall | Yup. I added some custom code in the Citrix CI that downloaded the patchset and conditionally skipped a test. But it was seriously ugly and quite error prone | 14:09 |
dansmith | BobBall: if this was going to persist for a long time, some mitigation would be good, but I assume this isn't far off from landing | 14:10 |
*** artom has quit IRC | 14:10 | |
BobBall | I hope it's not, agreed. | 14:10 |
ndipanov | bauzas, so what happens to the old way of forcing hosts once we introduce this? | 14:11 |
bauzas | ndipanov: you mean when migrating ? | 14:12 |
bauzas | ndipanov: or when booting ? sorry if unclear, but force_hosts is a hack for the boot command | 14:12 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Support Network objects in set_network_host https://review.openstack.org/192396 | 14:12 |
*** mdrabe has quit IRC | 14:12 | |
openstackgerrit | Ed Leafe proposed openstack/nova: Add documentation for block device mapping https://review.openstack.org/192651 | 14:13 |
ndipanov | right when booting | 14:13 |
edleafe | ndipanov: great doc writeup on block device mapping. I had some grammar nits, so I just fixed them and pushed a new patch. ^ | 14:13 |
openstackgerrit | garyk proposed openstack/nova: Compute: enable admin to decide action for conflicting states https://review.openstack.org/190047 | 14:13 |
ndipanov | edleafe, many thanks! | 14:14 |
edleafe | ndipanov: glad to help! | 14:14 |
*** kbyrne has joined #openstack-nova | 14:15 | |
ndipanov | bauzas, when booting... will that have the same behavior now | 14:15 |
ndipanov | or we leave that case as it was? | 14:15 |
*** zzzeek has joined #openstack-nova | 14:15 | |
openstackgerrit | garyk proposed openstack/nova-specs: Propose VMware limits, reservation and shares https://review.openstack.org/192675 | 14:17 |
*** nagyz has joined #openstack-nova | 14:17 | |
*** doron_afk is now known as doron_ | 14:18 | |
bauzas | ndipanov: I would like to do this as a separate spec | 14:18 |
bauzas | ndipanov: because I dislike force_hosts but I would prefer to have a distinct way to ask for forcing a destination than just using --az fake_az:host | 14:19 |
ndipanov | bauzas, I mean ok but how much more work is that on top of this. | 14:19 |
ndipanov | up to you | 14:19 |
bauzas | ndipanov: MHO is that we should provide a new option when booting (ie. the same --force_destination) with an optional destination host | 14:20 |
mriedem | danpb: semi interesting convo in -infra right now about libvirt and bug 1353939 | 14:20 |
openstack | bug 1353939 in OpenStack Compute (nova) "Rescue fails with 'Failed to terminate process: Device or resource busy' in the n-cpu log" [High,Fix committed] https://launchpad.net/bugs/1353939 - Assigned to Matt Riedemann (mriedem) | 14:20 |
mriedem | danpb: basically ceilometer has a compute agent that polls libvirt directly | 14:20 |
mriedem | which theoretically could be causing that ebusy maybe | 14:20 |
bauzas | ndipanov: but I think it should be done at M | 14:20 |
*** jog0 has quit IRC | 14:20 | |
bauzas | ndipanov: because just the current spec is enough large for Liberty only for me | 14:20 |
danpb | mriedem: hmm, i'd be surprised if ceilometer could do something that would block SIGKILL from working in reasonable timeframe | 14:21 |
* mriedem graps at straws | 14:21 | |
mriedem | *grasps | 14:21 |
*** iamjarvo has joined #openstack-nova | 14:22 | |
*** nkrinner has quit IRC | 14:23 | |
*** exploreshaifali has quit IRC | 14:24 | |
*** jog0 has joined #openstack-nova | 14:24 | |
*** alex_klimov has joined #openstack-nova | 14:25 | |
claudiub | danpb: hi | 14:26 |
*** Marga_ has joined #openstack-nova | 14:26 | |
ndipanov | bauzas, OK fair enough | 14:27 |
*** mdbooth has quit IRC | 14:27 | |
ndipanov | also I see that we merged the RequestSpec spec with all the duplication... | 14:27 |
ndipanov | ok let's see how that goes once you get further in | 14:28 |
ndipanov | maybe it'll be ok | 14:28 |
ndipanov | maybe it'll be bad | 14:28 |
bauzas | ndipanov: otp | 14:28 |
*** sudipto has joined #openstack-nova | 14:28 | |
*** mdrabe has joined #openstack-nova | 14:28 | |
*** VW has quit IRC | 14:28 | |
*** jlanoux has joined #openstack-nova | 14:29 | |
*** sudipto has quit IRC | 14:30 | |
*** zz_jgrimm is now known as jgrimm | 14:30 | |
nagyz | sup guys | 14:31 |
*** cdent has joined #openstack-nova | 14:31 | |
*** xyang1 has quit IRC | 14:31 | |
cdent | mriedem: and...go! | 14:31 |
mriedem | cdent: so for this libvirt pollster thing in ceilometer, was there ever a bp or spec in nova for doing something different with notifications from nova? | 14:32 |
sdague | mriedem: this stuff goes back a *long* ways | 14:32 |
cdent | Not that I'm aware of, but that's not the same as no. | 14:32 |
mriedem | the get_instance_diagnostics stuff in nova now provides a versioned serializable structured object with the vm data | 14:32 |
cdent | I think there's some historical fud that's been a blocker in the past. | 14:32 |
mriedem | sdague: i get that, which predates the get_instance_diagnostics rewrite for v2.1 | 14:32 |
sdague | cdent: what's the default poll interval here? | 14:33 |
cdent | sdague: 600 seconds | 14:33 |
openstackgerrit | Stefan Amann proposed openstack/nova: Support host type specific block volume attachment https://review.openstack.org/192556 | 14:33 |
cdent | which pretty much every operator says is far far far too low, and all the older devs say "you need to keep that as high as possible" | 14:33 |
mriedem | so ceilometer could be using the v2.1 API to get instance diagnostics on that big of an interval, or nova could just have a periodic task in the compute manager that emits the notification | 14:33 |
cdent | s/low/long/ | 14:33 |
mriedem | keep it high so you're not hammering the hypervisor? | 14:34 |
cdent | long term I want to kill all polling from ceilo and just hear notifications | 14:34 |
sdague | right, I think we moved that up because otherwise ceilometer was consuming over 50% of all resources on the gate test nodes | 14:34 |
mriedem | i'm not sure how you'd trigger a notification in the diags case | 14:34 |
danpb | yeah, the nova virt driver really should have a way to push performance data to ceilometer | 14:34 |
*** electrocucaracha has joined #openstack-nova | 14:34 | |
danpb | because libvirt can easily push out bulk stats data for every running guest in one go | 14:34 |
cdent | sdague: there's two different sources of load: one is hammering the hypervisor, the other is asking the nova-api for resource metadata | 14:35 |
mriedem | danpb: the compute manager could already do this regardless of virt driver | 14:35 |
cdent | there is a spec in progress to vastly the improve the latter (we nope) | 14:35 |
danpb | which will be waaaay more efficient than doing one rpc request on instance diagnostics for each guest | 14:35 |
mriedem | danpb: but i wonder if the other virt drivers can do that | 14:35 |
danpb | mriedem: you still don't want the compute manager querying the virt driver for each individual guest | 14:35 |
mriedem | especially vmware/ironic | 14:36 |
danpb | so it would at least need a new virt/driver.py api to request bulk stats | 14:36 |
cdent | resource metadata cachign: https://github.com/openstack/ceilometer-specs/blob/master/specs/liberty/resource-metadata-caching.rst | 14:36 |
sdague | mriedem: well, that's their own fault for being silly | 14:36 |
mriedem | true | 14:36 |
*** sudipto has joined #openstack-nova | 14:36 | |
cdent | getting the hypervisors to drop a regular notification onto messaging would be a huge win for any of the many things that like info, not just ceilo | 14:38 |
kashyap | mriedem: Unintentionally I've just reproduced the thing locally "error: Failed to terminate process 6426 with SIGKILL: Device or resource busy" | 14:39 |
sdague | cdent: agreed, so there was a failed attempt to get someone to write down what they wanted that schema to be a couple cycles ago when it looked like stacktac and ceilo would collab on it | 14:39 |
kashyap | While testing migration related things as I wanted to kill off an unwanted guest. | 14:39 |
danpb | anyway, insane architecture design decisions of the past aside, I really struggle to see how this ceilometer code would prevent the kenrel handling SIGKILL in a timely manner | 14:39 |
sdague | any idea if anyone is still trying to get there, because I think that's probably the missing next step | 14:39 |
mriedem | i think cdent said he had nothing to work on and this was right up his alley :) | 14:39 |
cdent | that's exactly right mriedem :) | 14:40 |
sdague | danpb: yeh, it was a shot in the dark. It was just surprising that we had 2 active clients. | 14:40 |
mriedem | maybe this jason myers on the ceilometer spec would be interested in helping on the nova side | 14:40 |
mriedem | defining the required spec | 14:40 |
cdent | sdague: It's remained sort of latent on the ceilometer agent for the past two cycles | 14:40 |
mriedem | *schema i mean | 14:40 |
danpb | mriedem: did i recall you saying this only happened on certain OS versions ? | 14:40 |
*** kiran-r has quit IRC | 14:41 | |
mriedem | danpb: the retry loop we added seems to work with newer libvirt/qemu in the centos6/fc21 jobs | 14:41 |
*** sambol has joined #openstack-nova | 14:41 | |
cdent | mriedem: jason's a good idea, this is right in his area of concern and interest | 14:41 |
mriedem | danpb: but those are also run far less often since they are experimental jobs | 14:41 |
sdague | mriedem: we run a lot less of those jobs, so are you sure it's statistically significant? | 14:41 |
mriedem | sdague: jinx | 14:41 |
mriedem | i'll jump over to -ceilometer and ping jasonamyers | 14:42 |
*** jasonamyers has joined #openstack-nova | 14:42 | |
danpb | mriedem: so you're saying the bulk of the failures are on the ubuntu images then | 14:43 |
cdent | I momentarily had some traction with sandywalsh on notifications-related stuff but a) he left the building b) there was stronger interest in formalizing notification schema | 14:43 |
cdent | b, to me, has felt rather like a distracting bike shed (after initial enthusiasm) | 14:43 |
*** VW has joined #openstack-nova | 14:44 | |
sdague | danpb: mriedem is saying that by absolute could, yes. But by absolute count we also run at least 100x more jobs on ubuntu images, so it's not really clear that means anything. | 14:44 |
sdague | fwiw, the most recent logstash fail is on f21 | 14:44 |
sdague | http://logs.openstack.org/29/192329/2/check/check-tempest-dsvm-f21/6bd72b0/logs/screen-n-cpu.txt | 14:44 |
mriedem | cdent: yeah (b) seems like it will never happen | 14:45 |
mriedem | b includes versioned notifications i assume | 14:45 |
*** belmoreira has quit IRC | 14:45 | |
cdent | yeah, which, yeah great whatever, but first we need some notifications | 14:45 |
danpb | sdague: yeah, agreed | 14:45 |
cdent | So it looks like jasonamyers and I are willing to do something here, can we define what that "something" is? | 14:46 |
sdague | so... actually | 14:46 |
sdague | http://logstash.openstack.org/#eyJmaWVsZHMiOltdLCJzZWFyY2giOiJtZXNzYWdlOihcImxpYnZpcnRFcnJvcjogRmFpbGVkIHRvIHRlcm1pbmF0ZSBwcm9jZXNzXCIgTk9UIFwiaW4gX3NodXRkb3duX2luc3RhbmNlXCIpIEFORCB0YWdzOlwic2NyZWVuLW4tY3B1LnR4dFwiIiwidGltZWZyYW1lIjoiNjA0ODAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJvZmZzZXQiOjAsInRpbWUiOnsidXNlcl9pbnRlcnZhbCI6MH0sInN0YW1wIjoxNDM0NTUyMzQzNDE1LCJtb2RlIjoic2NvcmUiLCJhbmFseXplX2ZpZWxkIjoiYnVpbGRfbmFtZSJ9 | 14:46 |
sdague | this is only faiing on f21, centos7, and the multinode jobs | 14:46 |
openstackgerrit | Jeffrey Zhang proposed openstack/nova: Fix the wrong address ref when the fixed_ip is invalid https://review.openstack.org/192707 | 14:46 |
openstackgerrit | Dan Smith proposed openstack/nova: WIP: Test removing flavors in sysmeta from compat_instance https://review.openstack.org/192708 | 14:46 |
sdague | mriedem: that's the right query, right? | 14:47 |
sdague | it was the one linked | 14:47 |
danpb | what OS does the multinode job run on ? | 14:47 |
mriedem | sdague: this is the e-r query http://goo.gl/kgNNKj | 14:47 |
mriedem | danpb: trusty | 14:48 |
sdague | mriedem: ok, the elastic recheck signature is stale then | 14:48 |
*** unicell has quit IRC | 14:48 | |
*** exploreshaifali has joined #openstack-nova | 14:48 | |
sdague | mriedem: we should get rid of that other er sig | 14:49 |
sdague | mriedem: http://goo.gl/kgNNKj is only 4 hits in 7 days? | 14:49 |
*** diogogmt has joined #openstack-nova | 14:50 | |
sdague | 3 of which are centos7 | 14:50 |
*** unicell has joined #openstack-nova | 14:50 | |
openstackgerrit | Chris Gacsal proposed openstack/nova: REST API changes to support multiple volume attach https://review.openstack.org/153038 | 14:50 |
*** Jeffrey4l has joined #openstack-nova | 14:50 | |
*** varunarya has joined #openstack-nova | 14:51 | |
*** tjones1 has joined #openstack-nova | 14:52 | |
Jeffrey4l | beagles, hi, I want to change the nova api current behavior. I was asked for some help from u. The detail can be found https://review.openstack.org/#/c/154068/11/nova/network/neutronv2/api.py . Could u give me some advice? | 14:52 |
*** sdake_ has joined #openstack-nova | 14:53 | |
mriedem | danpb: fyi, this is what ceilometer would be looking to replace with a bulk diag notification: http://docs.openstack.org/admin-guide-cloud/content/section_telemetry-compute-meters.html - anything with 'pollster' | 14:53 |
mriedem | sdague: looking | 14:53 |
*** dulek has quit IRC | 14:53 | |
*** xyang2 has joined #openstack-nova | 14:53 | |
openstackgerrit | Josh Durgin proposed openstack/nova-specs: RBD instance snapshots https://review.openstack.org/188244 | 14:53 |
beagles | Jeffrey4l, looking | 14:54 |
*** moshele has quit IRC | 14:54 | |
*** moshele has joined #openstack-nova | 14:54 | |
Jeffrey4l | thanks. | 14:54 |
*** moshele has quit IRC | 14:54 | |
*** moshele has joined #openstack-nova | 14:55 | |
danpb | mriedem: those don't look too unreasonable | 14:55 |
*** tonytan_brb has joined #openstack-nova | 14:55 | |
*** tonytan4ever has quit IRC | 14:55 | |
*** xylan_kong has left #openstack-nova | 14:55 | |
claudiub | danpb: hi | 14:56 |
*** tonytan_brb has quit IRC | 14:56 | |
*** Jeffrey4l has quit IRC | 14:56 | |
danpb | claudiub: hi ? | 14:56 |
*** tonytan4ever has joined #openstack-nova | 14:56 | |
*** ssurana has joined #openstack-nova | 14:56 | |
*** Jeffrey4l has joined #openstack-nova | 14:56 | |
claudiub | danpb: So, from what I understand, you have experience in libvirt using shared storage, right? | 14:57 |
sdague | danpb: so what would you envision as the flow to get all this stuff out as notifications? | 14:57 |
danpb | claudiub: i've used nfs once or two to test migration, if that counts as experiance | 14:57 |
*** jdurgin1 has joined #openstack-nova | 14:58 | |
danpb | sdague: i guess the nova compute manager would periodically query virt driver for bulk stats about all guests | 14:59 |
cdent | sdague, danpb: A big question for me (re flow) is whether the notifications would come as a big huge drop about all the current instances on a node, or whether there are efficiencies gained by the data gathering being granular? | 14:59 |
danpb | and then push that data out on the message bus to some target ? | 14:59 |
mriedem | danpb: is there a page somewhere that shows what a bulk data dump from libvirt looks like? | 14:59 |
*** nihilifer has joined #openstack-nova | 15:00 | |
claudiub | danpb: I see. Well, there is a thingy, that I was hoping you would have some advice on. So the scenario is that there are multiple compute nodes over the same shared storage. The issue is that they both report the same disk resources in this case, which can be problematic, if you spawn 1 instance on one one with 200GB storage, the other compute nodes will not see that 200 GB as used storage.. | 15:00 |
danpb | if we want to unicast the data on the bus, the act of deploying ceilometer would involve a nova.conf setting to tell it where to target the data | 15:00 |
claudiub | danpb: ... and report it as free. | 15:00 |
*** otter768 has joined #openstack-nova | 15:00 | |
danpb | claudiub: yes, the resource manager handling of shared storage is utterly fubar currently | 15:01 |
*** browne has joined #openstack-nova | 15:01 | |
sdague | cdent: good questions, I honestly expect it's more efficient to collect and send in bulk, but that's just a guess. | 15:01 |
jdurgin1 | danpb: dansmith: jaypipes: thanks for your comments on https://review.openstack.org/#/c/188244/ - ready for a final review now | 15:01 |
danpb | claudiub: last time i looked at it, i came to the conclusion it couldn't be fixed in its current form and would need majorly redesigning | 15:01 |
jaypipes | jdurgin: it's right at the top of my list of reviews to get to today. | 15:02 |
mriedem | sdague: cdent: danpb: jasonamyers: ildikov: i'll sign up to take notes and get those into the ML, then start working a rough draft spec with help from jasonamyers on the ceilometer particulars (what ceilometer needs out of nova) | 15:02 |
mriedem | ctrath: ^ | 15:02 |
danpb | cdent: given that list of data items that ceilometer wants - if we queried libvirt for each guest, for each type of data, that would require on the order of O(n*6) libvirt API calls | 15:02 |
jdurgin1 | jaypipes: thanks! | 15:02 |
*** mpaolino has quit IRC | 15:03 | |
danpb | and even though they're fast, that still doesn;t really scale nicely | 15:03 |
claudiub | danpb: I see... well, this is a problem I'm currently facing and I was hoping for some answers.. well, if by any chance I find a worthwhile solution, I'll let you know. | 15:03 |
cdent | danpb: I wasn't suggesting we considering doing one for each datapoint, just wondering what the options are | 15:03 |
danpb | newer libvirt has an API call that lets us query data for all running guests in a single api call | 15:03 |
sdague | mriedem: sounds great, thanks | 15:03 |
sdague | danpb: when did that get added? | 15:03 |
danpb | so my preference would be to design something that lets us take advantage of that | 15:03 |
jasonamyers | thank you mriedem | 15:03 |
cdent | danpb: makes sense | 15:04 |
cdent | mriedem++ | 15:04 |
danpb | sdague: about version 1.2.8 | 15:04 |
dansmith | jaypipes: you're the primary reviewer that needs to approve that I think, so let me know when you're happy with it and I'll hit it again | 15:04 |
phschwartz | Hey all, can I get some more people reviewing https://review.openstack.org/#/c/154521/ I want to try to get it in as soon as possible so we can begin testing it in a new release version he in Rax. | 15:04 |
danpb | sdague: but that's not a big deal - if we run against older libvirt we'd simply fallback to querying data the slow way | 15:04 |
mriedem | danpb: ok, so we'd have a version cap but could test this in the centos6/fc21 job at least | 15:04 |
jaypipes | dansmith: will do. | 15:04 |
mriedem | danpb: ah, or that | 15:04 |
*** xyang1 has joined #openstack-nova | 15:04 | |
sdague | danpb: ok, so we'd still need a per instance query in the fallback | 15:04 |
danpb | yep | 15:05 |
*** otter768 has quit IRC | 15:05 | |
sdague | but that seems still less expensive than the current way | 15:05 |
*** openstackgerrit has quit IRC | 15:05 | |
danpb | yeah, as you only have the overhead of the libvirt rpc system there which is low | 15:05 |
danpb | the real key is to avoid nova rpc overhead per instance | 15:05 |
*** openstackgerrit has joined #openstack-nova | 15:05 | |
*** amotoki_ has quit IRC | 15:06 | |
openstackgerrit | Ankit Agrawal proposed openstack/nova: Test patch race condition in delete (do not merge) https://review.openstack.org/191732 | 15:06 |
danpb | for reference, the libvirt api i'm referring to is http://libvirt.org/html/libvirt-libvirt-domain.html#virConnectGetAllDomainStats | 15:06 |
*** mdrabe has quit IRC | 15:07 | |
*** agireud has joined #openstack-nova | 15:08 | |
*** xyang2 has quit IRC | 15:08 | |
*** xyang2 has joined #openstack-nova | 15:08 | |
*** ssurana has quit IRC | 15:08 | |
*** amotoki_ has joined #openstack-nova | 15:08 | |
dansmith | phschwartz: what all have you changed in this since jerdfeldt's last version? Just rebasing? | 15:08 |
*** mdrabe has joined #openstack-nova | 15:09 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 15:09 |
*** iamjarvo has quit IRC | 15:09 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 15:09 |
openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 15:09 |
phschwartz | dansmith: Rebase + a few changes related to code review, things like changes of use of iteritems and compressing of a nested if statement that didn't need to be nested. | 15:09 |
dansmith | phschwartz: okay | 15:10 |
*** afazekas has quit IRC | 15:10 | |
dansmith | phschwartz: given the recent conversations and the fact that we know that contract is a little risky right now, I wonder if we should make the nova-manage contract command just return "not supported"? | 15:10 |
dansmith | since contract is completely optional | 15:11 |
danpb | dansmith: i was wondering what/where VersionedObjectSerializer is used ? | 15:11 |
dansmith | danpb: it's NovaObjectSerializer in our tree at the moment | 15:12 |
phschwartz | dansmith: I discussed that with johnthetubaguy to see if he felt if we should do just that, his thoughts were since it was experimental that allowing to still function would be fine. | 15:12 |
dansmith | danpb: it's plugged into our RPC layers so that objects get serialized automatically over the wire | 15:12 |
phschwartz | dansmith: I have no issues making it "not supported" at the moment though | 15:12 |
*** iamjarvo has joined #openstack-nova | 15:12 | |
*** armax has joined #openstack-nova | 15:12 | |
danpb | i was doing some hacking on oslo.messaging that would invole olso messaging depending on something that used vesioned objects | 15:12 |
dansmith | johnthetubaguy: what about a --no-really-I-mean-it flag? | 15:12 |
*** haypo has left #openstack-nova | 15:12 | |
*** iamjarvo has quit IRC | 15:12 | |
danpb | but because versioned objects has this class, i ended up with a circular dependancy between olso messaging & versionedobjects :-( | 15:12 |
dansmith | johnthetubaguy: just because I'm not sure we're exposing it as experimental enough before things could be destroyed | 15:13 |
*** Nikolay_St has quit IRC | 15:13 | |
*** iamjarvo has joined #openstack-nova | 15:13 | |
*** xyang2 has quit IRC | 15:13 | |
dansmith | danpb: yeah, not sure why messaging should know anything about objects.. it's a lower layer, right? | 15:13 |
phschwartz | dansmith: technically it is a "no op" unless someone removes a column from a modle. | 15:13 |
phschwartz | *model | 15:13 |
dansmith | danpb: the actual serializer interface is defined in oslo.messaging though | 15:13 |
dansmith | phschwartz: yep | 15:14 |
danpb | dansmith: well versioned objects aren't just used for things going over the message bus :-) | 15:14 |
*** xyang2 has joined #openstack-nova | 15:14 | |
danpb | dansmith: they're so useful i was trying to use them for instrumenting API calls and serializing them out to disk | 15:14 |
dansmith | danpb: sure, but... in oslo.messaging? | 15:14 |
danpb | and oslo messaging was one of the things i wanted to instrument | 15:14 |
danpb | i guess i'll have to think about how to restructure things | 15:15 |
danpb | so i can avoid creating a circular dep | 15:15 |
dansmith | danpb: well, I'd have to see an example I guess to help more | 15:15 |
dansmith | danpb: I'm both intrigued and terrified by what you're doing :D | 15:15 |
danpb | lol | 15:15 |
*** xyang2 has quit IRC | 15:16 | |
*** doron_ is now known as doron_afk | 15:17 | |
*** mtanino has joined #openstack-nova | 15:18 | |
dansmith | jaypipes: I think you should do the +W on this: https://review.openstack.org/#/c/154521/19 | 15:19 |
dansmith | (or the -1 of course) | 15:20 |
mriedem | dansmith: dims: what was the 1:1 clustered hypervisor change again? that was to enforce a single compute node/host for vmware/ironic right? | 15:22 |
*** doron_afk has quit IRC | 15:22 | |
*** xyang2 has joined #openstack-nova | 15:22 | |
mriedem | i think that might be required for this bulk stats thing to work for those driveres | 15:23 |
johnthetubaguy | dansmith: phschwartz: year. I think I requested an experimental flag at some point but forgot about that, --i-might-kill-your-cloud=True or whatever it is | 15:23 |
*** afazekas has joined #openstack-nova | 15:23 | |
phschwartz | johnthetubaguy: If you would like, I can add that now | 15:23 |
dansmith | johnthetubaguy: I think that'd be a real good idea, yeah | 15:24 |
dims | mriedem: to be able to expose the vcenter+multiple-esx hierarchy in nova API and CLI - so i can say deploy straight to esx-32 | 15:24 |
mriedem | dims: do you remember the patch that was going to enforce that at all? | 15:24 |
mriedem | i think it was abandoned in kilo | 15:24 |
dims | mriedem: never got started, so no code | 15:24 |
dansmith | phschwartz: throw a -1 on there so we know you're working on that and don't land it before you do | 15:25 |
johnthetubaguy | phschwartz: dansmith: +1 | 15:25 |
jaypipes | dansmith: done | 15:25 |
mriedem | dims: i remember there was some change that came out of the juno summit in ATL which was agreed to be a thing, basically deprecate or not allow multiple vcenters for a single compute service | 15:25 |
mriedem | something like that | 15:25 |
dansmith | doh | 15:25 |
dansmith | heh | 15:25 |
dansmith | phschwartz: well, maybe let that land and follow up with that flag? | 15:25 |
dims | mriedem: that's in already. one can only specify one vcenter cluster with one nova-compute process | 15:26 |
phschwartz | dansmith: yeah, he just beat me to it. I will wait for it to gate and then put up a follow up with the flag | 15:26 |
dims | earlier one could specify multpiple vcenter clusters using just one nova-compute | 15:26 |
dansmith | phschwartz: k | 15:26 |
mriedem | dims: is that this? https://review.openstack.org/#/c/163426/ | 15:27 |
*** afazekas has quit IRC | 15:28 | |
dims | mriedem: looking | 15:28 |
mriedem | i should have -1ed that :) we should have updated the options help text since that's what goes in the docs, and we should have had a DocImpact/UpgradeImpact so we updated the release notes | 15:28 |
mriedem | i can add that | 15:28 |
openstackgerrit | Jordan Pittier proposed openstack/nova: libvirt:when deleting snapshot use qemu-img to blockRebase if domain is stopped https://review.openstack.org/192736 | 15:29 |
dims | mriedem: yep | 15:29 |
*** oomichi has joined #openstack-nova | 15:29 | |
mriedem | oh it's in kilo, i can update the release notes for that yet | 15:30 |
mriedem | i'll still update the help text to say multiple is a no-no | 15:30 |
dims | right | 15:30 |
dims | garyk: we need to revive https://review.openstack.org/#/c/103916/ | 15:30 |
dims | mriedem: ^^ that's what we should get into liberty | 15:30 |
bauzas | alex_xu: still around by a very late time ? | 15:31 |
*** garyk has quit IRC | 15:31 | |
bauzas | or maybe an API subteam member ? | 15:31 |
bauzas | gilliard, sdague ? | 15:31 |
bauzas | my question is about https://review.openstack.org/#/c/179224/10/specs/liberty/approved/check-destination-on-migrations.rst,cm | 15:31 |
*** yamahata has joined #openstack-nova | 15:31 | |
*** Maike has quit IRC | 15:32 | |
bauzas | and what should be the behaviour if someone is specifying a flag like --force_destination but would not provide a destination host | 15:32 |
bauzas | (that's silly, but possible) | 15:32 |
bauzas | should we just drop the request by a 400 ? | 15:32 |
*** yamahata has quit IRC | 15:32 | |
bauzas | or ignore the flag ? | 15:32 |
bauzas | gilliard: sdague: alex_xu: ^ | 15:32 |
*** yamahata has joined #openstack-nova | 15:33 | |
johnthetubaguy | mriedem: good spot on that one, thanks, I guess it was missing the upgradeimpact tag :( | 15:33 |
* bauzas is considering calling 555-API-WG | 15:34 | |
bauzas | :p | 15:34 |
dansmith | too many numbers | 15:35 |
bauzas | huh, we have 10 digits here | 15:35 |
dansmith | well, the 555 in the US is an exchange, | 15:35 |
dansmith | which means only four digits after it | 15:35 |
edleafe | bauzas: yes, return a 400 | 15:35 |
bauzas | dansmith: I see :) | 15:36 |
edleafe | dansmith: unless it's an area code | 15:36 |
edleafe | dansmith: then it should be 555-API-WG55 | 15:36 |
* bauzas just remembers the Last Action Hero movie | 15:36 | |
dansmith | edleafe: well, the conventional example of 555 numbers are using it as an exchange | 15:36 |
edleafe | dansmith: yeah, but he's French, so... :) | 15:37 |
dansmith | edleafe: and 555 as an area code goes to actual things | 15:37 |
dansmith | edleafe: right, but I assumed 555 had no special significance there | 15:37 |
dansmith | assumed he was making a US-centric movie reference | 15:37 |
edleafe | dansmith: correct, of course | 15:37 |
dansmith | which he was | 15:37 |
bauzas | hum, 555 even has a wikipage https://en.wikipedia.org/wiki/555_%28telephone_number%29 | 15:38 |
bauzas | dansmith: yeah, like I said, 555 is well known for us :) | 15:38 |
dansmith | okay, that's sorted, what's next? :) | 15:38 |
*** jamielennox is now known as jamielennox|away | 15:38 | |
*** rajesht has quit IRC | 15:39 | |
*** jdurgin1 has quit IRC | 15:39 | |
bauzas | edleafe gave me his opinion, I take it :) | 15:39 |
bauzas | the magical number worked ! | 15:39 |
*** ssurana has joined #openstack-nova | 15:40 | |
edleafe | bauzas: there should be a magic API WG incantation you can chant | 15:40 |
dansmith | well, of course, because once you typed APIW, you were connected and the G was just superfluous | 15:40 |
*** vishwana_ has quit IRC | 15:40 | |
*** vishwanathj has joined #openstack-nova | 15:41 | |
*** lpetrut has quit IRC | 15:42 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova-specs: Proposes to check destination on migrations https://review.openstack.org/179224 | 15:44 |
sdague | bauzas: it should be an explicit 400 if the provide non sensible parameters | 15:44 |
mriedem | jasonamyers: ctrath: etherpad with notes on the thing https://etherpad.openstack.org/p/nova-hypervisor-bulk-stats-notify | 15:45 |
tdurakov | danpb: hi! | 15:45 |
bauzas | sdague: fair, ty | 15:45 |
*** ericksonsantos has joined #openstack-nova | 15:46 | |
tdurakov | danpb:about https://review.openstack.org/#/c/183057/3 VirtCPUModel uses VirtCPUFeature for feature list, do you think we should change that in api? | 15:46 |
*** jamielennox|away is now known as jamielennox | 15:47 | |
*** unicell1 has joined #openstack-nova | 15:47 | |
*** hemnafk is now known as hemna | 15:47 | |
*** unicell has quit IRC | 15:50 | |
*** Jeffrey4l has quit IRC | 15:51 | |
*** kiran-r has joined #openstack-nova | 15:52 | |
*** jecarey has quit IRC | 15:53 | |
*** exploreshaifali has quit IRC | 15:53 | |
*** tjones1 has quit IRC | 15:54 | |
*** tjones1 has joined #openstack-nova | 15:56 | |
*** kiran-r has quit IRC | 15:56 | |
*** cfriesen has quit IRC | 15:56 | |
*** kiran-r has joined #openstack-nova | 15:57 | |
*** jamielennox is now known as jamielennox|away | 15:58 | |
ctrath | mriedem: thanks | 15:58 |
*** tsekiyama has joined #openstack-nova | 15:58 | |
*** Sukhdev has joined #openstack-nova | 15:58 | |
*** tojuvone has joined #openstack-nova | 15:59 | |
beagles | Jeffrey4l, was there a specific thing you were looking for an assist on? | 15:59 |
*** krtaylor has quit IRC | 15:59 | |
*** haypo has joined #openstack-nova | 16:00 | |
*** SourabhP has joined #openstack-nova | 16:00 | |
*** iamjarvo has quit IRC | 16:00 | |
*** ypraveen has joined #openstack-nova | 16:01 | |
mriedem | btw, ctrath is new to nova and will be working on it, so feel free to dump everything on him | 16:01 |
*** dulek_home has joined #openstack-nova | 16:02 | |
*** tonytan4ever has quit IRC | 16:03 | |
*** kiran-r has quit IRC | 16:03 | |
*** kiran-r has joined #openstack-nova | 16:03 | |
*** exploreshaifali has joined #openstack-nova | 16:03 | |
ctrath | mriedem: thanks again. :-* | 16:04 |
*** jlanoux has quit IRC | 16:04 | |
*** rotbeard has quit IRC | 16:06 | |
*** jamielennox|away is now known as jamielennox | 16:06 | |
*** arnaud___ has joined #openstack-nova | 16:06 | |
*** vilobhmm has joined #openstack-nova | 16:06 | |
*** tpeoples has joined #openstack-nova | 16:08 | |
*** Jeffrey4l has joined #openstack-nova | 16:08 | |
openstackgerrit | Timofey Durakov proposed openstack/nova-specs: nova-api hypervisor.cpu_info change from string to json object https://review.openstack.org/183057 | 16:08 |
*** jcoufal has quit IRC | 16:10 | |
*** kiran-r has quit IRC | 16:10 | |
*** dims_ has joined #openstack-nova | 16:11 | |
*** krtaylor has joined #openstack-nova | 16:11 | |
*** garyk has joined #openstack-nova | 16:11 | |
*** achanda has joined #openstack-nova | 16:12 | |
*** electroc_ has joined #openstack-nova | 16:13 | |
*** dims has quit IRC | 16:13 | |
*** jecarey has joined #openstack-nova | 16:13 | |
*** unicell1 has quit IRC | 16:14 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: vmware: update help text for vmware.cluster_name option https://review.openstack.org/192753 | 16:15 |
*** nic has joined #openstack-nova | 16:16 | |
*** electrocucaracha has quit IRC | 16:16 | |
*** gszasz has quit IRC | 16:17 | |
openstackgerrit | Chris Gacsal proposed openstack/nova-specs: Add REST API impact to the multi-attach spec https://review.openstack.org/192755 | 16:18 |
*** rook has joined #openstack-nova | 16:18 | |
*** ptm is now known as PaulMurray | 16:19 | |
*** Mike_D_laptop has joined #openstack-nova | 16:19 | |
*** rook is now known as rook-spec | 16:19 | |
PaulMurray | ndipanov, | 16:21 |
*** achanda_ has joined #openstack-nova | 16:21 | |
*** gabor-t has joined #openstack-nova | 16:21 | |
PaulMurray | ndipanov, oops sorry - autocomplete failure | 16:21 |
ndipanov | PaulMurray, no worries | 16:21 |
*** neophy has joined #openstack-nova | 16:22 | |
*** agireud has quit IRC | 16:23 | |
lxsli | dansmith: before I go any further, is this the sort of thing you were expecting please? re: no-more-soft-delete | 16:23 |
lxsli | dansmith: http://paste.openstack.org/show/298813/ | 16:23 |
*** achanda has quit IRC | 16:24 | |
bauzas | ndipanov: around for discussing why I made https://review.openstack.org/#/c/188487/ this way ? | 16:24 |
dansmith | lxsli: that looks awesome.. it *might* be too much given how many instances we have.. I wonder if you could cut out some of the middle layers.. you'd just have to use judgment on which ones make sense | 16:24 |
dansmith | lxsli: maybe start with the fully verbose layout, | 16:25 |
*** sdake_ is now known as sdake | 16:25 | |
dansmith | and we can trim it down in the review too, if you'd prefer | 16:25 |
ndipanov | bauzas, ok | 16:25 |
lxsli | dansmith: OK I'll slog on! | 16:25 |
bauzas | ndipanov: okay, lemme try to be clear | 16:25 |
bauzas | ndipanov: so, I'm okay with creating a new select_nodes(RequestSpec) method | 16:26 |
dansmith | lxsli: let me edit that paste as an example | 16:26 |
bauzas | ndipanov: but this means I'll have to change all the southbound method signatures, right? | 16:26 |
ndipanov | what's southbound? | 16:26 |
*** dsanders has joined #openstack-nova | 16:26 | |
lxsli | dansmith: removing middle layers is easy, it's doing the trace that takes the time | 16:26 |
bauzas | ndipanov: the methods called by the API server methods | 16:27 |
*** markus_z has quit IRC | 16:27 | |
bauzas | ndipanov: so for example, given select_nodes(RequestSpec) | 16:27 |
dansmith | lxsli: http://paste.openstack.org/show/298814/ | 16:27 |
bauzas | ndipanov: it will call FilterScheduler.select_nodes(Spec) | 16:27 |
dansmith | lxsli: you can also probably stop at some layer in the trace once you know the semantics of what is going on | 16:27 |
lxsli | dansmith: ah! that's very helpful! | 16:27 |
bauzas | ndipanov: but I'll also have to create it select_nodes() for the ChanceScheduler | 16:28 |
*** afazekas has joined #openstack-nova | 16:28 | |
bauzas | ndipanov: then, I'll also have to modify the signature of nova.filters | 16:28 |
lxsli | dansmith: great, that often becomes obvious relatively early, but chasing through the layers of API code gets tedious | 16:28 |
bauzas | (or create a new method) | 16:28 |
ndipanov | yes - you add a completely new RPC call and method | 16:28 |
*** vilobhmm has quit IRC | 16:28 | |
dansmith | lxsli: that parenthetical comment is exactly what I think we need to evaluate in each case... knowing what it is doing, why, and if we *really* need read_deleted, or if we just do it to avoid a trace | 16:28 |
ndipanov | bauzas, yes - nova filters need to be changed in a single patch | 16:28 |
dansmith | lxsli: yep | 16:28 |
bauzas | ndipanov: then I'll have to create host_passes2(RequestSpec) | 16:29 |
bauzas | all of that will need to be done in a single patch, right? | 16:29 |
ndipanov | why? | 16:29 |
ndipanov | so select_destination_objectified() works with objects | 16:29 |
bauzas | ndipanov: I see, you're asking to duplicate all the calls | 16:29 |
openstackgerrit | Sudipta Biswas proposed openstack/nova: The 'scheduled_at' field is a part of the Nova instances. Currently, the 'scheduled_at' field is no longer updated by the Nova scheduler, instead the 'launched_at' field is updated for the same purpose. The 'scheduled_at' column is now redundant and shoul https://review.openstack.org/192760 | 16:29 |
*** lpetrut has joined #openstack-nova | 16:30 | |
ndipanov | select_destinations() gets dicts, forward ports them, and works with objects | 16:30 |
ndipanov | both of these only call filters which are converted in a single patch | 16:30 |
ndipanov | (but anyway they would have to be converted in a single patch or have some kind of a silly decorator that does isinstance blah | 16:30 |
bauzas | ndipanov: I'll have to change all the filters at once, right? | 16:30 |
*** achanda_ has quit IRC | 16:31 | |
ndipanov | yeah but it's like renaming stuff | 16:31 |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 16:31 |
ndipanov | it's not like there's million of t hings to change | 16:32 |
ndipanov | it's just filters | 16:32 |
ndipanov | (and tests) | 16:32 |
ndipanov | 250 line patch at most | 16:32 |
*** matrohon has quit IRC | 16:32 | |
ndipanov | that's stupid easy to review | 16:32 |
ndipanov | and the win is that there is no backporting to old format which is hell | 16:32 |
ndipanov | trust me | 16:32 |
ndipanov | downside is taht you get to write less patches and your stackalytics suffer from that :P | 16:33 |
alex_xu | bauzas: just +1, I think 400 is ok for me | 16:33 |
*** openstackgerrit has quit IRC | 16:33 | |
*** agireud has joined #openstack-nova | 16:34 | |
*** openstackgerrit has joined #openstack-nova | 16:34 | |
*** jamielennox is now known as jamielennox|away | 16:35 | |
*** Marga_ has quit IRC | 16:35 | |
*** oomichi has quit IRC | 16:36 | |
ndipanov | bauzas, fwiw - you could do all that without adding a new method | 16:36 |
ndipanov | but I see no reason not to | 16:36 |
*** jog0 has quit IRC | 16:37 | |
*** krtaylor has quit IRC | 16:37 | |
ndipanov | you would ghetto-ise all the cruft in hte old method and have a new shiny method to work with! | 16:37 |
*** agireud has quit IRC | 16:37 | |
*** sudipto has quit IRC | 16:38 | |
*** arnaud___ has quit IRC | 16:38 | |
dansmith | +1 for ghetto-ization | 16:38 |
*** artom has joined #openstack-nova | 16:38 | |
*** neophy has quit IRC | 16:38 | |
*** neophy has joined #openstack-nova | 16:39 | |
*** ypraveen has quit IRC | 16:39 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova-specs: Add REST API impact to the multi-attach spec https://review.openstack.org/192755 | 16:41 |
*** kylek3h_ has joined #openstack-nova | 16:43 | |
*** thangp_ has joined #openstack-nova | 16:43 | |
*** apuimedo_ has joined #openstack-nova | 16:43 | |
*** med_` has joined #openstack-nova | 16:43 | |
*** oro_ has joined #openstack-nova | 16:43 | |
*** gyee_ has joined #openstack-nova | 16:43 | |
*** cdent_ has joined #openstack-nova | 16:43 | |
*** pixelbeat_ has joined #openstack-nova | 16:43 | |
*** vladikr_ has joined #openstack-nova | 16:43 | |
*** changbl_ has joined #openstack-nova | 16:43 | |
*** fifieldt_ has joined #openstack-nova | 16:44 | |
*** wverdugo5001 has joined #openstack-nova | 16:44 | |
*** mtanino has quit IRC | 16:45 | |
*** kylek3h has quit IRC | 16:45 | |
*** vladikr has quit IRC | 16:45 | |
*** oro has quit IRC | 16:45 | |
*** thangp has quit IRC | 16:45 | |
*** cdent has quit IRC | 16:45 | |
*** pixelbeat has quit IRC | 16:45 | |
*** apuimedo has quit IRC | 16:45 | |
*** fifieldt has quit IRC | 16:45 | |
*** josecastroleon has quit IRC | 16:45 | |
*** changbl has quit IRC | 16:45 | |
*** operator99 has quit IRC | 16:45 | |
*** med_ has quit IRC | 16:45 | |
*** tcammann has quit IRC | 16:45 | |
*** cdent_ is now known as cdent | 16:45 | |
*** kylek3h_ is now known as kylek3h | 16:45 | |
*** josecastroleon1 has joined #openstack-nova | 16:45 | |
*** nic has quit IRC | 16:45 | |
*** nic has joined #openstack-nova | 16:46 | |
*** a| has joined #openstack-nova | 16:46 | |
*** a| has quit IRC | 16:46 | |
bauzas | ndipanov: sorry, was afk for a bit | 16:47 |
bauzas | ndipanov: had to fix something | 16:47 |
*** penick has joined #openstack-nova | 16:47 | |
*** mfisch has quit IRC | 16:47 | |
*** logan2 has joined #openstack-nova | 16:47 | |
bauzas | ndipanov: so, you're saying, just changing the filters by hydrating the object there, then doing backwards ? | 16:48 |
*** tcammann has joined #openstack-nova | 16:48 | |
*** sambol has quit IRC | 16:48 | |
*** krtaylor has joined #openstack-nova | 16:48 | |
*** mtanino has joined #openstack-nova | 16:49 | |
*** jamielennox|away is now known as jamielennox | 16:49 | |
*** mfisch has joined #openstack-nova | 16:50 | |
*** mfisch has quit IRC | 16:50 | |
*** mfisch has joined #openstack-nova | 16:50 | |
*** kiran-r has joined #openstack-nova | 16:50 | |
ndipanov | no | 16:51 |
*** unicell has joined #openstack-nova | 16:51 | |
*** Marga_ has joined #openstack-nova | 16:51 | |
*** alexpilotti has joined #openstack-nova | 16:51 | |
bauzas | ndipanov: sorry for being dumb, but I don't get your point :/ | 16:52 |
*** romainh has left #openstack-nova | 16:52 | |
*** exploreshaifali has quit IRC | 16:52 | |
ndipanov | you have one patch that moves the whole scheduler to use the new Objects | 16:52 |
bauzas | ndipanov: I'm not that sure it would be a +200/-200 one then | 16:52 |
ndipanov | more or less | 16:52 |
*** prashup has quit IRC | 16:52 | |
bauzas | ndipanov: just because of some cruftness in the HostManager for example | 16:52 |
ndipanov | and then old method when it gets the dicts - forward ports them (that already exists) | 16:53 |
*** sudipto has joined #openstack-nova | 16:53 | |
bauzas | ndipanov: okay, I can give a try | 16:54 |
*** yamahata has quit IRC | 16:54 | |
ndipanov | it literally does 3 lookups on filter properties | 16:54 |
bauzas | ndipanov: but my main worries is the size of the change for using the ReqSpec dot notation as a whole | 16:54 |
ndipanov | nothing else touches it | 16:54 |
ndipanov | meh | 16:54 |
mriedem | bauzas: do dot notation changes in a separate patch | 16:54 |
mriedem | bulk dot notation changes that is | 16:55 |
*** yamahata has joined #openstack-nova | 16:55 | |
ndipanov | that's better than having a ton of useles commits plus a bunch of code that is useless (backporting) | 16:55 |
mriedem | convert the unit tests to use objects rather than dicts and change to dot notation in the code at the same time to enforce that it's working | 16:55 |
ndipanov | mriedem, it can't be done like that | 16:55 |
ndipanov | it has to be one shot with the RPC change | 16:55 |
ndipanov | to avoid backporting to dicts | 16:55 |
ndipanov | which has been proven horrible in a number of cases | 16:55 |
mriedem | sorry, came in late to a thing i'm not following | 16:56 |
bauzas | mriedem: that was my initial series where ndipanov is -1 it | 16:56 |
ndipanov | and we actually don't need it here - it';s nowhere near as bit as for example instance in the compute manager | 16:56 |
ndipanov | it';s just filters! | 16:56 |
ndipanov | literally | 16:56 |
bauzas | ndipanov: and HostManager.consume_from_instance() again | 16:56 |
*** lucasagomes is now known as lucas-dinner | 16:56 | |
jaypipes | johnthetubaguy, ndipanov, danpb: https://review.openstack.org/#/c/183057/ could use a re-review from you. thx in advance. I know tdurakov is eager to get started on that work. | 16:57 |
bauzas | but I can give a test and see | 16:57 |
mriedem | ndipanov: literally? https://twitter.com/awfulannouncing/status/526754351264829440 | 16:57 |
*** belmoreira has joined #openstack-nova | 16:57 | |
ndipanov | :) | 16:57 |
*** achanda has joined #openstack-nova | 16:57 | |
*** abhishekk has quit IRC | 16:58 | |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 16:58 |
bauzas | ndipanov: anyway, I got your views, lemme see how I can achieve it | 16:58 |
*** derekh has quit IRC | 16:58 | |
*** vilobhmm has joined #openstack-nova | 16:58 | |
bauzas | alaski: cells meeting in 2mins right? | 16:58 |
alaski | correct | 16:59 |
bauzas | cool | 16:59 |
*** kiran-r has quit IRC | 16:59 | |
*** kiran-r has joined #openstack-nova | 16:59 | |
*** haypo has quit IRC | 16:59 | |
ndipanov | bauzas, one more thing you'll need to do | 16:59 |
*** dulek_home has quit IRC | 17:00 | |
bauzas | ndipanov: sure, any feedback is good to hear | 17:00 |
ndipanov | move all the junk from scheduler.utils to the object itself | 17:00 |
ndipanov | but we should all be super happy about that really | 17:00 |
bauzas | ndipanov: yeah, that's the target, again I just wanted some small patches | 17:00 |
ndipanov | yeah taht can be a separate patch of course | 17:00 |
bauzas | ndipanov: my stackalytics stats are good, ty :p | 17:01 |
bauzas | ndipanov: even if my review ratio is decreasing dramatically because of that stuff :p | 17:01 |
ndipanov | the fact that you even look at that makes me think less of you :P | 17:01 |
*** otter768 has joined #openstack-nova | 17:01 | |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 17:01 |
*** atuvenie has quit IRC | 17:01 | |
bauzas | ndipanov: meh, don't you do that every 1 hour ? | 17:01 |
bauzas | :p | 17:01 |
*** Marga_ has quit IRC | 17:01 | |
*** chaochin has joined #openstack-nova | 17:02 | |
*** aswadr has quit IRC | 17:02 | |
*** asrangne has joined #openstack-nova | 17:02 | |
*** achanda has quit IRC | 17:02 | |
*** dulek_home has joined #openstack-nova | 17:02 | |
*** dulek_home has quit IRC | 17:03 | |
*** exploreshaifali has joined #openstack-nova | 17:03 | |
*** haomaiw__ has joined #openstack-nova | 17:04 | |
*** rushiagr is now known as rushiagr_away | 17:04 | |
*** haomaiwang has quit IRC | 17:04 | |
*** asrangne__ has joined #openstack-nova | 17:05 | |
*** achanda has joined #openstack-nova | 17:05 | |
*** otter768 has quit IRC | 17:06 | |
*** alex_klimov has quit IRC | 17:07 | |
*** Marga_ has joined #openstack-nova | 17:08 | |
*** yamahata has quit IRC | 17:08 | |
ndipanov | ok bauzas need to run - but if you don't trust me ask dansmith... if we can get away with not doing the new -> legacy conversion and only always forwardport - it saves us a ton of headache | 17:08 |
bauzas | ndipanov: yeah, I got the rationale | 17:08 |
bauzas | ndipanov: which I appreciate | 17:08 |
*** asrangne has quit IRC | 17:08 | |
bauzas | ndipanov: I just need to see the level of complexity that it implies, but maybe I ubercared too much | 17:09 |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup https://review.openstack.org/192760 | 17:09 |
bauzas | overthinking is like a second nature for me :( | 17:09 |
ndipanov | I think the scheduler is the perfect place to do it | 17:09 |
*** moshele has quit IRC | 17:09 | |
ndipanov | we trade a beefier (but easy) patch for less and way simpler code | 17:09 |
ndipanov | obviously cannot be done for (for example) compute manager | 17:10 |
*** Marga_ has quit IRC | 17:11 | |
ndipanov | what might trip you up is that filter props get sent to the compute host... :( | 17:11 |
bauzas | ndipanov: I know | 17:11 |
*** achanda has quit IRC | 17:11 | |
bauzas | ... | 17:11 |
ndipanov | shite | 17:12 |
ndipanov | ah well | 17:12 |
ndipanov | I'm off - talk tomorrow :) | 17:12 |
bauzas | sure thign | 17:12 |
bauzas | for that, I was just thinking to hydate the Spec but leave the existing filt_props passed to the compute manager | 17:12 |
*** ndipanov has quit IRC | 17:12 | |
*** jamielennox is now known as jamielennox|away | 17:13 | |
bauzas | ie. we hydrate the object right before calling the scheduler API, but we leave the existing dicts, until someone enough foolish is doing that for the compute side | 17:13 |
openstackgerrit | Josh Durgin proposed openstack/nova-specs: RBD instance snapshots https://review.openstack.org/188244 | 17:13 |
bauzas | crap, nikola left :p | 17:13 |
openstackgerrit | Qin Zhao proposed openstack/nova: Make resize api of compute manager to send flavor object https://review.openstack.org/161860 | 17:14 |
openstackgerrit | Qin Zhao proposed openstack/nova: Support flavor object in migrate_disk_and_power_off https://review.openstack.org/192786 | 17:14 |
*** dims_ has quit IRC | 17:16 | |
*** ypraveen has joined #openstack-nova | 17:16 | |
*** dims has joined #openstack-nova | 17:17 | |
*** SourabhP has quit IRC | 17:18 | |
*** achanda has joined #openstack-nova | 17:18 | |
*** cfriesen has joined #openstack-nova | 17:19 | |
*** atuvenie has joined #openstack-nova | 17:21 | |
*** achanda_ has joined #openstack-nova | 17:22 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 17:25 |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 17:25 |
openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 17:25 |
*** achanda has quit IRC | 17:25 | |
*** ddutta has quit IRC | 17:25 | |
*** ildikov has quit IRC | 17:30 | |
*** sdake_ has joined #openstack-nova | 17:30 | |
*** ctrath has quit IRC | 17:31 | |
*** varunarya has quit IRC | 17:31 | |
*** VW has quit IRC | 17:32 | |
openstackgerrit | Ed Leafe proposed openstack/nova: Add logging when filtering returns nothing. https://review.openstack.org/192184 | 17:32 |
*** achanda_ has quit IRC | 17:33 | |
*** sdake has quit IRC | 17:34 | |
*** ctrath has joined #openstack-nova | 17:35 | |
*** rajesht has joined #openstack-nova | 17:35 | |
*** belmoreira has quit IRC | 17:38 | |
vilobhmm | jaypipes : ping | 17:38 |
*** yassine has quit IRC | 17:39 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova-specs: Allow list all instances for non-admin based on policy https://review.openstack.org/191151 | 17:39 |
*** rajesht has quit IRC | 17:39 | |
dims | alex_xu: johnthetubaguy: thanks for the review, i added exactly what the current is and new behavior will be | 17:40 |
openstackgerrit | Dan Smith proposed openstack/nova: Remove compat_instance() https://review.openstack.org/192708 | 17:40 |
openstackgerrit | Dan Smith proposed openstack/nova: Fix testing object fields with missing instance rows https://review.openstack.org/192805 | 17:40 |
*** afazekas has quit IRC | 17:41 | |
*** aix has quit IRC | 17:42 | |
*** danpb has quit IRC | 17:42 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: Switch to using os-brick https://review.openstack.org/175569 | 17:43 |
*** amotoki_ has quit IRC | 17:43 | |
mriedem | nova version 12 and keystone version 8 is going to be very confusing for awhile | 17:44 |
mgagne | I think an opinion on https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1248 | 17:44 |
mriedem | cinder version 7 | 17:44 |
mgagne | If the revert fails, the user won't get an InterfaceAttachFailed exception, is it the desired behavior? | 17:45 |
mgagne | AFAIK, he will get a 500 instead | 17:45 |
mriedem | mgagne: what will they get? 500 for ProcessExecutionError? | 17:45 |
mriedem | mgagne: put that in an excutils.save_and_reraise_exception context | 17:45 |
mriedem | for the unplug | 17:45 |
mriedem | actually no, since that would raise the libvirterror | 17:46 |
mriedem | just a try/except should be good | 17:46 |
mgagne | yes | 17:46 |
openstackgerrit | Qin Zhao proposed openstack/nova: Support flavor object in migrate_disk_and_power_off https://review.openstack.org/192786 | 17:46 |
vilobhmm | dansmith : regarding https://review.openstack.org/#/c/190322/ need your insights on what will be a better approach for us going ahead….irc://irc.freenode.net:6667/#1. should the services information and the service liveliness information be stored in same datastore and let the common API fetch/update data from same source of truth OR irc://irc.freenode.net:6667/#2. keep the service information always in DB and service liveliness information in | 17:46 |
mriedem | mgagne: remember to report a bug in case you want to backport that fix | 17:47 |
mgagne | mriedem: should I log the original exception and just fallback to a common InterfaceAttachFailed? | 17:48 |
*** Longgeek has quit IRC | 17:48 | |
mgagne | log (and mute) | 17:48 |
*** tdurakov_ has joined #openstack-nova | 17:50 | |
*** Marga__ has joined #openstack-nova | 17:50 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Make the guest console configurable https://review.openstack.org/122523 | 17:51 |
*** kiranr has joined #openstack-nova | 17:51 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: imagebackend: Split out resize_image logic from create_image https://review.openstack.org/187857 | 17:51 |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Use utf8_bin collation on the flavor extra-specs table in MySQL https://review.openstack.org/190300 | 17:51 |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 17:51 |
mgagne | mriedem: it looks like it's already handled in the manager | 17:52 |
mgagne | mriedem: https://git.openstack.org/cgit/openstack/nova/commit/?id=78f7c744f675d618048968850259422338dd4955 | 17:52 |
*** gabor-t has quit IRC | 17:53 | |
*** achanda has joined #openstack-nova | 17:54 | |
*** ijw_ has quit IRC | 17:54 | |
*** electroc_ has quit IRC | 17:55 | |
*** kiran-r has quit IRC | 17:55 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova-specs: Allow list all instances for non-admin based on policy https://review.openstack.org/191151 | 17:55 |
mriedem | mgagne: oh yeah that was added relatively recently | 17:55 |
mgagne | and it's part of kilo so I'm happy | 17:56 |
mriedem | if you're happy then i'm happy | 17:56 |
*** tonytan4ever has joined #openstack-nova | 17:56 | |
*** sambol has joined #openstack-nova | 17:56 | |
*** electrocucaracha has joined #openstack-nova | 17:57 | |
*** kiranr has quit IRC | 17:58 | |
*** sdake has joined #openstack-nova | 17:58 | |
*** Sukhdev has quit IRC | 17:58 | |
*** Nikolay_St has joined #openstack-nova | 17:59 | |
*** SourabhP has joined #openstack-nova | 18:00 | |
*** browne has quit IRC | 18:01 | |
*** Sukhdev has joined #openstack-nova | 18:01 | |
*** sdake_ has quit IRC | 18:01 | |
dims | so we begin the debate again - bug or feature - whether we need micro version :( https://review.openstack.org/#/c/191151/ | 18:02 |
dansmith | vilobhmm: okay | 18:02 |
*** Marga__ has quit IRC | 18:03 | |
*** browne has joined #openstack-nova | 18:03 | |
vilobhmm | dansmith : thanks :) | 18:03 |
*** ociuhandu has quit IRC | 18:06 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova-specs: Allow list all instances for non-admin based on policy https://review.openstack.org/191151 | 18:06 |
*** annashen has joined #openstack-nova | 18:07 | |
openstackgerrit | Ankit Agrawal proposed openstack/nova: Test patch race condition in delete (do not merge) https://review.openstack.org/191732 | 18:08 |
*** VW has joined #openstack-nova | 18:10 | |
mriedem | dims: i don't see that anyone asked for a microveresion | 18:10 |
mriedem | dims: seems like a bug fix | 18:10 |
*** sdake_ has joined #openstack-nova | 18:10 | |
*** IanGovett has joined #openstack-nova | 18:11 | |
mriedem | in general the code should work based on what's configured in the policy | 18:11 |
*** vladikr_ has quit IRC | 18:11 | |
*** isd has joined #openstack-nova | 18:11 | |
mriedem | dims: i'd probably ask alex_xu and sdague | 18:12 |
dims | mriedem: didn't we say we need microversion in the ipv6 bug? | 18:12 |
*** vladikr_ has joined #openstack-nova | 18:12 | |
mriedem | yeah... | 18:12 |
mriedem | so that's probably precedent i guess | 18:12 |
dims | right | 18:12 |
*** annashen has quit IRC | 18:13 | |
mriedem | i look forward to the day when i have a patchy gray beard and there is nova v2.439 | 18:13 |
dims | lol | 18:13 |
sdague | so a microversion would mean we would keep the old behavior regardless of policy | 18:13 |
*** sdake has quit IRC | 18:14 | |
sdague | this seems weird in that regard | 18:14 |
*** VW has quit IRC | 18:14 | |
mriedem | yeah, it seems as long as the policy is being enforced in the code we should be honoring it | 18:15 |
mriedem | but i'm getting tripped up with the old @require_admin_context stuff that we kept broken in v2 api | 18:15 |
*** annashen has joined #openstack-nova | 18:15 | |
mriedem | and we didn't have microversions for those changes in the v2.1 api | 18:15 |
mriedem | unless that's just, v2.1 itself | 18:16 |
*** agireud has joined #openstack-nova | 18:16 | |
mgagne | weighters or weights? :D | 18:17 |
*** ijw_ has joined #openstack-nova | 18:17 | |
*** achanda has quit IRC | 18:17 | |
mgagne | I see both in the code | 18:18 |
*** ijw_ has quit IRC | 18:18 | |
edmondsw | dims, I really think that should get fixed for all versions. | 18:18 |
*** ijw_ has joined #openstack-nova | 18:19 | |
edmondsw | not clear why anyone would need to be able to discover that they are talking to a fixed nova instance | 18:19 |
dims | edmondsw: happy to do the work. need consensus from the cores | 18:19 |
edmondsw | and if there is a good reason, then there should be some other way... | 18:19 |
edmondsw | sure, I get that... not a one man show here :) | 18:20 |
dims | edmondsw: one sec let me grab you the other review where the precedent was set | 18:20 |
dims | edmondsw: https://review.openstack.org/#/c/187835/ | 18:20 |
edmondsw | related question... we're not going to spin off multiple microversions per release, are well? Is there not already a new microversion for Liberty? | 18:21 |
*** sambol has quit IRC | 18:21 | |
edmondsw | s/well/we/ | 18:21 |
mriedem | edmondsw: yes there will be multiple microversions per release | 18:21 |
edmondsw | what's the point of that? | 18:21 |
mriedem | there were 3 in kilo, and already at least one new one in liberty | 18:21 |
*** cdent has quit IRC | 18:21 | |
mriedem | http://docs.openstack.org/developer/nova/api_microversion_history.html | 18:22 |
edmondsw | so kilo shipped with 2.1, 2.2, and 2.3? | 18:23 |
sdague | edmondsw: people deploy nova from source all the time | 18:23 |
sdague | from master | 18:23 |
sdague | they don't wait for releases | 18:23 |
edmondsw | sdague, ugh!!! | 18:23 |
sdague | edmondsw: not ugh, that's been part of the model since day one | 18:23 |
edmondsw | then they should pay the price in losing version discovery | 18:23 |
sdague | edmondsw: no, not at all, continuous delivery is a core value in Nova | 18:24 |
sdague | and most of the openstack projects | 18:24 |
*** markvoelker has quit IRC | 18:25 | |
*** markvoelker has joined #openstack-nova | 18:25 | |
mriedem | i'm still not sure how this particular change relates to microversions since if the admin doesn't change the policy it's still not going to work for non-admins | 18:27 |
*** isq_ has quit IRC | 18:27 | |
mriedem | so you don't know if it works (discoverable) because the admin has not configured it or because the nova you're talking to doesn't have that microversion | 18:27 |
*** VW has joined #openstack-nova | 18:27 | |
*** baoli has quit IRC | 18:28 | |
*** baoli has joined #openstack-nova | 18:28 | |
*** Marga_ has joined #openstack-nova | 18:29 | |
dansmith | edmondsw: no really, CDers should be the primary, and package-based people can deal with more funkiness :) | 18:30 |
sdague | mriedem: yeh, the fact that it's non default policy makes me feel like we probably don't need a bump for it | 18:31 |
edmondsw | sdague, ok... that's interesting... I'm all for continuous delivery, but I didn't think we were claiming that | 18:31 |
sdague | edmondsw: yes, since forever | 18:31 |
sdague | public clouds like rax are CD | 18:31 |
edmondsw | so why the convos about releasing more frequently? | 18:31 |
mriedem | edmondsw: rax used to be like every 2 weeks or something | 18:31 |
mriedem | 2 weeks from trunk | 18:31 |
mriedem | now that's like 4 drops a year or something right? | 18:32 |
mriedem | as was HP cloud | 18:32 |
*** VW has quit IRC | 18:32 | |
* mriedem reboots | 18:32 | |
*** mriedem has quit IRC | 18:32 | |
edmondsw | I agree with what mriedem said about about microversions not applying in this case because of the need for a policy change as well | 18:33 |
*** VW has joined #openstack-nova | 18:33 | |
sdague | if policy was discoverable, we'd need that, but right now that's all black box and not, so it's a shortcoming of our discovery | 18:34 |
*** penick has quit IRC | 18:34 | |
*** agireud has quit IRC | 18:35 | |
*** baoli has quit IRC | 18:35 | |
melwitt | I think the only possible concern would be if someone has left the policy as default, which is allow all_tenants for everyone, and doesn't know it. current behavior only admin can do all_tenants, then if it changes, suddenly everyone can do it | 18:36 |
*** penick has joined #openstack-nova | 18:37 | |
openstackgerrit | Dan Smith proposed openstack/nova: Remove compat_instance() https://review.openstack.org/192708 | 18:37 |
edmondsw | melwitt, that would be a deployer's mistake... I wouldn't expect our code to have to account for that | 18:37 |
*** mingy has joined #openstack-nova | 18:37 | |
sdague | melwitt: the default policy blocks this as well though, right? | 18:40 |
*** Marga_ has quit IRC | 18:40 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Don't fetch kernel/ramdisk files if you already have them. https://review.openstack.org/163661 | 18:41 |
melwitt | edmondsw: yeah. I admit I get a little confused about when it's okay to fix wrong behavior without a microversion | 18:41 |
*** afazekas has joined #openstack-nova | 18:41 | |
*** ZZelle_ has joined #openstack-nova | 18:41 | |
melwitt | sdague: I guess I'm not sure what default policy means. our policy.json in tree has "compute:get_all_tenants": "", which means all can do | 18:42 |
edmondsw | default policy has all kinds of security issues... hopefully nobody who cares about security actually uses those | 18:43 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Drop scheduler RPC 3.x support https://review.openstack.org/192836 | 18:43 |
melwitt | edmondsw: fair point | 18:44 |
edmondsw | sadly... | 18:44 |
sdague | melwitt: so... that's a good point. dims can you explain the change a bit more? I'm confused now. | 18:45 |
sdague | edmondsw: lots of people use default policy, if there are issues we should tighten up, we should do that. | 18:46 |
sdague | morganfainberg told me they had an ops session on policy, and in the room the number of folks that had successfully modified their policy without breaking the world was a small handful. People assume we're sending them sane defaults. | 18:47 |
edmondsw | yes, we should clean that up | 18:47 |
*** markvoelker_ has joined #openstack-nova | 18:47 | |
*** markvoelker has quit IRC | 18:47 | |
edmondsw | I'm trying to find time to work on that | 18:47 |
morganfainberg | sdague: 2 or 3 people raised their hands. | 18:48 |
morganfainberg | Not even a handful. | 18:48 |
dims | sdague: see old and new behavior lines 68-77 - https://review.openstack.org/#/c/191151/6/specs/liberty/approved/allow-list-all-instances-for-non-admin.rst,cm | 18:48 |
edmondsw | I'd have raised mine if I'd been there | 18:48 |
sdague | ok, less than a handful. And one of them was Nate from the NSA. | 18:48 |
morganfainberg | Yep. | 18:48 |
*** nihilifer has quit IRC | 18:48 | |
*** Longgeek has joined #openstack-nova | 18:48 | |
dansmith | sdague: important qualifier :D | 18:48 |
morganfainberg | And just 403ing a single api call isn't what we were asking. But I'm sure the hands would have been the same. | 18:49 |
morganfainberg | Even just looking at blocking a single api call. | 18:49 |
edmondsw | that is extremely sad | 18:50 |
*** mtecer has joined #openstack-nova | 18:50 | |
dims | sdague: i'll be happy not to do a microversion for this :) | 18:50 |
morganfainberg | It's a sign we need to make default policy a rich / usable set of definitions. | 18:50 |
morganfainberg | And not assume everyone will customize it. | 18:51 |
*** ildikov has joined #openstack-nova | 18:51 | |
edmondsw | morganfainberg, the starting point should have been that policy doesn't allow much, and you have to open things up if you need them | 18:52 |
samueldmq | morganfainberg, ++ with different meanings to different admins (cloud, domain and project) :) | 18:52 |
edmondsw | instead we've left so much open, that people would need to customize to close things off | 18:52 |
edmondsw | but in either event, people will need to customize.. we can't have defaults that cover every environment | 18:53 |
*** Longgeek has quit IRC | 18:53 | |
morganfainberg | edmondsw: I am not making that judgement call. I'm just saying we need a better set of defaults because clearly most deployments use the defaults. And updates from defaults need to be better. | 18:53 |
sdague | dims: ok, so I -1ed it because I think we need to talk about the security impact, because there is one for existing default install envs | 18:53 |
sdague | but once you get that updated, I don't think it needs a microversion | 18:53 |
edmondsw | if we don't need a microversion, we don't even need a spec, do we? | 18:54 |
*** mriedem has joined #openstack-nova | 18:54 | |
*** ociuhandu has joined #openstack-nova | 18:54 | |
dims | sdague: default policy allows these calls, but does not return the right result... | 18:55 |
sdague | dims: right | 18:55 |
*** oro_ has quit IRC | 18:55 | |
sdague | but that combo of behavior is what people expect | 18:55 |
sdague | we're blocking all_tenants in code | 18:55 |
dims | should i be fixing the policy too? | 18:55 |
sdague | dims: absolutely | 18:55 |
sdague | otherwise you just openned a new security hole for people | 18:56 |
dims | so "is_admin:True" | 18:56 |
sdague | yep | 18:56 |
*** jog0 has joined #openstack-nova | 18:56 | |
*** saza has joined #openstack-nova | 18:56 | |
saza | hello | 18:56 |
sdague | and because policy isn't code, I think UpgradeImpact SecurityImpact should be used on the patch, so people know they should make a change at the same time to policy | 18:56 |
saza | nova network | 18:56 |
dims | sdague: still need a spec and blueprint? | 18:57 |
*** sudipto has quit IRC | 18:57 | |
*** sahid has joined #openstack-nova | 18:57 | |
sdague | dims: you've already got one, I'd say yes. You can ask johnthetubaguy in the morning what he thinks. | 18:57 |
*** krtaylor has quit IRC | 18:57 | |
saza | i have question nova-network | 18:57 |
dims | sdague: ack thanks | 18:58 |
saza | can i ask querstion on this channel ? | 18:58 |
mriedem | saza: just ask | 18:58 |
samueldmq | sdague, does not restrict operations on a project to tokens scoped to that specific project ? for any calls | 18:58 |
samueldmq | sdague, https://github.com/openstack/nova/blob/master/nova/api/openstack/wsgi.py#L804-L812 | 18:58 |
samueldmq | s/not/nova | 18:58 |
mriedem | saza: if it's a support question, see topic | 18:58 |
saza | br100: 172.16.0.254/24 | 18:58 |
saza | nova flat network | 18:58 |
saza | this gateway for subnet. | 18:59 |
saza | on compute note. | 18:59 |
saza | on vm can ping this ip <172.16.0.254> | 18:59 |
sdague | samueldmq: so that's a little different, that's asking specifically for a different project than yours | 18:59 |
*** Marga_ has joined #openstack-nova | 18:59 | |
sdague | but all_tenants wouldn't hit that check | 18:59 |
saza | but extenal network can't ping this ip . | 19:00 |
sdague | because you didn't set project_id to anything | 19:00 |
mriedem | saza: you're asking a support question, that's not for this channel, which is a dev channel | 19:00 |
samueldmq | sdague, k .. I was wondering in the update quota case | 19:00 |
mriedem | saza: you want the #openstack channel or ask.openstack.org | 19:00 |
mriedem | saza: or the general openstack mailing list | 19:00 |
saza | but external network ( ex. 172.16.0.100 can ping vm ( | 19:00 |
samueldmq | sdague, where someone updating a givne project's quota would be a domain admin (or someone in a higher project in the hierarhcy) | 19:00 |
saza | thx | 19:00 |
sdague | samueldmq: yeh, I don't know how all this works in a domains context | 19:01 |
*** VW has quit IRC | 19:01 | |
*** ctrath has quit IRC | 19:01 | |
edmondsw | samueldmq, I believe the domain admin would need to get a project-scoped token to update that project's quotas | 19:02 |
*** mtecer has quit IRC | 19:02 | |
*** saza has left #openstack-nova | 19:02 | |
*** otter768 has joined #openstack-nova | 19:02 | |
edmondsw | domain-scoped tokens aren't good for much | 19:02 |
samueldmq | edmondsw, the domain admin may even not have a role assigned on that project | 19:02 |
edmondsw | samueldmq, then they're out of luck | 19:03 |
samueldmq | edmondsw, domains are about identity, managing users and groups separately, and contain projects as well | 19:03 |
samueldmq | edmondsw, I mean, he could be able to delegate quotas, but couldn't list servers, for ex | 19:03 |
edmondsw | samueldmq, yes, I'm familiar... more familiar with keystone than with nova | 19:03 |
samueldmq | edmondsw, nice | 19:03 |
edmondsw | I didn't say I like the way domain tokens work.. ;) | 19:04 |
samueldmq | edmondsw, tokens behavior is controlled depending on the scope and roles | 19:05 |
samueldmq | edmondsw, but all depends on your policy | 19:05 |
samueldmq | edmondsw, RBAC :) | 19:05 |
edmondsw | samueldmq, well... yes and no. I wish that were really true | 19:05 |
samueldmq | edmondsw, why aren't they? | 19:06 |
*** tojuvone has quit IRC | 19:06 | |
edmondsw | because there is too much that is coded in a way that doesn't allow the policy to really control it to that extent | 19:06 |
*** otter768 has quit IRC | 19:07 | |
samueldmq | edmondsw, we're improving that | 19:09 |
edmondsw | good! | 19:09 |
samueldmq | edmondsw, dynamic policies is coming ot start improving our access control mechanism | 19:09 |
samueldmq | edmondsw, https://wiki.openstack.org/wiki/DynamicPolicies | 19:09 |
samueldmq | sdague, cc ^ | 19:09 |
*** Sukhdev_ has joined #openstack-nova | 19:10 | |
edmondsw | samueldmq, I'm familiar with the dynamic policy work... I think it's actually putting the cart before the horse... | 19:10 |
samueldmq | edmondsw, sdague that's still wip, we're still working with people to get the right solution | 19:10 |
edmondsw | we need to fix all these policy issues throughout our code and then worry about how to configure policy dynamically | 19:10 |
samueldmq | sdague, I am considerign nova requirements in my first try in the 'Overview Solution - Liberty Scope' section | 19:10 |
samueldmq | sdague, where warns are logged in the case someone change policy from defaults | 19:11 |
*** cfriesen has quit IRC | 19:11 | |
samueldmq | edmondsw, what are the issues throughout our code? give me an example ? | 19:11 |
samueldmq | not sure I follow | 19:12 |
*** fifieldt_ has quit IRC | 19:12 | |
*** Sukhdev has quit IRC | 19:13 | |
edmondsw | e.g., nova and cinder have a single policy for a class of things in some places, rather than for the individual CRUD actions | 19:13 |
*** IanGovett has quit IRC | 19:13 | |
edmondsw | ceilometer has policy hardcoded for probably half of their APIs | 19:13 |
edmondsw | etc. | 19:13 |
*** ijw_ has quit IRC | 19:14 | |
edmondsw | there are a lot of issues around ownership... where policy really needs to allow you to specify that only the owner of the element can do something, but the service isn't recording ownership for that element | 19:14 |
edmondsw | etc. | 19:14 |
*** rook-spec is now known as rook | 19:15 | |
*** openstackgerrit has quit IRC | 19:16 | |
samueldmq | edmondsw, we're improving the mechanisms around our curent policy.json, if people don't respect that policy that's a separate issue | 19:16 |
samueldmq | edmondsw, we will educate people to get better default policies | 19:17 |
samueldmq | edmondsw, which includes going away from hard-coded | 19:17 |
*** openstackgerrit has joined #openstack-nova | 19:17 | |
edmondsw | don't get me wrong, though... I will be glad to see dynamic policy | 19:17 |
edmondsw | and maybe working on it will bring more attention to some of these other things | 19:17 |
samueldmq | edmondsw, sure, no worries .. you're more than welcome to come and help, everyone is | 19:18 |
*** matrohon has joined #openstack-nova | 19:19 | |
samueldmq | edmondsw, I am working now on having some agreements internally on keystone, will be sendng a message in the ML this week | 19:19 |
edmondsw | samueldmq, yep, I believe I added some comments to that spec the other day | 19:19 |
samueldmq | edmondsw, we need to involve other projects on this (nova started) .. we need a weekly meeting for this subject | 19:19 |
samueldmq | edmondsw, I am trying to run with those things .. the clock is ticking, and we plan to get something valuable in Liberty | 19:20 |
*** afazekas has quit IRC | 19:21 | |
*** ctrath has joined #openstack-nova | 19:22 | |
*** nihilifer has joined #openstack-nova | 19:23 | |
*** mtecer has joined #openstack-nova | 19:27 | |
mriedem | dhellmann: what are your thoughts on all of the deprecation warning code in nova that is referring to 2015.2 and 2016.1 for when things are removed, as ericwb points out in https://review.openstack.org/#/c/192406/ ? | 19:27 |
mriedem | since i have one of those proposed here https://review.openstack.org/#/c/192753/1/nova/virt/vmwareapi/driver.py | 19:27 |
dhellmann | mriedem: I suppose those should be updated? | 19:28 |
mriedem | dhellmann: yeah, but in your change? | 19:28 |
mriedem | i'd think yes | 19:28 |
*** krtaylor has joined #openstack-nova | 19:28 | |
mriedem | and 2016.1 M will be 13.0.0 M right? | 19:28 |
mriedem | maybe not even called M anymore? i don't know the latest on wtf is going on. | 19:29 |
*** tjones1 has quit IRC | 19:29 | |
dhellmann | mriedem: I will happily rebase my change on top of that, but I really can't do all of that for you guys. | 19:29 |
*** agireud has joined #openstack-nova | 19:29 | |
mriedem | dhellmann: yeah, understand, since you have 50 of these cagts | 19:29 |
mriedem | *cats | 19:29 |
dhellmann | right | 19:29 |
ijuwang | Do we open a bug for implemented spec that contains api that's not implemented? For example, http://specs.openstack.org/openstack/nova-specs/specs/juno/implemented/user-defined-shutdown.html | 19:29 |
*** fifieldt_ has joined #openstack-nova | 19:30 | |
dhellmann | mriedem: we have some similar issues in heat, fwiw | 19:30 |
mriedem | ijuwang: if anything it'd just be a spec update probably | 19:30 |
mriedem | dhellmann: yeah, and novaclient | 19:30 |
mriedem | https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/shell.py#L66 | 19:31 |
*** Marga_ has quit IRC | 19:31 | |
mriedem | ok.... | 19:31 |
mriedem | so maybe i'll just take one for the team here and update that | 19:31 |
*** Marga_ has joined #openstack-nova | 19:31 | |
dhellmann | mriedem: that's very sporting of you | 19:32 |
*** Mike_D_laptop has quit IRC | 19:32 | |
mriedem | ctrath: if you could legally push code this would be a good one for you :) | 19:32 |
*** Nikolay_St has quit IRC | 19:32 | |
ijuwang | mriedem: ok, thanks | 19:34 |
sahid | mriedem: hello, we have a backport ready for kilo, if you are happy with it https://review.openstack.org/#/c/189350/ | 19:37 |
*** electrocucaracha has quit IRC | 19:37 | |
mriedem | sahid: i saw that before, i wanted to dig into the original change on master to make sure i understood it first | 19:37 |
mriedem | since that seems too easy | 19:37 |
sahid | mriedem: ah ok | 19:38 |
sahid | mriedem: please let me know of the review if you need any information | 19:38 |
*** arnaud___ has joined #openstack-nova | 19:39 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: cells: remove deprecated mute_weight_value option https://review.openstack.org/192853 | 19:39 |
sahid | O_o - what i wanted to say... | 19:39 |
mriedem | i understood :) | 19:39 |
mriedem | let you know if i have questoins | 19:39 |
*** electrocucaracha has joined #openstack-nova | 19:40 | |
*** Marga_ has quit IRC | 19:40 | |
mriedem | alaski: dansmith: sdague: ^ removing that option will help with the semver change patch | 19:40 |
sahid | ahah cool thank you | 19:40 |
*** thedodd has joined #openstack-nova | 19:40 | |
dansmith | bauzas: jaypipes: https://review.openstack.org/#/c/192708 | 19:41 |
*** isd has quit IRC | 19:42 | |
dansmith | I think I'm clear to do that ^^ based on the justification in the commit message | 19:42 |
*** VW has joined #openstack-nova | 19:42 | |
dansmith | but would like you guys to check my work :) | 19:42 |
*** IanGovett has joined #openstack-nova | 19:42 | |
bauzas | dansmith: looking | 19:43 |
*** arnaud___ has quit IRC | 19:43 | |
*** electrocucaracha has quit IRC | 19:44 | |
*** beagles is now known as beagles|afk | 19:45 | |
*** agireud has quit IRC | 19:45 | |
*** nkrinner has joined #openstack-nova | 19:46 | |
*** agireud has joined #openstack-nova | 19:48 | |
*** Longgeek has joined #openstack-nova | 19:49 | |
dansmith | bauzas: no rush, it's at the end of a set | 19:51 |
edleafe | dansmith: in https://review.openstack.org/#/c/192708/3/nova/compute/resource_tracker.py,cm L913, should that parameter still be 'object_or_dict'? | 19:51 |
edleafe | dansmith: or should it always be an object at this point? | 19:51 |
*** penick has quit IRC | 19:51 | |
dansmith | edleafe: not changed because of this patch | 19:52 |
dansmith | and I think it's still valid anyway | 19:52 |
edleafe | so you could still pass an instance dict? | 19:52 |
dansmith | edleafe: one could, yes | 19:53 |
dansmith | whether that can still happen I'm not sure, but I haven't seen anything lately that would change that | 19:53 |
dansmith | I think this has to do with the fact that we take in an object, and convert it to a dict, and some things will re-pass that dict to us | 19:53 |
edleafe | dansmith: ok, thanks. I wasn't sure that obj_to_primitive() would accept a dict | 19:54 |
*** nihilifer_ has joined #openstack-nova | 19:54 | |
dansmith | edleafe: we won't call obj_to_primitive() with a dict in this code anyway | 19:54 |
dansmith | because of the isinstance() | 19:54 |
edleafe | dansmith: yep, got it | 19:55 |
bauzas | edleafe: the main problem is that Instance or Flavor can be either a dict or not, depending on the caller | 19:55 |
bauzas | edleafe: just because sometimes the instance is primitived before that | 19:55 |
*** isd has joined #openstack-nova | 19:56 | |
bauzas | edleafe: so there are many possibilities where it can be different, hence nikola trying to provide that helper method for managing all the cases | 19:56 |
edleafe | bauzas: yeah, like dansmith pointed out, I missed the isinstance() qualifier | 19:56 |
*** nihilifer has quit IRC | 19:56 | |
bauzas | edleafe: yeah, I just tried to explain why we need this helper | 19:57 |
*** exploreshaifali has quit IRC | 19:57 | |
bauzas | just because we need to fix all the callers before removing that | 19:57 |
*** baoli has joined #openstack-nova | 19:57 | |
*** penick has joined #openstack-nova | 19:58 | |
*** penick has quit IRC | 19:58 | |
*** Nikolay_St has joined #openstack-nova | 19:58 | |
bauzas | dansmith: so, I maybe have a question https://review.openstack.org/#/c/192708/3/nova/scheduler/utils.py,cm | 19:58 |
dansmith | bauzas: that's the bit I figured would be concerning, yes :D | 19:58 |
bauzas | dansmith: build_request_spec is for calling the Scheduler RPC API, but we didn't removed the Scheduler RPC v3 yet | 19:59 |
dansmith | um | 19:59 |
dansmith | hmm | 19:59 |
dansmith | I was thinking this went to compute for build_and_run_instances(), and it doesn't use anything there | 19:59 |
bauzas | dansmith: so, I don't think it's really a problem, because the instance is mainly consumed at the right named consume_from_instance() method | 19:59 |
bauzas | dansmith: nope, it goes thru select_dests() too | 20:00 |
*** electrocucaracha has joined #openstack-nova | 20:00 | |
dansmith | does anything actually use the flavor stuff in sysmeta there? | 20:00 |
*** tonytan4ever has quit IRC | 20:00 | |
bauzas | dansmith: that's my point | 20:00 |
bauzas | dansmith: I don't think so | 20:00 |
dansmith | hmm, okay | 20:01 |
dansmith | if it doesn't we could either (a) just do this or (b) put it back in just in this location I guess | 20:01 |
bauzas | dansmith: I mean, I'm 99% sure about that since I'm currently working on the HostManager atm :D | 20:01 |
dansmith | okay | 20:01 |
bauzas | dansmith: but I'm not sure the instance could be checked for the flavor stuff in a filter, I need to check that | 20:01 |
*** penick has joined #openstack-nova | 20:02 | |
bauzas | dansmith: anyway, all of that build_req_spec() will be doomed once someone does his job | 20:02 |
*** tjones1 has joined #openstack-nova | 20:02 | |
dansmith | bauzas: okay | 20:03 |
bauzas | dansmith: but we still need to care about that given we would still leave the old select_dest() method for the compat | 20:03 |
bauzas | so, checking now who calls instance_props in the scheduler, and if the flavor stuff is checked then | 20:03 |
dansmith | bauzas: so what we want to avoid is these things being in the instance_properties and then ending up creating instances with that in sysmeta | 20:04 |
*** neophy has quit IRC | 20:04 | |
bauzas | dansmith: as the scheduler doesn't save the instances, we're fine IMHO | 20:04 |
dansmith | if we can reliably add it into scheduler/utils and then make sure it is removed before we create the instance in the db, then we can do that until your object work is done | 20:04 |
dansmith | bauzas: well, it passes those bits along to someone that does (conductor) right? | 20:04 |
bauzas | dansmith: the conductor calls build_req_spec() and then select_dest() | 20:05 |
bauzas | dansmith: but the spec is not then used | 20:05 |
bauzas | I mean, after the scheduler call | 20:05 |
*** Marga_ has joined #openstack-nova | 20:06 | |
dansmith | well, whatever creates the instance does it from instance_properties, as a template, right? | 20:06 |
bauzas | dansmith: which is different from filter_properties, which is passed to the compute IIRC | 20:06 |
*** Longgeek has quit IRC | 20:06 | |
bauzas | dansmith: the instance is created before the sched call, right? | 20:06 |
alaski | the instance is created in the compute api | 20:06 |
bauzas | alaski: yeah, my point | 20:07 |
*** tonytan4ever has joined #openstack-nova | 20:07 | |
alaski | for now... | 20:07 |
dansmith | bauzas: ah, okay | 20:07 |
bauzas | alaski: don't freak me out :) | 20:07 |
dansmith | here is where I insert a hilarious "gosh I should know this" comment where we all laugh about how it's nearly impossible to follow all this madness | 20:07 |
*** nic has quit IRC | 20:08 | |
bauzas | dansmith: and wait for cells v2 | 20:09 |
bauzas | ... :) | 20:09 |
alaski | it's pretty crazy | 20:09 |
bauzas | the only reasonable reason I'm part of cells v2 is to lurk at the design discussions | 20:09 |
*** baoli has quit IRC | 20:10 | |
bauzas | dansmith: so, given that, I'm pretty okay with your change | 20:10 |
bauzas | dansmith: there is no flavor-ish thing needed | 20:11 |
*** baoli has joined #openstack-nova | 20:11 | |
bauzas | I mean by the scheudler | 20:11 |
*** thiagop has joined #openstack-nova | 20:11 | |
*** tdurakov_ has quit IRC | 20:12 | |
dansmith | alaski: bauzas: okay, so I'll slide the crap back into that one call, and make sure that it doesn't end up with stuff back in sysmeta in the actual cleaned instances | 20:12 |
bauzas | dansmith: that's pretty isolated on the scheduler side | 20:13 |
*** thiagop has left #openstack-nova | 20:13 | |
bauzas | dansmith: I mean, what's given by that utils.build_req_spec() method goes to the scheduler as a dead-end, just for consuming fields | 20:14 |
*** jog0 has quit IRC | 20:14 | |
dansmith | yeah, but if filters could be looking at it... | 20:14 |
bauzas | dansmith: and if you see the RequestSpec object that I modeled, I only provided some instance fields | 20:14 |
bauzas | dansmith: not the whole Instance object | 20:15 |
dansmith | right | 20:15 |
*** artom has quit IRC | 20:15 | |
bauzas | dansmith: I checked that, and no filters are looking at it | 20:15 |
bauzas | dansmith: and if I had found one, I would have slapfaced me | 20:15 |
*** artom has joined #openstack-nova | 20:15 | |
bauzas | dansmith: just because I made that review earlier on the spec writing when I exploded the Instance stuff | 20:16 |
*** dane-fichter has quit IRC | 20:16 | |
dansmith | bauzas: well, in-tree filters anyway | 20:17 |
bauzas | dansmith: right | 20:17 |
dansmith | if it's easy for me to add it in without any other work, I will | 20:17 |
*** artom has quit IRC | 20:17 | |
dansmith | if it starts getting difficult, I'll recant :) | 20:17 |
*** artom has joined #openstack-nova | 20:17 | |
bauzas | dansmith: but since we're working on providing a clear API for what's going to the scheduler, out-of-tree filters are also impacted anyway | 20:18 |
dansmith | yeah | 20:18 |
bauzas | just because it won't be that easier to pass something unversioned to the sched now | 20:18 |
bauzas | easy* | 20:18 |
*** achanda has joined #openstack-nova | 20:19 | |
*** artom has quit IRC | 20:19 | |
*** artom has joined #openstack-nova | 20:20 | |
*** jog0 has joined #openstack-nova | 20:20 | |
*** dpaterson has quit IRC | 20:20 | |
mriedem | ctrath: this is that bug i was talking about https://bugs.launchpad.net/nova/+bug/1186354 - i think we agreed that if you're hitting those quota APIs and you're using neutron for networking, just fail since it's not going to be correct and we don't want to add the proxy code, even though we could | 20:22 |
openstack | Launchpad bug 1186354 in OpenStack Compute (nova) "Limits API doesn't work with Neutron" [Medium,Confirmed] | 20:22 |
mriedem | should maybe bring that one up in the ML first | 20:22 |
*** achanda_ has joined #openstack-nova | 20:23 | |
*** mleroy_ has joined #openstack-nova | 20:23 | |
mriedem | this was the original change way back in havana i think to proxy the calls to neutron, so some form of that could be resurrected if we wanted to proxy https://review.openstack.org/#/c/43822/ | 20:23 |
*** thangp_ has quit IRC | 20:24 | |
mriedem | although http://docs.openstack.org/developer/nova/project_scope.html?highlight=proxy#no-more-api-proxies | 20:24 |
openstackgerrit | Andrew Laski proposed openstack/nova: Cells: cell scheduler anti-affinity filter https://review.openstack.org/192875 | 20:24 |
*** whenry has joined #openstack-nova | 20:26 | |
*** achanda has quit IRC | 20:26 | |
*** ivasev has quit IRC | 20:26 | |
*** thangp has joined #openstack-nova | 20:27 | |
*** VW has quit IRC | 20:28 | |
*** lpetrut has quit IRC | 20:29 | |
*** ctrath has quit IRC | 20:29 | |
*** ctrath has joined #openstack-nova | 20:32 | |
*** mtecer has quit IRC | 20:33 | |
*** cfriesen has joined #openstack-nova | 20:34 | |
*** mwagner_afk has quit IRC | 20:34 | |
*** sahid has quit IRC | 20:35 | |
openstackgerrit | Ming Yang proposed openstack/nova: Refactor get_filtered_hosts in filter host manager https://review.openstack.org/192878 | 20:37 |
mingy | (I'm just doing this while I'm reading the code... it helps me understand what's going on. I hope no one minds.) | 20:37 |
*** comstud has joined #openstack-nova | 20:37 | |
*** achanda_ has quit IRC | 20:38 | |
*** baoli has quit IRC | 20:39 | |
*** whenry has quit IRC | 20:39 | |
*** baoli has joined #openstack-nova | 20:40 | |
*** vladikr_ has quit IRC | 20:41 | |
*** tlbr has quit IRC | 20:47 | |
*** tlbr has joined #openstack-nova | 20:47 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: Switch to using os-brick https://review.openstack.org/175569 | 20:47 |
*** READ10 has quit IRC | 20:47 | |
*** lan has quit IRC | 20:47 | |
openstackgerrit | Qin Zhao proposed openstack/nova: Support flavor object in migrate_disk_and_power_off https://review.openstack.org/192786 | 20:48 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Update version for Liberty https://review.openstack.org/192406 | 20:50 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Remove hv_type translation shim for powervm https://review.openstack.org/192887 | 20:50 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: remove deprecated allow_duplicate_networks config option https://review.openstack.org/192888 | 20:50 |
*** lan has joined #openstack-nova | 20:52 | |
*** josecastroleon1 has quit IRC | 20:53 | |
*** yamahata has joined #openstack-nova | 20:54 | |
SourabhP | beagles|afk: ping | 20:54 |
*** josecastroleon has joined #openstack-nova | 20:54 | |
*** jskinner has quit IRC | 20:54 | |
*** haypo has joined #openstack-nova | 20:54 | |
bauzas | dansmith: quick question, what would be better than a StringField for a JSON property, saying that I can't make sure that it would be a DictOfStrings or rather a DictofDicts etc. | 20:58 |
bauzas | ? | 20:58 |
*** artom has quit IRC | 20:58 | |
*** nkrinner has quit IRC | 20:58 | |
*** penick has quit IRC | 20:59 | |
openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds Hyper-V Shielded VMs spec https://review.openstack.org/192892 | 20:59 |
bauzas | dansmith: see https://github.com/openstack/nova/blob/master/nova/scheduler/filters/json_filter.py#L143 for the question | 20:59 |
*** dulek_home has joined #openstack-nova | 21:00 | |
*** dulek_home has quit IRC | 21:01 | |
*** dprince has quit IRC | 21:01 | |
*** dulek has joined #openstack-nova | 21:02 | |
mriedem | dhellmann: i updated https://review.openstack.org/#/c/192406/ but does that not work until we actually tag the 12.0.0 release for liberty? | 21:02 |
*** agireud has quit IRC | 21:02 | |
*** otter768 has joined #openstack-nova | 21:03 | |
*** annashen has quit IRC | 21:05 | |
*** edmondsw has quit IRC | 21:06 | |
*** electrocucaracha has quit IRC | 21:07 | |
openstackgerrit | Philip Marc Schwartz proposed openstack/nova: Add flag to force experimental run of db contract https://review.openstack.org/192894 | 21:07 |
*** otter768 has quit IRC | 21:08 | |
*** electrocucaracha has joined #openstack-nova | 21:09 | |
*** mleroy_ has quit IRC | 21:09 | |
*** Sukhdev_ has quit IRC | 21:10 | |
*** penick has joined #openstack-nova | 21:10 | |
*** haomaiwang has joined #openstack-nova | 21:13 | |
*** READ10 has joined #openstack-nova | 21:14 | |
*** haomaiw__ has quit IRC | 21:15 | |
*** dulek has quit IRC | 21:18 | |
*** atuvenie has quit IRC | 21:20 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Kicking the oslo.service tires https://review.openstack.org/192900 | 21:20 |
*** isd has quit IRC | 21:20 | |
*** haomaiw__ has joined #openstack-nova | 21:21 | |
dansmith | bauzas: well, anything other than a JSONfield would be better | 21:21 |
dansmith | bauzas: like: GiraffeField | 21:21 |
*** haomaiwang has quit IRC | 21:21 | |
*** mwagner_afk has joined #openstack-nova | 21:22 | |
mriedem | GarField? | 21:23 |
*** Marga_ has quit IRC | 21:24 | |
dansmith | bah dum pish | 21:24 |
*** openstackgerrit has quit IRC | 21:24 | |
mriedem | i'm actually here all week | 21:24 |
dansmith | heh | 21:24 |
mriedem | i guess openstackgerrit couldn't handle that level of pun | 21:24 |
*** Marga_ has joined #openstack-nova | 21:24 | |
*** openstackgerrit has joined #openstack-nova | 21:24 | |
dims | Lol | 21:25 |
*** dulek_home has joined #openstack-nova | 21:25 | |
*** thangp has quit IRC | 21:25 | |
*** sdake_ is now known as sdake | 21:25 | |
*** dulek_home has quit IRC | 21:25 | |
*** dulek has joined #openstack-nova | 21:26 | |
*** nihilifer_ is now known as nihilifer | 21:26 | |
openstackgerrit | Matt Riedemann proposed openstack/python-novaclient: Change future nova version number references based on new values https://review.openstack.org/192902 | 21:27 |
*** erlon has quit IRC | 21:30 | |
openstackgerrit | Matt Riedemann proposed openstack/python-novaclient: Change future nova version number references based on new values https://review.openstack.org/192903 | 21:30 |
*** VW has joined #openstack-nova | 21:32 | |
*** angdraug has joined #openstack-nova | 21:32 | |
*** penick has quit IRC | 21:33 | |
*** Marga_ has quit IRC | 21:33 | |
*** Marga_ has joined #openstack-nova | 21:34 | |
*** achanda has joined #openstack-nova | 21:36 | |
*** tonytan4ever has quit IRC | 21:36 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: vmware: update help text for vmware.cluster_name option https://review.openstack.org/192753 | 21:37 |
*** baoli_ has joined #openstack-nova | 21:37 | |
*** yamahata has quit IRC | 21:37 | |
*** radek_ has quit IRC | 21:38 | |
*** baoli has quit IRC | 21:40 | |
mriedem | SourabhP: i think you're looking for this https://review.openstack.org/#/c/162468/ | 21:40 |
*** jlk has joined #openstack-nova | 21:40 | |
jlk | Quick question, can somebody help me find in the nova code where a hypervisor's hostname is determined for the sake of creating the service entry, and whether or not that can be configured in nova's config file? | 21:41 |
openstackgerrit | Philip Marc Schwartz proposed openstack/nova: Add flag to force experimental run of db contract https://review.openstack.org/192894 | 21:41 |
mriedem | jlk: isn't that my_host in nova.conf? | 21:41 |
mriedem | sorry my_ip | 21:42 |
jlk | ah, I knew about my_ip, didn't about my_host | 21:42 |
jlk | no, my_ip is a bit different. | 21:42 |
mriedem | b/c i made it up | 21:42 |
SourabhP | mriedem: thanks for the pointer! | 21:42 |
jlk | that's about how to /reach/ the host, for the sake of things like migration | 21:42 |
*** achanda has quit IRC | 21:42 | |
jlk | but I'm more concerned about what hostname gets written into the service table when nova-compute comes online | 21:42 |
*** annashen has joined #openstack-nova | 21:42 | |
*** alex_klimov has joined #openstack-nova | 21:43 | |
jlk | or if it's hard set to whatever it can resolve from the system hostname | 21:43 |
*** matrohon has quit IRC | 21:44 | |
mriedem | jlk: should be able to trace that to whatever is using the resource tracker in the compute manager | 21:44 |
jlk | ah, maybe it's just host= | 21:44 |
jlk | nicely undocumented :) | 21:46 |
mriedem | patches welcome | 21:46 |
mriedem | if not host: | 21:47 |
mriedem | host = CONF.host | 21:47 |
mriedem | there you go | 21:47 |
dansmith | tweets unwelcome | 21:47 |
jlk | n/m, i tis documented | 21:47 |
mriedem | nova.service.Service.create | 21:47 |
mriedem | the nova.cmd.compute entry point creates the compute service and doesn't pass a host, so that comes from CONF.host | 21:48 |
jlk | I think I failed ot find it because I was looking for "hostname" | 21:48 |
mriedem | host = localhost(StrOpt) Name of this node. This can be an opaque identifier. It is not necessarily a hostname, FQDN, or IP address. However, the node name must be valid within an AMQP key, and if using ZeroMQ, a valid hostname, FQDN, or IP address | 21:48 |
jlk | yup, thanks, I've found the bits I was looking for. Always a pleasure! | 21:48 |
mriedem | http://docs.openstack.org/kilo/config-reference/content/list-of-compute-config-options.html | 21:48 |
mriedem | \o/ | 21:48 |
mriedem | on that i can leave for the day | 21:48 |
*** tjones1 has quit IRC | 21:51 | |
mriedem | ctrath: you are now the proud owner of https://bugs.launchpad.net/nova/+bug/1186354 | 21:52 |
openstack | Launchpad bug 1186354 in OpenStack Compute (nova) "Limits API doesn't work with Neutron" [Medium,Confirmed] - Assigned to Cale Rath (ctrath) | 21:52 |
mriedem | ctrath: you should probably also join the nova bug team if you want to triage stuff https://launchpad.net/~nova-bugs | 21:53 |
*** agireud has joined #openstack-nova | 21:53 | |
mriedem | s/want/need/ | 21:53 |
*** ianbrown has joined #openstack-nova | 21:53 | |
mriedem | just dont sign up to get bug email for all new bugs | 21:53 |
*** xyang2 has quit IRC | 21:57 | |
*** ZZelle_ has quit IRC | 21:57 | |
*** mriedem has quit IRC | 21:58 | |
*** jog0 is now known as jogo | 21:58 | |
*** burt has quit IRC | 22:00 | |
*** jlk has left #openstack-nova | 22:00 | |
*** thedodd has quit IRC | 22:01 | |
*** Longgeek has joined #openstack-nova | 22:03 | |
*** jogo has quit IRC | 22:03 | |
*** wverdugo5001 has quit IRC | 22:06 | |
*** jgrimm is now known as zz_jgrimm | 22:07 | |
*** mdrabe has quit IRC | 22:11 | |
*** VW has quit IRC | 22:11 | |
*** alex_klimov has quit IRC | 22:12 | |
*** VW has joined #openstack-nova | 22:13 | |
*** agireud has quit IRC | 22:14 | |
*** baoli_ has quit IRC | 22:22 | |
rook | wrt Kilo & SRIOV - should the databse be populated with the pci information? and if so, what table? | 22:23 |
rook | database* | 22:23 |
*** haypo has quit IRC | 22:24 | |
*** penick has joined #openstack-nova | 22:24 | |
*** dims has quit IRC | 22:26 | |
*** jecarey has quit IRC | 22:30 | |
*** baoli has joined #openstack-nova | 22:31 | |
*** Longgeek has quit IRC | 22:36 | |
*** ijuwang has quit IRC | 22:36 | |
*** baoli has quit IRC | 22:38 | |
*** kylek3h has quit IRC | 22:42 | |
*** aysyanne has quit IRC | 22:42 | |
*** Nikolay_St has quit IRC | 22:42 | |
*** krtaylor has quit IRC | 22:44 | |
*** zzzeek has quit IRC | 22:44 | |
*** jasondotstar has joined #openstack-nova | 22:45 | |
*** IanGovett has quit IRC | 22:45 | |
*** josecastroleon has quit IRC | 22:47 | |
*** josecastroleon has joined #openstack-nova | 22:48 | |
*** ctrath has quit IRC | 22:49 | |
*** ociuhandu has quit IRC | 22:49 | |
*** Marga_ has quit IRC | 22:49 | |
*** ctrath has joined #openstack-nova | 22:50 | |
*** Marga_ has joined #openstack-nova | 22:50 | |
*** oro_ has joined #openstack-nova | 22:51 | |
*** asselin has joined #openstack-nova | 22:55 | |
*** dboik_ has quit IRC | 22:56 | |
*** IanGovett has joined #openstack-nova | 22:56 | |
*** dboik has joined #openstack-nova | 22:56 | |
*** ctrath has quit IRC | 22:56 | |
*** _diana_ has joined #openstack-nova | 22:59 | |
*** tpeoples has quit IRC | 22:59 | |
*** nihilifer has quit IRC | 23:01 | |
*** oro_ has quit IRC | 23:02 | |
*** otter768 has joined #openstack-nova | 23:04 | |
*** IanGovett has quit IRC | 23:06 | |
*** Marga_ has quit IRC | 23:07 | |
*** otter768 has quit IRC | 23:08 | |
_diana_ | Hi folks, I have what is hopefully an easy PCI virtual function vs physical function question. Does anyone have a couple of minutes to explain why is_physical_function returns true when it matches a virtual function? https://github.com/openstack/nova/blob/master/nova/pci/utils.py#L97 I would have assumed that a virtual function match would mean that is_physical_function would return false, but I've obviously m | 23:09 |
*** tpeoples has joined #openstack-nova | 23:09 | |
*** obondarev has quit IRC | 23:11 | |
*** obondarev has joined #openstack-nova | 23:12 | |
*** tjones1 has joined #openstack-nova | 23:13 | |
*** yamahata has joined #openstack-nova | 23:15 | |
*** puniuee has joined #openstack-nova | 23:15 | |
puniuee | @all I need help in figuring out something | 23:16 |
puniuee | two of my nova.tests.unit.compute.test_compute_cells.CellsComputeAPITestCase are failing | 23:16 |
puniuee | i am stuck on this from 2 days and I don't know, how the mock calls and stub calls actually work | 23:17 |
*** park_ has joined #openstack-nova | 23:22 | |
*** dims has joined #openstack-nova | 23:24 | |
*** zzzeek has joined #openstack-nova | 23:25 | |
*** Marga_ has joined #openstack-nova | 23:27 | |
*** VW has quit IRC | 23:28 | |
*** park_ has quit IRC | 23:28 | |
*** dims has quit IRC | 23:28 | |
melwitt | puniuee: I recommend looking through the docs to learn how they work https://code.google.com/p/pymox/wiki/MoxDocumentation or https://mock.readthedocs.org/en/latest/index.html depending on whether the tests in question are using mox or mock | 23:28 |
*** dims has joined #openstack-nova | 23:28 | |
*** park_ has joined #openstack-nova | 23:29 | |
*** krtaylor has joined #openstack-nova | 23:32 | |
*** dims has quit IRC | 23:33 | |
*** claudiub has quit IRC | 23:34 | |
puniuee | thanks @melwitt | 23:35 |
*** yamahata has quit IRC | 23:40 | |
*** mriedem has joined #openstack-nova | 23:42 | |
*** diogogmt has quit IRC | 23:45 | |
*** mriedem1 has joined #openstack-nova | 23:45 | |
*** _diana_ has quit IRC | 23:45 | |
*** jskinner has joined #openstack-nova | 23:46 | |
*** mriedem has quit IRC | 23:47 | |
*** lucas-dinner has quit IRC | 23:51 | |
puniuee | please someone also tell me an example of usage of NoopQuotaDriver | 23:51 |
puniuee | it says there" with nova compute cells, the parent cell should do quota checking, but the child cell should not" | 23:51 |
*** park_ has quit IRC | 23:52 | |
*** otter768 has joined #openstack-nova | 23:53 | |
puniuee | i mean a flow where parent cell checks the quota and child does not | 23:57 |
*** pixelbeat_ has quit IRC | 23:59 | |
*** sdake_ has joined #openstack-nova | 23:59 | |
*** oomichi has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!