| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-07 | |||
| 15:18:01 | mriedem | didn't check the n-cpu logs but the test passed | |
| 15:26:07 | ildikov | mriedem: this looks a little scary: Do NOT duplicate iSCSI InitiatorNames | |
| 15:45:12 | ildikov | I also saw this with the second detach: Couldn't find iSCSI nodes because iscsiadm err: iscsiadm: No records found | |
| 15:46:15 | ildikov | so we might still have issues with lvm detach or I just freaked out by an irrelevant line | |
| 21:31:42 | mriedem | ildikov: would have to ask hemna probably, i think os-brick might just log some stuff that is normal if a thing it's trying to remove is already gone | |
| 21:31:49 | mriedem | i don't know about the duplicate initiator names | |
| 22:32:09 | ildikov | mriedem: we had a similar log message before with the second detach, but here it looks like a simple debug log | |
| 22:32:34 | ildikov | mriedem: I just instinctively freaked out about it... | |
| 22:32:46 | ildikov | Will ask around tomorrow | |
| #openstack-nova - 2018-01-08 | |||
| 01:54:04 | Rambo | hello,everyone | |
| 01:54:29 | Rambo | Can you help me to review it ?The link is https://review.openstack.org/#/c/531524/ | |
| 01:54:50 | Rambo | Another is :https://review.openstack.org/#/c/528740/ | |
| 01:54:58 | Rambo | Thank you very much | |
| 01:55:35 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [DNM] add uuid in server migration https://review.openstack.org/531633 | |
| 02:17:42 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add index(updated_at) on migrations table. https://review.openstack.org/531132 | |
| 02:21:47 | alex_xu_ | yikun: sorry for reply late https://review.openstack.org/#/c/330406/78/api-ref/source/parameters.yaml@4148 | |
| 02:54:04 | yikun | alex_xu_, ok, thanks, saw it, I will update and add the uuid on server migrations in this patch. | |
| 03:00:40 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429 | |
| 03:00:40 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117 | |
| 03:11:05 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [DNM] add uuid in server migration https://review.openstack.org/531633 | |
| 03:13:17 | yikun | ^ I submit the separate one, just for convenient reviewing, and if all is ok, I will merge it into parent patch. | |
| 03:54:42 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904 | |
| 03:54:43 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | |
| 03:54:43 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | |
| 03:54:44 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [DNM] add uuid in server migration https://review.openstack.org/531633 | |
| 04:43:00 | gcb | alex_xu_, could you please help review https://review.openstack.org/528620 a simple adoption of oslo function | |
| 04:43:39 | alex_xu_ | gcb: will check it after lunch | |
| 04:45:03 | gcb | alex_xu_: thanks | |
| 04:45:10 | alex_xu_ | gcb: np | |
| 05:44:16 | openstackgerrit | Rajesh Tailor proposed openstack/nova master: Allow force-delete even if task_state is not None https://review.openstack.org/530879 | |
| 05:56:46 | openstackgerrit | Kevin Zhao proposed openstack/nova master: Modify the test case of get_disk_mapping_rescue_with_config https://review.openstack.org/494156 | |
| 06:10:19 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601 | |
| 06:35:23 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 06:37:18 | openstackgerrit | Kevin Zhao proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | |
| 06:52:26 | bhagyashris | ianw: Hi, are you around ? | |
| 07:24:36 | openstackgerrit | Merged openstack/nova master: Use method validate_integer from oslo.utils https://review.openstack.org/528620 | |
| 07:51:58 | openstackgerrit | Sriharsha Basavapatna proposed openstack/os-vif master: Fix VF-rep lookup routine to use parent PF number https://review.openstack.org/531674 | |
| 08:03:55 | hrw | morning | |
| 08:35:04 | openstackgerrit | Mr Rambo proposed openstack/nova master: Support volume-backed server rebuild https://review.openstack.org/528740 | |
| 08:44:23 | bauzas | good morning Nova | |
| 08:44:37 | gibi_away | good morning bauzas and Nova | |
| 08:48:30 | hrw | speadup zuul, speed up... | |
| 09:08:47 | bauzas | just in case people wanna give me a Christmas present, here is a series loving reviews https://review.openstack.org/#/c/514763/ | |
| 09:08:49 | bauzas | :) | |
| 09:19:55 | openstackgerrit | Kevin Zhao proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | |
| 09:24:47 | openstackgerrit | Mr Rambo proposed openstack/nova master: Support volume-backed server rebuild https://review.openstack.org/528740 | |
| 09:50:38 | ameeda | hello | |
| 09:51:10 | ameeda | can you please review my code here https://review.openstack.org/#/c/528069/ ? and let me know if I have to consider force_delete and soft_delete | |
| 10:16:51 | openstackgerrit | Lajos Katona proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728 | |
| 10:17:36 | openstackgerrit | Jie Li proposed openstack/nova master: Support volume-backed server rescue https://review.openstack.org/531524 | |
| 10:22:27 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386 | |
| 10:48:49 | openstackgerrit | chenpengzi proposed openstack/nova master: For a pre-generated example of the latest policy.yaml https://review.openstack.org/531713 | |
| 11:10:27 | openstackgerrit | chenpengzi proposed openstack/nova master: For a pre-generated example of the latest policy.yaml https://review.openstack.org/531713 | |
| 11:41:28 | openstackgerrit | chenpengzi proposed openstack/nova master: Add reference to policy sample https://review.openstack.org/531713 | |
| 11:45:56 | openstackgerrit | Jianle He proposed openstack/nova-specs master: Back up volume-backed-instance https://review.openstack.org/530214 | |
| 12:09:26 | gibi | bauzas: left some questions / nits in https://review.openstack.org/#/c/514763/ Merry Christmas! ;) | |
| 12:14:09 | openstackgerrit | Merged openstack/nova master: Remove 'nova-manage host' and 'nova-manage agent' https://review.openstack.org/521837 | |
| 12:20:12 | ameeda | bauzas: can you please review my code here https://review.openstack.org/#/c/526900/ | |
| 12:45:37 | hrw | stephenfin: https://review.openstack.org/#/c/530965/ got +1 from zuul, has test coverage. can you +2 again? | |
| 12:47:01 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/531737 | |
| 12:47:52 | ameeda | Hello | |
| 12:48:10 | ameeda | FutureWarning: fake is an invalid UUID. Using UUIDFields with invalid UUIDs is no longer supported, and will be removed in a future release. Please update your code to input valid UUIDs or accept ValueErrors for invalid UUIDs. See https://docs.openstack.org/oslo.versionedobjects/latest/reference/fields.html#oslo_versionedobjects.fields.UUIDField for further details | |
| 12:48:10 | ameeda | I got this error when I run tox -e py27 | |
| 12:48:23 | ameeda | any idea ? | |
| 12:49:27 | sean-k-mooney | ameeda: when using oslo versioned objects the validator for uuid fields check that the value is really a uuid. fake is not a uuid hence the error | |
| 12:49:44 | sean-k-mooney | ameeda: is that from https://review.openstack.org/#/c/526900 | |
| 12:51:51 | ameeda | sean-k-mooney: thanks for reply, no its from this https://review.openstack.org/#/c/528385/ when I move my code to _cleanup_volumes and modify the unit test for that | |
| 12:53:40 | ameeda | sean-k-mooney: here is my edit http://paste.openstack.org/show/640858/ | |
| 12:53:41 | sean-k-mooney | oh ok you can use uuidgen on the commandline to generate a uuid for the tests for example e21d0851-3c56-4b9d-a133-a2c1ba59778a whcich you can use instead of fake to resolve the warning | |
| 12:54:49 | ameeda | sean-k-mooney: so I should use this uuid inside my code ? since the error appears in another file | |
| 12:56:03 | sean-k-mooney | only if its cause by your code. if the warning is present without your patch you dont need to fix it. | |
| 12:56:09 | ktibi | Hi, do you think it's good if nova doesn't check if neutron have free IP before spawn instance ? | |
| 12:57:02 | ktibi | because I have a strange case where user spawn 200 vms with 50 free IP, and all my compute become disabled with "consecutive_build_service_disable_threshold" | |
| 12:57:46 | sean-k-mooney | ameeda: oslo versioned objects uuid filed not emit this warning if invalid uuids are used but at present its only a warning to allow projects to fix there tests. in the future it will be changed to an error but only once most project have updated there tests | |
| 12:58:55 | ameeda | sean-k-mooney: when I remove my code, the test success completely, and the error appears at another file where I add my code | |
| 12:59:53 | sean-k-mooney | ktibi: strictly speaking nova has no requirement that a vm will have an ip. it just requires that vms have mac adressses. | |
| 13:00:11 | ameeda | sean-k-mooney: if I do this http://paste.openstack.org/show/640870/ it will success ? | |
| 13:00:48 | sean-k-mooney | ameeda: i would guess that uuids.instance on line 6 of http://paste.openstack.org/show/640858/ is the root of the warning | |
| 13:02:03 | sean-k-mooney | ameeda: try using e21d0851-3c56-4b9d-a133-a2c1ba59778a instead and see if it resolves the issue | |
| 13:02:33 | ameeda | sean-k-mooney: thanks alot, I will try that and let you know | |
| 13:03:08 | ameeda | sean-k-mooney: I have another issue with openstack overcloud image build in dib, can you help me with that ? | |
| 13:04:00 | sean-k-mooney | ktibi: regarding your issue i dont think the compute nodes should become disabled just because a user ran our of ips in there subnet. was there any error reported in the n-cpu or n-sch logs? | |
| 13:04:12 | ktibi | sean-k-mooney so, with the default value for consecutive_build_service_disable_threshold, any user can disable all compute ? | |
| 13:06:05 | sean-k-mooney | ktibi: interesting looks like that is a new option in pike https://review.openstack.org/#/c/463597/ | |
| 13:06:57 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Fix possible TypeError in VIF.fixed_ips https://review.openstack.org/531745 | |
| 13:06:58 | sean-k-mooney | ktibi: it is intended to disable a compute node if there is an issue with the hypervisor like libvirt crashed or something like that. this would seam to be an edgecase | |
| 13:07:25 | ktibi | compute node fail to spawn instance with error ==>Failed to allocate the network(s), not rescheduling | |
| 13:07:38 | bauzas | gibi: thanks for the review, will upload a new change in 1 hour-ish | |
| 13:08:13 | ktibi | sean-k-mooney user spawn 200 instances, so I have 20 fail on each compute, all my compte become disable !! very dangerous !! | |
| 13:08:14 | sean-k-mooney | ktibi: as a work around you can set the config value to 0 to disable auto disable. | |
| 13:08:21 | ktibi | I think this option need to be disable by default | |
| 13:09:53 | sean-k-mooney | ktibi: i actully tend to agree that this probably should not be enabled by default. that said i understand why you might want this | |
| 13:10:50 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ocata: Fix possible TypeError in VIF.fixed_ips https://review.openstack.org/531746 | |
| 13:11:42 | ameeda | sean-k-mooney: same error after change instance.uuids to e21d0851-3c56-4b9d-a133-a2c1ba59778a . see the error here http://paste.openstack.org/show/640881/ | |
| 13:12:37 | sean-k-mooney | stephenfin regarding https://review.openstack.org/#/c/463597/6 do you recall why consecutive_build_service_disable_threshold is not disabled by default. | |
| 13:22:10 | sean-k-mooney | ameeda: so it looke like the varning changed form fake is an invalid UUID to fake_vol so im guessing the volume id is not set in the bdms object | |
| 13:22:20 | sean-k-mooney | ameeda: you are extending https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute_mgr.py#L278-L304 | |
| 13:25:22 | ameeda | sean-k-mooney: I try to do this http://paste.openstack.org/show/640891/, but it show me error said that instance doesn't has get_by_uuid attribute. | |
| 13:27:49 | ameeda | sean-k-mooney: what you guess me to do ? if I remove my changes, everything working fine | |
| 13:30:02 | ameeda | sean-k-mooney: when I do this http://paste.openstack.org/show/640894/ "note line 5" that will work fine. | |