| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-19 | |||
| 21:48:56 | mriedem | well, | |
| 21:48:58 | mnaser | couldnt nova confirm that it doesnt exist in the db | |
| 21:49:05 | mriedem | not if it's a race | |
| 21:49:06 | mnaser | and technically just delete it when it polls | |
| 21:49:09 | mnaser | ah | |
| 21:49:21 | mnaser | i guess we'd have to check N times that it's not there but that starts becoming silly i guess | |
| 21:49:26 | mriedem | as noted, we create the allocations in scheduling before the instance is created in the cell db | |
| 21:49:33 | mriedem | right | |
| 21:49:47 | mnaser | or if its been running for a while | |
| 21:49:53 | mriedem | putting it into some kind of faq could work, but people might not see that and come here asking the same thing | |
| 21:49:59 | mnaser | like 300s age | |
| 21:50:08 | mnaser | 300s isn't a race condition at that point i guess | |
| 21:50:24 | mnaser | but i guess log is the easiest path | |
| 21:50:31 | mriedem | rather than nova auto-delete your stuff, i'd prefer to just document weird edge cases for manual intervention | |
| 21:51:07 | mriedem | we've also talked about (last week it came up too) a command to compare placement allocations against nova instances, | |
| 21:51:13 | mriedem | the opposite of heal_allocations, | |
| 21:51:24 | mriedem | more of a "tell me what garbage is in the placement db b/c nova didn't clean up after itself" | |
| 21:51:54 | mriedem | but we can't reliably do that until we have consumer types in placement... | |
| 21:54:47 | mriedem | takashin: is this todo still valid https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3509 given we check the specified host is in the same cell above? https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3405 | |
| 21:55:55 | mriedem | i think we can remove that todo; when we call objects.ComputeNode.get_first_node_by_host_for_old_compat the context is targeted to the cell in which the instance currently exists | |
| 21:56:08 | mriedem | so if the specified host is in another cell, we'll get ComputeHostNotFound there | |
| 21:58:57 | mriedem | johnthetubaguy: we should remove this rebuild parameter from the conductor migrate_server task API :) https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L263 | |
| 21:59:12 | mriedem | that must have been something someone talked about way back in 2013 to unify the various move APIs | |
| 22:10:08 | takashin | mriedem: you are right. We can remove the 'todo'. | |
| 22:10:42 | mriedem | takashin: please push a change to remove it and i'll +2 | |
| 22:11:11 | takashin | mriedem: okay. I will submit the patch. | |
| 22:13:52 | vishwanathj | hi looking for guidance on how to limit the number of instances on certain set of hosts... | |
| 22:15:06 | vishwanathj | not clear on sequence of steps to be executed on controller and compute nodes to get this working | |
| 22:31:47 | mriedem | there is a NumInstancesFilter | |
| 22:32:11 | mriedem | vishwanathj: https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#numinstancesfilter | |
| 22:32:25 | mriedem | and https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#aggregatenuminstancesfilter for aggregates | |
| 22:33:16 | openstackgerrit | Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots https://review.openstack.org/603844 | |
| 22:33:53 | openstack | Launchpad bug 1740320 in OpenStack Compute (nova) "nova-scheduler does not honor max_instances_per_host set to a host aggregate" [Undecided,Confirmed] | |
| 22:33:53 | mriedem | there is a known bug with those filters https://bugs.launchpad.net/nova/+bug/1740320 | |
| 22:34:01 | mriedem | race bug | |
| 22:47:40 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove an unnecessary comment https://review.openstack.org/603924 | |
| 22:49:34 | takashin | mriedem: I submitted https://review.openstack.org/#/c/603924 . | |
| 22:57:15 | sorrison | Anyone know how to re kick a gate job that failed due to a timeout. https://review.openstack.org/#/c/526558/ I tried a recheck but that aint it | |
| 22:57:42 | sorrison | mriedem: ^ | |
| 23:02:34 | mriedem | sorrison: the gate has been eating a ton of dairy | |
| 23:02:49 | mriedem | plug 526558 into http://zuul.openstack.org/ | |
| 23:03:06 | mriedem | it's queued up | |
| 23:03:54 | mriedem | http://lists.openstack.org/pipermail/openstack-dev/2018-September/134867.html | |
| 23:04:48 | sorrison | ok thanks | |
| 23:19:24 | mriedem | takashin: got it, thanks | |
| 23:20:11 | takashin | mriedem: Thank you for your review. | |
| 23:40:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930 | |
| 23:40:39 | mriedem | ship it! | |
| 23:52:34 | gmann | lbragstad: i will check the ML today and get back to you. | |
| #openstack-nova - 2018-09-20 | |||
| 00:57:24 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Add support changes-before for microversion 2.66 https://review.openstack.org/603549 | |
| 01:07:32 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Spec: allocation candidates in tree https://review.openstack.org/603585 | |
| 01:12:57 | openstackgerrit | Tao Li proposed openstack/python-novaclient master: Remove the unused instance-name https://review.openstack.org/602520 | |
| 01:35:04 | lbragstad | gmann sounds good | |
| 02:33:47 | openstackgerrit | Sundar Nadathur proposed openstack/nova-specs master: Nova Cyborg interaction specification. https://review.openstack.org/603955 | |
| 02:50:24 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Add support changes-before for microversion 2.66 https://review.openstack.org/603549 | |
| 02:57:36 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104 | |
| 03:12:07 | openstackgerrit | fupingxie proposed openstack/python-novaclient master: Do not translate log messages. https://review.openstack.org/603962 | |
| 04:16:51 | openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: Spec for API inconsistency cleanup https://review.openstack.org/603969 | |
| 04:28:21 | openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: Spec for API inconsistency cleanup https://review.openstack.org/603969 | |
| 04:47:10 | gmann | lyarwood: i replied on this - https://review.openstack.org/#/c/602018/ | |
| 04:47:30 | gmann | lyarwood: stable branch patches needs to get in before openstack-infra/openstack-zuul-jobs one | |
| 06:28:44 | openstackgerrit | huanhongda proposed openstack/nova stable/ocata: Cleanup RP and HM records while deleting a compute service. https://review.openstack.org/603749 | |
| 06:29:14 | openstackgerrit | fupingxie proposed openstack/nova master: Avoid Forcing the Translation of Translatable Variables https://review.openstack.org/603993 | |
| 06:47:48 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Only update cells' transport_url and database_connection when needed. https://review.openstack.org/603998 | |
| 06:53:45 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | |
| 07:38:16 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Add support changes-before for microversion 2.66 https://review.openstack.org/603549 | |
| 07:44:23 | bauzas | good morning | |
| 07:45:33 | sean-k-mooney | good morning :) | |
| 07:46:10 | gibi | good morning folks | |
| 08:26:23 | openstackgerrit | Merged openstack/nova master: libvirt: Drop MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472 | |
| 08:49:15 | kashyap | Hey folks, I wonder if anyone here knows Linux systems where only /dev/random exists | |
| 08:49:19 | kashyap | And not /dev/urandom. | |
| 08:50:20 | kashyap | Because I'm considering to add a default 'virtio-rng' device for Nova guests, whic uses /dev/urandom. | |
| 08:50:24 | openstack | Launchpad bug 1789868 in OpenStack Compute (nova) "RFE: Add a virtio-rng device to Nova instances by default" [Medium,Triaged] - Assigned to Kashyap Chamarthy (kashyapc) | |
| 08:50:24 | kashyap | https://bugs.launchpad.net/nova/+bug/1789868 | |
| 08:52:03 | kashyap | (If anyone is wondering why '/dev/urandom'? It is non-blocking, and doesn't have the same limitations of `/dev/random`, which is a legacy interface. | |
| 09:02:20 | openstackgerrit | huanhongda proposed openstack/nova stable/ocata: Cleanup the caches when deleting a resource provider https://review.openstack.org/604021 | |
| 09:13:35 | kashyap | stephenfin: The last clean up (IIRC) in that series: https://review.openstack.org/#/c/562313/ (libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD) | |
| 09:30:26 | stephenfin | kashyap: cool, I'll take a look now | |
| 09:34:08 | kashyap | stephenfin: Thanks; it's not urgent. Just a boring thing to tick off, when you have time | |
| 09:51:40 | openstackgerrit | Yikun Jiang proposed openstack/nova master: WIP: Add nova database migration script(psql version) https://review.openstack.org/604026 | |
| 09:51:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION https://review.openstack.org/565517 | |
| 09:52:30 | kevko_home | Hello guys , can i have simple question on you :) ? | |
| 09:57:51 | openstackgerrit | Yikun Jiang proposed openstack/nova master: WIP: Add nova database migration script(psql version) https://review.openstack.org/604026 | |
| 10:06:34 | kashyap | kevko_home: Just ask, someone might respond. | |
| 10:24:19 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Add support changes-before for microversion 2.66 https://review.openstack.org/603549 | |
| 10:47:31 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Spec for API policy updates https://review.openstack.org/547850 | |
| 10:51:25 | gibi | bauzas, tetsuro: I managed to re-use the existing ServerMovingTests functional test set to verify the nested allocations during move operations. I found that force livemigrate and every evacuate case are faulty. | |
| 10:51:51 | gibi | bauzas, tetsuro: I will push the patch soon | |
| 10:52:16 | gibi | bauzas, tetsuro: I mean the fault reproduction patch, then I will look at how to solve the problem | |
| 10:58:12 | openstackgerrit | huanhongda proposed openstack/nova stable/ocata: Cleanup RP and HM records while deleting a compute service. https://review.openstack.org/603749 | |
| 10:58:25 | openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: Spec for API inconsistency cleanup https://review.openstack.org/603969 | |
| 11:07:40 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/pike: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/604039 | |
| 11:22:15 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for examples https://review.openstack.org/604060 | |
| 11:48:44 | kevko_home | Problem is that i have error when api_database connection is missing in nova-placement-api package, does it mean that nova-placement-api needs both nova_api and placement connection settings filled ? Shouldn't be command separate for nova_api and placement ... something like nova-manage api_db sync , nova-mange api_placement sync ? | |
| 11:48:44 | kevko_home | Each of package is doing his work, for example nova-common creating database users, etc, and also populating nova db. I have also nova-placement-api and nova-api which are doing same stuff with populating db, nova-placement-api creating database placement , users, grants ..etc ..fill connection in config and run nova-manage api_db sync , same in nova-api create db nova , users , grants, run nova-manage api_db sync. | |
| 11:48:44 | kevko_home | Guys, i am packaging nova for debian. Current situation is that i have several packages , nova-common , nova-api, nova-placement-api, nova-conductor ...etc | |
| 11:49:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing specifying doctrees directory https://review.openstack.org/604068 | |
| 11:55:41 | bauzas | gibi: hola, what's the fault ? | |
| 11:58:49 | tssurya | kevko_home: which release are you using ? | |