| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-01 | |||
| 16:11:27 | bauzas | like I said in the bug report, the bug was fixed implicitly by https://review.openstack.org/#/c/469037/ but we can't just backport it | |
| 16:12:10 | mriedem | bauzas: if it's fixed on master, then yes it's a stable only change | |
| 16:12:32 | bauzas | okay, lemme upload then and we'll discuss | |
| 16:12:54 | mriedem | unless you just write the regression test for master but no fix | |
| 16:13:03 | mriedem | as you said, the backport of the test would be weird | |
| 16:15:05 | bauzas | anyway, looks difficult to discuss unless I'm uploading it | |
| 16:15:09 | bauzas | so I'll do | |
| 16:18:00 | mriedem | does that mean sylvain's wife is variable? | |
| 16:18:08 | openstackgerrit | Sean Dague proposed openstack/nova master: add redirects for existing broken docs urls https://review.openstack.org/489650 | |
| 16:18:14 | sdague | mriedem: ^^^ | |
| 16:18:41 | sdague | mriedem: also, here are real 404s from today, which include stuff going way back - http://paste.openstack.org/show/617142/ | |
| 16:18:44 | bauzas | mriedem: oh oops, WIFE + $idea | |
| 16:18:52 | bauzas | $ikea even | |
| 16:18:54 | mdbooth | mriedem: *potentially* variable. In practise it may not actually be modified after assignment. | |
| 16:19:36 | mriedem | sdague: can those tabs be fixed? | |
| 16:19:49 | sdague | mriedem: sure | |
| 16:20:14 | openstackgerrit | Dan Smith proposed openstack/nova master: Add some more cellsv2 doc goodness https://review.openstack.org/487183 | |
| 16:20:43 | openstackgerrit | Sean Dague proposed openstack/nova master: add redirects for existing broken docs urls https://review.openstack.org/489650 | |
| 16:21:15 | mdbooth | So I can rebase 7 patches on top of the revert function tests | |
| 16:22:28 | sdague | mriedem: I'm going to walk away for a bit, but I can rebase sfinucan's patches later today probably, and add any more redirects if we need them | |
| 16:22:51 | sdague | although I think most of the moves already happened | |
| 16:24:08 | openstackgerrit | Matthew Booth proposed openstack/nova master: Split Compute.errors_out_migration into a separate contextmanager https://review.openstack.org/485734 | |
| 16:24:08 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in resize_instance https://review.openstack.org/487495 | |
| 16:24:09 | openstackgerrit | Matthew Booth proposed openstack/nova master: fake_notifier: allow waiting on either notification type https://review.openstack.org/489637 | |
| 16:24:09 | openstackgerrit | Matthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination https://review.openstack.org/462521 | |
| 16:24:10 | openstackgerrit | Matthew Booth proposed openstack/nova master: Ensure errors_out_migration errors out migration https://review.openstack.org/479802 | |
| 16:24:10 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in finish_resize https://review.openstack.org/487515 | |
| 16:24:11 | openstackgerrit | Matthew Booth proposed openstack/nova master: Functional test for auto revert resize https://review.openstack.org/489654 | |
| 16:24:25 | mdbooth | Meh, at least now I have something to point people at | |
| 16:33:42 | openstackgerrit | Merged openstack/python-novaclient master: Remove custom autodoc implementation https://review.openstack.org/480262 | |
| 16:39:06 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/489604 | |
| 16:39:31 | mriedem | dansmith: jaypipes: i'm +2 on gibi's test now https://review.openstack.org/#/c/487958/ - should i just push that through? | |
| 16:39:39 | mriedem | since you both worked on it and sean is afk | |
| 16:39:53 | dansmith | mriedem: I'm not likely to +W it in its current form, so probably | |
| 16:39:59 | mriedem | ok | |
| 16:40:01 | jaypipes | mriedem: actually I'd like to add one more assertion in there. | |
| 16:40:28 | mriedem | jaypipes: can it go on top? | |
| 16:40:35 | jaypipes | mriedem: sure I suppose | |
| 16:40:47 | mriedem | i just want to stop iterating on this one | |
| 16:40:55 | jaypipes | ok with me | |
| 16:40:57 | mriedem | but if you're ready like now then i can wait | |
| 16:41:18 | jaypipes | mriedem: please proceed. | |
| 16:41:57 | mriedem | done | |
| 16:43:00 | cdent | jaypipes: has a plan (for migrations) distilled out of today’s conversations yet, or is that still pending? | |
| 16:43:32 | jaypipes | cdent: 1707071 is the primary bug I'm tracking right now. | |
| 16:44:29 | cdent | jaypipes: yes, but I thought any potential solutions to that impinge on the wider migration picture? | |
| 16:45:06 | jaypipes | cdent: not quite following you on that... | |
| 16:47:30 | cdent | jaypipes: maybe I misunderstood your answer to my question. Are you saying “yes, that’s what 1707071 is about” or “no, I’m focused on the details of making sure that we create the right allocations”? Looks like the former? | |
| 16:48:05 | jaypipes | cdent: I'm focused on trying to not have Pike compute node RTs fight each other writing and overwriting allocation records. | |
| 16:48:06 | cdent | Earlier today you said that overnight you fought with the code enough to think that you had to rejigger everything again. | |
| 16:48:42 | jaypipes | cdent: what I said earlier we mostly referring to needing to essentially put a bunch of conditionals in to keep being broken for Ocata compute hosts migrating to Pike compute hosts. | |
| 16:49:43 | cdent | So sounds like you have some kind of plan. That’s great. As before, if there’s something I can do, let me know. | |
| 16:50:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Accept any scheduler driver entrypoint https://review.openstack.org/484828 | |
| 16:50:17 | openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Refactor get_endpoint_data to accept service_type https://review.openstack.org/489666 | |
| 16:50:28 | jaypipes | cdent: will do. | |
| 16:56:16 | bks | Hi all, i wanted to run a clou-init script in centos 6.6 os to change the root password, can someone please pass me the cloud-init to do it. Thanks | |
| 16:56:34 | cfriesen_ | in Newton would we expect a deleted instance (which is in the "instances" table) to always have an entry in instance_extra? | |
| 16:56:59 | bks | and is used_data option is used in the nova create server api to pass the cloud-init? | |
| 16:57:18 | bks | can someone please help me here | |
| 16:58:04 | cfriesen_ | bks: your question should probably be asked on #openstack | |
| 17:00:19 | bks | @cfriesen_ i asked there too, no response | |
| 17:00:54 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add ``HostPortProfileInfo`` class https://review.openstack.org/441590 | |
| 17:02:08 | mriedem | yes user_data is passed through to cloud-init | |
| 17:02:12 | cfriesen | bks: many of the nova folks are busy with release-critical stuff. have you looked at: http://blog.scottlowe.org/2015/11/09/changing-passwords-cloud-init/ | |
| 17:05:03 | bks | @cfriesen ohh sorry, iam trying the same script but couldnot get it worked | |
| 17:05:37 | bks | @cfriesen the userdata option in createserver is used to pass the cloud-init? | |
| 17:06:06 | bks | as mentioned in the https://developer.openstack.org/api-ref/compute/#create-server | |
| 17:06:54 | mriedem | https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/user-guide/source/cli-config-drive.rst | |
| 17:08:53 | cburgess | Because.....? | |
| 17:10:29 | openstackgerrit | Eric Fried proposed openstack/nova master: Adopt new pypowervm power_off APIs https://review.openstack.org/476945 | |
| 17:20:21 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/457711 | |
| 17:42:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: See if we can squash ComputeHostNotFound_Remote on startup https://review.openstack.org/489683 | |
| 17:42:50 | mriedem | dansmith: ^ destroys unit tests for the objects b/c of the fake indirection api, but is that something that's even possible? ^ | |
| 17:43:26 | mriedem | i want to get rid of these http://logs.openstack.org/54/487954/5/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/4412f5d/logs/screen-n-cpu.txt.gz?level=TRACE#_Jul_28_19_41_38_466972 | |
| 17:43:37 | dansmith | mriedem: no because those aren't actually the rpc calls | |
| 17:43:45 | cdent | Is stephenfin around this week, I sense his absence | |
| 17:43:46 | dansmith | well, I'm guessing that won't work because ^ | |
| 17:43:57 | mriedem | cdent: i'm told he's back tomorrow | |
| 17:44:04 | cdent | thanks mriedem | |
| 17:45:48 | dansmith | mriedem: that line you quote is intentional log.warning in the libvirt driver | |
| 17:46:02 | dansmith | mriedem: and it looks like we're getting a stack trace automagically from the logging layer (?) | |
| 17:46:05 | mriedem | i know, but the trace is coming from oslo.messaging isn't it? | |
| 17:46:11 | dansmith | no I don't think so | |
| 17:47:06 | dansmith | mriedem: see the line below.. you need exc_info=False maybe ? | |
| 17:47:09 | dansmith | maybe a default changed somehow? | |
| 17:47:12 | dansmith | oh | |
| 17:47:14 | dansmith | ooooh | |
| 17:47:17 | dansmith | you're hitting the lower one | |
| 17:47:19 | dansmith | with exc_info=True | |
| 17:47:25 | dansmith | because it is _Remote | |
| 17:47:52 | mriedem | dansmith: yeah there is that, | |
| 17:47:54 | dansmith | so maybe in that handler, check the exception name and if it looks remoteish then you can not pass the exc_info flag? | |
| 17:47:57 | mriedem | but that doesn't explain this one http://logs.openstack.org/54/487954/5/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/4412f5d/logs/screen-n-cpu.txt.gz?level=TRACE#_Jul_28_19_41_40_927881 | |
| 17:47:59 | dansmith | it's a little icky | |
| 17:48:33 | mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6624 | |
| 17:48:40 | mriedem | oh well, | |
| 17:48:51 | mriedem | maybe we don't hit that code b/c of the same thing? | |
| 17:49:45 | dansmith | well, you log that message, so it seems like you are | |
| 17:49:50 | dansmith | not sure why you get a trace there then | |
| 17:49:53 | mriedem | but where does the traceback come from? | |