| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-04 | |||
| 21:07:57 | edleafe | crap, looks like cdent has left | |
| 21:07:59 | mtreinish | johnsom: that's independent. THat's just service logs, because the services are running under systemd | |
| 21:08:12 | mtreinish | console logs are grabbed from the virt driver | |
| 21:08:18 | edleafe | I don't see where he is referring to in his comment. dansmith? | |
| 21:08:30 | johnsom | Yeah, I wouldn't have expected it. Especially because there is a zero byte console.log file sitting there | |
| 21:08:45 | mtreinish | johnsom: I would check the n-cpu log | |
| 21:09:00 | edleafe | oh, nvm | |
| 21:09:17 | edleafe | I had the unit test test_resource_tracker.py open, not the functional | |
| 21:09:22 | openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor instance-exists comment and test https://review.openstack.org/462738 | |
| 21:10:51 | dansmith | edleafe: I didn't look close, I just read what he said and thought "mocks, okay, complicated" | |
| 21:11:03 | edleafe | heh | |
| 21:11:18 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | |
| 21:11:23 | openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor instance-exists comment and test https://review.openstack.org/462738 | |
| 21:11:31 | johnsom | Yeah, I started there. No ERROR records and it shows 2 vcpus allocated. It looks normal to me. Ok, I will dig in. I just didn't want to burn a bunch of time if this was a known issue. | |
| 21:12:13 | dansmith | edleafe: assume he means this: https://pastebin.com/x6MJut7P | |
| 21:12:32 | openstackgerrit | Ed Leafe proposed openstack/nova master: Don't delete allocation if instance being scheduled https://review.openstack.org/460177 | |
| 21:12:33 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add a convenience attribute for reportclient https://review.openstack.org/462679 | |
| 21:12:45 | edleafe | dansmith: yep ^^ | |
| 21:18:07 | johnsom | mtreinish Ok, I am suspecting the new qemu packages from ubuntu. Those processes are sitting at 120% in top | |
| 21:37:33 | mtreinish | johnsom: that could be it | |
| 21:37:49 | johnsom | Yeah, digging, will try a rollback | |
| 21:47:14 | openstackgerrit | melanie witt proposed openstack/nova master: DNM: Revert "Use six.text_type() when logging Instance object" https://review.openstack.org/462744 | |
| 21:48:10 | openstackgerrit | Ed Leafe proposed openstack/nova master: WIP: Delete existing allocations for move operations or retries https://review.openstack.org/460433 | |
| 21:48:10 | openstackgerrit | Ed Leafe proposed openstack/nova master: WIP: Allocate instance by the conductor https://review.openstack.org/460455 | |
| 21:49:45 | mriedem | melwitt: that's not going to work | |
| 21:49:57 | mriedem | unless we have a ci job that uses LIBS_FROM_GIT=oslo.log | |
| 21:50:00 | mriedem | the depends-on won't work | |
| 21:50:20 | melwitt | mriedem: it won't make a new oslo.log? oh. I wasn't sure which things Depends-On works with | |
| 21:50:38 | mriedem | efried had the same question earlier this week | |
| 21:50:40 | mriedem | basically, | |
| 21:50:50 | mriedem | can't depends-on to project-config changes or libraries/clients | |
| 21:50:58 | mriedem | the latter requires LIBS_FROM_GIT | |
| 21:52:00 | efried | Good news is that it seems fairly straightforward to get a new release cut and through requirements. | |
| 21:52:39 | melwitt | mriedem: does Depends-On work on changes in other projects like cinder for example? | |
| 21:52:45 | mriedem | wtf are we not running the releasenotes job anymore? | |
| 21:52:49 | mriedem | melwitt: yes | |
| 21:53:24 | melwitt | okay. oh, right because devstack always pulls those from git | |
| 21:54:45 | melwitt | I was just curious what the log would look like with the Instance object if we didn't do anything to convert it, with the oslo.log change | |
| 21:55:09 | mriedem | melwitt: you could do a devstack change that sets LIBS_FROM_GIT=oslo.log, | |
| 21:55:13 | mriedem | and have that depends on the nova and oslo.log changes | |
| 21:55:14 | mriedem | i think | |
| 21:55:35 | melwitt | because the ovo repr encodes to ascii, which has the chars with the \x escapes in it. I guess it would just look like in my unit test, with \x in it | |
| 21:56:36 | johnsom | mtreinish Yep, downgrade packages work. | |
| 21:56:46 | johnsom | apt-get install seabios=1.8.2-1ubuntu1 qemu-system-sparc=1:2.5+dfsg-5ubuntu10.11 qemu-system-ppc=1:2.5+dfsg-5ubuntu10.11 qemu-system-arm=1:2.5+dfsg-5ubuntu10.11 qemu-kvm=1:2.5+dfsg-5ubuntu10.11 qemu-system-x86=1:2.5+dfsg-5ubuntu10.11 qemu-system-mips=1:2.5+dfsg-5ubuntu10.11 qemu-system-misc=1:2.5+dfsg-5ubuntu10.11 | |
| 21:56:53 | johnsom | restart nova and life is better | |
| 22:06:07 | mriedem | sdague: in https://review.openstack.org/#/c/435432/11/releasenotes/notes/project_id_validation-568d31c13c3ef735.yaml what does "If the user has enough permissions in user" mean? | |
| 22:06:10 | mriedem | the 'in user' part | |
| 22:14:09 | mriedem | nvm i'll clean it up | |
| 22:28:45 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Verify project id for flavor access calls https://review.openstack.org/435432 | |
| 22:30:56 | mriedem | oomichi: ^ i cleaned that one up | |
| 22:34:44 | oomichi | mriedem: thanks, lets close the bug | |
| 22:36:41 | dansmith | sdague: melwitt: shockingly, I now have logs for that instance | |
| 22:37:27 | melwitt | you mean with sdague's change you do? | |
| 22:38:24 | dansmith | melwitt: checkit, like I thought: http://logs.openstack.org/37/458537/10/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0601a5e/logs/screen-n-cauth.txt.gz#_May_04_20_47_48_620783 | |
| 22:38:37 | dansmith | melwitt: yeah | |
| 22:39:25 | melwitt | validate_console_port | |
| 22:40:02 | dansmith | dinnuh I say? | |
| 22:40:11 | melwitt | yeah | |
| 22:40:25 | dansmith | I shall flog that tomorrow | |
| 22:41:25 | melwitt | cool. somehow instance must not be targeted to the right MQ even though there's a target_cell call in there | |
| 22:44:16 | dansmith | something. | |
| 22:47:29 | melwitt | strange. not seeing what's wrong there | |
| 22:48:00 | mnaser | i've been flying around the openstack api but this might be a little crazy. is it possible to remove an existing ephermal drive and replace it by cinder volume... on an existing instance? | |
| 22:48:13 | melwitt | the context gets wiped out after the context manager exits, but all of the calls are under it | |
| 22:48:31 | mnaser | so converting to boot from cinder for an existing VM (bdm stuff relevant at all here?) | |
| 23:16:06 | jaypipes | mriedem, dansmith, edleafe, alex_xu: your prompt attention to https://review.openstack.org/#/q/topic:cleanups+project:openstack/os-traits would be appreciated. I'd like to cut a 0.3 os-traits release ASAP because I need to use it in some of the new placement work... | |
| 23:18:37 | dansmith | jaypipes: I -1'ed the fsck out of that last patch | |
| 23:18:38 | dansmith | utter garbage! | |
| 23:20:15 | jaypipes | dansmith: fixed. :) | |
| 23:23:43 | dansmith | jaypipes: that was almost a very embarrassing event | |
| 23:23:50 | jaypipes | :) | |
| 23:29:07 | mriedem | ha | |
| 23:29:13 | mriedem | got you both | |
| 23:30:21 | dansmith | mriedem: luckily I don't care about my ratio on os-traits, so impede progress all you want buddy :) | |
| 23:32:04 | mriedem | i'll always remember | |
| 23:32:17 | mriedem | like your love of pizza and cookies and complete sentences | |
| 23:32:50 | dansmith | heh | |
| 23:33:19 | mriedem | jaypipes: on the last one, | |
| 23:33:27 | mriedem | that regex isn't enforced in the api schema from what i can tell | |
| 23:33:32 | mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/trait.py#L29 | |
| 23:33:50 | mriedem | or does that just mean for custom traits? | |
| 23:34:04 | jaypipes | yeah, that's just custom traits. | |
| 23:34:25 | mriedem | well, https://review.openstack.org/#/c/462759/2/os_traits/hw/cpu/x86.py is a standard trait no? | |
| 23:35:22 | jaypipes | no | |
| 23:35:26 | jaypipes | that's a module. | |
| 23:35:56 | jaypipes | hw.cpu.x86.SSE42 is a standard trait *symbol*, but the trait string is HW_CPU_X86_SSE42 | |
| 23:36:46 | mriedem | but based on the api schema, | |
| 23:36:57 | mriedem | i could create a trait that's named HW_CPU_X86_AES-NI right? | |
| 23:38:12 | mriedem | or are you only allowed to create custom traits via the API? https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/trait.py#L99 | |
| 23:38:21 | mriedem | and standard traits come from the lib? | |
| 23:44:12 | jaypipes | mriedem: correct. | |
| 23:47:35 | jaypipes | mriedem: meh, fixed that README typo. | |
| 23:51:32 | jaypipes | mriedem: I just want to keep the standard trait naming consistent. | |
| 23:52:45 | mriedem | alright all done | |
| #openstack-nova - 2017-05-05 | |||
| 00:07:15 | openstackgerrit | melanie witt proposed openstack/nova master: Make NovaException format errors fatal for tests https://review.openstack.org/411615 | |
| 00:07:16 | openstackgerrit | melanie witt proposed openstack/nova master: Fix unit test exception KeyErrors https://review.openstack.org/462764 | |
| 00:07:54 | melwitt | mriedem: I revived my old attempt at the exception KeyErrors detection ^ I ordered the fixes after the detection patch for now to demonstrate that it works | |
| 00:13:34 | openstackgerrit | melanie witt proposed openstack/nova master: Fix unit test exception KeyErrors https://review.openstack.org/462764 | |
| 00:15:40 | jaypipes | mriedem: ty sir. | |
| 00:15:58 | jaypipes | mriedem: to cut a release, do I just git tag -a it? | |