| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-11 | |||
| 14:22:09 | jangutter | I was more thinking of writing my life story with an acrostic, but those suggestions are good too. | |
| 14:23:23 | aspiers | X-D | |
| 14:43:08 | openstackgerrit | Jan Gutter proposed openstack/os-vif master: Reflow docs to 79 columns https://review.openstack.org/609726 | |
| 14:43:59 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | |
| 14:44:00 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Handle allocations consuming only from the child RPs https://review.openstack.org/608298 | |
| 14:45:00 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Handle allocations consuming only from the child RPs https://review.openstack.org/608298 | |
| 14:57:16 | jangutter | I think the only time formatting and whitespace changes are legit allowed to be mixed is in the IOCCC | |
| 14:57:25 | sean-k-mooney | jangutter: for the record i also hate that we enforce a 79 charater colum limit | |
| 14:58:06 | jangutter | sean-k-mooney: you prefer wider or narrower? | |
| 14:58:39 | sean-k-mooney | well one wider but too the limit was ment to be 80 charaters but someone decided to enforce 79 instead | |
| 14:59:14 | jangutter | sean-k-mooney: could be worse, could be Fortran. | |
| 15:00:03 | sean-k-mooney | frotrans standard is 132 i would be fine with that | |
| 15:01:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup https://review.openstack.org/606050 | |
| 15:01:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject forced move with nested source allocation https://review.openstack.org/605785 | |
| 15:01:40 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | |
| 15:01:40 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Handle allocations consuming only from the child RPs https://review.openstack.org/608298 | |
| 15:08:17 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware: OVA and StrOpt images as VM templates https://review.openstack.org/609736 | |
| 15:30:42 | artom | So, I'm trying to write a new func tests that boots a server with a volume using the Cinder fixture, and I'm hitting 401 errors: http://paste.openstack.org/show/731904/ | |
| 15:31:09 | artom | As far as I can tell I'm going the same things as other existing tests | |
| 15:32:30 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add support for microversion 2.67: BDMv2 volume_type https://review.openstack.org/609743 | |
| 15:47:05 | mriedem | dansmith: huh, something must have regressed devstack's setup for nova-cpu.conf because it has db setup in it now... http://logs.openstack.org/87/604687/6/check/tempest-full/5e2a539/controller/logs/etc/nova/nova-cpu_conf.txt.gz | |
| 15:47:29 | dansmith | mriedem: hmm, I just installed a fresh devstack the other day and it didn't | |
| 15:47:46 | mriedem | i noticed this on a devstack i just created this morning | |
| 15:47:58 | dansmith | and pointing at cell0 | |
| 15:48:51 | mriedem | right - not that nova-compute should be trying to hit a db either way, but those shouldn't be in the cpu config | |
| 15:49:00 | dansmith | maybe melwitt's console patch? | |
| 15:49:05 | dansmith | definitely | |
| 15:49:25 | mriedem | https://github.com/openstack-dev/devstack/blob/master/lib/nova#L918 ? | |
| 15:49:49 | mriedem | cell1 conf looks good http://logs.openstack.org/87/604687/6/check/tempest-full/5e2a539/controller/logs/etc/nova/nova_cell1_conf.txt.gz | |
| 15:50:27 | dansmith | that line is old | |
| 15:50:33 | mriedem | yeah, | |
| 15:50:39 | mriedem | it's also wrong in nova-cpu.conf on queens http://logs.openstack.org/55/606155/1/check/tempest-full/dcd0005/controller/logs/etc/nova/nova-cpu_conf.txt.gz | |
| 15:50:45 | mriedem | so it's not mel's console config change | |
| 15:50:49 | mriedem | maybe just always busted? | |
| 15:51:03 | dansmith | mm, I dunno | |
| 15:51:06 | mriedem | looks like that function is copying nova-cpu.conf from $NOVA_CONF which is nova.conf | |
| 15:51:09 | mriedem | which is the controller config | |
| 15:51:46 | mriedem | anywho, not a huge deal since we don't have local conductor | |
| 15:51:50 | mriedem | i was just confused | |
| 15:51:58 | mriedem | b/c i was able to sync the cell db using nova-cpu.conf | |
| 15:52:00 | mriedem | which shouldn't work | |
| 15:52:36 | dansmith | definitely wrong though.. I can't imagine that's been broken like that for so long | |
| 15:53:29 | dansmith | it's wrong in a result from sep 26 at least | |
| 15:55:52 | dansmith | I was probably looking at my subnode cpu conf recently actually | |
| 15:56:15 | dansmith | but surely looks like it's been this way | |
| 15:56:46 | mriedem | yeah, reported a bug anyway https://bugs.launchpad.net/devstack/+bug/1797413 | |
| 15:56:46 | openstack | Launchpad bug 1797413 in devstack "nova-cpu.conf shouldn't have database config in it" [Low,Triaged] | |
| 15:56:48 | mriedem | for an intrepid contributor | |
| 15:57:13 | dansmith | I shall propose a fix | |
| 15:57:21 | mriedem | i just wouldn't want someone new to spin up devstack and look and think "oh nova-compute needs db access" | |
| 15:57:44 | dansmith | well, we also want to make sure we're not cheating on this somehow | |
| 15:57:52 | mriedem | right | |
| 16:08:06 | mriedem | you know, the service catalog clearly needs more cinder entries | |
| 16:22:43 | melwitt | ||
| 16:25:13 | aspiers | melwitt: I should have the spec submitted in the next few hours | |
| 16:25:26 | melwitt | ok, cool | |
| 16:32:44 | mriedem | heh oops http://paste.openstack.org/show/731907/ | |
| 16:33:06 | mriedem | installed new versioned object code, restarted computes to new code but forgot to restart conductor | |
| 16:33:42 | openstackgerrit | Georg Hoesch proposed openstack/nova master: handle logfiles for tcp-based consoles https://review.openstack.org/575735 | |
| 16:38:01 | mriedem | welp, bfv with volume type works http://paste.openstack.org/show/731908/ | |
| 16:43:04 | mriedem | others might want to start reviewing that series, | |
| 16:43:09 | mriedem | it ends on the 14th | |
| 16:43:31 | melwitt | oof ok | |
| 16:45:41 | sean-k-mooney | mriedem_afk: that is booting with a precreated voluem or you are asking for a new volume to be created? i never do boot from volume via cli just horizon so just interested in understanding how it works | |
| 16:46:35 | sean-k-mooney | im guessin --block-device id=b9cdd407-14fc-42fb-beb9-d53988d866e3,source=image,dest=volume,size=1,bootindex=0,shutdown=remove,volume_type=lvmdriver-1 mean create an volume of 1GB form this image and boot with it | |
| 16:47:04 | melwitt | yes | |
| 16:48:39 | sean-k-mooney | cool the shutdown=remove is sligtly odd. that is so we can swap the root disk when the insance is shutdown im guessing? | |
| 16:49:08 | dansmith | shutdown=remove means delete the volume when you delete the instance, and that's been around a long time | |
| 16:49:29 | melwitt | yeah. it maps to delete_on_termination=True in the code | |
| 16:50:01 | sean-k-mooney | i see delete_on_termination=True convayes that much more cleanly | |
| 16:58:36 | openstackgerrit | melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068 | |
| 16:59:10 | melwitt | mriedem_afk: I added the code comment ^ | |
| 17:26:48 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517 | |
| 17:35:59 | dansmith | mriedem_afk: I feel like you could probably look at this and judge it quickly: https://review.openstack.org/#/c/609517/4 | |
| 17:36:14 | dansmith | error handling in volume detach during live migration.. two of your favorite things :) | |
| 17:48:00 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517 | |
| 18:03:40 | aspiers | following up from the whitespace discussion earlier, are typo fixes to documentation more welcomed than hated? | |
| 18:05:11 | melwitt | typo fixes to user-facing documentation are helpful | |
| 18:05:19 | jaypipes | aspiers: I think it depends on the reviewer. Personally, I'm fine with such fixes and encourage them. Increasing the quality of our docs is important to me. | |
| 18:05:41 | aspiers | thanks, submitting :) | |
| 18:06:21 | openstackgerrit | Adam Spiers proposed openstack/nova master: fix "you" typo https://review.openstack.org/609777 | |
| 18:08:03 | jaypipes | aspiers: -3. | |
| 18:08:11 | jaypipes | aspiers: j/k :) | |
| 18:08:14 | aspiers | lol | |
| 18:08:21 | imacdonn | wow .. -3 ... yikes! | |
| 18:08:34 | jaypipes | :) | |
| 18:08:34 | aspiers | thanks, now I have to change my underwear | |
| 18:08:38 | jaypipes | +Wallaby'd | |
| 18:08:47 | aspiers | X-D | |
| 18:09:49 | aspiers | that'll push me up from being # 97,381 in the stackalytics rankings to # 97,380 \o/ | |
| 18:10:05 | imacdonn | congrats ;) | |
| 18:10:16 | aspiers | and we all know success in life is defined by stackalytics | |
| 18:10:59 | mriedem | dansmith: i'll be the judge of how quickly i can judge something | |
| 18:11:11 | dansmith | aspiers: yeah my last loan officer said "aand, just one more thing, what's your stackalytics ranking?" | |
| 18:11:19 | aspiers | hahaha | |
| 18:11:29 | dansmith | mriedem: I should hope so | |
| 18:12:10 | aspiers | I barely squeaked through CBP at Denver airpor because my stackalytics were so poor | |
| 18:13:55 | imacdonn | dansmith sean-k-mooney mriedem: please rereview ... note that I had to tweak the logic a bit (see comment in review) ... https://review.openstack.org/#/c/608091/ | |
| 18:14:45 | aspiers | dansmith: luckily my UK passport will let me into Berlin for the next few months at least :-/ | |
| 18:14:55 | dansmith | heh | |