Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-04
15:45:33 kashyap Abandoned; will check the question on the CMT one.
15:45:41 kashyap Thanks for review.
15:45:58 kashyap s/for/"for the"/
16:30:55 sean-k-mooney johnthetubaguy: regarding the osinfo issue we coudl merge you small fix as a stepping stone but i think we should file a seperate bug to adress the larger issue later
16:31:15 sean-k-mooney e.g. we should unbreak your customers and then figure out where to go from there.
16:31:24 johnthetubaguy sean-k-mooney: cool, that aligns with my thinking
16:32:21 johnthetubaguy certainly, a heap of issues in there... its tempting to just drop the libosinfo stuff honestly, I am not sure its helping us any more, if we always send "virtio" expect for very old distros
16:33:36 sean-k-mooney ya i considerd that too but was not sure why it was originally added e.g. what would we be regressing if we did
16:38:06 sean-k-mooney by the way os_distro=centos7.0 is not correct it should be os_distro=centos and os_version=7
16:38:18 sean-k-mooney see https://github.com/openstack/glance/blob/master/etc/metadefs/operating-system.json
16:39:36 sean-k-mooney which i now realise that module does not handel correctly either
16:43:34 sean-k-mooney https://github.com/openstack/nova/blob/master/nova/virt/osinfo.py#L123-L124 should be checking both values and technically we should be rejecting centos7.0 as an invalde disto name
16:44:52 sean-k-mooney but we treat it as a sting filed here https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py#L456 since we dont know the full set of os we could be running
16:53:02 openstackgerrit Merged openstack/nova master: vif: Skip most of 'get_base_config' if not using virtio https://review.opendev.org/666629
17:10:52 openstackgerrit Merged openstack/nova stable/queens: Init HostState.failed_builds https://review.opendev.org/668911
17:24:04 openstackgerrit Ghanshyam Mann proposed openstack/nova master: WIP:Introduce scope_types in os-services https://review.opendev.org/645427
18:00:20 johnthetubaguy sean-k-mooney: right, it only seems to work with centos7.0 right now, else it gets rejected, go figure
18:00:52 sean-k-mooney right but the os_disto is ment to be the os name without the version
18:00:59 sean-k-mooney so it was broken from the start
18:01:47 johnthetubaguy maybe, it spits out a warning unless you smoosh them together
18:02:12 sean-k-mooney libosinfo?
18:02:28 sean-k-mooney looking at the spec the spec was using it wrongly form the start https://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/libvirt-hardware-policy-from-libosinfo.html
18:02:30 johnthetubaguy no, Nova
18:02:43 sean-k-mooney nova should not spit out an error
18:02:46 johnthetubaguy the spec wasn't updated vs the implementation
18:03:05 sean-k-mooney os_distro=centos is the correct form
18:03:08 johnthetubaguy it always, had, well only in the operators logs as a warning
18:03:14 johnthetubaguy saying osdistro not found
18:03:23 openstackgerrit Merged openstack/nova stable/stein: Add functional recreate test for regression bug 1825537 https://review.opendev.org/666959
18:03:23 openstack bug 1825537 in OpenStack Compute (nova) stein "finish_resize failures incorrectly revert allocations" [Medium,In progress] https://launchpad.net/bugs/1825537 - Assigned to Matt Riedemann (mriedem)
18:03:28 openstackgerrit Merged openstack/nova stable/stein: Drop source node allocations if finish_resize fails https://review.opendev.org/667155
18:03:34 openstackgerrit Merged openstack/nova stable/rocky: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/666854
18:03:38 johnthetubaguy sorry, I need ot run
18:03:39 sean-k-mooney yes but os_distro was not added for libosinfo
18:03:41 openstackgerrit Merged openstack/nova stable/rocky: Disable limit if affinity(anti)/same(different)host is requested https://review.opendev.org/659246
18:03:49 sean-k-mooney johnthetubaguy: sure o/
18:03:59 johnthetubaguy I thought it existed before
18:04:05 sean-k-mooney it did
18:04:16 sean-k-mooney for other drivers for other things that expect jsut the os name
18:04:25 sean-k-mooney not the os name + version
18:07:05 openstackgerrit Merged openstack/nova master: Refactor bandwidth related functional tests https://review.opendev.org/641679
#openstack-nova - 2019-07-05
01:01:06 openstackgerrit Merged openstack/nova stable/stein: Ignore hw_vif_type for direct, direct-physical vNIC types https://review.opendev.org/667354
02:18:11 openstackgerrit Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963
03:13:03 lewe Hi, Maybe someone can help me, I am trying to cold migrate an instance but it does not seems to work right. The instance was newly created on the destination hosts rather than copying the original disk from the instance original host. Thank you.
03:36:16 openstackgerrit wangwei1 proposed openstack/nova master: fix spelling error in nova/api/validation/__init__.py https://review.opendev.org/669244
04:26:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix a broken link https://review.opendev.org/669248
04:45:38 openstackgerrit Seyeong Kim proposed openstack/nova stable/queens: Share snapshot image membership with instance owner https://review.opendev.org/661667
06:00:33 openstackgerrit wangwei1 proposed openstack/nova stable/stein: fix spelling error in nova/api/validation/__init__.py https://review.opendev.org/669251
06:32:07 openstackgerrit Ghanshyam Mann proposed openstack/nova-specs master: Fix the cross_az_attach value in AZ-unshelved spec https://review.opendev.org/669261
06:39:10 openstackgerrit Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021
07:18:21 openstackgerrit Yongli He proposed openstack/nova master: Clean up orphan instances virt driver https://review.opendev.org/648912
07:18:21 openstackgerrit Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765
07:24:02 openstackgerrit Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765
08:12:55 openstackgerrit Kashyap Chamarthy proposed openstack/nova-specs master: Fix indentation in the "Secure Boot for KVM ... guests" spec https://review.opendev.org/669284
08:13:24 kashyap stephenfin: johnthetubaguy: A trivial cosmetic fix --^ (rST syntax related)
08:25:32 openstackgerrit Lee Yarwood proposed openstack/nova master: Get rid of args to RBDDriver.__init__() https://review.opendev.org/668564
08:35:40 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove native LUKS compat code https://review.opendev.org/669121
09:12:29 openstackgerrit Stephen Finucane proposed openstack/nova master: Fix spelling error in 'nova/api/validation/__init__.py' https://review.opendev.org/669244
09:14:08 openstackgerrit Stephen Finucane proposed openstack/nova stable/stein: Fix spelling error in 'nova/api/validation/__init__.py' https://review.opendev.org/669251
09:15:41 kashyap stephenfin: Hope I answered your question there: https://review.opendev.org/#/c/669129/
09:16:02 kashyap (Not an entirely satisfactory one; but it's the truth :-))
09:16:26 kashyap New rev upcoming.
09:17:30 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_PERF_VERSION https://review.opendev.org/669128
09:17:30 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove support for Intel CMT `perf` events https://review.opendev.org/669129
09:17:31 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: vif: Remove MIN_LIBVIRT_MACVTAP_PASSTHROUGH_VLAN https://review.opendev.org/669176
09:25:17 openstackgerrit Merged openstack/nova-specs master: Fix indentation in the "Secure Boot for KVM ... guests" spec https://review.opendev.org/669284
10:38:17 openstackgerrit Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969
10:40:40 bauzas gibi: remember me, do we document in the API docs something like https://review.opendev.org/#/c/630723/4/nova/api/openstack/compute/migrate_server.py ?
10:40:48 bauzas not the API guide I mean
10:41:23 bauzas I can read https://docs.openstack.org/neutron/latest/admin/config-qos-min-bw.html tho
10:41:45 bauzas and https://developer.openstack.org/api-guide/compute/port_with_resource_request.html document what's missing
10:42:02 bauzas but I wonder if within the API docs, we said that too
10:42:15 bauzas I mean api-ref
10:42:19 bauzas lemme check actually
10:44:13 bauzas nope, we don't (eg. https://developer.openstack.org/api-ref/compute/?expanded=resize-server-resize-action-detail#resize-server-resize-action )
10:44:31 bauzas anyway, not a big deal, it's just for internal docs
10:44:49 bauzas gibi: FWIW, ping me if you have any changes about ^ btw.
11:04:47 mdbooth >>> x.config_drive = False
11:04:47 mdbooth >>> x.config_drive
11:04:47 mdbooth u'False'
11:04:47 mdbooth >>> x.config_drive = True
11:04:56 mdbooth Don't you just love python?
11:05:06 mdbooth And that field has the wrong type, of course
11:06:06 mdbooth >>> x.config_drive = False
11:06:06 mdbooth >>> bool(x.config_drive)
11:06:06 mdbooth True
11:06:33 openstackgerrit Merged openstack/nova master: api-ref: Fix a broken link https://review.opendev.org/669248
11:47:51 stephenfin kashyap: Replied to https://review.opendev.org/#/c/669129/
11:48:12 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: vif: Remove MIN_LIBVIRT_MACVTAP_PASSTHROUGH_VLAN https://review.opendev.org/669176
11:51:04 gibi bauzas: we have the limitation separately, not in the api-ref. I think it was intentional as it is expected to be a temporary limitation
11:51:34 gibi bauzas: the actual implementation of server move operation still needs my time. it is started up here https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/support-move-ops-with-qos-ports
11:52:00 gibi bauzas: but I was busy with https://review.opendev.org/#/c/637955/
12:29:27 kashyap stephenfin: Will respond (saddled with a couple off meetings); thanks for the thoughtful reply.
12:41:34 openstackgerrit Lee Yarwood proposed openstack/nova master: Get rid of args to RBDDriver.__init__() https://review.opendev.org/668564
12:47:29 bauzas gibi: ack ok
13:05:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Error out interrupted builds https://review.opendev.org/666857
14:12:42 mriedem lyarwood: have you actually hit a recreate for this? https://review.opendev.org/#/q/I15a7c13edf78884ec223fd531a78a341106b41b8
14:15:04 lyarwood mriedem: did you mean reason?
14:15:32 mriedem no,
14:15:40 mriedem when that was reported there was no recreate in the bug for it,

Earlier   Later