Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-13
12:05:29 johnthetubaguy alex_xu: I think the behaviour is the key thing
12:05:39 johnthetubaguy alex_xu: so we should be good
12:05:54 alex_xu johnthetubaguy: ++, the behaviour is everyone expected
12:07:37 openstackgerrit sahid proposed openstack/nova: libvirt: ensure live-migration success https://review.openstack.org/409261
12:07:37 openstackgerrit sahid proposed openstack/nova: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260
12:13:48 openstackgerrit jichenjc proposed openstack/nova: make limit=0 means query 0 data https://review.openstack.org/419913
12:17:45 mdbooth lyarwood: Any reason for https://review.openstack.org/#/c/399735/ to depend on the previous 2 patches?
12:18:04 mdbooth Seems the middle 2 are about fixing something Rbd related?
12:18:44 lyarwood mdbooth: other than using self._guest.get_disk_by_target_device no I don't think so
12:32:04 openstackgerrit Stephen Finucane proposed openstack/nova: scheduler: Don't modify RequestSpec in filtering https://review.openstack.org/419479
12:36:47 openstackgerrit Emilien Macchi proposed openstack/nova: CI test - never merge https://review.openstack.org/419926
12:39:47 openstack Launchpad bug 1656276 in tripleo "Error running nova-manage cell_v2 simple_cell_setup when configuring nova with puppet-nova" [Critical,Triaged]
12:39:47 openstackgerrit Emilien Macchi proposed openstack/nova: Debug https://bugs.launchpad.net/tripleo/+bug/1656276 https://review.openstack.org/419926
12:41:45 openstack Launchpad bug 1656276 in tripleo "Error running nova-manage cell_v2 simple_cell_setup when configuring nova with puppet-nova" [Critical,Triaged]
12:41:45 EmilienM hey nova folks, we're having a new issue with a patch merged in Nova yesterday, https://review.openstack.org/#/c/409890/ - details are reported here: https://bugs.launchpad.net/tripleo/+bug/1656276 - any feedback is welcome
12:43:51 openstackgerrit Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540
12:55:04 dtp good morning. are quotas enforced in the database?
12:56:16 openstackgerrit Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413
13:13:19 openstackgerrit Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413
13:27:31 sahid JayF:
13:45:48 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] try py35 functional with a clean slate https://review.openstack.org/419968
13:45:51 openstackgerrit Vladyslav Drok proposed openstack/nova: Generate necessary network metadata for portgroups https://review.openstack.org/388756
13:48:34 cdent i'm back dims, in case that's of any use
13:50:10 dims cdent : hehe, sorry i had gotten distracted with some shiny stuff. so yes, "EncodeUTF8 middleware" sounds awesome. we could do it as a 2 step tango too, let this change in and then do the middleware. right?
13:50:49 dims cdent : i'am currently poking at what functional tests start working with this change
13:53:23 cdent dims: yeah two step tango would work well
13:54:30 dims cool thanks cdent. let me un-black list stuff some tests and get it ready later today.
13:54:44 cdent \o/
13:57:29 openstackgerrit Ed Leafe proposed openstack/nova: Fix missing RP generation update https://review.openstack.org/419137
13:59:10 openstackgerrit Vasyl Saienko proposed openstack/nova: WIP implement interface attach/detach in ironic virt driver https://review.openstack.org/419975
13:59:31 openstackgerrit Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488
14:06:42 openstackgerrit sahid proposed openstack/nova: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260
14:06:43 openstackgerrit sahid proposed openstack/nova: libvirt: ensure live-migration success https://review.openstack.org/409261
14:08:44 openstackgerrit John Garbutt proposed openstack/nova: WIP: Call preallocate_for_instance before scheduler https://review.openstack.org/413124
14:08:45 openstackgerrit John Garbutt proposed openstack/nova: Tidy up preallocate_for_instance https://review.openstack.org/419978
14:08:45 openstackgerrit John Garbutt proposed openstack/nova: WIP: introduce preallocate_for_instance method. https://review.openstack.org/418881
14:08:46 openstackgerrit John Garbutt proposed openstack/nova: Fix no networks preallocate_for_instance return https://review.openstack.org/419979
14:10:42 lyarwood mdbooth: just looking at the swap volume fix again, I think we do need the previous 2 patches still. _swap_volume only provides a file path so we can't lookup the disk in the domain using guest.get_disk. I'll squash these down now for review.
14:11:37 openstack Launchpad bug 1656276 in tripleo "Error running nova-manage cell_v2 simple_cell_setup when configuring nova with puppet-nova" [Critical,Triaged]
14:11:37 EmilienM bauzas: have you looked https://bugs.launchpad.net/nova/+bug/1656276 ?
14:24:00 openstackgerrit garyk proposed openstack/nova: VMware: retry network lookup if network not ready https://review.openstack.org/391778
14:25:26 openstackgerrit Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413
14:30:45 bauzas EmilienM: hola, sorry was afk due to 10cm of snow and my frontyark needing cleanup :)
14:30:51 bauzas frontyard* even
14:30:58 bauzas EmilienM: owalsh pointed me the issue
14:31:03 EmilienM bauzas: 10 cm only? :-P
14:31:22 bauzas we're in France, and you know how 10cm of snow can be terrible for us :p
14:31:29 bauzas (even living in the French alps)
14:31:39 bauzas anyway
14:31:40 EmilienM bauzas: ok cool, let me know if there is something we need to change in the tripleo workflow or if you spot something in nova itself
14:31:52 bauzas so, owalsh pointed me the sequence that puppet does
14:32:13 bauzas https://github.com/openstack/puppet-nova/blob/master/manifests/db/sync_cell_v2.pp#L41 means that it runs after API DB sync, right?
14:32:23 bauzas EmilienM: ^
14:32:24 lyarwood mdbooth: correction, we need them still thanks to _live_snapshot, I've updated the review with a comment.
14:32:35 EmilienM bauzas: right
14:32:53 bauzas EmilienM: okay, so you populated the DB, which is good
14:33:25 bauzas EmilienM: but you haven't started the compute nodes, right?
14:33:35 bauzas EmilienM: so they aren't registered in the Nova DB, correct?
14:33:39 EmilienM bauzas: that is true
14:33:41 bauzas okay
14:33:43 openstackgerrit Vasyl Saienko proposed openstack/nova: WIP implement interface attach/detach in ironic virt driver https://review.openstack.org/419975
14:33:56 bauzas EmilienM: in theory, that shouldn't be a problem
14:34:34 EmilienM bauzas: that was 2 weeks ago, wanna help? https://twitter.com/EmilienMacchi/status/814838163654975488
14:34:54 bauzas heh
14:36:38 bauzas EmilienM: the error occurs on a fresh new install, not an upgrade ?
14:36:44 openstackgerrit Vladyslav Drok proposed openstack/nova: Generate necessary network metadata for portgroups https://review.openstack.org/388756
14:36:47 EmilienM bauzas: fresh
14:36:56 bauzas ok
14:36:59 EmilienM bauzas: we haven't the CI job in place for upgrade yet (it's WIP)
14:37:08 bauzas ack
14:37:13 EmilienM actually, we have but it fails because of this cellv2 thing
14:37:18 EmilienM and mwhahaha is working on it
14:37:39 EmilienM https://review.openstack.org/#/q/topic:bug/1649341
14:39:16 bauzas EmilienM: so, the goal of that simple_cells command is to populate your cellsv2 database with the necessary mappings for the compute nodes being in cells
14:39:37 bauzas EmilienM: if you don't have nodes already, then you have no cell mapping AFAICT
14:40:06 bauzas but when you add a node, then you have to run again the simple cells command in order to assign it a cell
14:40:17 EmilienM why isn't it dynamic?
14:40:30 EmilienM why conductor or something wouldn't do it for us?
14:40:35 bauzas because it hasn't been done yet
14:40:42 EmilienM ah ok, it's in the roadmap
14:40:46 bauzas well
14:40:49 bauzas the fact is
14:41:14 bauzas only the operator can know what cell the node should be added to, nova can't guess for it except putting the node into a default cell
14:41:59 EmilienM we could have the cell name in the nova.conf on each compute node
14:42:15 EmilienM and compute would send this name to conductor which write in db
14:42:36 EmilienM so you can scale up/down cells dynamically without running nova-manage
14:42:50 bauzas that's a possibility but like I said, it's not written yet :)
14:42:56 EmilienM cool np
14:43:00 bauzas so
14:43:16 bauzas I'm trying to understand where exactly puppet fails
14:43:26 bauzas when running the simple cell command
14:43:35 bauzas EmilienM: do you have some trace that I can look at ?
14:43:49 openstack Launchpad bug 1656276 in tripleo "Error running nova-manage cell_v2 simple_cell_setup when configuring nova with puppet-nova" [Critical,Triaged]
14:43:49 bauzas I guess only https://bugs.launchpad.net/nova/+bug/1656276 ?
14:43:50 EmilienM it's running "nova-manage cell_v2 simple_cell_setup" as part of the nova database initialization when no compute nodes have been created and it returns 1
14:44:05 openstackgerrit Vladyslav Drok proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413
14:44:10 EmilienM yes there is no trace but I can easily give one to you, let me do something quickly
14:44:40 openstackgerrit Emilien Macchi proposed openstack/nova: Debug https://bugs.launchpad.net/tripleo/+bug/1656276 https://review.openstack.org/419926
14:45:25 EmilienM bauzas: i'm preparing a CI job with more debug, let me 5 min
14:45:50 bauzas EmilienM: anyway, I think the bug description has a first point to look

Earlier   Later