| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-15 | |||
| 15:41:48 | adrianc | thanks mriedem, appreciated | |
| 15:41:55 | mriedem | what you need to avoid in the BM case is the update to the port | |
| 15:41:56 | sean-k-mooney | i know we cna do boot form volume with ironic now | |
| 15:42:06 | sean-k-mooney | do we supprot resize/cold migrate | |
| 15:42:46 | sean-k-mooney | in principal if we did then yes we coudl have the same failure cases | |
| 15:43:36 | mriedem | no we don't | |
| 15:45:42 | sean-k-mooney | ok there is no reason that should not be supported for boot form volume ironic instance right? | |
| 15:46:09 | sean-k-mooney | i guess there is the quetion of ephemeral storage | |
| 15:46:36 | mriedem | there was a spec for supporting migration of ironic instances, you could read the details there | |
| 15:47:13 | mriedem | https://review.openstack.org/#/c/449155/ | |
| 15:47:20 | sean-k-mooney | if it was a pure bfv ironic node with no local disk it should work. no im jsut trying to reson about if we would need to validate neutron port for ironic instance in the future | |
| 15:47:22 | mriedem | anyway, that doesn't really have anything to do with this change IMO | |
| 15:47:35 | mriedem | just avoid the port binding update if ironic to get back to the pre-regression behavior | |
| 15:47:57 | sean-k-mooney | ya i was leaving a comment to say somthign similar | |
| 15:48:15 | sean-k-mooney | just wondering if we will need to do something more involed in the future | |
| 15:48:40 | sean-k-mooney | but to fix the regress we jus tneed to not to the port binding update here https://review.openstack.org/#/c/649345/4/nova/compute/manager.py@73311 | |
| 15:49:25 | sean-k-mooney | sorry here https://review.openstack.org/#/c/649345/4/nova/compute/manager.py@7311 | |
| 15:49:31 | sean-k-mooney | extra 3 | |
| 15:50:00 | adrianc | a noob's question here, but why would nova attempt to fix something that ironic did ? that is, ironic is the once who does the actual port binding today right ? | |
| 15:50:07 | mriedem | melwitt: you should review this part of the cells v2 removal series regarding console stuff https://review.openstack.org/#/c/651295/ | |
| 15:50:30 | mriedem | adrianc: the regression was just an oversight | |
| 15:50:42 | sean-k-mooney | adrianc: it depends on the workflow. for spawn yes the virt dirver does the binding | |
| 15:51:00 | sean-k-mooney | adrianc: this was trying to fix move opertion for virt dirver that supprot them | |
| 15:51:30 | sean-k-mooney | for live migration the conductor does the binidng | |
| 15:52:11 | adrianc | understood, sean-k-mooeny, was just addressing what you said about, "just wondering if we will need to do something more involed in the future" | |
| 15:53:15 | sean-k-mooney | adrianc: ah well if we support move operation maybe we shoudl be honestly this is only here because we dont cleanup thing correctly on revert or failed migration in all cases | |
| 15:53:31 | sean-k-mooney | adrianc: if we did we would not need to heal the info cache | |
| 15:55:03 | openstackgerrit | Dan Smith proposed openstack/nova master: Add image type capability flags and trait conversions https://review.openstack.org/652710 | |
| 15:55:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Server move operations with ports having resource request https://review.openstack.org/652608 | |
| 16:07:27 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Standardize CPU resource tracking https://review.openstack.org/555081 | |
| 16:07:47 | mriedem | bauzas: could take a look at these backports? https://review.openstack.org/#/q/topic:bug/1821594+branch:stable/rocky | |
| 16:11:33 | openstackgerrit | Merged openstack/nova stable/ocata: Update port device_owner when unshelving https://review.openstack.org/626413 | |
| 16:23:58 | sean-k-mooney | the periodic task is ment to heal all instance that need to be healed on each invocation right? | |
| 16:24:26 | sean-k-mooney | because if it is, that is not what it actully does | |
| 16:25:30 | sean-k-mooney | oh actully reading the doc string that was intentional? | |
| 16:31:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest https://review.openstack.org/528104 | |
| 16:37:58 | openstackgerrit | Merged openstack/nova stable/queens: Ignore VolumeAttachmentNotFound exception in compute.manager https://review.openstack.org/637391 | |
| 16:53:37 | tssurya | dansmith: about the power update event spec (https://review.openstack.org/#/c/636132/4/specs/train/approved/nova-support-instance-power-update.rst) whenever you have the time could you confirm the things discussed at L77 and L74 ? main concern being the notifications/action record requirement | |
| 16:53:43 | tssurya | thanks! :) | |
| 17:22:03 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove ComputeDriver.macs_for_instance method https://review.openstack.org/652737 | |
| 17:30:27 | pacharya | a quick question, does creation of a batch instance delete API fall under https://docs.openstack.org/nova/latest/contributor/project-scope.html#no-more-orchestration ? | |
| 17:31:45 | _erlon_ | Folks, Im trying to debug a failure on the instance creation on Nova, on a Pike release, but all I get is " Got no allocation candidates from the Placement API" and no explanation on what resource is lacking: | |
| 17:31:51 | _erlon_ | http://paste.openstack.org/show/749318/ | |
| 17:32:02 | _erlon_ | look at the placement API logs also does not help | |
| 17:32:53 | _erlon_ | what is the procudure now? | |
| 17:32:53 | _erlon_ | I didn't had this problem before the placement API was introduced | |
| 18:07:24 | mriedem | there was debug logging added to the placement GET /allocation_candidates code but i'm not sure it made it back to pike | |
| 18:07:38 | mriedem | jaypipes: ^? | |
| 18:10:22 | mriedem | _erlon_: https://review.openstack.org/#/q/If9ddb8a6d2f03392f3cc11136c4a0b026212b95b | |
| 18:10:42 | mriedem | and https://review.openstack.org/#/q/status:merged+project:openstack/nova+branch:master+topic:one-resource-per-query | |
| 18:10:48 | mriedem | that would take some work to get back to pike | |
| 18:11:12 | mriedem | _erlon_: how many computes do you have? | |
| 18:11:54 | mriedem | first thing to do would be to make sure there is a resource provider for each compute node | |
| 18:22:28 | openstackgerrit | Merged openstack/python-novaclient stable/rocky: Revert "Fix crashing console-log" https://review.openstack.org/650364 | |
| 18:22:29 | openstackgerrit | Merged openstack/python-novaclient stable/rocky: Add test for console-log and docs for bug 1746534 https://review.openstack.org/652033 | |
| 18:22:31 | openstack | bug 1746534 in python-novaclient "encoding error when doing console-log" [High,Fix released] https://launchpad.net/bugs/1746534 - Assigned to Thomas Goirand (thomas-goirand) | |
| 18:28:21 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove macs kwarg from allocate_for_instance https://review.openstack.org/652749 | |
| 18:33:08 | edleafe | vim | |
| 18:33:12 | edleafe | doh! | |
| 18:34:26 | sean-k-mooney | edleafe: password: | |
| 18:34:38 | sean-k-mooney | :) | |
| 18:35:43 | edleafe | too many windows open - now *that* would make an easy-to-remember password | |
| 18:37:27 | sean-k-mooney | im kind of sad the cirros passward is nolonger "cubswin:)" | |
| 18:37:45 | sean-k-mooney | anyway time for dinner | |
| 18:37:47 | sean-k-mooney | o/ | |
| 18:58:33 | _erlon_ | mriedem: only 1 | |
| 18:58:40 | _erlon_ | its a devstack node | |
| 19:00:02 | mriedem | _erlon_: oh, well, make sure it's reporting into placement with the correct inventory | |
| 19:00:15 | mriedem | https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-list | |
| 19:00:20 | mriedem | https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-inventory-list | |
| 19:00:34 | _erlon_ | mriedem: its a devstack node and should be working fine | |
| 19:00:46 | _erlon_ | I can like create up to 3 instances | |
| 19:00:59 | _erlon_ | the 4th fails for some reason | |
| 19:02:25 | mriedem | well, you can use ^ to see the inventory of the provider, | |
| 19:02:32 | mriedem | and https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-usage-show to see the usage of the project which you used to create instances | |
| 19:02:42 | mriedem | and then see if there is any remaining capacity for the 4th instance you're trying to create | |
| 19:34:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: conf: Undeprecate and move the 'dhcp_domain' option https://review.openstack.org/480616 | |
| 19:34:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle unsetting '[DEFAULT] dhcp_domain' https://review.openstack.org/652662 | |
| 19:49:32 | mriedem | edleafe: error response status code question! https://review.openstack.org/#/c/645458/17/specs/train/approved/add-host-and-hypervisor-hostname-flag-to-create-server.rst@141 | |
| 19:50:01 | mriedem | do those still exist? | |
| 19:50:08 | edleafe | jink | |
| 19:50:11 | edleafe | jinx | |
| 19:50:16 | aspiers | we still have some | |
| 19:50:21 | aspiers | but I don't think they tend to work | |
| 19:50:30 | aspiers | unless you're a timelord | |
| 19:50:37 | jaypipes | _erlon_: did you find the answer to your query? | |
| 19:52:25 | artom | Just quick sanity check to be 100% sure, but the resource stats we provide in https://developer.openstack.org/api-ref/compute/?expanded=show-hypervisor-details-detail#show-hypervisor-details come from the resource tracker via the ComputeNode object, right? And specifically, not from placement? | |
| 19:52:31 | edleafe | mriedem: that's not really a response code question, is it? If the app logic is to let the scheduler pick, then 2xx. If you treat the value as missing, then 400 | |
| 19:53:02 | mriedem | in the same request, the API will return a 409 HTTP Conflict error." | |
| 19:53:02 | mriedem | (``az:host:node``) mechanisms are mutually exclusive. If both are specified | |
| 19:53:02 | mriedem | edleafe: this one? "The new (``host`` and/or ``hypervisor_hostname``) and the old | |
| 19:53:11 | mriedem | they are saying the api will return an error and we don't get to the scheduler | |
| 19:53:16 | mriedem | i'm just saying that's not a 409 | |
| 19:53:49 | edleafe | mriedem: huh, I clicked that link and L125 was front and center | |
| 19:54:01 | mriedem | artom: everything in the hypervisors api comes from the compute node yeah | |
| 19:54:08 | artom | mriedem, cheerS! | |
| 19:54:27 | edleafe | Yeah, that's not a 409. There's nothing conflicting about the state of the system and the request. It's just a bad request | |
| 19:55:41 | mriedem | artom: interestingly that API doesn't use the ComputeNode.stats object https://github.com/openstack/nova/blob/78e742662edd164c46382c31e106884762fed029/nova/db/sqlalchemy/api.py#L730 | |
| 19:55:58 | mriedem | https://github.com/openstack/nova/blob/78e742662edd164c46382c31e106884762fed029/nova/objects/compute_node.py#L81 | |
| 19:56:46 | mriedem | which is this happy thing https://github.com/openstack/nova/blob/78e742662edd164c46382c31e106884762fed029/nova/compute/stats.py | |