*** baoli has quit IRC | 00:00 | |
*** david-lyle_ has joined #openstack-nova | 00:00 | |
*** tjones has left #openstack-nova | 00:08 | |
*** david-lyle has joined #openstack-nova | 00:09 | |
*** salv-orlando has quit IRC | 00:10 | |
*** gcb has quit IRC | 00:10 | |
*** Guest72572 has quit IRC | 00:10 | |
*** baoli has joined #openstack-nova | 00:17 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | 00:18 |
---|---|---|
*** breitz has quit IRC | 00:18 | |
*** gongysh has joined #openstack-nova | 00:19 | |
*** dimtruck is now known as zz_dimtruck | 00:19 | |
*** ducttape_ has quit IRC | 00:19 | |
*** david-lyle has quit IRC | 00:20 | |
*** adrian_otto has joined #openstack-nova | 00:24 | |
*** david-lyle_ has quit IRC | 00:25 | |
*** iceyao has joined #openstack-nova | 00:26 | |
*** yingjun has joined #openstack-nova | 00:27 | |
*** baoli has quit IRC | 00:27 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 00:27 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 00:29 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 00:29 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 00:29 |
*** takashin has joined #openstack-nova | 00:30 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: wait for interface detach from the guest https://review.openstack.org/349014 | 00:33 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: fix and break up _test_attach_detach_interface https://review.openstack.org/431778 | 00:33 |
*** mriedem has quit IRC | 00:39 | |
*** mlavalle has quit IRC | 00:40 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/426636 | 00:41 |
*** hoangcx has joined #openstack-nova | 00:41 | |
*** ducttape_ has joined #openstack-nova | 00:50 | |
*** nic has quit IRC | 00:50 | |
*** yuntongjin has joined #openstack-nova | 00:50 | |
*** baoli has joined #openstack-nova | 00:53 | |
*** kevinz has joined #openstack-nova | 00:54 | |
openstackgerrit | yuhui proposed openstack/nova master: Correct some spelling errors https://review.openstack.org/430646 | 01:00 |
*** litao has joined #openstack-nova | 01:01 | |
*** yuntongjin has quit IRC | 01:01 | |
*** yuntongjin has joined #openstack-nova | 01:02 | |
openstackgerrit | melanie witt proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521 | 01:02 |
openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 01:02 |
openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 01:02 |
*** thorst_ has joined #openstack-nova | 01:03 | |
*** phuongnh has joined #openstack-nova | 01:04 | |
*** jamesdenton has joined #openstack-nova | 01:05 | |
*** thorst_ has quit IRC | 01:06 | |
*** yuntongjin has quit IRC | 01:06 | |
*** jamielennox is now known as jamielennox|away | 01:07 | |
*** sdague has quit IRC | 01:08 | |
*** blaisebool has quit IRC | 01:08 | |
*** crushil has joined #openstack-nova | 01:09 | |
*** grumpycatt has quit IRC | 01:10 | |
*** grumpycatt has joined #openstack-nova | 01:11 | |
*** dave-mccowan has joined #openstack-nova | 01:11 | |
*** blaisebool has joined #openstack-nova | 01:11 | |
*** oomichi has quit IRC | 01:12 | |
*** oomichi has joined #openstack-nova | 01:12 | |
*** zhurong has joined #openstack-nova | 01:13 | |
*** amotoki has joined #openstack-nova | 01:14 | |
*** amotoki has quit IRC | 01:17 | |
*** amotoki has joined #openstack-nova | 01:17 | |
*** amotoki_ has joined #openstack-nova | 01:18 | |
*** gongysh has quit IRC | 01:20 | |
*** amotoki has quit IRC | 01:21 | |
*** jamielennox|away is now known as jamielennox | 01:21 | |
*** ducttape_ has quit IRC | 01:22 | |
*** lucas_ has joined #openstack-nova | 01:29 | |
openstackgerrit | Li Jingjing proposed openstack/nova master: Fix typo in nova/network/neutronv2/api.py https://review.openstack.org/431796 | 01:30 |
*** david-lyle_ has joined #openstack-nova | 01:30 | |
*** Guest81164 has quit IRC | 01:36 | |
*** Guest81164 has joined #openstack-nova | 01:36 | |
*** thorst_ has joined #openstack-nova | 01:37 | |
*** thorst_ has quit IRC | 01:38 | |
*** Daisy has joined #openstack-nova | 01:41 | |
*** delattec has quit IRC | 01:44 | |
*** hamzy has joined #openstack-nova | 01:45 | |
*** zz_dimtruck is now known as dimtruck | 01:47 | |
*** tbachman has joined #openstack-nova | 01:48 | |
*** zhurong has quit IRC | 01:52 | |
*** david-lyle_ has quit IRC | 01:55 | |
*** Sukhdev has quit IRC | 01:56 | |
*** gcb has joined #openstack-nova | 01:56 | |
*** jamesdenton has quit IRC | 01:57 | |
*** sree has quit IRC | 01:58 | |
*** adrian_otto has quit IRC | 01:58 | |
*** esberglu has joined #openstack-nova | 02:10 | |
*** esberglu has quit IRC | 02:10 | |
*** esberglu has joined #openstack-nova | 02:10 | |
*** esberglu has quit IRC | 02:11 | |
*** Apoorva has quit IRC | 02:11 | |
*** zhurong has joined #openstack-nova | 02:12 | |
*** jamesdenton has joined #openstack-nova | 02:13 | |
*** unicell has quit IRC | 02:14 | |
*** ducttape_ has joined #openstack-nova | 02:15 | |
*** lucas_ has quit IRC | 02:23 | |
*** Daisy has quit IRC | 02:24 | |
*** Daisy has joined #openstack-nova | 02:24 | |
*** zhurong has quit IRC | 02:25 | |
*** crushil has quit IRC | 02:27 | |
*** zhurong has joined #openstack-nova | 02:29 | |
*** mriedem has joined #openstack-nova | 02:31 | |
*** ducttape_ has quit IRC | 02:34 | |
*** zhurong has quit IRC | 02:38 | |
*** thorst_ has joined #openstack-nova | 02:38 | |
*** baoli has quit IRC | 02:39 | |
alex_xu | mriedem: our team are working on enable CAT(CPU L3 cache allocation tech) on openstack, currently the kernel support is ready, the team are working the libvirt interface. is there any chance we can submit the spec or raise some discussion in Nova even the libvirt interface not ready? at least some discussion, what kind of usage-model from nova point of view. | 02:40 |
mriedem | alex_xu: are you going to be at the PTG? | 02:41 |
*** jamesdenton has quit IRC | 02:41 | |
alex_xu | mriedem: yea, I will go to the PTG | 02:41 |
mriedem | that's probably a decent place to discuss it, but i'd recommend starting a thread in the mailing list, or maybe a draft spec since i've never heard of that before | 02:42 |
alex_xu | mriedem: yeah, got it | 02:43 |
*** lucas_ has joined #openstack-nova | 02:43 | |
mriedem | i'm assuming this would be some kind of image metadata thing + minimum libvirt version check in the driver? | 02:44 |
mriedem | although, if it's not in libvirt we can't even check a minimum version | 02:44 |
*** Daisy has quit IRC | 02:44 | |
alex_xu | mriedem: yes, but I think the hard thing is how much detail expose to the user, I guess we may not hope we let the user choice 20MB or 30MB l3 cache they need | 02:44 |
mriedem | that seems like flavor extra specs | 02:45 |
mriedem | like how qos is handled | 02:45 |
alex_xu | maybe user just need to specify gold or liver | 02:45 |
alex_xu | yea | 02:45 |
alex_xu | s/liver/sliver/ | 02:45 |
*** Daisy has joined #openstack-nova | 02:46 | |
mriedem | there is a joke in there somewhere about a liver-flavored flavor | 02:47 |
alex_xu | I guess we didn't want to add more thing to the extra spec, thinking of whether we can match the propose into resource provider | 02:48 |
*** thorst_ has quit IRC | 02:48 | |
*** Daisy has quit IRC | 02:48 | |
* alex_xu try to understand liver-flavored... | 02:48 | |
*** kevinz has quit IRC | 02:48 | |
*** thorst_ has joined #openstack-nova | 02:48 | |
*** kevinz has joined #openstack-nova | 02:49 | |
*** zhurong has joined #openstack-nova | 02:52 | |
*** ijw has quit IRC | 02:52 | |
*** thorst_ has quit IRC | 02:53 | |
*** harlowja has quit IRC | 02:53 | |
*** claudiub has quit IRC | 02:54 | |
*** lucas_ has quit IRC | 02:54 | |
*** zhangqiankun has joined #openstack-nova | 02:54 | |
zhangqiankun | I want to use guacamole connection openstack create vm for vnc,what can i do? | 02:57 |
*** zhurong has quit IRC | 02:59 | |
*** sudipto_ has joined #openstack-nova | 03:02 | |
*** sudipto has joined #openstack-nova | 03:02 | |
*** ducttape_ has joined #openstack-nova | 03:04 | |
mriedem | alex_xu: liver-flavored means something that tastes like liver | 03:05 |
mriedem | alex_xu: so a liver-flavored flavor, is a joke :) | 03:05 |
*** Daisy has joined #openstack-nova | 03:06 | |
mriedem | alex_xu: i think with traits we'd expose the libvirt version, so a request spec could require a certain minimum version of the hypervisor for this capability, or the libvirt driver could just say it has this capability and we decouple it from the version in the request spec | 03:07 |
mriedem | but i would think you would still need something in the flavor extra specs to tell what size of cache is being requested | 03:07 |
*** amotoki_ has quit IRC | 03:08 | |
*** thorst_ has joined #openstack-nova | 03:09 | |
*** thorst_ has quit IRC | 03:09 | |
alex_xu | mriedem: maybe we can create ResourceClass called l3_cache, then track the usage of cache in the scheduler? | 03:09 |
alex_xu | at least we need figure which host can have enough cache, or need some ways to balance the cache | 03:10 |
*** gongysh has joined #openstack-nova | 03:10 | |
alex_xu | emm...whatever there should be somewhere talk about the size | 03:11 |
alex_xu | or gold/sliver stuff | 03:11 |
alex_xu | half years ago, we have initial thought using extra specs, and think about shared/isolate cache, and maybe need to bind with NUMA node. maybe that is over complex | 03:12 |
alex_xu | try to rethink of all of those, try to find something more easy | 03:12 |
*** amotoki has joined #openstack-nova | 03:13 | |
alex_xu | in China, there are a lot of food made by liver...I guess we may have different understanding of the taste of liver... | 03:13 |
alex_xu | I remember people won't expose libvirt version into the traits, but the libvirt version will answer the question of capabilitis API | 03:18 |
*** huanxie has joined #openstack-nova | 03:20 | |
*** Daisy has quit IRC | 03:24 | |
*** ssurana has quit IRC | 03:28 | |
*** nicolasbock has quit IRC | 03:28 | |
*** ducttape_ has quit IRC | 03:28 | |
*** ssurana has joined #openstack-nova | 03:28 | |
*** crushil has joined #openstack-nova | 03:29 | |
*** Daisy has joined #openstack-nova | 03:30 | |
*** edmondsw has joined #openstack-nova | 03:31 | |
*** diga has quit IRC | 03:31 | |
*** lucas_ has joined #openstack-nova | 03:32 | |
*** ssurana has quit IRC | 03:32 | |
*** jose-phillips has joined #openstack-nova | 03:33 | |
*** Daisy has quit IRC | 03:35 | |
*** thorst_ has joined #openstack-nova | 03:35 | |
*** thorst_ has quit IRC | 03:35 | |
*** edmondsw has quit IRC | 03:36 | |
*** Daisy has joined #openstack-nova | 03:36 | |
*** mriedem has quit IRC | 03:39 | |
*** Daisy has quit IRC | 03:41 | |
*** timello has quit IRC | 03:42 | |
*** dave-mccowan has quit IRC | 03:42 | |
*** ducttape_ has joined #openstack-nova | 03:42 | |
*** tovin07 has quit IRC | 03:46 | |
*** Daisy has joined #openstack-nova | 03:46 | |
*** links has joined #openstack-nova | 03:50 | |
*** tovin07 has joined #openstack-nova | 03:51 | |
*** ducttape_ has quit IRC | 03:52 | |
*** timello has joined #openstack-nova | 03:55 | |
*** markvoelker has quit IRC | 03:56 | |
*** dimtruck is now known as zz_dimtruck | 03:56 | |
*** tuanluong has joined #openstack-nova | 03:56 | |
*** markvoelker has joined #openstack-nova | 03:58 | |
*** Daisy has quit IRC | 04:07 | |
*** brault|away has joined #openstack-nova | 04:08 | |
*** hongbin_ has joined #openstack-nova | 04:08 | |
*** nmathew has joined #openstack-nova | 04:09 | |
*** david-lyle has joined #openstack-nova | 04:12 | |
*** brault|away has quit IRC | 04:12 | |
*** jose-phillips has quit IRC | 04:12 | |
*** Daisy has joined #openstack-nova | 04:13 | |
*** psachin has joined #openstack-nova | 04:14 | |
*** sudipto has quit IRC | 04:16 | |
*** sudipto_ has quit IRC | 04:16 | |
*** Daisy has quit IRC | 04:17 | |
*** Daisy has joined #openstack-nova | 04:19 | |
*** sree has joined #openstack-nova | 04:20 | |
*** zz_dimtruck is now known as dimtruck | 04:20 | |
*** ratailor has joined #openstack-nova | 04:22 | |
*** sree_ has joined #openstack-nova | 04:23 | |
*** Daisy has quit IRC | 04:23 | |
*** sree_ is now known as Guest62768 | 04:24 | |
*** sree has quit IRC | 04:24 | |
*** udesale has joined #openstack-nova | 04:24 | |
*** mdnadeem has joined #openstack-nova | 04:26 | |
*** tovin07 has quit IRC | 04:27 | |
*** shaner has quit IRC | 04:28 | |
*** faizy has joined #openstack-nova | 04:28 | |
*** shaner has joined #openstack-nova | 04:29 | |
*** gouthamr has quit IRC | 04:33 | |
*** tuanluong has quit IRC | 04:35 | |
*** lucas_ has quit IRC | 04:37 | |
*** amotoki has quit IRC | 04:42 | |
*** adisky_ has joined #openstack-nova | 04:42 | |
*** sudipto has joined #openstack-nova | 04:42 | |
*** sudipto_ has joined #openstack-nova | 04:42 | |
*** ducttape_ has joined #openstack-nova | 04:43 | |
*** hongbin_ has quit IRC | 04:49 | |
*** gcb has quit IRC | 04:51 | |
*** gcb has joined #openstack-nova | 04:53 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/431906 | 04:53 |
*** gouthamr has joined #openstack-nova | 04:55 | |
*** gouthamr_ has joined #openstack-nova | 04:56 | |
*** yamahata has quit IRC | 04:56 | |
*** dikonoor has joined #openstack-nova | 05:00 | |
*** gouthamr has quit IRC | 05:00 | |
*** esmiurium has quit IRC | 05:01 | |
*** diga has joined #openstack-nova | 05:01 | |
gmann | alex_xu: johnthetubaguy sfinucan it would not raise exception if more than 1 port right ?- https://review.openstack.org/#/c/424931/4/nova/network/neutronv2/api.py@1676 | 05:03 |
gmann | but yea it will return diff port as mentioned by sfinucan | 05:03 |
alex_xu | gmann: johnthetubaguy prefer to not change anything | 05:03 |
alex_xu | actually I both to ok for change or not, that is rare case I think | 05:04 |
gmann | alex_xu: yea that we should keep same and may be previous logic in PS2 is good ? | 05:04 |
*** ducttape_ has quit IRC | 05:05 | |
alex_xu | gmann: PS2 is same logic with current one I think? | 05:05 |
*** esmiurium has joined #openstack-nova | 05:05 | |
alex_xu | both of them stop the loop when a port found? | 05:05 |
gmann | alex_xu: yea, that also stop on first finding. | 05:07 |
gmann | got it. thanks :) | 05:07 |
alex_xu | gmann: np :) | 05:09 |
alex_xu | gmann: are you going to PTG? | 05:09 |
gmann | alex_xu: yea, but need to book flight today. | 05:10 |
gmann | alex_xu: you will be there from monday or Wed ? | 05:11 |
alex_xu | gmann: I will be there Monday | 05:11 |
alex_xu | gmann: looking for see you there | 05:11 |
gmann | alex_xu: nice, you can join QA on Monday and Tuesday :) | 05:11 |
alex_xu | gmann: yeah, i'm just worry what I should do in Monday and Tuesday....since i already booked the flight... | 05:12 |
*** mlakat has quit IRC | 05:12 | |
*** mlakat has joined #openstack-nova | 05:13 | |
gmann | alex_xu: nice, you are welcome to join QA, I have put topic of compute microversion testing - https://etherpad.openstack.org/p/qa-ptg-pike | 05:13 |
gmann | L63 | 05:14 |
alex_xu | gmann: thanks :) | 05:14 |
*** prateek has joined #openstack-nova | 05:15 | |
gmann | alex_xu: also to dicsuss to run all compute tests with 'latest' microversion so that we can find bugs in advance like tag one | 05:15 |
gmann | but in experimental job or something as all tempest compute tests would not pass with 'latest' | 05:15 |
alex_xu | gmann: that's nice | 05:16 |
alex_xu | acutally bug only can be found, there is another bump | 05:16 |
alex_xu | but that is enough | 05:16 |
alex_xu | i'm still plan to take a look at what we can do in nova side | 05:16 |
alex_xu | gmann: or you think tempest is enough to help that? | 05:17 |
gmann | yea that is also required | 05:17 |
gmann | alex_xu: nova one is always needed to have close eyes, | 05:17 |
alex_xu | gmann: got it | 05:17 |
gmann | alex_xu: something like this, we planned earlier - https://blueprints.launchpad.net/nova/+spec/nova-microversion-functional-tests | 05:18 |
*** abalutoiu has joined #openstack-nova | 05:18 | |
alex_xu | gmann: sounds like we already talk about that long time ago... | 05:19 |
gmann | yea but did not get time, may be good to do in pike | 05:19 |
alex_xu | gmann: yea, maybe check with johnthetubaguy and sdague in next meeting | 05:20 |
gmann | yea | 05:20 |
gmann | alex_xu: want to discuss 1 more thing, abort cold migration - https://review.openstack.org/#/c/334732/ | 05:21 |
gmann | alex_xu: can we take this for Pike if so I can add on etherpad | 05:21 |
alex_xu | gmann: I guess that won't be too hard to get re-approve | 05:21 |
gmann | yea | 05:22 |
*** amotoki has joined #openstack-nova | 05:22 | |
alex_xu | gmann: and the code already in the gerrit, I think it isn't a problem | 05:22 |
gmann | and nice feature, in NEC also we need that as live migration have capability to abort | 05:22 |
gmann | ok, ll put on etherpad | 05:22 |
alex_xu | gmann: cool, I will reorganize the etherpad later, then raise that in next meeting | 05:24 |
gmann | alex_xu: cool, thanks | 05:24 |
alex_xu | gmann: np | 05:24 |
*** amotoki has quit IRC | 05:24 | |
*** amotoki has joined #openstack-nova | 05:25 | |
*** david-lyle has quit IRC | 05:27 | |
*** gcb has quit IRC | 05:29 | |
*** gcb has joined #openstack-nova | 05:30 | |
*** satyar has joined #openstack-nova | 05:36 | |
*** thorst_ has joined #openstack-nova | 05:36 | |
*** owalsh_ has joined #openstack-nova | 05:38 | |
*** owalsh has quit IRC | 05:39 | |
*** thorst_ has quit IRC | 05:41 | |
*** lpetrut has joined #openstack-nova | 05:41 | |
*** salv-orlando has joined #openstack-nova | 05:44 | |
*** avolkov has joined #openstack-nova | 05:46 | |
*** zerda3 has quit IRC | 05:49 | |
*** salv-orlando has quit IRC | 05:50 | |
*** unicell has joined #openstack-nova | 05:50 | |
*** zerda3 has joined #openstack-nova | 05:51 | |
*** gouthamr_ has quit IRC | 05:52 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/432001 | 05:55 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/395347 | 05:57 |
*** lucas_ has joined #openstack-nova | 05:57 | |
*** trinaths has joined #openstack-nova | 05:58 | |
*** unicell1 has joined #openstack-nova | 05:59 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/431906 | 05:59 |
*** unicell has quit IRC | 05:59 | |
*** jdurgin1 has joined #openstack-nova | 06:07 | |
*** Jack_I has joined #openstack-nova | 06:11 | |
*** tiendc has joined #openstack-nova | 06:11 | |
*** armax has quit IRC | 06:15 | |
*** armax has joined #openstack-nova | 06:16 | |
*** armax has quit IRC | 06:16 | |
*** litao has quit IRC | 06:16 | |
*** litao has joined #openstack-nova | 06:17 | |
*** gcb has quit IRC | 06:18 | |
*** gcb has joined #openstack-nova | 06:18 | |
*** ayogi has joined #openstack-nova | 06:19 | |
*** lucas_ has quit IRC | 06:23 | |
*** sacharya has quit IRC | 06:24 | |
*** phuongnh has quit IRC | 06:25 | |
*** takashin has left #openstack-nova | 06:27 | |
*** crushil has quit IRC | 06:28 | |
*** vsaienko has joined #openstack-nova | 06:28 | |
*** sridharg has joined #openstack-nova | 06:29 | |
*** Daisy has joined #openstack-nova | 06:36 | |
*** Daisy has quit IRC | 06:41 | |
*** Daisy has joined #openstack-nova | 06:42 | |
*** salv-orlando has joined #openstack-nova | 06:46 | |
*** Daisy has quit IRC | 06:47 | |
*** lpetrut has quit IRC | 06:48 | |
*** Daisy has joined #openstack-nova | 06:48 | |
*** lpetrut has joined #openstack-nova | 06:50 | |
*** salv-orlando has quit IRC | 06:50 | |
*** Jinxing has joined #openstack-nova | 06:53 | |
*** Daisy has quit IRC | 06:55 | |
*** Daisy has joined #openstack-nova | 06:56 | |
*** mtanino has quit IRC | 07:00 | |
*** sandanar has joined #openstack-nova | 07:00 | |
*** Daisy has quit IRC | 07:01 | |
*** salv-orlando has joined #openstack-nova | 07:02 | |
*** markvoelker has quit IRC | 07:02 | |
*** ltomasbo|away is now known as ltomasbo | 07:02 | |
*** markvoelker has joined #openstack-nova | 07:02 | |
*** Daisy has joined #openstack-nova | 07:02 | |
*** rcernin has joined #openstack-nova | 07:03 | |
*** andreas_s has joined #openstack-nova | 07:04 | |
*** ducttape_ has joined #openstack-nova | 07:05 | |
*** markvoelker has quit IRC | 07:06 | |
*** cfriesen has quit IRC | 07:07 | |
*** edmondsw has joined #openstack-nova | 07:07 | |
*** Daisy has quit IRC | 07:07 | |
*** lpetrut has quit IRC | 07:09 | |
*** tiendc has quit IRC | 07:11 | |
*** edmondsw has quit IRC | 07:11 | |
*** ociuhandu has joined #openstack-nova | 07:11 | |
*** ducttape_ has quit IRC | 07:11 | |
*** ociuhandu has quit IRC | 07:16 | |
*** lpetrut has joined #openstack-nova | 07:20 | |
*** nkrinner_afk is now known as nkrinner | 07:22 | |
*** vsaienko has quit IRC | 07:22 | |
*** faizy has quit IRC | 07:28 | |
*** Jeffrey4l__ has joined #openstack-nova | 07:29 | |
*** pcaruana has joined #openstack-nova | 07:30 | |
*** Jeffrey4l_ has quit IRC | 07:30 | |
*** faizy has joined #openstack-nova | 07:31 | |
*** fragatina has quit IRC | 07:36 | |
*** brault|away has joined #openstack-nova | 07:36 | |
*** thorst_ has joined #openstack-nova | 07:37 | |
openstackgerrit | Tomi Juvonen proposed openstack/nova-specs master: Add Host Details https://review.openstack.org/428070 | 07:37 |
*** lpetrut has quit IRC | 07:40 | |
*** mnaser has quit IRC | 07:41 | |
*** thorst_ has quit IRC | 07:42 | |
*** afazekas has quit IRC | 07:43 | |
*** lpetrut has joined #openstack-nova | 07:44 | |
*** tiendc has joined #openstack-nova | 07:45 | |
*** tuanluong has joined #openstack-nova | 07:45 | |
*** foexle has joined #openstack-nova | 07:45 | |
*** slaweq has quit IRC | 07:48 | |
*** lpetrut has quit IRC | 07:49 | |
*** yamahata has joined #openstack-nova | 07:51 | |
*** slaweq has joined #openstack-nova | 07:54 | |
*** brault|away has quit IRC | 07:56 | |
*** markus_z has joined #openstack-nova | 07:57 | |
*** snikitin has quit IRC | 07:59 | |
*** Daisy has joined #openstack-nova | 08:00 | |
*** faizy_ has joined #openstack-nova | 08:02 | |
*** markvoelker has joined #openstack-nova | 08:03 | |
*** faizy has quit IRC | 08:05 | |
*** faizy__ has joined #openstack-nova | 08:05 | |
*** Daisy has quit IRC | 08:08 | |
*** markvoelker has quit IRC | 08:09 | |
*** faizy_ has quit IRC | 08:09 | |
*** Daisy has joined #openstack-nova | 08:09 | |
*** faizy has joined #openstack-nova | 08:09 | |
*** faizy__ has quit IRC | 08:12 | |
*** amoralej|off is now known as amoralej | 08:13 | |
*** anilvenkata has joined #openstack-nova | 08:17 | |
anilvenkata | Hi, need help. Trying to setup devstack multi node. I am getting this nova error in compute node http://paste.openstack.org/show/598346/ | 08:19 |
*** unicell1 has quit IRC | 08:19 | |
*** brault|away has joined #openstack-nova | 08:19 | |
*** Daisy has quit IRC | 08:19 | |
anilvenkata | services I have enabled in compute node are ENABLED_SERVICES=n-cpu,n-api-meta,c-vol | 08:19 |
*** Jeffrey4l__ has quit IRC | 08:21 | |
*** Jeffrey4l__ has joined #openstack-nova | 08:22 | |
*** yamahata has quit IRC | 08:24 | |
*** mptacekx has joined #openstack-nova | 08:28 | |
*** dimtruck is now known as zz_dimtruck | 08:30 | |
*** foexle has quit IRC | 08:33 | |
*** fragatina has joined #openstack-nova | 08:46 | |
*** faizy has quit IRC | 08:46 | |
*** faizy has joined #openstack-nova | 08:47 | |
*** fragatina has quit IRC | 08:51 | |
*** ralonsoh has joined #openstack-nova | 08:51 | |
*** litao has quit IRC | 08:52 | |
*** salv-orlando has quit IRC | 08:53 | |
*** mjura has joined #openstack-nova | 08:58 | |
*** zzzeek has quit IRC | 09:00 | |
*** jpena|away is now known as jpena | 09:01 | |
*** amotoki has quit IRC | 09:01 | |
*** zzzeek has joined #openstack-nova | 09:03 | |
*** salv-orlando has joined #openstack-nova | 09:06 | |
*** ygl has joined #openstack-nova | 09:06 | |
bhagyashris | anilvenkata: Hi, enabled the placement service on your compute node by adding the [placement] section in compute node nova.conf. Means simply copy the [placement] section from the /etc/nova/nova.conf from Controller + compute node in to the compute node's /etc/nova/nova.conf | 09:07 |
*** derekh has joined #openstack-nova | 09:07 | |
bhagyashris | anilvenkata: and restart the n-cpu service | 09:08 |
*** ducttape_ has joined #openstack-nova | 09:08 | |
ygl | Hi all | 09:08 |
ygl | i have installed devstack on ubuntu | 09:08 |
ygl | but I am unable to access horizon | 09:09 |
ygl | i am following this guide | 09:09 |
ygl | http://docs.openstack.org/developer/devstack/guides/single-machine.html | 09:09 |
ygl | can anyone help me please | 09:09 |
*** zz_dimtruck is now known as dimtruck | 09:09 | |
*** ducttape_ has quit IRC | 09:13 | |
*** lpetrut has joined #openstack-nova | 09:14 | |
*** lpetrut1 has joined #openstack-nova | 09:17 | |
*** dimtruck is now known as zz_dimtruck | 09:18 | |
*** lpetrut has quit IRC | 09:19 | |
*** lpetrut1 is now known as lpetrut | 09:19 | |
*** claudiub has joined #openstack-nova | 09:23 | |
robcresswell | ygl: You should try #openstack This channel is for Nova devs. | 09:24 |
*** snikitin has joined #openstack-nova | 09:27 | |
*** lpetrut has quit IRC | 09:32 | |
anilvenkata | bhagyashris, thanks Bhagyashri, I will try | 09:33 |
bhagyashris | anilvenkata: refer the mailing thread update regarding the placement service : http://lists.openstack.org/pipermail/openstack-dev/2017-January/111295.html | 09:37 |
anilvenkata | bhagyashris, thanks a lot, it is working, also able to create instance | 09:37 |
bhagyashris | anilvenkata: Welcome :) | 09:38 |
*** thorst_ has joined #openstack-nova | 09:38 | |
*** lucas-afk is now known as lucasagomes | 09:38 | |
*** cloud-liang has joined #openstack-nova | 09:39 | |
*** faizy_ has joined #openstack-nova | 09:42 | |
*** thorst_ has quit IRC | 09:43 | |
*** salv-orlando has quit IRC | 09:45 | |
*** faizy has quit IRC | 09:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/432001 | 09:46 |
*** fragatina has joined #openstack-nova | 09:49 | |
*** fragatina has quit IRC | 09:55 | |
*** ygl has quit IRC | 09:59 | |
*** yingjun has quit IRC | 09:59 | |
*** iceyao has quit IRC | 10:03 | |
*** markvoelker has joined #openstack-nova | 10:03 | |
*** satyar has quit IRC | 10:05 | |
*** cloud-liang has quit IRC | 10:06 | |
*** zz_dimtruck is now known as dimtruck | 10:09 | |
*** mkoderer has joined #openstack-nova | 10:09 | |
*** kevinz has quit IRC | 10:10 | |
*** lpetrut has joined #openstack-nova | 10:15 | |
*** openstackgerrit has quit IRC | 10:18 | |
*** openstackgerrit has joined #openstack-nova | 10:18 | |
openstackgerrit | John Garbutt proposed openstack/nova master: Stop leaking ports on port_create timeout https://review.openstack.org/347183 | 10:18 |
*** dimtruck is now known as zz_dimtruck | 10:19 | |
*** liusheng has quit IRC | 10:20 | |
*** liusheng has joined #openstack-nova | 10:20 | |
*** alex_xu has quit IRC | 10:21 | |
*** Jinxing has quit IRC | 10:21 | |
frickler | is there a description somewhere of the difference between nova-api-os-compute and nova-api services? | 10:26 |
*** alex_xu has joined #openstack-nova | 10:26 | |
*** sambetts|afk is now known as sambetts | 10:30 | |
*** udesale has quit IRC | 10:33 | |
*** ociuhandu has joined #openstack-nova | 10:35 | |
*** gszasz has joined #openstack-nova | 10:41 | |
*** owalsh_ is now known as owalsh | 10:43 | |
*** edmondsw has joined #openstack-nova | 10:43 | |
*** tuanluong has quit IRC | 10:44 | |
*** hoangcx has quit IRC | 10:46 | |
*** edmondsw has quit IRC | 10:48 | |
*** tiendc has quit IRC | 10:48 | |
*** rfolco has joined #openstack-nova | 10:57 | |
johnthetubaguy | frickler: are they not two names for the same thing? | 10:58 |
*** Jeffrey4l__ has quit IRC | 11:00 | |
*** Jeffrey4l__ has joined #openstack-nova | 11:01 | |
*** nicolasbock has joined #openstack-nova | 11:05 | |
*** prateek has quit IRC | 11:06 | |
*** ducttape_ has joined #openstack-nova | 11:09 | |
*** zz_dimtruck is now known as dimtruck | 11:10 | |
*** ducttape_ has quit IRC | 11:14 | |
*** salv-orlando has joined #openstack-nova | 11:16 | |
*** alex_xu has quit IRC | 11:18 | |
frickler | johnthetubaguy: I'm not sure, that is why I'm asking. Used to always deploy nova-api-os-compute and now found out that the Newton installation guide asks to install nova-api instead. and if I install that pkg on ubuntu, it will remove all these: nova-api-metadata nova-api-os-compute nova-placement-ap | 11:19 |
*** faizy_ has quit IRC | 11:20 | |
*** dimtruck is now known as zz_dimtruck | 11:20 | |
*** salv-orlando has quit IRC | 11:20 | |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Fix additional mistakes in the NUMA spec document https://review.openstack.org/323252 | 11:21 |
*** alex_xu has joined #openstack-nova | 11:22 | |
*** trinaths has quit IRC | 11:23 | |
*** mjura has quit IRC | 11:24 | |
lyarwood | any cores able to review https://review.openstack.org/#/c/431595/ on stable/ocata for mriedem so I can then ack the stable/newton backport? | 11:24 |
*** smatzek has joined #openstack-nova | 11:27 | |
*** diga has quit IRC | 11:31 | |
*** tbachman has quit IRC | 11:34 | |
*** thorst_ has joined #openstack-nova | 11:39 | |
*** thorst_ has quit IRC | 11:43 | |
*** sdague has joined #openstack-nova | 11:46 | |
*** yingjun has joined #openstack-nova | 11:47 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Use min parameter to restrict live-migration config options https://review.openstack.org/323313 | 11:48 |
*** tesseract has joined #openstack-nova | 11:50 | |
*** vsaienko has joined #openstack-nova | 11:54 | |
*** bkopilov has quit IRC | 11:55 | |
*** bkopilov_ has quit IRC | 11:55 | |
*** owalsh is now known as owalsh-afk | 11:57 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova master: Remove usage of config option verbose https://review.openstack.org/432271 | 11:57 |
*** wllabs has quit IRC | 11:57 | |
*** nkrinner has quit IRC | 11:58 | |
*** yingjun has quit IRC | 12:00 | |
*** nkrinner has joined #openstack-nova | 12:01 | |
*** panda|zZ is now known as panda | 12:04 | |
*** satyar has joined #openstack-nova | 12:05 | |
openstackgerrit | John Garbutt proposed openstack/nova master: Make it easier to avoid live-migration timeouts https://review.openstack.org/408002 | 12:05 |
*** dave-mccowan has joined #openstack-nova | 12:08 | |
*** cdelatte has joined #openstack-nova | 12:08 | |
openstackgerrit | keerthivasan selvaraj proposed openstack/nova master: Getting cpu_allocation_ratio from database https://review.openstack.org/432274 | 12:09 |
*** sudipto has quit IRC | 12:09 | |
*** sudipto_ has quit IRC | 12:09 | |
*** fragatina has joined #openstack-nova | 12:10 | |
openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Make it easier to avoid live-migration timeouts https://review.openstack.org/408002 | 12:10 |
*** bkopilov has joined #openstack-nova | 12:10 | |
*** zz_dimtruck is now known as dimtruck | 12:11 | |
*** bkopilov_ has joined #openstack-nova | 12:11 | |
*** fragatina has quit IRC | 12:14 | |
johnthetubaguy | frickler: oh, I am unsure whats in all the ubuntu packages I am afraid. I would ask in an ubuntu channel | 12:16 |
sdague | johnthetubaguy: are you going to hit the newton/mitaka versions of that same patch? | 12:17 |
*** salv-orlando has joined #openstack-nova | 12:17 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'shell' commands https://review.openstack.org/432280 | 12:18 |
*** dimtruck is now known as zz_dimtruck | 12:20 | |
*** salv-orlando has quit IRC | 12:21 | |
*** ducttape_ has joined #openstack-nova | 12:23 | |
*** salv-orlando has joined #openstack-nova | 12:25 | |
*** faizy_ has joined #openstack-nova | 12:31 | |
*** edmondsw has joined #openstack-nova | 12:32 | |
*** dave-mccowan has quit IRC | 12:32 | |
*** gszasz has quit IRC | 12:32 | |
*** dave-mccowan has joined #openstack-nova | 12:33 | |
*** thorst_ has joined #openstack-nova | 12:33 | |
*** bvanhav_ has joined #openstack-nova | 12:35 | |
*** edmondsw has quit IRC | 12:36 | |
*** gongysh has quit IRC | 12:37 | |
*** carthaca_ has quit IRC | 12:38 | |
*** jpena is now known as jpena|lunch | 12:38 | |
*** carthaca_ has joined #openstack-nova | 12:39 | |
*** vsaienko has quit IRC | 12:40 | |
*** jamesdenton has joined #openstack-nova | 12:40 | |
*** jamesden_ has joined #openstack-nova | 12:45 | |
*** markvoelker has quit IRC | 12:46 | |
*** markvoelker has joined #openstack-nova | 12:46 | |
*** jamesdenton has quit IRC | 12:46 | |
*** jamesden_ has quit IRC | 12:52 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/432001 | 12:52 |
*** mdnadeem has quit IRC | 12:53 | |
*** gcb has quit IRC | 12:55 | |
openstackgerrit | John Garbutt proposed openstack/nova master: Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259 | 12:55 |
*** ayogi has quit IRC | 12:55 | |
*** ociuhandu has quit IRC | 12:56 | |
*** faizy_ has quit IRC | 12:56 | |
*** panda is now known as panda|afk | 12:58 | |
*** vsaienko has joined #openstack-nova | 12:59 | |
*** salv-orl_ has joined #openstack-nova | 12:59 | |
frickler | has anyone seen this py35 error in novnc testing? seems to happen since about a week http://logs.openstack.org/67/351367/11/check/gate-tempest-dsvm-py35-ubuntu-xenial-nv/fa03be0/console.html.gz#_2017-02-03_18_25_10_353578 | 13:00 |
frickler | johnthetubaguy: thx, I'll try to walk through the devstack steps from http://docs.openstack.org/developer/nova/placement.html#id1 and see if that helps | 13:02 |
*** salv-orlando has quit IRC | 13:02 | |
*** Dinesh_Bhor has quit IRC | 13:04 | |
*** owalsh-afk has quit IRC | 13:04 | |
*** ZZelle has joined #openstack-nova | 13:05 | |
*** ZZelle has quit IRC | 13:06 | |
*** edmondsw has joined #openstack-nova | 13:06 | |
*** mtanino has joined #openstack-nova | 13:10 | |
*** ZZelle_ has joined #openstack-nova | 13:11 | |
snikitin | sfinucan: Hi! If you have a free time can you please look at PCI-NUMA affinity spec? https://review.openstack.org/#/c/361140 | 13:11 |
*** zz_dimtruck is now known as dimtruck | 13:12 | |
*** Guest21681 has joined #openstack-nova | 13:12 | |
*** lucasagomes is now known as lucas-hungry | 13:12 | |
*** zzelle__ has joined #openstack-nova | 13:14 | |
*** Guest21681 has quit IRC | 13:14 | |
*** zzelle__ has quit IRC | 13:14 | |
*** owalsh-afk has joined #openstack-nova | 13:15 | |
*** owalsh-afk is now known as owalsh | 13:15 | |
*** gszasz has joined #openstack-nova | 13:15 | |
*** ZZelle__ has joined #openstack-nova | 13:16 | |
*** ZZelle_ has quit IRC | 13:17 | |
*** mkoderer has quit IRC | 13:17 | |
*** ducttape_ has quit IRC | 13:18 | |
*** faizy has joined #openstack-nova | 13:19 | |
*** ZZelle__ has quit IRC | 13:20 | |
*** ZZelle_ has joined #openstack-nova | 13:20 | |
*** ZZelle_ has quit IRC | 13:20 | |
*** ZZelle_ has joined #openstack-nova | 13:21 | |
*** ZZelle_ is now known as Guest90071 | 13:21 | |
*** dimtruck is now known as zz_dimtruck | 13:21 | |
*** diana_clarke has joined #openstack-nova | 13:22 | |
*** panda|afk is now known as panda | 13:23 | |
*** Guest90071 has quit IRC | 13:26 | |
*** ZZelle__ has joined #openstack-nova | 13:26 | |
*** ZZelle__ is now known as idalgo | 13:30 | |
*** idalgo is now known as zzelle_hello | 13:30 | |
sfinucan | snikitin: Sure. I was waiting on your replies to sgordon before weighing in :) | 13:31 |
*** mtanino has quit IRC | 13:35 | |
*** tbachman has joined #openstack-nova | 13:36 | |
*** baoli has joined #openstack-nova | 13:37 | |
*** tblakes has joined #openstack-nova | 13:37 | |
*** baoli has quit IRC | 13:39 | |
*** baoli has joined #openstack-nova | 13:40 | |
snikitin | sfinucan, Oh. Ok. I'll add my comments | 13:44 |
*** jpena|lunch is now known as jpena | 13:47 | |
*** mriedem has joined #openstack-nova | 13:50 | |
*** zz_dimtruck is now known as dimtruck | 13:53 | |
*** links has quit IRC | 13:55 | |
*** bvanhav__ has joined #openstack-nova | 13:55 | |
*** bvanhav__ is now known as bvanhav | 13:55 | |
*** Guest62768 has quit IRC | 13:56 | |
*** gouthamr has joined #openstack-nova | 13:56 | |
*** bvanhav_ has quit IRC | 13:57 | |
*** ducttape_ has joined #openstack-nova | 13:57 | |
*** ratailor has quit IRC | 14:00 | |
diana_clarke | hmm... on devstack stable/ocata I'm seeing warnings when I do nova commands: http://paste.openstack.org/show/598407/ | 14:01 |
*** gokhan has quit IRC | 14:01 | |
mriedem | diana_clarke: that's from novaclient 7.0 | 14:02 |
mriedem | diana_clarke: does your localrc have OS_TENANT_ID set instead of OS_PROJECT_ID? | 14:03 |
mriedem | i think that's what it's complaining about | 14:03 |
diana_clarke | mriedem: maybe, I'll so check after my meeting (that machine is downstairs) | 14:03 |
*** READ10 has joined #openstack-nova | 14:03 | |
*** ratailor has joined #openstack-nova | 14:04 | |
diana_clarke | mriedem: I didn't change anything though. I just sourced openrc like I always have. | 14:06 |
*** ociuhandu has joined #openstack-nova | 14:06 | |
mriedem | diana_clarke: sure, but novaclient changed | 14:06 |
mriedem | in 7.0 it wants project_id instead of tenant_id | 14:06 |
diana_clarke | mriedem: Yup, I reviewed those changes, mostly just wondering if we need to change something though so we don't see those warnings by default. | 14:07 |
kashyap | Hmm, just looking at this spec ("Re-Propose Expose quiesce/unquiesce API"), it seems to be approved for Mitaka, but not for master (with some rationale, & strong commentary from mriedem) | 14:08 |
kashyap | https://review.openstack.org/#/c/295595/ | 14:08 |
kashyap | I can't find the approved Mitaka Gerrit review for historical discussion | 14:08 |
mriedem | diana_clarke: i think you would need to replace OS_TENANT_ID with OS_PROJECT_ID in your openrc file | 14:08 |
mriedem | diana_clarke: if you're using the openrc from devstack, it might just be something we need to fix in devstack | 14:08 |
diana_clarke | mriedem: exactly, I'm using the openrc from devstack, so perhaps it needs to be fixed. | 14:09 |
*** vsaienko has quit IRC | 14:09 | |
mriedem | kashyap: http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/expose-quiesce-unquiesce-api.html | 14:09 |
*** adisky_ has quit IRC | 14:09 | |
kashyap | mriedem: Yeah, I saw that, but the Gerrit discussion itself. Maybe it's linked in there | 14:09 |
* kashyap looks | 14:09 | |
*** sudipto has joined #openstack-nova | 14:10 | |
*** sudipto_ has joined #openstack-nova | 14:10 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Enable global hacking checks and removed local checks https://review.openstack.org/431030 | 14:10 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Update hacking version https://review.openstack.org/389338 | 14:10 |
kashyap | No, it isn't. I only see the other, related spec that was approved https://review.openstack.org/#/c/126966/ ("Quiesce filesystems with QEMU guest agent during image snapshot") | 14:10 |
*** mdrabe has joined #openstack-nova | 14:10 | |
mriedem | kahshttps://review.openstack.org/#/c/237881/ | 14:11 |
mriedem | kashyap: ^ | 14:11 |
*** fragatina has joined #openstack-nova | 14:11 | |
* kashyap clicks | 14:11 | |
kashyap | mriedem: Perfect! How did you find it? | 14:12 |
gabor_antal | johnthetubaguy: can you please give your +2 back? I had to rebase https://review.openstack.org/#/c/431030 | 14:12 |
mriedem | kashyap: looked up the hash on the file in the nova-specs repo | 14:12 |
*** ducttape_ has quit IRC | 14:12 | |
*** vsaienko has joined #openstack-nova | 14:12 | |
*** jamesdenton has joined #openstack-nova | 14:13 | |
* kashyap nods, thanks. | 14:13 | |
gabor_antal | Vek: could please give your +2 back too? https://review.openstack.org/#/c/431030 | 14:13 |
*** fragatina has quit IRC | 14:16 | |
*** lucas-hungry is now known as lucasagomes | 14:17 | |
*** READ10 has quit IRC | 14:18 | |
*** amoralej is now known as amoralej|lunch | 14:19 | |
*** READ10 has joined #openstack-nova | 14:23 | |
*** jamesdenton has quit IRC | 14:23 | |
*** jamesdenton has joined #openstack-nova | 14:24 | |
*** sandanar has quit IRC | 14:24 | |
*** ratailor has quit IRC | 14:25 | |
lyarwood | mriedem: just reading over the notes from the nova meeting yesterday that I missed thanks to my ISP having an outage. Do you think you will have time next week to cut a stable/newton release before the PTG? | 14:29 |
*** dansmith is now known as superdan | 14:29 | |
mriedem | lyarwood: hopefully | 14:30 |
lyarwood | mriedem: cool, I'm out Monday but if there's anything I can do to help that long just let me know | 14:31 |
lyarwood | along* | 14:31 |
*** smatzek has quit IRC | 14:31 | |
mriedem | will do | 14:31 |
*** jose-phillips has joined #openstack-nova | 14:33 | |
jroll | mriedem: I think we'll have time for this in pike, if you want to approve it. made decent progress on the ironic side in ocata. https://blueprints.launchpad.net/nova/+spec/ironic-boot-from-volume | 14:35 |
mriedem | superdan: want to hit this ocata backport and the one after it? https://review.openstack.org/#/c/430306/ | 14:35 |
superdan | yup | 14:35 |
*** jaypipes is now known as leakypipes | 14:36 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Initiate network changes from source during live migration https://review.openstack.org/404607 | 14:36 |
*** dikonoor has quit IRC | 14:36 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs master: Send service token along with user tokens https://review.openstack.org/426933 | 14:38 |
*** rfolco_ has joined #openstack-nova | 14:39 | |
*** rfolco has quit IRC | 14:40 | |
*** ducttape_ has joined #openstack-nova | 14:40 | |
mriedem | jroll: done | 14:40 |
mriedem | johnthetubaguy: you can drop the -2 on this now https://review.openstack.org/#/c/215385/ | 14:41 |
johnthetubaguy | mriedem: jroll: -2 removed | 14:41 |
jroll | mriedem: johnthetubaguy: you rock, thanks | 14:42 |
mriedem | lyarwood: i think i'm going to hold https://review.openstack.org/#/c/431530/ out of rc2 since it's latent | 14:43 |
mriedem | lyarwood: but good if that fixes it, i remember takashi tried to fix that and we had to revert his fix | 14:43 |
*** CristinaPauna has quit IRC | 14:43 | |
*** nmathew has quit IRC | 14:44 | |
lyarwood | mriedem: yup np and thanks, hopefully that's enough this time. https://review.openstack.org/#/c/427808/ hasn't landed yet btw, I'll try to get another tempest core to ack now. | 14:45 |
*** CristinaPauna has joined #openstack-nova | 14:46 | |
mriedem | lyarwood: well.... | 14:46 |
mriedem | oh, | 14:46 |
mriedem | i guess we only enable that test starting in ocata | 14:46 |
*** amoralej|lunch is now known as amoralej | 14:46 | |
mriedem | i was going to say, you'd break stable/mitaka, but we don't run the test there | 14:46 |
*** cfriesen has joined #openstack-nova | 14:47 | |
*** mtanino has joined #openstack-nova | 14:47 | |
lyarwood | mriedem: yup, that reminds me I still haven't landed https://review.openstack.org/#/c/427602/ | 14:49 |
mriedem | ah so that's why the ceph job has been red for weeks :) | 14:50 |
mriedem | good thing we made it non-voting for other brokenness before that | 14:50 |
*** ducttape_ has quit IRC | 14:51 | |
lyarwood | yeah sorry about that, the lack of fire from the job no longer voting means I keep dropping this | 14:52 |
*** ducttape_ has joined #openstack-nova | 14:53 | |
*** udesale has joined #openstack-nova | 14:55 | |
*** smatzek has joined #openstack-nova | 14:56 | |
*** crushil has joined #openstack-nova | 14:58 | |
openstackgerrit | Chris Friesen proposed openstack/nova-specs master: add spec for instance flavor API https://review.openstack.org/431775 | 15:01 |
cfriesen | ^ the "flavor information in nova show output" spec from yesterday's discussion | 15:02 |
*** mlavalle has joined #openstack-nova | 15:02 | |
*** ratailor has joined #openstack-nova | 15:02 | |
mriedem | cfriesen: can you please just cleanup the existing review? | 15:03 |
mriedem | rather than post a new one | 15:03 |
*** david-lyle has joined #openstack-nova | 15:03 | |
mriedem | https://review.openstack.org/#/c/265282 | 15:04 |
mriedem | also, did you know there was a claude friesen? | 15:04 |
*** andreas_s has quit IRC | 15:07 | |
*** dimtruck is now known as zz_dimtruck | 15:08 | |
*** sudipto_ has quit IRC | 15:08 | |
*** sudipto has quit IRC | 15:08 | |
*** mptacekx has quit IRC | 15:09 | |
cfriesen | mriedem: I started by cherry-picking the existing review...oh, did I change the Change-Id maybe? | 15:10 |
*** lucas_ has joined #openstack-nova | 15:10 | |
mriedem | cfriesen: most likely | 15:10 |
mriedem | although cherry-picking should preserve that | 15:10 |
mriedem | i would have just done a git review -d | 15:10 |
mriedem | but either words | 15:10 |
mriedem | *works | 15:10 |
cfriesen | didn't even know about "git review -d". :) | 15:11 |
openstackgerrit | Chris Friesen proposed openstack/nova-specs master: add spec for instance flavor API https://review.openstack.org/265282 | 15:13 |
*** tbachman has quit IRC | 15:13 | |
artom | cfriesen, you're not alone, I was like a year into my Openstack work when I saw a colleague use it | 15:13 |
cfriesen | I cherrypicked and then squashed some stuff on top and forgot to preserve the old change-id. That newer one ^ should be the right one. | 15:14 |
*** faizy_ has joined #openstack-nova | 15:14 | |
*** vsaienko has quit IRC | 15:16 | |
*** bhagyashris has quit IRC | 15:16 | |
mriedem | cool thanks | 15:16 |
mriedem | i'm nowhere near ready to review pike specs yet, | 15:16 |
mriedem | but it's good to get those ducks in a row | 15:16 |
cfriesen | speaking of specs...did anything happen in Ocata for this? https://blueprints.launchpad.net/nova/+spec/memory-bw | 15:17 |
*** faizy has quit IRC | 15:17 | |
*** tbachman has joined #openstack-nova | 15:17 | |
cfriesen | looks like parts of it got merged and parts were abandoned | 15:17 |
openstackgerrit | Silvan Kaiser proposed openstack/nova master: Switch Nova Quobyte volume driver to mount via systemd-run https://review.openstack.org/432344 | 15:20 |
*** jose-phillips has quit IRC | 15:21 | |
*** sacharya has joined #openstack-nova | 15:23 | |
*** CristinaPauna has quit IRC | 15:23 | |
mriedem | first, did we have an ocata midcycle? i'm totally drawing a blank | 15:23 |
mriedem | i guess not | 15:23 |
mriedem | because of the short release | 15:23 |
mriedem | cfriesen: we said at the newton midcycle that we weren't going to do the memory bandwidth tracking stuff in nova | 15:23 |
mriedem | cfriesen: see http://lists.openstack.org/pipermail/openstack-dev/2016-August/100600.html | 15:25 |
*** nkrinner is now known as nkrinner_Afk | 15:25 | |
mriedem | "* Future of compute host metrics monitoring in Nova" | 15:25 |
*** nkrinner_Afk is now known as nkrinner_afk | 15:25 | |
mriedem | http://docs.openstack.org/developer/nova/policies.html?highlight=monitoring#metrics-gathering | 15:25 |
*** sacharya has quit IRC | 15:27 | |
cfriesen | mriedem: thanks for the links | 15:28 |
*** dims_ has quit IRC | 15:28 | |
*** zz_dimtruck is now known as dimtruck | 15:30 | |
cfriesen | mriedem: is there any coalescing around a new standard metrics gathering tool? | 15:32 |
*** sacharya has joined #openstack-nova | 15:33 | |
*** dims has joined #openstack-nova | 15:33 | |
mriedem | cfriesen: not that i'm aware of, | 15:34 |
mriedem | there are lots of metrics gathering tools, | 15:34 |
mriedem | i'm sure there are probably at least 3 competing big tent projects | 15:34 |
*** esberglu has joined #openstack-nova | 15:35 | |
mriedem | cfriesen: might be worth asking the telemetry and/or watcher teams, | 15:36 |
mriedem | edleafe is involved with watcher so he might have some tips | 15:36 |
edleafe | cfriesen: I'd ask in #openstack-watcher | 15:37 |
edleafe | oops! my manners | 15:37 |
*** edleafe is now known as figleaf | 15:37 | |
* cfriesen averts his eyes | 15:37 | |
figleaf | heh | 15:38 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove flavor_* functions from db/api https://review.openstack.org/431752 | 15:40 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove main_db flavor compatibility https://review.openstack.org/431722 | 15:40 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove straggling use of main db flavors in cellsv1 code https://review.openstack.org/432352 | 15:40 |
mriedem | random question - why do we even have this decorator on compute_api.delete when it won't enforce anything? https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2152 | 15:40 |
*** raj_singh_ has joined #openstack-nova | 15:40 | |
*** ralonsoh_ has joined #openstack-nova | 15:40 | |
mriedem | i think everything in here will eval to False https://github.com/openstack/nova/blob/master/nova/compute/api.py#L130 | 15:40 |
mriedem | so we'll never get InstanceInvalidState | 15:40 |
cfriesen | mriedem: is there no way to assert that the vm_state/task_state must be None? | 15:42 |
mriedem | that's not what None means there | 15:42 |
cfriesen | but yeah, that does seem pointless on the delete operation | 15:42 |
mriedem | None for that decorator means, i don't care about vm_state or task_state | 15:42 |
cfriesen | mriedem: yeah, I see that. | 15:42 |
cfriesen | mriedem: but what if I wanted to ensure that it actually was None? | 15:43 |
*** ralonsoh has quit IRC | 15:43 | |
cfriesen | wait, I guess vm_state should be something else | 15:43 |
mriedem | then you pass in (None,) | 15:44 |
*** udesale has quit IRC | 15:44 | |
sfinucan | mriedem: We can't remove the floating IP nova-manage commands until we remove cells v1 and nova-network, correct? https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L305-L308 | 15:45 |
mriedem | sfinucan: corect | 15:45 |
mriedem | *correct | 15:45 |
mriedem | i already tried | 15:45 |
mriedem | and gave up | 15:45 |
mriedem | b/c devstack uses those | 15:46 |
sfinucan | mriedem: aww :( | 15:46 |
cfriesen | mriedem: maybe it serves to assert that we don't care about the state? | 15:46 |
mriedem | cfriesen: then we shouldn't even use the decorator right? | 15:46 |
sfinucan | OK, I'll update that comment to reflect the new reality so | 15:46 |
cfriesen | mriedem: an argument could be made to that effect | 15:46 |
*** sridharg has quit IRC | 15:47 | |
*** mdrabe has quit IRC | 15:48 | |
* leakypipes back in an hour. need to boot into windows to use webex :( | 15:52 | |
*** leakypipes has quit IRC | 15:52 | |
cfriesen | someone should tell leakypipes about VMs. | 15:52 |
*** marst has quit IRC | 15:52 | |
*** crushil has quit IRC | 15:54 | |
*** mdrabe has joined #openstack-nova | 15:55 | |
*** dims has quit IRC | 15:55 | |
*** dims has joined #openstack-nova | 15:55 | |
*** dims is now known as Guest12406 | 15:56 | |
*** ijw has joined #openstack-nova | 15:58 | |
*** adrian_otto has joined #openstack-nova | 15:59 | |
*** mnaser has joined #openstack-nova | 16:00 | |
*** jwcroppe has joined #openstack-nova | 16:01 | |
*** marst has joined #openstack-nova | 16:02 | |
*** markus_z has quit IRC | 16:04 | |
*** jdurgin1 has quit IRC | 16:04 | |
*** Guest12406 has quit IRC | 16:05 | |
*** karimb has joined #openstack-nova | 16:05 | |
*** ratailor has quit IRC | 16:05 | |
*** rcernin has quit IRC | 16:06 | |
*** crushil has joined #openstack-nova | 16:06 | |
artom | cfriesen, I hear there's a thing called OvenShack or something that can even manage VMs for you | 16:08 |
*** acabot has quit IRC | 16:09 | |
superdan | hah | 16:09 |
*** Oku_OS is now known as Oku_OS-away | 16:10 | |
*** lucas_ has quit IRC | 16:13 | |
*** lucas_ has joined #openstack-nova | 16:13 | |
*** zzelle has joined #openstack-nova | 16:15 | |
*** stvnoyes has quit IRC | 16:15 | |
*** stvnoyes has joined #openstack-nova | 16:16 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Extract _post_live_migratiion_network_setup https://review.openstack.org/413021 | 16:16 |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: [WIP] Network binding based on events https://review.openstack.org/432370 | 16:17 |
*** armax has joined #openstack-nova | 16:18 | |
*** zzelle has quit IRC | 16:19 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Update deprecation timeline https://review.openstack.org/432373 | 16:19 |
*** bvanhav has quit IRC | 16:19 | |
*** bvanhav has joined #openstack-nova | 16:20 | |
*** gszasz has quit IRC | 16:21 | |
*** adrian_otto1 has joined #openstack-nova | 16:22 | |
*** stvnoyes has left #openstack-nova | 16:24 | |
*** dims_ has joined #openstack-nova | 16:24 | |
*** adrian_otto has quit IRC | 16:25 | |
*** adrian_otto1 has quit IRC | 16:27 | |
*** lucas_ has quit IRC | 16:27 | |
*** adrian_otto has joined #openstack-nova | 16:28 | |
*** raj_singh_ has quit IRC | 16:29 | |
*** sudipto_ has joined #openstack-nova | 16:29 | |
*** sudipto has joined #openstack-nova | 16:29 | |
*** anilvenkata has quit IRC | 16:30 | |
mriedem | melwitt: some feedback in https://review.openstack.org/#/c/429075/ | 16:30 |
melwitt | cool, thanks | 16:31 |
*** hos has joined #openstack-nova | 16:32 | |
*** afazekas has joined #openstack-nova | 16:32 | |
*** sree has joined #openstack-nova | 16:34 | |
kashyap | mriedem: [OT?] Your response on that IRC thread made me chuckle, and do a `grep` for the said word (not going to repeat here, lest it ends up archives again) | 16:36 |
kashyap | So, it's not really _that_ bad: | 16:36 |
kashyap | $ grep $WORD ~/irclogs/FN-B/#openstack-nova.log | wc -l | 16:36 |
kashyap | 12 | 16:36 |
kashyap | I checked that all entries were from you :-) | 16:37 |
mriedem | kashyap: heh | 16:38 |
mriedem | yeah i wasn't making it up | 16:38 |
* kashyap knows | 16:38 | |
*** sree has quit IRC | 16:38 | |
kashyap | Oh, should mention the archives were starting only from 22-MAR-2015 | 16:39 |
*** zzelle has joined #openstack-nova | 16:39 | |
kashyap | mriedem: On a diff. topic, from your email on min libv version | 16:40 |
kashyap | Should the next one be 1.3.1, then? | 16:40 |
kashyap | FWIW, /me wrote this http://lists.openstack.org/pipermail/openstack-dev/2017-February/111933.html | 16:40 |
mriedem | kashyap: well, i know you always want bleeding edge so i'm not sure i can trust you here :) | 16:41 |
mriedem | i saw your response | 16:41 |
mriedem | your general comment on anything libvirt version related is "but that version was released 2 years ago?!" | 16:41 |
kashyap | mriedem: No, please don't be so harsh that I don't care about operators who don't care about bleeding edge. | 16:42 |
*** tlian has joined #openstack-nova | 16:42 | |
kashyap | mriedem: If you saw, I was careful to not mention any newer release on the response :-) | 16:42 |
mriedem | heh | 16:42 |
mriedem | so you mean bump to 1.2.9 and mark 1.3.1 as 'next'? | 16:43 |
kashyap | I know you'd chide me for that. Learnt from the burnt. | 16:43 |
*** ssurana has joined #openstack-nova | 16:43 | |
kashyap | Yeah. | 16:43 |
mriedem | it sounded like sgordon was ok with going to 1.2.9 | 16:43 |
*** bkopilov_ has quit IRC | 16:43 | |
*** bkopilov has quit IRC | 16:43 | |
mriedem | 1.3.1 would be nice since we have lots of checks we could drop after that i think | 16:44 |
mriedem | but that's queens at the earliest | 16:44 |
kashyap | Okay, maybe we need hard data here: to evaluate pros & cons w.r.t. features / critical bug fixes with 1.3.1 compared to 1.2.9 | 16:44 |
mriedem | it's less about that, | 16:44 |
mriedem | and more about distro support | 16:44 |
*** tesseract has quit IRC | 16:45 | |
sgordon | mriedem, from an RDO and RHOSP pov we always recommend folks use the latest RHEL anyway | 16:45 |
kashyap | Me nods. I am not ignoring the support matrix | 16:45 |
sgordon | mriedem, so right now while we still support OSP all the way back to icehouse, the recommended host OS for that is RHEL 7.3 | 16:45 |
sgordon | mriedem, and that is the combination we currently (re)validate | 16:45 |
mriedem | which is libvirt 2.0 | 16:46 |
mriedem | ballsy | 16:46 |
sgordon | correct | 16:46 |
cfriesen | mriedem: which distros do we care about | 16:46 |
kashyap | For upstream, we should also see what is available in Debian, too | 16:46 |
sgordon | (and of course anyone still on icehouse, isnt impacted by us changing the minimum for ocata anyway) | 16:46 |
mriedem | cfriesen: https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | 16:46 |
*** jose-phillips has joined #openstack-nova | 16:47 | |
mriedem | so rhel would be ok with 1.3.1 | 16:48 |
mriedem | i'm less sure about debian and sles/suse | 16:48 |
mriedem | maybe i should ask ajaeger | 16:48 |
kashyap | Yeah, if we can get the data from Debian and SUSE folks, we can settle on 1.3.1 -- as the cross-distro supported version, for upstream | 16:49 |
mriedem | zigo would know about debian | 16:49 |
kashyap | Yeah, would be better if they responded on the list, so the record is archived | 16:50 |
mriedem | i'll follow up with those guys | 16:51 |
kashyap | Thanks. | 16:51 |
mordred | debian stable has 1.2.9 - debian testing has 3.0.0 | 16:52 |
*** AJaeger has joined #openstack-nova | 16:52 | |
AJaeger | hi nova team | 16:52 |
cfriesen | do we only care about LTS versions of Ubuntu? | 16:52 |
mordred | cfriesen: yah | 16:53 |
AJaeger | mriedem: what's up? | 16:53 |
*** pcaruana has quit IRC | 16:53 | |
kashyap | AJaeger: See the mailing list post: | 16:53 |
mriedem | AJaeger: see -infra now :) | 16:53 |
AJaeger | ;) | 16:53 |
*** david-lyle has quit IRC | 16:53 | |
kashyap | AJaeger: http://lists.openstack.org/pipermail/openstack-dev/2017-February/111902.html | 16:54 |
mordred | cfriesen: or, rather, we must care about at least the latest LTS- and we only test the latest LTS in the gate. our policy _technically_ is to target the latest ubuntu unless doing so would make backporting to latest lts unreasonably difficult ... but since in practice we gate on latest LTS ... | 16:54 |
johnthetubaguy | mriedem: superdan: is there anyone looking at the mod-wsgi service version break thing already? sneti has been working on a related bug when using the eventlet stack | 16:54 |
AJaeger | mriedem: libvirt 2.0.0 should be new enough, isn't it? | 16:55 |
mriedem | johnthetubaguy: for pike | 16:55 |
superdan | johnthetubaguy: I was planning to do it but haven't started yet | 16:56 |
superdan | johnthetubaguy: figured we probably want to discuss plans at the ptg at least | 16:56 |
mriedem | AJaeger: 2.0.0 would mean nova in pike wouldn't work with xenial | 16:56 |
cfriesen | would there be any interest in being able to specify a vif_model as part of the neutron port creation, then have nova use that vif_model when booting/attaching with that port? | 16:56 |
johnthetubaguy | mriedem: yeah, I was thinking both in terms of the devstack changes and the service reporting | 16:56 |
mriedem | AJaeger: we're talking 1.2.9 minimum libvirt for nova in pike, and 1.3.1 libvirt minimum for nova in queens | 16:56 |
*** alaski is now known as lascii | 16:57 | |
lascii | good morning everyone | 16:57 |
dims_ | hey lascii ! | 16:57 |
melwitt | lascii o/ | 16:58 |
johnthetubaguy | superdan: OK, yeah, yeah certainly ptg fodder | 16:58 |
johnthetubaguy | lascii: hello hello | 16:58 |
*** lucas_ has joined #openstack-nova | 16:58 | |
*** tlian has quit IRC | 16:59 | |
AJaeger | mriedem: I let our experts answer on the mailing... AFAIK we have libvirt 2.0.0 version currently and thus a minimum requirement of any 1.x should be fine for SUSE | 17:00 |
mriedem | AJaeger: ok thanks | 17:01 |
*** bnemec is now known as beekneemech | 17:02 | |
*** panda is now known as panda|off | 17:03 | |
AJaeger | mriedem: thanks for asking! | 17:04 |
*** yamahata has joined #openstack-nova | 17:05 | |
*** dave-mccowan has quit IRC | 17:05 | |
*** harlowja has joined #openstack-nova | 17:05 | |
*** harlowja has quit IRC | 17:05 | |
*** harlowja has joined #openstack-nova | 17:05 | |
*** ralonsoh_ has quit IRC | 17:07 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: PowerVM Driver Integration (Pike) https://review.openstack.org/432387 | 17:07 |
*** AJaeger has left #openstack-nova | 17:07 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: PowerVM Driver Integration (Pike) https://review.openstack.org/432387 | 17:09 |
*** Apoorva has joined #openstack-nova | 17:09 | |
*** stevemar has left #openstack-nova | 17:10 | |
efried | mriedem FYI ^^ for continuation of https://blueprints.launchpad.net/nova/+spec/powervm-nova-compute-driver | 17:10 |
*** hos has quit IRC | 17:11 | |
*** efried has quit IRC | 17:12 | |
superdan | mriedem: why does the 14.0.1 section talk about 14.0.0 things here? http://docs.openstack.org/releasenotes/nova/newton.html | 17:15 |
mriedem | it's a bug | 17:16 |
mriedem | oh, no, | 17:16 |
mriedem | that's because we didn't get the prelude into 14.0.0 | 17:16 |
mriedem | remember bauzas added that after we released newton | 17:16 |
mriedem | so it got into 14.0.1 | 17:16 |
superdan | oh | 17:16 |
*** hos has joined #openstack-nova | 17:16 | |
*** psachin has quit IRC | 17:16 | |
*** faizy_ has quit IRC | 17:17 | |
*** mriedem is now known as mriedem_lunch | 17:17 | |
*** fragatina has joined #openstack-nova | 17:19 | |
*** aarefiev is now known as aarefiev_afk | 17:19 | |
*** lpetrut has quit IRC | 17:20 | |
*** fragatina has quit IRC | 17:20 | |
*** fragatina has joined #openstack-nova | 17:20 | |
*** rwmjones has quit IRC | 17:21 | |
*** dikonoor has joined #openstack-nova | 17:21 | |
*** rwmjones has joined #openstack-nova | 17:23 | |
*** nic has joined #openstack-nova | 17:24 | |
*** ygl has joined #openstack-nova | 17:26 | |
ygl | hi all | 17:27 |
*** qwebirc83974 has joined #openstack-nova | 17:28 | |
*** jaypipes has joined #openstack-nova | 17:28 | |
*** qwebirc83974 has quit IRC | 17:29 | |
*** qwebirc21796 has joined #openstack-nova | 17:30 | |
*** david-lyle has joined #openstack-nova | 17:31 | |
*** qwebirc21796 has quit IRC | 17:31 | |
*** hos has quit IRC | 17:31 | |
*** mnestratov has quit IRC | 17:31 | |
*** hos has joined #openstack-nova | 17:32 | |
*** jaypipes is now known as leakypipes | 17:34 | |
*** bvanhav has quit IRC | 17:36 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove flavor_* functions from db/api https://review.openstack.org/431752 | 17:37 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove main_db flavor compatibility https://review.openstack.org/431722 | 17:37 |
superdan | hmm, where can i see the release notes as rendered like they would for a release for the current tree if it were to happen now? | 17:39 |
superdan | dhellmann: dims_ mriedem_lunch ^ ? | 17:39 |
*** derekh has quit IRC | 17:40 | |
*** bvanhav has joined #openstack-nova | 17:41 | |
melwitt | superdan: you mean this? http://docs.openstack.org/releasenotes/nova/unreleased.html | 17:42 |
superdan | melwitt: yeah | 17:42 |
superdan | melwitt: didn't see it in a docs job run, figured it'd be there but thanks | 17:42 |
*** raj_singh_ has joined #openstack-nova | 17:42 | |
melwitt | oh, yeah I dunno where/if it exists in any per patch runs | 17:44 |
*** sambetts is now known as sambetts|afk | 17:44 | |
*** sdague has quit IRC | 17:44 | |
superdan | this is what I want anyway | 17:45 |
superdan | so thanks | 17:45 |
*** cdent has joined #openstack-nova | 17:45 | |
*** breitz has joined #openstack-nova | 17:52 | |
*** bvanhav has quit IRC | 17:53 | |
*** raj_singh_ has quit IRC | 17:55 | |
dims_ | superdan : y was lunching :) back now | 17:58 |
*** jose-phillips has quit IRC | 17:58 | |
*** jaosorior has joined #openstack-nova | 18:03 | |
*** unicell has joined #openstack-nova | 18:04 | |
*** lucasagomes is now known as lucas-afk | 18:06 | |
*** bvanhav has joined #openstack-nova | 18:11 | |
*** adrian_otto has quit IRC | 18:11 | |
*** salv-orl_ has quit IRC | 18:12 | |
*** lucas_ has quit IRC | 18:14 | |
*** lucas_ has joined #openstack-nova | 18:14 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 18:15 |
openstackgerrit | Ed Leafe proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 18:15 |
openstackgerrit | Ed Leafe proposed openstack/nova master: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | 18:15 |
openstackgerrit | Ed Leafe proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | 18:15 |
openstackgerrit | Ed Leafe proposed openstack/nova master: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | 18:15 |
openstackgerrit | Ed Leafe proposed openstack/nova master: placement: adds ProviderTree for nested resources https://review.openstack.org/415920 | 18:15 |
openstackgerrit | Ed Leafe proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 18:15 |
openstackgerrit | Ed Leafe proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 18:15 |
*** sdague has joined #openstack-nova | 18:16 | |
*** sudipto has quit IRC | 18:16 | |
*** sudipto_ has quit IRC | 18:16 | |
*** ociuhandu has quit IRC | 18:19 | |
leakypipes | mriedem_lunch, johnthetubaguy, superdan: I think https://review.openstack.org/#/c/404472/ is ready for your eyeballs again now. | 18:19 |
*** yamahata has quit IRC | 18:20 | |
*** crushil has quit IRC | 18:20 | |
superdan | leakypipes: I don't think I've reviewed that yet at all, but | 18:24 |
superdan | leakypipes: does that not seem like a major layering violation? maybe that has already been discussed? | 18:24 |
*** jpena is now known as jpena|away | 18:24 | |
leakypipes | superdan: sorry, I'm not quite following you. do you mean that that code is in the wrong place? | 18:25 |
*** abalutoiu_ has joined #openstack-nova | 18:26 | |
*** mriedem_lunch is now known as mriedem | 18:27 | |
superdan | leakypipes: the resource tracker having things like "if ironic, do this other thing" | 18:27 |
superdan | we try not to even do that in compute manager, which directly consumes the compute drivers | 18:27 |
superdan | and we nak people having tried to do stuff like that in the past | 18:27 |
leakypipes | superdan: yeah, it's a temporary thing until we can be sure that the request from the user just contains a list of resources to consume as opposed to the existing situation where we need to tell if the instance is a baremetal instance or not and then do something different to track Ironic resources. | 18:28 |
leakypipes | superdan: the dilemma we have is a chicken and egg one. | 18:28 |
superdan | I'm not sure what that means, request wise, but I assume that means temporary ... for a while? | 18:29 |
*** abalutoiu has quit IRC | 18:29 | |
mriedem | i had made a comment at one point about subclasses the resource tracker for ironic specific resource class massaging | 18:30 |
mriedem | since there were big conditionals in that patch at one point which were all just for ironic | 18:30 |
leakypipes | superdan: an OpenStack admin may have set the node.resource_class attribute for some or all of their Ironic baremetal nodes, but a) there may be records in Nova already existing that refer to the consumed resources like VCPU, MEMORY_MB and DISK_GB. We need to continue to track those "resources" like we do now but also track the new node.resource_class consumption. All of that code is basically the "auto-heal" code for Ironic installations in | 18:30 |
leakypipes | Newton and Ocatra | 18:30 |
mriedem | fwiw ironic + resource providers is broken in newton anyway i think | 18:31 |
*** ociuhandu has joined #openstack-nova | 18:31 | |
leakypipes | superdan: yes, that code and those conditionals will not be necessary once the "flavor" for Ironic nodes no longer requests VCPU, MEMORY_MB and the like and instead just requests the custom resource class like "CUSTOM_IRON_SILVER" or whatnot. | 18:31 |
mriedem | w/o https://review.openstack.org/#/c/416762/ anyway | 18:31 |
leakypipes | mriedem: ah, yes, but that's a bit of a different thing, right? | 18:32 |
superdan | mriedem: yeah, I'd like that to be less hacky.. I'd have to think about the best way to do it though | 18:32 |
mriedem | yeah | 18:32 |
*** bswartz has quit IRC | 18:32 | |
ygl | can we access the ovs controller of devstack ? | 18:32 |
ygl | is it a opendaylight controller ? | 18:33 |
*** cdent has quit IRC | 18:34 | |
*** ygl has quit IRC | 18:35 | |
*** crushil has joined #openstack-nova | 18:37 | |
*** vsaienko has joined #openstack-nova | 18:37 | |
superdan | jroll: is ironic making "set your node class" a requirement for people deploying ocata? | 18:37 |
superdan | meaning, is there a "you will break in pike if you haven't set this property" warning? | 18:38 |
jroll | superdan: not yet no | 18:38 |
jroll | we also support ironic without nova so we can never make it strictly mandatory | 18:38 |
superdan | jroll: so we have to tolerate not having this set until queens? | 18:38 |
jroll | or rather, doing that would be painful for people that don't care | 18:38 |
jroll | nah, we should be able to enforce it nova side in pike | 18:39 |
jroll | we can make it a pre-pike upgrade requirement | 18:39 |
superdan | jroll: okay presumably you have to note your pre-pike requirements in ocata, no? | 18:39 |
superdan | or are you just going to say in pike release notes you're broken if you don't? | 18:40 |
*** yamahata has joined #openstack-nova | 18:40 | |
jroll | superdan: was planning on pike relnotes + upgrade docs | 18:40 |
*** slaweq has quit IRC | 18:40 | |
superdan | okay | 18:40 |
superdan | so since we're in pike now, | 18:40 |
mriedem | in newton we said placement was going to be required in ocata, which was kind of true, but we didn't make it a hard break in the scheduler | 18:40 |
superdan | can nova not just dispense with this ironic-specific RT stuff leakypipes was talking about? | 18:40 |
jroll | I don't want to strictly enforce it at any time with log.warn() or whatever, because standalone | 18:40 |
superdan | sure, I get the standalone thing | 18:41 |
mriedem | jroll: i mean strictly enforce in nova | 18:41 |
mriedem | if you don't have node.resource_class set in nova, it's a an error | 18:41 |
*** ducttape_ has quit IRC | 18:41 | |
superdan | mriedem: we strictly enforce placement in ocata, just not until you're fully upgraded | 18:41 |
jroll | mriedem: yes I would like to do that thing in pike | 18:41 |
jroll | superdan: so, we still need ironic-specific RT stuff, right | 18:41 |
jroll | because it auto-creates a custom resource class and such | 18:41 |
superdan | jroll: leakypipes said not once we can depend on that no/ | 18:41 |
jroll | oh there's scrollback involved | 18:42 |
*** slaweq has joined #openstack-nova | 18:42 | |
superdan | I guess he's also saying something about existing legacy resources | 18:42 |
mriedem | if the nova RT isn't going to create the custom resource classes from the node.resource_class, what is? something/someone outside nova? | 18:42 |
mriedem | pre-pike? | 18:42 |
superdan | I guess it seems like we should be refactoring the interface between RT and the resources the driver returns, | 18:43 |
jroll | yeah, so there's a couple parts | 18:43 |
superdan | and make the ironic driver return the node class types if appropriate, | 18:43 |
superdan | and let the RT be generic | 18:43 |
mriedem | i was going to say, the get_available_resources() call to the driver could return the node.resource_class couldn't it? | 18:43 |
superdan | and it would then delete the legacy resources at that point, but we want that | 18:43 |
jroll | 1) something needs to put these resources in the placement engine correctly | 18:43 |
superdan | mriedem: right | 18:43 |
jroll | 2) flavors need to be able to request custom resource classes | 18:43 |
jroll | 3) I'm blanking, totally had something | 18:44 |
artom | 3) ??? | 18:44 |
artom | 4) Profit! | 18:44 |
jroll | superdan: I think leakypipes was saying we can drop it once flavors can be converted over | 18:44 |
jroll | people will need to be able to use the new and old flavors in pike | 18:45 |
jroll | so they can transition | 18:45 |
*** esberglu has quit IRC | 18:45 | |
jroll | and then we can drop the cpu/memory/disk tracking in queens | 18:45 |
superdan | jroll: so "temporary" means queens at the soonest yes? | 18:45 |
jroll | I think I have that right | 18:45 |
mriedem | leakypipes: i'll create you a bp for resource class work in pike | 18:45 |
jroll | yeah, for dropping the old stuff | 18:45 |
jroll | superdan: I'm planning to like, write up a plan here before PTG sometime | 18:46 |
jroll | but our ocata release is a bit delayed so I've been dealing with that | 18:46 |
mriedem | leakypipes: oh i guess i already did https://blueprints.launchpad.net/nova/+spec/custom-resource-classes-pike | 18:46 |
*** Tanvir has joined #openstack-nova | 18:46 | |
*** Tanvir is now known as Guest54279 | 18:47 | |
jroll | I want to spend a bunch more time here during pike and get this shit over with | 18:47 |
mriedem | jroll: i'll throw it on our etherpad | 18:47 |
jroll | mriedem: ptg? it's there :) | 18:47 |
jroll | just not fleshed out | 18:47 |
superdan | so, I still think we should be able to do this by refactoring the interface between the RT and the driver that returns resources, | 18:47 |
superdan | and make the ironic driver encapsulate this different-for-ironic behavior right? | 18:47 |
jroll | mriedem: line 88 | 18:47 |
jroll | mriedem: or what you wrote, I'll delete mine | 18:48 |
mriedem | jroll: ok jay had something too | 18:48 |
Guest54279 | Hi, anyone here has worked on pci_whitelist_entry inside nova.conf (pci passthrough) in nova ? | 18:48 |
*** breitz has quit IRC | 18:49 | |
jroll | superdan: I'm honestly not sure | 18:49 |
jroll | superdan: I guess we could say any virt driver that passes back a resource_class field back from get_available_resource gets that resource class created/written | 18:49 |
jroll | that seems sane to me | 18:49 |
jroll | idk what else is special case | 18:50 |
*** breitz has joined #openstack-nova | 18:50 | |
superdan | we kinda do that today, but in old format, | 18:50 |
jroll | write all the resources returned into the placement api | 18:50 |
superdan | I would think we should be converting the drivers to communicate in the new format anyway | 18:50 |
mriedem | Guest54279: you want moshele who isn't around right now | 18:50 |
superdan | and since we're pike now, we need to rip out the old reporting format anyway | 18:50 |
jroll | so | 18:51 |
jroll | in pike we need ironic nodes reporting both cpu/disk/ram AND a resource class | 18:52 |
jroll | in queens we want just a resource class | 18:52 |
jroll | I feel like just writing everything returned by the driver is the right thing to do | 18:52 |
*** breitz has quit IRC | 18:52 | |
jroll | and we'll just change what the driver returns each cycle | 18:52 |
superdan | jroll: right so the ironic driver would return that extra thing right now and then stop later | 18:52 |
jroll | not sure why we need all this stuff to remove allocations for cpus, etc | 18:53 |
jroll | I need to buy a whiteboard for my office already :| | 18:53 |
leakypipes | sorry guys, was on a call. | 18:53 |
jroll | superdan: I know we're hyper to get this code done, but I'd almost rather map this plan out on the ML (or at the PTG if we still don't have a plan by then) | 18:54 |
jroll | and code according to that plan | 18:54 |
superdan | jroll: yeah | 18:55 |
superdan | jroll: I just haven't had time to look at this stuff until now and so I'm asking questions based on that | 18:55 |
jroll | superdan: right, I haven't had enough time either, tbh | 18:55 |
*** ltomasbo is now known as ltomasbo|away | 18:56 | |
jroll | I kinda have it in my head, need to write it down and have people tell me where I'm wrong | 18:56 |
jroll | but I need to get ocata out the door first >.> | 18:56 |
superdan | I can do the latter part | 18:56 |
jroll | I'm sure you can | 18:56 |
mriedem | so maybe we do wed morning all resource providers/placement stuff at the ptg? | 18:57 |
mriedem | 10-noon is nova/cinder on thursday | 18:57 |
mriedem | jroll: or would you rather do wed afternoon? | 18:57 |
mriedem | maybe you don't care | 18:57 |
jroll | mriedem: yeah I think afternoon would be better | 18:57 |
mriedem | ok | 18:57 |
mriedem | let you guys get your ironic-specific stuff started on wed morning | 18:57 |
jroll | yeah | 18:57 |
jroll | I know at least a couple people wanted to float over for placement stuff | 18:58 |
*** breitz has joined #openstack-nova | 18:58 | |
leakypipes | superdan: what you're talking about there (fix the stuff in the virt driver's get_available_resource()) call is EXACTLY what I had originally proposed and that approach was shot down as too invasive of the virt layer :) | 18:59 |
mriedem | scottda: so for whatever reason, at 3am last night, i was thinking maybe we should just change the default catalog_info for nova/cinder to be v3 in pike | 18:59 |
leakypipes | superdan: lemme get a link ... one sec. | 18:59 |
superdan | leakypipes: well, this is too invasive to the RT layer, IMHO :D | 18:59 |
mriedem | scottda: anyone with older config would be using cinder v2, new nova installs would be cinder v3 by default | 18:59 |
mriedem | scottda: that would make us test nova + cinder v3 by default in the gate in pike | 18:59 |
*** jaosorior has quit IRC | 19:00 | |
leakypipes | superdan: see here for an original proposed solution to this: https://review.openstack.org/#/c/334038/ | 19:00 |
superdan | leakypipes: we might also haveanother method for just RP stuff, and just stop calling the old one at some point | 19:00 |
leakypipes | back in the good ole days of Mitaka :) | 19:00 |
superdan | to avoid mixing the two | 19:00 |
leakypipes | superdan: ya, totes agreed. | 19:00 |
superdan | leakypipes: where is the giant nak from zeus on that patch? | 19:00 |
leakypipes | superdan: I think we have to live with some more cruft int he RT for a time (until Queens at earliest). | 19:00 |
mriedem | ha yeah it's all +1s in there | 19:01 |
*** burt has quit IRC | 19:01 | |
jroll | the get_inventory() method landed but not the implementations for the drivers, wat | 19:02 |
superdan | leakypipes: cruft being the reporting of both things? or what/ | 19:02 |
mriedem | leakypipes: no that was also abandoned https://review.openstack.org/#/c/334035/ | 19:02 |
mriedem | i don't know why it takes you to git.o.o | 19:02 |
jroll | mriedem: oh, weird | 19:02 |
jroll | thanks | 19:02 |
cfriesen | has anyone run into problems in the field due to heat trying to scale out with a flavor that has been deleted? Or is that sort of thing usually dealt with by issuing notifications of upcoming flavor changes? | 19:03 |
*** Guest81164 has quit IRC | 19:03 | |
*** Guest54279 has quit IRC | 19:03 | |
mriedem | cfriesen: no one uses heat in the field | 19:03 |
cfriesen | lol | 19:03 |
mriedem | btw, let me just say, trying to sort this ptg topics etherpad into some kind of agenda is giving me heartburn | 19:03 |
jroll | mriedem: just watching our new ptl sort ironic's topics gave me heartburn | 19:04 |
leakypipes | pretty sure it was bauzas that didn't like the get_inventory() stuff, but I can't find his comments now. maybe comments were only on IRC>... | 19:04 |
*** jaosorior has joined #openstack-nova | 19:05 | |
leakypipes | ah, no.. it was figleaf :) | 19:05 |
*** owalsh is now known as owalsh-afk | 19:05 | |
leakypipes | this patch here, comment from figleaf on PS3: https://review.openstack.org/#/c/415920/ | 19:05 |
mriedem | i'll go fill out foundation marketing surveys instead, that'll be much more fun | 19:05 |
superdan | leakypipes: well, regardless, I think that's the way to go IMHO | 19:06 |
leakypipes | that patch series originally added an update_inventory() method to the virt driver interface. | 19:06 |
mriedem | update_inventory() in each virt driver does seem gross | 19:07 |
*** tojuvone has quit IRC | 19:07 | |
mriedem | but get_resource_thingies() seems fine | 19:07 |
mriedem | if only ironic needs to return anything, then so be it | 19:07 |
*** lpetrut has joined #openstack-nova | 19:07 | |
mriedem | that's why i was thinking, just add a 'resource_class' key to the get_available_resources() dict | 19:07 |
*** dikonoor has quit IRC | 19:07 | |
superdan | mriedem: yeah, just "show me your resources, beeyotch" in the virt drviers | 19:08 |
superdan | not "update 'em too" | 19:08 |
leakypipes | mriedem: err, that already exists :) | 19:08 |
* jroll wonders if it would work out if get_resource_thingies() just returned {'VCPUS': 2, 'CUSTOM_RESOURCE_FOO': 1, ...} and the RT handled the rest | 19:08 | |
superdan | jroll: that's what I'm talking about | 19:08 |
leakypipes | https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L331 | 19:08 |
*** cdelatte has quit IRC | 19:09 | |
mriedem | ok, | 19:09 |
leakypipes | that's how things *already* work :) only for Ironic of course, though. | 19:09 |
mriedem | so why do we need the extra 'if resource_class: do special ironic stuff' in RT? | 19:09 |
*** owalsh-afk has quit IRC | 19:09 | |
superdan | because it's the wrong layer | 19:09 |
superdan | instead of returning the node class, | 19:09 |
superdan | we should return resources | 19:09 |
mriedem | because it's doing the whole drop old types, add new resource class | 19:09 |
leakypipes | mriedem: because Ironic doesn't require the node.resource_class to be present. | 19:09 |
superdan | so that the RT doesn't need to translate | 19:09 |
superdan | the ironic driver can return the VCPU, DISK, and IRONIC_FOO things when appropriate | 19:10 |
leakypipes | Take a look at the first PS on this: https://review.openstack.org/#/c/415921/ | 19:10 |
*** karimb has quit IRC | 19:10 | |
superdan | leakypipes: why not just "return inventory"? | 19:11 |
leakypipes | specifically, go here: | 19:11 |
leakypipes | https://review.openstack.org/#/c/415921/8..1/nova/virt/libvirt/driver.py | 19:11 |
leakypipes | line 5471 | 19:11 |
superdan | I don't like the semantics of this, but the general idea is what I'm saying | 19:11 |
*** dharinic is now known as dharinic|lunch | 19:12 | |
leakypipes | superdan: because there was a complaint that "the virt driver then needs to understand the internals of the resource provider and inventory modeling of the placement API"./ | 19:12 |
*** esberglu has joined #openstack-nova | 19:12 | |
jroll | I don't think it does | 19:12 |
superdan | it just needs to know about resources, which is completely reasonable IMHO | 19:12 |
jroll | it just needs to know what a VCPUS is, what a MEMORY_MB is, etc | 19:12 |
superdan | right | 19:12 |
superdan | and it knows about those today | 19:12 |
superdan | we just have to translate them from useless to usefulish in the RT | 19:13 |
superdan | and then we get shit like "if ironic, do other stuff" too high in the stack | 19:13 |
*** burt has joined #openstack-nova | 19:14 | |
superdan | leakypipes: okay so I have sufficiently shit all over your thing, but I think that the end result is getting to do it like you originally wanted right? | 19:14 |
leakypipes | superdan: and hierarchies of providers, and traits, etc... | 19:15 |
melwitt | mriedem: on that max_count = 50 thing on the map_instances, I was trying to say I think because of the map_all = True it will keep looping doing batches of 50 until all are mapped, and then it will return 0 | 19:15 |
* mriedem looks | 19:15 | |
leakypipes | superdan: heh, who knows? I think I've actually had about 3 different patch series that use three different strategies to get there. :) | 19:15 |
mriedem | melwitt: you're right | 19:15 |
mriedem | i'm dumb | 19:15 |
* jroll wishes he could sketch this code out the way he sees it working, but is too naive on stuff like nested RP and such | 19:16 | |
* figleaf reads scrollback | 19:16 | |
leakypipes | jroll: just go for it :) | 19:17 |
jroll | leakypipes: I will try next week sometime | 19:17 |
superdan | jroll: well, we're two weeks from being able to do that right? | 19:17 |
jroll | or at least pseudo-code it on the list | 19:17 |
*** Sukhdev has joined #openstack-nova | 19:17 | |
jroll | superdan: from being able to do what now | 19:17 |
superdan | leakypipes: well, sorry I dunno what to say.. you asked me to review it | 19:17 |
superdan | jroll: draw shit on the walls | 19:17 |
* superdan notes that he's using lots of foul language today | 19:18 | |
jroll | superdan: yeah that too | 19:18 |
figleaf | superdan: my objection was that there was too much in terms of placement engine implementation details in the virt layer | 19:18 |
*** vsaienko has quit IRC | 19:18 | |
jroll | f your language | 19:18 |
figleaf | superdan: not that the virt layer reported its inventory | 19:18 |
cfriesen | leakypipes: regarding https://review.openstack.org/#/c/265282/4/specs/pike/approved/instance-flavor-api.rst , mordred specifically asked for the flavor "id" to be included as a historical data point about what was used to boot the instance. | 19:18 |
leakypipes | superdan: no worries man :) | 19:18 |
superdan | figleaf: the resource class and inventory part of placement is very much the business of the virt driver, IMHO.. | 19:19 |
cfriesen | leakypipes: even though that id might represent something else now. | 19:19 |
figleaf | jroll: I think that we *all* need a more visual representation of the flow of information through the code | 19:19 |
superdan | figleaf: reporting it to placement is not, but reporting it up the stack to compute manager and RT is | 19:19 |
jroll | figleaf: I want one built in VR where I can swim through it | 19:19 |
leakypipes | cfriesen: nobody listens to mordred. nothing to see here. move along. | 19:19 |
figleaf | jroll: you might drown | 19:19 |
mordred | leakypipes: :) | 19:19 |
jroll | ha | 19:20 |
*** satyar has quit IRC | 19:20 | |
dims_ | LOL | 19:20 |
leakypipes | here is a visualization of the data flow between the systems: http://www.27bslash6.com/images/logo_for_simon_edhouse.gif | 19:20 |
*** vsaienko has joined #openstack-nova | 19:21 | |
superdan | now I'm paying attention | 19:21 |
jroll | wow | 19:21 |
leakypipes | ok, back to work people. | 19:21 |
superdan | wait, which one is ironic and which one is nova | 19:21 |
mriedem | https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/resource-providers-scheduler-db-filters.html#other-deployer-impact needs to be updated... | 19:21 |
superdan | I think ironic is behind, no? | 19:22 |
jroll | I thought front was nova devs and back was the resource tracker | 19:22 |
jroll | that's been my experience | 19:22 |
superdan | jroll: well, certainly has been true some of the time | 19:22 |
jroll | mriedem: what about it | 19:23 |
mriedem | jroll: last paragraph isn't correct | 19:23 |
mriedem | i can update it | 19:23 |
jroll | ah ok | 19:23 |
mriedem | but it's going in my ocata marketing survey so it needs to be spotless | 19:23 |
leakypipes | mriedem: what about the last paragraph is incorrect? | 19:23 |
mriedem | leakypipes: we don't even attempt to use placement until all of your computes are ocata | 19:24 |
mriedem | and they can't start in ocata w/o reporting into placement | 19:24 |
jroll | mriedem: trying to make readers' minds melt so they don't see that you don't care about marketing surveys? | 19:24 |
mriedem | the fallback described in there says that you can get like one compute / RP back | 19:24 |
*** raunak has joined #openstack-nova | 19:25 | |
mriedem | specifically, "There may be a short period of time while the scheduler receives a smaller-than-accurate set of resource providers that meet the requested resource amounts." | 19:25 |
leakypipes | mriedem: gotcha. you want me to push up a fix? | 19:25 |
mriedem | i think the rest of it, while not correct per the implementation, is generic enough that it's still truthy | 19:26 |
mriedem | leakypipes: if you want | 19:26 |
* mordred has officially responded to leakypipes on the review :) | 19:26 | |
leakypipes | thx mordred I see that :) | 19:27 |
superdan | mordred: you don't think it's confusing to say that this instance is from flavor foo when foo exists an is totally different now? | 19:27 |
superdan | mordred: like you booted this instance with flavor foo, and that flavor had 24TB of disk, but foo now has no disk and 64MB of ram, but we report that the instance's flavor is "foo" ? | 19:28 |
mordred | yah. I mean, the operator who did that is an operator who hates their users | 19:29 |
sdague | superdan: in the conversation the other day we said we'd document it as "potentially highly fictional" | 19:29 |
mordred | but as a user who booted the server in the first place, I booted a "foo" server | 19:29 |
superdan | mordred: for what it's worth, horizon makes the above trivial to do | 19:29 |
mordred | superdan: that's really sad for everyone | 19:29 |
sdague | superdan: it does, but there are also lots of people that don't change the meanings of their flavors | 19:30 |
mordred | I think of flavors sort of like server model numbers | 19:30 |
mordred | when you order a DL380 from HPE, you get a DL380 | 19:30 |
superdan | mordred: right, horizon says "you can edit flavors" but what it really means is "I will delete and recreate it for you" | 19:30 |
mordred | it doesn't stop being a dl380 | 19:30 |
mordred | but next year, hpe will sell you a new thing called a dl380 | 19:30 |
sdague | so, it feels fine to include it, but say that the underlying definition may have changed, hence why we give you the definition of what it once was as well | 19:31 |
leakypipes | mordred: even though you can order 10 DL380s and you'll get seven different motherboard manufacturers? ;) | 19:31 |
mordred | ++ | 19:31 |
mordred | (to both of you) | 19:31 |
sdague | I also think mordred should open a bug with horizon and get them to remove that feature | 19:31 |
sdague | because, it's terrible as an easy thing to tell users to do | 19:31 |
mordred | sdague: can we just make flavors undeletable instead? | 19:31 |
superdan | we've had bugs reported because they changed the size of the flavor foo and the instances didn't change and that was surprising to them | 19:32 |
*** cdelatte has joined #openstack-nova | 19:32 | |
mordred | _wow_ | 19:32 |
sdague | superdan: right, given the way horizon exposes it, I'm not surprised | 19:32 |
superdan | yeah | 19:32 |
sdague | it's actually extra terrible UX | 19:32 |
*** zzelle has quit IRC | 19:33 | |
superdan | leakypipes: before we go expose flavors in their current form, are we not supposed to at least consider reporting them in an RP-friendly way? like so that when we get rid of the current notion of a flavor we're not pinned to this format? | 19:33 |
*** lucas_ has quit IRC | 19:33 | |
superdan | leakypipes: like, does it matter if we just tell them how much DISK_GB they got and not necessarily report the split between swap and root and ephemeral? | 19:33 |
sdague | mordred: so even if we decided to make flavors undeletable (which I think has other fallout) it would have to be in a microversion, which means it would be a very long time before that was fundamental behavior | 19:34 |
superdan | we can't make them undeletable, IMHO | 19:34 |
sdague | mordred: honestly, it is better to get the horizon expectations reset | 19:34 |
mordred | yah. I see that ship as having sailed | 19:34 |
superdan | that's also punching ops in the face | 19:34 |
mordred | fwiw, I do not feel _strongly_ about name or id. I have a contract with shade users that I'll always be providing an id key in the flavor dict of a server, but I can easily just plop a none in there if it goes away from the server record | 19:34 |
superdan | mordred: in some sense, your flavor id becomes the uuid of the instance itself after boot | 19:35 |
mordred | so, in the realm of "mordred will scream and yell and rant for years" and "mordred will just sigh and have a beer" 0 this is WAY much more in the beer stage than the other | 19:35 |
superdan | mordred: if we later allow you to "buy an extra gig of ram".. that's what it'll be | 19:35 |
mordred | superdan: I can see that argument | 19:35 |
* superdan can see mordred took his pills "on account of his RAGE" | 19:35 | |
sdague | superdan: so, while it could some day | 19:35 |
sdague | it really doesn't today | 19:36 |
superdan | sdague: what doesn't? | 19:36 |
*** ZZelle has joined #openstack-nova | 19:36 | |
sdague | it feels like there is definitely a common case of "I want another like that one" | 19:36 |
*** abalutoiu has joined #openstack-nova | 19:36 | |
superdan | sdague: that's even worse, because we know that id can be wrong | 19:36 |
superdan | if you teach people to ask for another just like it, | 19:36 |
sdague | and if people didn't rename all their flavors, or hot resize | 19:36 |
sdague | then it can work | 19:37 |
superdan | they're going to get a 0disk 64mb instance | 19:37 |
*** lucas_ has joined #openstack-nova | 19:37 | |
superdan | didn't johnthetubaguy tell me that RAX has renamed flavors before? | 19:37 |
*** READ10 has quit IRC | 19:37 | |
sdague | it feels like the moment you hot added anything to the instance, the flavor id flips to uuid of the instance | 19:37 |
sdague | so we actively decouple there | 19:37 |
superdan | sdague: but if you change/delete a flavor, it's the same thing, and we can't really query for any instances that need changing if you alter the flavor | 19:38 |
superdan | that's hugely expensive | 19:38 |
sdague | superdan: sure, and that's why we document it as "possibly fiction" | 19:38 |
superdan | bigly expensive I mean | 19:38 |
*** ducttape_ has joined #openstack-nova | 19:38 | |
mriedem | SO | 19:38 |
mriedem | besides placement and cells v2, | 19:38 |
mriedem | what's the #3 newsworthy feature delivered in ocata | 19:38 |
mriedem | ? | 19:38 |
superdan | sdague: well, I just don't see the point in documenting a thing as "likely wrong" when we could just not expose it, but whatever | 19:38 |
mriedem | https://specs.openstack.org/openstack/nova-specs/specs/ocata/index.html | 19:39 |
*** fragatina has quit IRC | 19:39 | |
*** ZZelle has quit IRC | 19:39 | |
superdan | mriedem: "leakypipes posted an inappropriate picture in IRC" is a pike event I guess | 19:39 |
*** abalutoiu_ has quit IRC | 19:39 | |
cfriesen | superdan: I kind of feel like without either name or id, we don't really have a good starting point for "find me another flavor like this one" | 19:39 |
mriedem | there was a lot of random feature parity stuff that got in for hyper-v, xen and ironic virt drivers, but i can't really say that's one thing, unless i group them together | 19:40 |
superdan | cfriesen: you compare the resources not the id.. the id _is_ fiction | 19:40 |
mriedem | there is actually a lot in https://specs.openstack.org/openstack/nova-specs/specs/ocata/index.html which didn't make the release | 19:40 |
superdan | cfriesen: to put it in a way that mriedem and sdague will love: how do you know if your cloud provider doesn't dickpunch people by changing flavors? | 19:40 |
superdan | cfriesen: you don't, so you can never rely on it, thus we should never expose it | 19:41 |
superdan | er, I meant s/mriedem/mordred/ above | 19:41 |
mriedem | yeah... | 19:41 |
cfriesen | superdan: you'd still have to check it it matches, but it gives you something to check rather than starting from scratch and searching all the flavors. | 19:41 |
mriedem | i love a good dickpunch | 19:41 |
mriedem | makes me feel alive | 19:41 |
*** ZZelle has joined #openstack-nova | 19:41 | |
superdan | cfriesen: but you don't know if your provider is good, so you always have to check the resources anyway | 19:41 |
* cfriesen winces | 19:41 | |
*** Swami has joined #openstack-nova | 19:42 | |
cfriesen | superdan: you can start by checking whether that particular flavor still matches, rather than starting from scratch. | 19:42 |
superdan | mriedem: we re-released device tags? :D | 19:42 |
mriedem | superdan: that was for attach | 19:42 |
mriedem | attach vif, attach volume + tags | 19:42 |
mriedem | which didn't happen | 19:42 |
mriedem | the original spec covered boot and attach | 19:42 |
superdan | mriedem: well, I was just trying to be funny anyway, so thanks for ruining it | 19:42 |
superdan | mriedem: no, I thought the re-release thing was for boot | 19:43 |
mriedem | oh | 19:43 |
superdan | mriedem: because get me a network stepped on us | 19:43 |
mriedem | i'm talking about this one https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/virt-device-role-tagging.html | 19:43 |
superdan | I'm talking about the bugfix | 19:43 |
superdan | anyway | 19:43 |
mriedem | yeah https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/fix-tag-attribute-disappearing.html | 19:43 |
mriedem | har har | 19:43 |
mriedem | nova-status upgrade check was a thing | 19:43 |
superdan | yeah | 19:44 |
mriedem | came in late, seems fun | 19:44 |
mriedem | no bp | 19:44 |
superdan | and it's good press | 19:44 |
mriedem | yeah, because no one cares about being able to soft reboot ironic instances :) | 19:44 |
superdan | heh | 19:45 |
superdan | mriedem: so this is the cellsv1 regression I found whilst removing the flavor compat, which I pulled out into a separate fix: https://review.openstack.org/#/c/432352/1 | 19:51 |
*** hos has quit IRC | 19:51 | |
superdan | i think we need that back in newton | 19:51 |
*** efried has joined #openstack-nova | 19:51 | |
mriedem | file a bug? | 19:53 |
mriedem | leakypipes: we don't support scheduling decisions based on shared storage in ocata right? we never leveraged the aggregates stuff yet from whati remmeber | 19:54 |
mriedem | *remember | 19:54 |
leakypipes | mriedem: that is correct. | 19:55 |
* mriedem is writing up the list of pike things | 19:55 | |
leakypipes | mriedem: this merged: https://review.openstack.org/#/c/407309/. but it's only the beginning of what is needed. | 19:55 |
leakypipes | mriedem: just wrapping up that ocata scheduler-db-filters spec patch now... | 19:56 |
superdan | mriedem: I will if you're cool with that going back | 19:56 |
mriedem | superdan: yeah that's fine | 19:56 |
mriedem | i'm not the one you have to convince | 19:57 |
mriedem | lyarwood is the gate keeper | 19:57 |
mriedem | i'm only the key master | 19:57 |
sdague | an alternative is to provide a much more extensive search interface for flavors, including, give me one like server={id} | 19:58 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove straggling use of main db flavors in cellsv1 code https://review.openstack.org/432352 | 19:59 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove flavor_* functions from db/api https://review.openstack.org/431752 | 19:59 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove main_db flavor compatibility https://review.openstack.org/431722 | 19:59 |
openstackgerrit | melanie witt proposed openstack/nova master: Add Cells V1 -> Cells V2 step-by-step example https://review.openstack.org/429075 | 19:59 |
superdan | mriedem: I can make lyarwood very uncomfortable at team social events, so I think it'll be approved | 20:00 |
openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Record reality of placement API usage from n-sched https://review.openstack.org/432442 | 20:01 |
leakypipes | mriedem: ^ | 20:01 |
*** karimb has joined #openstack-nova | 20:01 | |
*** dharinic|lunch is now known as dharinic | 20:01 | |
*** david-lyle has quit IRC | 20:03 | |
*** scarab_ has joined #openstack-nova | 20:03 | |
mriedem | ocata, pike and queens marketing and roadmaps are all done, whew | 20:06 |
*** raunak has quit IRC | 20:06 | |
*** abalutoiu_ has joined #openstack-nova | 20:06 | |
*** abalutoiu has quit IRC | 20:10 | |
*** dave-mccowan has joined #openstack-nova | 20:12 | |
mriedem | superdan: +2 | 20:13 |
superdan | thanks | 20:15 |
mriedem | leakypipes: see my comment in https://review.openstack.org/#/c/432442/ - up to you if you want to adjust anything there | 20:16 |
mriedem | if not, i'm your huckleberry | 20:17 |
*** Sukhdev has quit IRC | 20:17 | |
*** adrian_otto has joined #openstack-nova | 20:17 | |
*** hos has joined #openstack-nova | 20:18 | |
cfriesen | sdague: what would "one like server={id}" imply? at least as good in all respects? what about stuff like aggregate_instance_extra_specs keys? | 20:18 |
scottda | mriedem: Just saw your message re: Nova using cinder v3 as default. That seems ok to me. | 20:19 |
scottda | mriedem: And we've plenty of time to test before next release | 20:19 |
mriedem | scottda: the only thing is i'd still want at least one job running with cinder v2 just to make sure we don't break something there | 20:19 |
mriedem | well, nvm | 20:20 |
mriedem | grenade would test that | 20:20 |
*** rfolco_ has quit IRC | 20:20 | |
*** rfolco_ has joined #openstack-nova | 20:20 | |
mriedem | scottda: let me air out the idea in the ML and see if john can poke hole sin it | 20:20 |
scottda | mriedem: Sounds good. | 20:20 |
*** rfolco_ has quit IRC | 20:20 | |
smcginnis | I'd love to just have the one at some point too. | 20:21 |
sdague | cfriesen: right, it's a whole other API for sure | 20:21 |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: PowerVM Driver Integration (Pike) https://review.openstack.org/432387 | 20:21 |
openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Record reality of placement API usage from n-sched https://review.openstack.org/432442 | 20:25 |
leakypipes | mriedem: try ^ on for size. | 20:25 |
mriedem | leakypipes: only if it says "juicy" on my ass | 20:26 |
*** amoralej is now known as amoralej|off | 20:26 | |
*** bvanhav has quit IRC | 20:27 | |
*** rfolco_ has joined #openstack-nova | 20:29 | |
*** rfolco_ has quit IRC | 20:29 | |
leakypipes | mriedem: lol | 20:31 |
*** marst has quit IRC | 20:34 | |
superdan | mriedem: just realized if we want to remove all that flavor code in pike and the tables/models in queens we need a blocker migration | 20:34 |
mriedem | yeah | 20:35 |
mriedem | pretty simple isn't it? if there are any flavors with delete==0 in the main db we fail right? | 20:36 |
mriedem | *deleted | 20:36 |
superdan | also, we only had two actual migrations in ocata and they were small | 20:36 |
superdan | mriedem: yep, but flavors, aggregates, server_groups, keypairs at least | 20:36 |
superdan | oh wait | 20:37 |
superdan | that was in newton | 20:37 |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova master: Placement api: set custom json_error_formatter in resource_class https://review.openstack.org/430463 | 20:37 |
*** lucas_ has quit IRC | 20:37 | |
openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: resolved final todos in libvirt conf https://review.openstack.org/429042 | 20:37 |
superdan | so we did that already in the pike migration.. sweet, nevermind, I lost track | 20:37 |
mriedem | ocata migration...? | 20:39 |
*** Jack_I has quit IRC | 20:39 | |
*** abalutoiu_ has quit IRC | 20:39 | |
superdan | mriedem: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/345_require_online_migration_completion.py | 20:39 |
openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: resolved final todos in libvirt conf https://review.openstack.org/429042 | 20:39 |
mriedem | yeah | 20:39 |
*** baoli has quit IRC | 20:40 | |
*** baoli has joined #openstack-nova | 20:40 | |
mriedem | efried: you need to run py27 on your spec thing | 20:41 |
mriedem | zuul is unhappy | 20:41 |
efried | mriedem Roger that. (Who knew an RST needed python validation) | 20:42 |
efried | (I'm sure it's documented, isn't it) | 20:42 |
mriedem | leakypipes: https://review.openstack.org/#/c/432442/ fits, thanks | 20:43 |
*** lucas_ has joined #openstack-nova | 20:43 | |
dims_ | mriedem : "cat woke me up at 3am" :) | 20:43 |
mriedem | superdan: want to get that? | 20:43 |
superdan | mriedem: gross. | 20:44 |
mriedem | dims_: yes, well, i was awake at 2, tossed and turned for 45 minutes, fell asleep, and then the cat was running around making noise and woke me up again | 20:44 |
mriedem | then i proceeded to think about cinder v3 for some reason | 20:44 |
superdan | mriedem: I have to -1 that | 20:45 |
mriedem | superdan: the spec or the mid morning cinder v3? | 20:45 |
superdan | mriedem: the spec | 20:45 |
superdan | well, I guess maybe he's copy/pasting a bit.. grr, I will note my complaint and proceed I guess | 20:46 |
mriedem | because of "behaviour" with a "u"? | 20:49 |
superdan | eff yeah man | 20:49 |
mriedem | you know jay eats blood pudding for breakfast right? | 20:50 |
mriedem | breakfaust | 20:50 |
superdan | heh | 20:50 |
*** ducttape_ has quit IRC | 20:53 | |
*** ijw has quit IRC | 20:53 | |
*** marst has joined #openstack-nova | 20:53 | |
*** baoli has quit IRC | 20:53 | |
*** baoli has joined #openstack-nova | 20:54 | |
*** fragatina has joined #openstack-nova | 20:56 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: PowerVM Driver Integration (Pike) https://review.openstack.org/432387 | 20:56 |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova master: Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259 | 20:57 |
*** karimb has quit IRC | 20:59 | |
*** dimtruck is now known as zz_dimtruck | 21:02 | |
openstackgerrit | Maciej Szankin proposed openstack/nova master: conf/compute: remove invalid TODOs https://review.openstack.org/432453 | 21:03 |
*** karimb has joined #openstack-nova | 21:04 | |
efried | mriedem Done and done. Thanks for the feedback. | 21:06 |
openstackgerrit | Maciej Szankin proposed openstack/nova master: conf/libvirt: remove invalid TODOs https://review.openstack.org/432455 | 21:07 |
*** raunak has joined #openstack-nova | 21:08 | |
*** coreywright has joined #openstack-nova | 21:11 | |
*** jamesdenton has quit IRC | 21:14 | |
*** ducttape_ has joined #openstack-nova | 21:15 | |
*** jamesdenton has joined #openstack-nova | 21:16 | |
*** crushil has quit IRC | 21:17 | |
*** crushil has joined #openstack-nova | 21:18 | |
*** sacharya has quit IRC | 21:18 | |
*** sacharya has joined #openstack-nova | 21:18 | |
*** Sukhdev has joined #openstack-nova | 21:21 | |
*** crushil has quit IRC | 21:23 | |
*** liverpooler has quit IRC | 21:24 | |
*** erlon has quit IRC | 21:25 | |
*** zz_dimtruck is now known as dimtruck | 21:30 | |
*** smatzek has quit IRC | 21:32 | |
*** karimb has quit IRC | 21:33 | |
*** lx_ has joined #openstack-nova | 21:33 | |
*** crushil has joined #openstack-nova | 21:35 | |
*** crushil has quit IRC | 21:43 | |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix a couple of spec names https://review.openstack.org/432469 | 21:43 |
*** avolkov has quit IRC | 21:43 | |
*** salv-orlando has joined #openstack-nova | 21:45 | |
*** crushil has joined #openstack-nova | 21:46 | |
*** alexpilo_ has quit IRC | 21:51 | |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix paths when moving specs https://review.openstack.org/432470 | 21:52 |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Move implemented specs for Ocata https://review.openstack.org/432471 | 21:52 |
*** gouthamr has quit IRC | 21:56 | |
*** lucas_ has quit IRC | 21:56 | |
*** vsaienko has quit IRC | 21:57 | |
*** fragatina has quit IRC | 21:59 | |
openstackgerrit | Sujitha proposed openstack/nova master: Change os-server-tags default policy https://review.openstack.org/396420 | 21:59 |
mriedem | melwitt: https://review.openstack.org/#/c/429075/ looks good - leakypipes want to +W that? | 22:00 |
*** lucas_ has joined #openstack-nova | 22:01 | |
leakypipes | mriedem: done | 22:03 |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova master: Placement api: set custom json_error_formatter in inventory https://review.openstack.org/432478 | 22:03 |
*** ijw has joined #openstack-nova | 22:04 | |
*** raunak has quit IRC | 22:07 | |
*** alexpilotti has joined #openstack-nova | 22:07 | |
*** thorst_ has quit IRC | 22:09 | |
*** baoli has quit IRC | 22:11 | |
*** dave-mccowan has quit IRC | 22:12 | |
*** jamesdenton has quit IRC | 22:13 | |
*** raunak has joined #openstack-nova | 22:13 | |
*** crushil has quit IRC | 22:13 | |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: PowerVM Driver Integration (Pike) https://review.openstack.org/432387 | 22:17 |
*** lucas_ has quit IRC | 22:18 | |
*** esberglu has quit IRC | 22:19 | |
*** esberglu has joined #openstack-nova | 22:19 | |
efried | mriedem Thanks! | 22:22 |
mriedem | np | 22:23 |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova master: Adopts keystoneauth with glance client. https://review.openstack.org/412634 | 22:23 |
*** esberglu has quit IRC | 22:24 | |
mriedem | superdan: i'm going to start bugging you about stable/newton things, starting with https://review.openstack.org/#/c/408725/ | 22:25 |
*** tblakes has quit IRC | 22:27 | |
superdan | mriedem: hmm, so this one is in ocata and we're just backporting it if they want to get a jump on the cleanup right? | 22:28 |
*** thorst_ has joined #openstack-nova | 22:29 | |
mriedem | superdan: yeah, it was reportd by someone moving to newton | 22:30 |
mriedem | so i fixed on ocata with the intent of backporting | 22:30 |
mriedem | hence the comments in there | 22:30 |
*** baoli has joined #openstack-nova | 22:31 | |
*** ssurana has quit IRC | 22:32 | |
superdan | yeah, reading, I just want to get my head wrapped around it, since this is fairly strange | 22:33 |
*** esberglu has joined #openstack-nova | 22:33 | |
superdan | people would have run the online migrations when they upgraded to newton and wouldn't again until the next time | 22:33 |
*** thorst_ has quit IRC | 22:34 | |
*** efried has quit IRC | 22:36 | |
mriedem | this is to resolve the issue for anyone upgrading from mitaka before the hit the issue, unless i'm not catching your point | 22:36 |
superdan | yeah, I think I get it | 22:37 |
*** esberglu has quit IRC | 22:37 | |
mriedem | in mitaka we had the build request mirror the instance, | 22:37 |
*** Apoorva has quit IRC | 22:37 | |
mriedem | failed instance build in the api (i think) which didn't get deleted, | 22:38 |
mriedem | something like that | 22:38 |
mriedem | then in newton, we serialized the instance into the build request, | 22:38 |
mriedem | but when you try to load that old build request from mitaka, it's data format is different and blows up on load | 22:38 |
*** sree_ has joined #openstack-nova | 22:38 | |
superdan | yeah | 22:38 |
superdan | why didn't we just do the delete.filter(instance_uuid=null()).limit($limit) ? | 22:39 |
superdan | instead of the pre-query? | 22:39 |
superdan | I mean too late now, just thinking | 22:39 |
*** sree_ is now known as Guest55885 | 22:39 | |
mriedem | this https://review.openstack.org/#/c/306023/ | 22:39 |
*** scarab_ has quit IRC | 22:39 | |
mriedem | because it's sticking with the limit/count semantis in the online migrations cli | 22:39 |
mriedem | we talked about just doing the delete straight up on master | 22:39 |
mriedem | https://review.openstack.org/#/c/408727/2/nova/objects/build_request.py | 22:40 |
superdan | I think we'd still be able to do the same thing with just delete, but anyway | 22:40 |
*** cmart has joined #openstack-nova | 22:40 | |
superdan | did we also make it not blow up on load? | 22:40 |
superdan | this makes instance list blow up until you run this right? | 22:41 |
*** mdrabe has quit IRC | 22:41 | |
mriedem | i'd have to dig into all of that context again | 22:42 |
mriedem | you were +2 on the ocata change fwiw :) | 22:42 |
superdan | I know, I just don't have this in my head anymore and.. it's a weird thing to backport | 22:42 |
mriedem | the whole idea was to backport it from the start | 22:43 |
mriedem | the blow up on load is here https://review.openstack.org/#/c/298455/16/nova/objects/build_request.py@110 | 22:43 |
*** Guest55885 has quit IRC | 22:43 | |
mriedem | with a mitaka db record that doesn't have the instance_uuid, that's going to blow up either way, so i'm not sure how we'd handle it, unless we checked for a uuid recored? | 22:44 |
superdan | I know, I know | 22:44 |
mriedem | *record | 22:44 |
superdan | um | 22:44 |
mriedem | https://github.com/openstack/nova/blob/stable/mitaka/nova/db/sqlalchemy/api_models.py#L143 | 22:45 |
superdan | did we not bump the version on that because it wasn't sent over RPC? | 22:45 |
superdan | but it was stored in the db... | 22:45 |
mriedem | "Note that the BuildRequest object version is not being bumped because this object is not yet passed over the wire anywhere or looked up anywhere, so there are no backwards compatibility concerns." | 22:45 |
superdan | yeah, I'm just trying to figure out why we didn't try to handle the explosion | 22:46 |
superdan | but I'll just stop asking questions I guess | 22:47 |
mriedem | i'm assuming no one just ever thought of it | 22:48 |
superdan | in newton, we were listing buildreqs and putting them in front of the instance list, | 22:50 |
cfriesen | I'm trying to run devstack on master and getting this: | 22:50 |
cfriesen | pkg_resources.VersionConflict: (packaging 16.5 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('packaging>=16.8')) | 22:50 |
superdan | and presumably this but means BuildRequestList.get_all() would fail to yield anything until you run this migration | 22:50 |
cfriesen | anyone know what's going on there? | 22:50 |
superdan | which makes the api broken until you get this online migration | 22:50 |
superdan | but we could have just made BuildRequestList.get_all() ignore a known exception there and log | 22:51 |
superdan | which seems like an obvious fix, so I'm just wondering why we didn't do it | 22:51 |
*** burt has quit IRC | 22:55 | |
*** david-lyle has joined #openstack-nova | 22:55 | |
mriedem | i also backported https://review.openstack.org/#/c/420051/ and i'm not sure if we care or not now | 22:55 |
*** lpetrut has quit IRC | 22:58 | |
mriedem | i think we want all of these in newton if we want people to use placement before ocata https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:bug/1647316 | 22:58 |
mriedem | which, maybe we care less about now | 22:58 |
mriedem | anyway, gotta run now | 22:58 |
*** jamielennox is now known as jamielennox|away | 22:58 | |
superdan | I care bigly | 22:59 |
*** Swami has quit IRC | 22:59 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova-specs master: Add prep-for-network-aware-scheduling-pike spec https://review.openstack.org/432489 | 23:01 |
*** catintheroof has quit IRC | 23:03 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Add and use ProcessedNetworkRequest object https://review.openstack.org/418025 | 23:04 |
*** david-lyle has quit IRC | 23:04 | |
*** catintheroof has joined #openstack-nova | 23:04 | |
*** ijw has quit IRC | 23:06 | |
*** ijw has joined #openstack-nova | 23:07 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Pass objects into _validate_requested_network_ids https://review.openstack.org/418027 | 23:07 |
*** Apoorva has joined #openstack-nova | 23:07 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Pass objects into _create_ports_for_instance https://review.openstack.org/418512 | 23:08 |
*** catintheroof has quit IRC | 23:08 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Pass objects into _update_ports_for_instance https://review.openstack.org/418513 | 23:09 |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Extract preallocate_for_instance https://review.openstack.org/418876 | 23:10 |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Tidy up preallocate_for_instance https://review.openstack.org/419978 | 23:11 |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Fix no networks preallocate_for_instance return https://review.openstack.org/419979 | 23:12 |
*** figleaf is now known as edleafe | 23:13 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: WIP: introduce preallocate_for_instance method. https://review.openstack.org/418881 | 23:13 |
*** nic has quit IRC | 23:13 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: WIP: Call preallocate_for_instance before scheduler https://review.openstack.org/413124 | 23:14 |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: WIP: Add processed_requests to allocate_for_instance https://review.openstack.org/420103 | 23:15 |
*** jaosorior has quit IRC | 23:15 | |
*** gouthamr has joined #openstack-nova | 23:21 | |
*** marst has quit IRC | 23:22 | |
*** lx_ has quit IRC | 23:23 | |
*** dimtruck is now known as zz_dimtruck | 23:24 | |
cfriesen | anyone around? I'm getting this in current devstack: pkg_resources.VersionConflict: (packaging 16.5 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('packaging>=16.8')) and I'm not sure where the requirement for 16.8 is coming from given that /opt/stack/requirements/upper-constraints.txt says 16.5 | 23:28 |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662 | 23:34 |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662 | 23:34 |
*** jwcroppe has quit IRC | 23:34 | |
*** jwcroppe has joined #openstack-nova | 23:35 | |
*** crushil has joined #openstack-nova | 23:35 | |
*** ssurana has joined #openstack-nova | 23:37 | |
*** jwcroppe has quit IRC | 23:39 | |
*** Swami has joined #openstack-nova | 23:43 | |
*** slaweq has quit IRC | 23:45 | |
*** adreznec has quit IRC | 23:47 | |
*** adreznec has joined #openstack-nova | 23:47 | |
*** slaweq has joined #openstack-nova | 23:48 | |
*** jamielennox|away is now known as jamielennox | 23:50 | |
*** ducttape_ has quit IRC | 23:51 | |
mriedem | cfriesen: usually the thing being installed right before it is what's pulling it in | 23:54 |
*** ssurana has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!