Earlier  
Posted Nick Remark
#openstack-nova - 2019-10-10
19:50:37 mriedem the api checks port quota but not volume
19:51:00 mriedem it would also help with some other stuff like az
19:51:23 mriedem i.e. if cross_az_attach=False, creating the volume earlier will tell us if we can't create a volume in a given az
19:52:00 mriedem mnaser: also note that (semi-recently) check cinder quota stuff during volume-backed snapshot
19:52:34 mriedem https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3105
19:54:04 mriedem doing that during bfv in the api or conductor seems pretty trivial
19:54:26 mriedem we already know (a) how many instances we're going to create and (b) how many volumes we're going to create for those instances
19:54:52 mriedem it'd be racy but so is the port quota check
20:05:52 mnaser mriedem: yeah and honestly i think its better to fail at the api layer than go through X reschedules..
20:06:23 mriedem fun fact,
20:06:34 mriedem we don't reschedule for prep_block_device failures in the compute
20:06:38 mriedem you're welcome
20:07:17 mriedem https://bugs.launchpad.net/nova/+bug/1488111
20:07:17 openstack Launchpad bug 1488111 in OpenStack Compute (nova) "Boot from volumes that fail in initialize_connection are not rescheduled" [Wishlist,Confirmed]
20:08:53 mriedem dansmith: https://review.opendev.org/#/c/687140/ is failing a few jobs (functional, api-ref and pep8) - did you want me to wait on re-reviewing that?
20:09:25 dansmith mriedem: ah yeah I didn't run a full slammajamma on that one, so yeah I'll hit that up in the mo'nan
20:09:42 mriedem ok i've wip'ed it
20:09:47 mriedem wip'ed it good
20:09:57 dansmith thanks
20:10:14 mriedem a couple of nits in the conductor one but +2
20:10:25 dansmith ack, thanks I'll look tomorrow as well
20:44:51 mriedem oh my https://www.cnn.com/2019/10/09/us/pge-power-outage-wednesday/index.html
20:44:55 mriedem had to google
20:45:19 mriedem are they the same ones that were responsible for all the faulty lines that caused the last wild fires?
20:45:56 melwitt yeah
20:46:30 melwitt I don't think their lines caused all of them but one was due to faulty equipment
20:47:28 melwitt and instead of fixing things during all the months prior to now, they shut off everything to prevent a problem
20:52:11 melwitt their faulty equipment caused Camp Fire last year, the largest wildfire in CA history. san bruno in 2010 was them too. other than that, I am not sure there were more
20:53:56 mriedem a track record of success
20:56:43 melwitt yeah... -_-
21:27:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Move Destination object tests to their own test class https://review.opendev.org/683017
21:28:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-guide: Fix available info in handling down cells https://review.opendev.org/686520
21:28:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix unit of hw_rng:rate_period https://review.opendev.org/683051
21:28:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update keypairs in saving an instance object https://review.opendev.org/683043
#openstack-nova - 2019-10-11
04:32:31 openstackgerrit Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243
04:36:43 openstackgerrit Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243
04:37:19 openstackgerrit Brin Zhang proposed openstack/nova master: Fix legacy issues in Filter migrations by user_id/project_id https://review.opendev.org/682198
04:38:58 openstackgerrit Brin Zhang proposed openstack/nova master: Fix legacy issues in filter migrations by user_id/project_id https://review.opendev.org/682198
06:00:26 openstackgerrit ya.wang proposed openstack/nova master: libvirt: Change _compare_cpu to raise InternalError https://review.opendev.org/687808
06:20:32 openstackgerrit Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856
06:45:38 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Change the primary assignee to the mainly contributor https://review.opendev.org/688042
06:46:55 openstackgerrit Yongli He proposed openstack/nova master: Clean up orphan instances virt driver https://review.opendev.org/648912
06:46:55 openstackgerrit Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765
08:07:08 openstackgerrit Merged openstack/nova master: Fix exception translation when creating volume https://review.opendev.org/678991
08:17:42 openstackgerrit Merged openstack/python-novaclient master: PDF documentation build https://review.opendev.org/687966
08:20:32 openstackgerrit Merged openstack/nova master: Fix unit of hw_rng:rate_period https://review.opendev.org/683051
08:32:14 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: Stop sending bad values from libosinfo to libvirt https://review.opendev.org/688067
08:33:52 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Stop sending bad values from libosinfo to libvirt https://review.opendev.org/688068
08:34:28 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: Stop sending bad values from libosinfo to libvirt https://review.opendev.org/688069
08:39:49 trident Any thoughts on https://bugs.launchpad.net/nova/+bug/1741364 ?
08:39:49 openstack Launchpad bug 1741364 in OpenStack Compute (nova) "ceph ephemeral info not updated during live migrate" [Medium,Confirmed]
08:41:19 openstackgerrit Fan Zhang proposed openstack/nova stable/train: Fix exception translation when creating volume https://review.opendev.org/688072
08:43:05 openstackgerrit Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073
08:43:53 openstackgerrit Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073
09:03:58 openstackgerrit Takashi NATSUME proposed openstack/nova stable/train: Fix unit of hw_rng:rate_period https://review.opendev.org/688083
09:08:42 openstackgerrit ya.wang proposed openstack/nova master: libvirt: Change _compare_cpu to raise InvalidCPUInfo https://review.opendev.org/687808
09:21:27 openstackgerrit Fan Zhang proposed openstack/nova stable/stein: Fix exception translation when creating volume https://review.opendev.org/688073
09:26:59 openstackgerrit Fan Zhang proposed openstack/nova stable/rocky: Fix exception translation when creating volume https://review.opendev.org/688089
09:31:40 openstackgerrit Nicolas Belouin proposed openstack/nova master: Add support for Xen PVH VM Mode in libvirt driver https://review.opendev.org/687827
09:39:13 stephenfin did we not have a meeting last night
09:41:10 openstackgerrit Fan Zhang proposed openstack/nova stable/queens: Fix exception translation when creating volume https://review.opendev.org/688090
11:20:27 kashyap Any stable cores, mind having a look at this stable/queens thing? It has also merged in other stable branches: https://review.opendev.org/#/c/668111/ (libvirt: Rework 'EBUSY' (SIGKILL) error handling code path
11:20:31 kashyap )
11:21:03 kashyap (It has merged in Rocky and Stein.)
11:34:26 lyarwood kashyap: yup sorry looking now
11:34:47 kashyap Gracias
11:56:27 openstackgerrit Balazs Gibizer proposed openstack/nova stable/pike: Error out interrupted builds https://review.opendev.org/687918
11:59:21 kashyap Needs a second stable core +2 here: https://review.opendev.org/#/c/668111/
12:21:39 frickler this bug is targeted on tempest, but I'd like to know from nova ppl how valuable testing the hard reboot is to you, as I'm considering to propose dropping this test if it can't be made more reliable https://bugs.launchpad.net/tempest/+bug/1847759
12:21:39 openstack Launchpad bug 1847759 in tempest "reboot_server_hard can trigger corrupted file system" [Undecided,New]
12:25:46 lyarwood frickler: IMHO it's useful and that sounds worrying as a bug report
12:28:02 frickler lyarwood: I'm even more worried after seeing how many matches my logstash query has, though I haven't verified yet how many of those match the bug http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22self._test_reboot_server('HARD')%5C%22
12:29:41 lyarwood frickler: I wonder if we even need to boot an instance using an image to test hard reboot
12:30:25 lyarwood frickler: if corruption of the image by the guest OS dying suddenly is the issue here
12:36:06 frickler lyarwood: o.k., I checked some of the failure logs. 2/3 seem to fail to connect to the instance on the initial boot already, 1/3 looks like fs corruption after the hard reboot, so still a relevant set. if you have any idea how to make that more stable, I think that would be pretty useful
12:57:42 bauzas stephenfin: nope, the nova meeting was cancelled, see ML
13:23:30 stephenfin bauzas: ah, yes, thanks
13:24:02 stephenfin bauzas: Would you do me the honours? https://review.opendev.org/#/c/684402/ https://review.opendev.org/#/c/685929/ https://review.opendev.org/#/c/687264// ?
13:25:30 openstackgerrit Matthew Booth proposed openstack/nova master: Don't take a lock when fetching a cached cell mapping https://review.opendev.org/688122
13:29:15 stephenfin bauzas: ta
13:35:40 openstackgerrit Merged openstack/nova master: api-guide: Fix available info in handling down cells https://review.opendev.org/686520
13:41:54 openstackgerrit Merged openstack/nova-specs master: Change the primary assignee to the mainly contributor https://review.opendev.org/688042
13:41:55 openstackgerrit Merged openstack/nova-specs master: Fix followup comments of policy-defaults-refresh spec https://review.opendev.org/669196
13:54:13 mdbooth lyarwood: Re https://code.engineering.redhat.com/gerrit/182841 GAH! and thanks. Mind if we don't wait for tempest after I update the commit message?
13:54:59 lyarwood mdbooth: wrong window but yeah of course
13:55:28 mdbooth lyarwood: Hah, so it is
14:11:18 openstackgerrit Balazs Gibizer proposed openstack/nova master: Avoid using image with kernel in BDM large request func test https://review.opendev.org/688132
14:40:08 gibi stephenfin: quick question in https://review.opendev.org/#/c/686802/6//COMMIT_MSG@18
14:47:00 stephenfin gibi: replied
14:47:51 gibi stephenfin: +2
14:47:58 stephenfin ta
14:55:54 stephenfin gibi: If you're on a reviewing streak, I'd appreciate your eyes on the following patch too since it's NeutronFixture'y and you know that stuff, heh https://review.opendev.org/#/c/684344/
14:56:04 stephenfin Feel free to chuck something my way too
14:56:10 gibi stephenfin: reviewing it right no
14:56:13 gibi w
14:56:20 stephenfin nice :D
14:56:43 dansmith gibi: there are a few patches in this stack for which you reviewed the spec that you could +W if you want :)
14:56:44 dansmith gibi: https://review.opendev.org/#/c/687137/4
14:57:05 gibi dansmith: ack, I will look at it after stephenfin's patch
14:57:28 dansmith gibi: thanks

Earlier   Later