| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-22 | |||
| 23:41:27 | melwitt | (corrects self) er, or there shouldn't be two cells with dupe DB connection or dupe MQ URL | |
| 23:41:28 | mriedem | sean-k-mooney: well you'll be happy to know it's just a json blob of whackiness so it doesn't matter https://github.com/openstack/nova/blob/master/nova/objects/instance_pci_requests.py#L32 | |
| 23:41:54 | openstackgerrit | Merged openstack/nova master: Generalize DB conf group copying https://review.openstack.org/484908 | |
| 23:41:55 | sean-k-mooney | mriedem: thats what im changing at stephenfin request | |
| 23:42:17 | openstackgerrit | Merged openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | |
| 23:42:32 | sean-k-mooney | mriedem: which is why my tests are failing because i dont allow device id since the spec says it product_id | |
| 23:43:08 | mriedem | ok | |
| 23:43:10 | mriedem | ... | |
| 23:43:53 | sean-k-mooney | mriedem: im reworking https://review.openstack.org/#/c/449257 with stephenfin comments since rodolfo had to move on to opnfv work | |
| 23:44:30 | openstackgerrit | melanie witt proposed openstack/nova master: Detach volume after deleting instance with no host https://review.openstack.org/340614 | |
| 23:45:00 | sean-k-mooney | i think this is really just at test bug but it still hurts to look at the git blame. anyway i think ill get back to this in the morning since its almost midnight my time | |
| 23:57:37 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | |
| #openstack-nova - 2018-01-23 | |||
| 00:07:38 | mriedem | i'm +2 on the OSC CLI change for show/set/delete allocations https://review.openstack.org/#/c/457534/ - i think that's a good one to get in before the final client release freeze this week (same day as FF) | |
| 00:08:12 | mriedem | best way to review these i've found is pull them down to your devstack and kick the tires | |
| 00:10:31 | melwitt | mriedem: is there something special that has to be done to use the osc-placement plugin with OSC? I tried to use it last week and failed to figure it out. there were no 'openstack placement' or 'openstack resource' commands available | |
| 00:10:54 | mriedem | melwitt: git clone the repo, | |
| 00:10:57 | mriedem | then pip install it | |
| 00:11:02 | mriedem | then just: openstack resource provider -? | |
| 00:11:05 | mriedem | for the list of commands | |
| 00:11:36 | melwitt | okay. in my devstack it was installed and I even tried pip removing and pip installing it again with no luck. I'll try it again | |
| 00:12:14 | melwitt | that is, the osc-placement package got installed by devstack by itself (I didn't do anything to install it manually) | |
| 00:12:20 | mriedem | hmm, did you have the latest version of the osc-placement repo? | |
| 00:12:25 | mriedem | oh... | |
| 00:12:33 | melwitt | no, it took whatever was on pypi | |
| 00:12:34 | mriedem | i always git clone it into /opt/stack/ | |
| 00:12:41 | melwitt | good to know | |
| 00:12:45 | mriedem | the pypi version doesn't have any actual CLIs yet | |
| 00:12:55 | melwitt | oh, no wonder. hah | |
| 00:13:01 | mriedem | i'm waiting to get at least this last allocations one merged before we release for queens | |
| 00:14:13 | mriedem | ideally we could get to https://review.openstack.org/#/c/511182/ yet for queens | |
| 00:14:29 | mriedem | because not being able to work with resource classes in the CLI makes the other existing CLIs can't of hard | |
| 00:14:31 | mriedem | *kind of hard | |
| 00:14:58 | melwitt | ack | |
| 00:45:03 | openstackgerrit | Gao Fei proposed openstack/nova master: Redirects to a valid https link from the old one https://review.openstack.org/535244 | |
| 01:54:37 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918 | |
| 02:12:18 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | |
| 02:15:50 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | |
| 02:19:33 | Kevin_Zheng | mriedem https://review.openstack.org/525505 <- rebased due to conflict in test_compute_api.py | |
| 02:24:52 | openstackgerrit | Merged openstack/osc-placement master: CLI for allocations https://review.openstack.org/457534 | |
| 02:30:14 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918 | |
| 02:32:37 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types https://review.openstack.org/430839 | |
| 02:59:52 | openstackgerrit | Merged openstack/nova master: Fix update_cell to ignore existing identical cells https://review.openstack.org/536546 | |
| 03:00:01 | openstackgerrit | Merged openstack/nova master: Start moving users of parted to privsep. https://review.openstack.org/519011 | |
| 03:59:03 | openstackgerrit | Merged openstack/nova master: Remove redundant swap_volume tests https://review.openstack.org/531179 | |
| 05:19:13 | takashin | ||
| 05:52:53 | fanzhang | hi anyone here | |
| 05:56:33 | fanzhang | I hit a H202 error while running tox pep8 check, it's because my unit test using something like self.assertRaises(Exception,... I was wondering is H202 a mandatory rule or not? Please, if anyone knows about that, please do me a favor :) | |
| 06:02:51 | jianghuaw | fanzhang, it's mandatory I think. You should define a specific exception instead of using the general Exception. | |
| 06:03:44 | fanzhang | cool, thanks a lot. jianghuaw :) | |
| 06:03:54 | jianghuaw | np | |
| 06:30:51 | Spazmotic | You're actually here jianghuaw? :D Didn't think you hung out on IRC | |
| 06:31:08 | Spazmotic | I saw you put some XenAPI notes on the PTG, Did you plan on attending Dublin? | |
| 06:31:39 | Spazmotic | Hoping to get Corporate approval to attend, if so, would be nice to talk to another person interested in the XenAPI drivers | |
| 06:31:43 | jianghuaw | Spazmotic, yes. I'm planning to attend the PTG. | |
| 06:32:00 | jianghuaw | Spazmotic, thanks. | |
| 06:32:45 | jianghuaw | Spazmotic, hope I can see you in Dublin. | |
| 06:32:54 | Spazmotic | Well wonderful to meet you. If you ever want to talk i'm around this time of night. I remain on US Night shift / Seoul time :p | |
| 06:33:46 | jianghuaw | nice to know:-) | |
| 06:59:03 | Spazmotic | Some Tuesday morning KPOP before we start waking up and working on actual serious things. Enjoy your days friends, i'll be around. https://www.youtube.com/watch?v=Ri6wvGjuoOg | |
| 07:51:43 | ameeda | Morning | |
| 07:52:04 | ameeda | what can I do with this ERROR log "http://paste.openstack.org/show/650903/" | |
| 08:15:05 | fanzhang | hi ameeda, I met this error some days earlier, updating your sqlite to sqlite-3.11.0 and your sqlite-libs to sqlite-libs-3.11.0 fixed my problem | |
| 08:15:46 | fanzhang | I mean it could help :) | |
| 08:31:52 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899 | |
| 08:39:15 | ameeda | fanzhang: thanks , but how can I update that ? | |
| 08:46:42 | fanzhang | ameeda: this part is where you should google it. :) I'll suggest that you get the right rpm packages and just install them. | |
| 08:58:35 | ameeda | fanzhang: hehe, ok thank you very much ! | |
| 08:59:05 | fanzhang | ameeda: np | |
| 08:59:17 | ameeda | fanzhang: but you know, when I add my code the error appears, is that caused by my code ? | |
| 09:00:50 | fanzhang | ameeda: I'm not sure. Have you tested before adding your codes? I don't know, you can surely try to find out. :) | |
| 09:01:25 | bauzas | good morning Novza | |
| 09:01:27 | ameeda | yes I test it without my code, I didn't see the error. | |
| 09:04:58 | openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | |
| 09:06:22 | fanzhang | ameeda: well, actually, I never test it before addming my codes, but updating sqlite related rpms solved my problem. | |
| 09:06:52 | ameeda | fanzhang: so let me check that, thanks ! | |
| 09:07:21 | fanzhang | ameeda: :) | |
| 09:08:24 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status https://review.openstack.org/536722 | |
| 09:13:31 | Spazmotic | Hmm I like that | |
| 09:13:38 | Spazmotic | Would help with scheduler failures as well. | |
| 09:20:16 | ameeda | jaypipes: are you around? | |
| 09:21:26 | openstackgerrit | Gao Fei proposed openstack/nova master: Replace Chinese punctuation with English punctuation https://review.openstack.org/536727 | |
| 09:22:19 | ameeda | any work-flow can check this for me ? "https://review.openstack.org/#/c/526900/" | |
| 09:25:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Replace curly quotes with straight quotes https://review.openstack.org/536727 | |
| 09:27:09 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: WIP: VMware: implement update_provider_tree() https://review.openstack.org/536348 | |
| 09:33:03 | bauzas | jianghuaw: around ? | |
| 09:33:40 | bauzas | jianghuaw: I don't know if you tested your vGPU patches with all the instance actions | |
| 09:34:23 | bauzas | jianghuaw: but see my relnote for libvirt https://review.openstack.org/#/c/533818/5/releasenotes/notes/add-support-for-vgpu-libvirt-91d2983e643f5ff1.yaml | |
| 09:34:41 | bauzas | jianghuaw: IMHO, you should be impacted by possibly the same problems | |
| 09:34:47 | bauzas | like for migrating | |
| 09:34:55 | bauzas | or resizing | |
| 09:35:54 | gibi | esberglu_: thanks for the update, I'm +2 on the SEA patch | |
| 09:37:17 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: using the dict format for the allocations https://review.openstack.org/536083 | |
| 09:37:18 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085 | |
| 09:40:00 | Spazmotic | Morning UK Nova | |
| 09:40:04 | Spazmotic | good to see you all :) | |
| 09:40:26 | alex_xu | gibi: bauzas ^, I just fix the functional tests | |
| 09:40:41 | Spazmotic | jianghuaw had some meetings, fyi bauzas | |
| 09:41:00 | alex_xu | bauzas: also there need one more +2 https://review.openstack.org/#/c/535642/4 :) appreciate you can help it | |
| 09:42:30 | Spazmotic | If anyone gets a chance with XenAPI knowledge, would love a look at https://review.openstack.org/#/c/533168/. Been sitting idle for about 12 days | |
| 09:46:11 | bauzas | Spazmotic: ok, thanks | |