openstackgerrit | Matt Riedemann proposed openstack/nova master: Retry on MessagingTimeout to init compute RPC API during n-cpu start https://review.openstack.org/597330 | 00:44 |
---|---|---|
*** mriedem has quit IRC | 00:44 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Retry on MessagingTimeout to init compute RPC API during n-cpu start https://review.openstack.org/597330 | 00:46 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Retry on MessagingTimeout to init compute RPC API during n-cpu start https://review.openstack.org/597330 | 00:48 |
*** tetsuro has joined #openstack-placement | 00:52 | |
*** alex_xu has joined #openstack-placement | 01:14 | |
*** lei-zh has joined #openstack-placement | 01:57 | |
*** lei-zh has quit IRC | 02:00 | |
*** lei-zh has joined #openstack-placement | 02:00 | |
*** openstack has joined #openstack-placement | 02:52 | |
*** ChanServ sets mode: +o openstack | 02:53 | |
*** openstackstatus has joined #openstack-placement | 03:01 | |
*** ChanServ sets mode: +v openstackstatus | 03:01 | |
*** nicolasbock has quit IRC | 03:58 | |
*** tetsuro has quit IRC | 04:04 | |
*** tetsuro has joined #openstack-placement | 04:26 | |
*** tetsuro has quit IRC | 05:04 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/597378 | 05:54 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [placement] Use osloutils uuidsentinel https://review.openstack.org/594144 | 06:15 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix a broken conf file description in networking doc https://review.openstack.org/597391 | 06:47 |
*** takashin has left #openstack-placement | 06:48 | |
*** tssurya has joined #openstack-placement | 06:53 | |
gibi | good morning. I happy to see that the issue about killing the nova service from a periodic has been settled during the night. I'm a bit suprised that we adapted our intention instead of adapted our code but at the end I agree that if it breaks later we can fix later | 06:59 |
*** ttsiouts has joined #openstack-placement | 07:45 | |
*** ttsiouts has quit IRC | 07:49 | |
*** e0ne has joined #openstack-placement | 07:56 | |
openstackgerrit | Merged openstack/nova master: Mention (unused) RP generation in POST /allocs/{c} https://review.openstack.org/597304 | 07:57 |
*** e0ne has quit IRC | 08:00 | |
openstackgerrit | Stephen Finucane proposed openstack/nova stable/rocky: Don't use '_TransactionContextManager._async' https://review.openstack.org/597421 | 08:08 |
*** ttsiouts has joined #openstack-placement | 08:16 | |
*** e0ne has joined #openstack-placement | 08:38 | |
*** cdent has joined #openstack-placement | 08:39 | |
openstackgerrit | Merged openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | 09:22 |
openstackgerrit | Merged openstack/nova master: Document no content on POST /reshaper 204 https://review.openstack.org/596494 | 09:22 |
cdent | \o/ | 09:22 |
openstackgerrit | huanhongda proposed openstack/nova master: List soft-deleted instances by "--status" option https://review.openstack.org/597434 | 09:31 |
cdent | hmmm, bit less \o/. that merge has broken placecat, investigating | 09:38 |
*** ttsiouts has quit IRC | 10:07 | |
gibi | cdent: do we need to create a new placement-specs repo along with the new placement repo or we keep doing the spec reviews for Stein in nova-specs? | 10:34 |
gibi | I would prefer the later as we already have spec discussed for placement in the nova-specs | 10:35 |
cdent | gibi: I think the latter is probably right, if the plan is wait on any governance changes until later | 10:35 |
gibi | cdent: then we are in agreement. thanks | 10:36 |
cdent | ✔ | 10:36 |
cdent | I do, however, think that once we are able to think about specs in terms of their placement bits and their not placement bits it will at least make the placement half of spec review a lot easier | 10:36 |
openstackgerrit | Chen proposed openstack/nova master: Fix filter servers with SOFT_DELETED status https://review.openstack.org/597443 | 10:40 |
openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Placement: any traits in allocation_candidate query https://review.openstack.org/565730 | 10:46 |
openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Placement: support mixing required traits with any traits https://review.openstack.org/565741 | 10:46 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova master: doc: add info how to troubleshoot vmware specific problems https://review.openstack.org/597446 | 10:49 |
*** nicolasbock has joined #openstack-placement | 11:11 | |
*** ttsiouts has joined #openstack-placement | 11:20 | |
cdent | jaypipes: bug you might be interested in: the new aggregate write handling is causing some issues with sqlite (when using files), which I wonder might be masking an issue for mysql: https://bugs.launchpad.net/nova/+bug/1789633 | 11:29 |
openstack | Launchpad bug 1789633 in OpenStack Compute (nova) "placement aggregate handling has lock trouble with sqlite files" [Undecided,New] | 11:29 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | 11:40 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | 11:40 |
openstackgerrit | Merged openstack/nova master: Refix disk size during live migration with disk over-commit https://review.openstack.org/536351 | 11:42 |
openstackgerrit | Merged openstack/nova master: Fix a broken conf file description in networking doc https://review.openstack.org/597391 | 11:42 |
*** alex_xu has quit IRC | 11:49 | |
jaypipes | cdent: left question for you on bug. not sure if this is an oslo.db enginefacade bug or whether we're just using it improperly. | 11:51 |
cdent | thanks jaypipes, reading | 11:54 |
openstackgerrit | Jay Pipes proposed openstack/os-traits master: clean up CUDA traits https://review.openstack.org/597170 | 11:54 |
jaypipes | cdent: for placecat travis jobs, is there a place I can look at the placement-api logs? | 11:57 |
cdent | not really but I can either paste you some, or if you've got a working docker install it takes about 5 minutes to make your own go (see the README in placecat) | 11:59 |
openstackgerrit | Jay Pipes proposed openstack/os-traits master: clean up CUDA traits https://review.openstack.org/597170 | 11:59 |
cdent | jaypipes: search for 'status: 500' in http://paste.openstack.org/show/729046/ and then go back a few lines | 12:00 |
jaypipes | cdent: ack, np, I can do that. just wondered if there were artifacts on travis. | 12:00 |
cdent | jaypipes: not that I'm aware of, but lemme look a bit closer | 12:01 |
cdent | as that would be handy. mostly I've been using the travis + placecat stuff as an alarm to go check elsewhere | 12:01 |
jaypipes | cdent: I don't see a status: 500 in that paste... right link | 12:02 |
jaypipes | ? | 12:02 |
cdent | jaypipes: ah it got truncated, let me make a new one | 12:02 |
openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/os-traits master: clean up CUDA traits https://review.openstack.org/597170 | 12:03 |
cdent | jaypipes: here we go, this should have two: http://paste.openstack.org/show/729047/ | 12:07 |
cdent | brb | 12:07 |
cdent | jaypipes: if I shell in and create the aggregate in placement_agregates before running the gabbits, they're fine | 12:23 |
jaypipes | cdent: ack. so I'm pretty sure it's the use (or how we're using) the writer.independent context manager there. | 12:28 |
cdent | yeah, I've just tried it without that, and it seems to be okay | 12:28 |
cdent | or rather: it doesn't break for this | 12:28 |
jaypipes | cdent: I can try to push a couple patches later today but I'm a bit swamped with $worky stuff today and tomorrows. | 12:28 |
cdent | I don't recall why independent was used there in the first place? | 12:29 |
jaypipes | cdent: yeah, I'm afraid it might "fix" it for this, but break it for mysql/pg again :) | 12:29 |
cdent | How about I push a dnm and see what happens? | 12:29 |
jaypipes | cdent: certainly worth just pushing a patch that removes independent and seeing what blows up. | 12:29 |
jaypipes | cdent: heh, jinx | 12:29 |
cdent | can do america | 12:29 |
jaypipes | cdent: remember that patch series from me was a few "throw some shit against a wall and see what sticked" approaches. Not sure whether we thought independent writer context was the thing that fixed the race problems for mysql, or it was just an unintended casualty of the throwing against a wall | 12:31 |
* cdent nods | 12:31 | |
*** mriedem has joined #openstack-placement | 12:42 | |
openstackgerrit | Chris Dent proposed openstack/nova master: DNM: [placement] Make _ensure_aggregate context not independent https://review.openstack.org/597486 | 12:42 |
openstackgerrit | Chen proposed openstack/nova master: Fix filter server list with SOFT_DELETED status https://review.openstack.org/597443 | 13:04 |
*** ttsiouts has quit IRC | 13:34 | |
*** ttsiouts has joined #openstack-placement | 13:40 | |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Random names for functional tests https://review.openstack.org/542745 | 13:43 |
openstackgerrit | Merged openstack/osc-placement master: Update reno for stable/rocky https://review.openstack.org/586115 | 14:11 |
openstackgerrit | Merged openstack/osc-placement master: Add image link in README.rst https://review.openstack.org/586839 | 14:20 |
openstackgerrit | Merged openstack/osc-placement master: Resource provider examples https://review.openstack.org/553461 | 14:26 |
cdent | jaypipes: that canary patch seems to be doing alright in zuul. The nova-next job (not yet finished but I was watching the console) doesn't have any issue create aggregates, and other jobs are green, it might be a goer: https://review.openstack.org/#/c/597486/ | 14:28 |
openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Placement model for passthrough devices https://review.openstack.org/591037 | 14:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Log the operation when updating generation in ProviderTree https://review.openstack.org/597553 | 14:43 |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Making instance listing skipping down cells configurable https://review.openstack.org/592428 | 14:54 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add debug logs for when provider inventory changes https://review.openstack.org/597560 | 14:56 |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add scatter-gather-single-cell utility https://review.openstack.org/594947 | 15:00 |
*** N3l1x has joined #openstack-placement | 15:03 | |
openstackgerrit | Merged openstack/osc-placement master: Random names for functional tests https://review.openstack.org/542745 | 15:04 |
openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Placement model for passthrough devices https://review.openstack.org/591037 | 15:05 |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Make _ensure_aggregate context not independent https://review.openstack.org/597486 | 15:10 |
*** ttsiouts has quit IRC | 15:11 | |
jaypipes | cdent: how often was it happening without the patch? | 15:23 |
cdent | jaypipes: in placecat, I was able to make it happen every time | 15:23 |
jaypipes | cdent: ack. and with the patch, how many runs has placecat had? | 15:24 |
openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Tester for grenade job https://review.openstack.org/597566 | 15:24 |
cdent | in the functional tests, not all the time, but I'm not sure how much of that was my hack for "using real files with sqlite" wasn't quite right | 15:24 |
cdent | jaypipes: how many do you want? | 15:24 |
jaypipes | cdent: enough for you to feel comfortable? :) honestly, I'm not sure. | 15:27 |
cdent | jaypipes: one sec, I'm cooking up a one line to do it many times in a fresh container | 15:28 |
*** ttsiouts has joined #openstack-placement | 15:28 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Make _ensure_aggregate context not independent https://review.openstack.org/597486 | 15:30 |
cdent | jaypipes: 20 clean runs so far | 15:30 |
cdent | (with the patch) | 15:30 |
jaypipes | cdent: cool. I'm happy with that. | 15:30 |
cdent | for i in {1..10} ; do foo=$(docker run -t -d -p 127.0.0.1:8080:80 --env-file dockerenv placetest) && sleep 1 && gabbi-run http://127.0.0.1:8080 -- gabbits/*.yaml && docker kill $foo ; done # did the trick | 15:32 |
cdent | efried: can you remember more about how the independent transaction on _ensure_aggregate entered into the picture? | 15:37 |
efried | cdent: Yeah, I was trying to dredge that up. | 15:37 |
efried | ugh, clearly that context isn't in the patch comments. Must have been in IRC. | 15:41 |
efried | cdent: http://eavesdrop.openstack.org/irclogs/%23openstack-placement/%23openstack-placement.2018-08-16.log.html#t2018-08-16T21:12:15 read along with me... | 15:42 |
*** ttsiouts has quit IRC | 15:43 | |
efried | I think it was one of two (apparently separate and independent (so to speak)) approaches to fixing the problem. | 15:44 |
efried | One was removing the flush(); the other was the independent transacton. | 15:45 |
*** cdent has quit IRC | 15:47 | |
*** cdent has joined #openstack-placement | 15:51 | |
cdent | thanks efried | 15:57 |
*** efried is now known as efried_rollin | 15:57 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Restart scheduler in TestNovaManagePlacementHealAllocations https://review.openstack.org/597571 | 16:00 |
*** tssurya has quit IRC | 16:04 | |
jaypipes | cdent: +W on https://review.openstack.org/#/c/597486/ | 16:17 |
cdent | cool, thanks | 16:19 |
*** e0ne has quit IRC | 16:56 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Resource provider - request group mapping in allocation candidate https://review.openstack.org/597601 | 17:02 |
gibi | efried_rollin, cdent, jaypipes: I think we needs to spend some time on this ^^ on the PTG | 17:04 |
cdent | gibi: thanks for pointing that out. I think I've used up all my brain today, but will try to digest it tomorrow morning | 17:05 |
gibi | efried_rollin, cdent, jaypipes: and I'm sorry dragging you into a long and most probably controversial discussion | 17:06 |
gibi | cdent: good decision as it pretty long and complex | 17:06 |
cdent | oh dear. I made the mistake of skimming the intro. this will be interesting. | 17:07 |
cdent | I at least have the idea in my head for some background processing | 17:10 |
jaypipes | gibi: you want the discussion on the review itself, yeah? | 17:11 |
gibi | jaypipes: review would be better for me as I will have to leave suddenly soon | 17:12 |
jaypipes | gibi: np | 17:13 |
gibi | jaypipes: and of course face to face in Denver | 17:13 |
jaypipes | gibi: I shall be working on my Hungarian. | 17:13 |
gibi | jaypipes: I can give a short Hungarian lessons in a pub during the PTG if you are interested :) | 17:14 |
jaypipes | gibi: ++ | 17:14 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: (Re)start caching scheduler after starting computes in tests https://review.openstack.org/597606 | 17:17 |
*** e0ne has joined #openstack-placement | 18:19 | |
*** e0ne_ has joined #openstack-placement | 18:23 | |
*** e0ne has quit IRC | 18:25 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add contributor guide for upgrade status checks https://review.openstack.org/596902 | 18:36 |
cdent | jaypipes: if you've got time this and its child are light and easy improvements: https://review.openstack.org/#/c/591850/ | 18:38 |
cdent | (the is already +W, parent is one +2 | 18:39 |
*** e0ne_ has quit IRC | 18:39 | |
*** e0ne has joined #openstack-placement | 18:41 | |
jaypipes | cdent: "pirate on crack output". <-- bueno. | 18:42 |
cdent | that's an efried_rollin -ism | 18:42 |
*** e0ne has quit IRC | 18:44 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add debug logs for when provider inventory changes https://review.openstack.org/597560 | 18:51 |
*** e0ne has joined #openstack-placement | 18:56 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add debug logs for when provider inventory changes https://review.openstack.org/597560 | 19:12 |
*** efried_rollin is now known as efried | 19:26 | |
*** mriedem has quit IRC | 19:31 | |
*** mriedem has joined #openstack-placement | 19:31 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Revert "libvirt: add method to configure migration speed" https://review.openstack.org/590814 | 19:41 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for live migrate with anti-affinity group https://review.openstack.org/588935 | 19:55 |
*** e0ne has quit IRC | 19:56 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete instance_group_member records from API DB during archive https://review.openstack.org/588943 | 20:16 |
*** mgagne has joined #openstack-placement | 20:20 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete instance_group_member records from API DB during archive https://review.openstack.org/588943 | 20:21 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old check_attach version check in API https://review.openstack.org/588348 | 20:28 |
*** cdent has quit IRC | 20:28 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add encrypted volume support to feature matrix docs https://review.openstack.org/570255 | 21:00 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Default AZ for instance if cross_az_attach=False and checking from API https://review.openstack.org/469675 | 21:18 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Time how long select_destinations() takes in conductor https://review.openstack.org/517108 | 21:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Combine error handling blocks in _do_build_and_run_instance https://review.openstack.org/545960 | 21:40 |
*** mriedem is now known as mriedem_away | 21:44 | |
*** mriedem_away has quit IRC | 21:49 | |
*** takashin has joined #openstack-placement | 21:51 | |
openstackgerrit | Merged openstack/nova master: doc: add info how to troubleshoot vmware specific problems https://review.openstack.org/597446 | 22:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Document differences and similaries between extra specs and hints https://review.openstack.org/581410 | 23:09 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.openstack.org/576758 | 23:51 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!