Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-22
13:24:36 sean-k-mooney - Failed: 0
13:24:38 sean-k-mooney Sum of execute time for each test: 0.4916 sec.
13:24:40 sean-k-mooney not quite
13:25:25 sean-k-mooney and the time output
13:25:29 sean-k-mooney real 0m5.681s
13:25:30 sean-k-mooney user 1m33.494s
13:25:32 sean-k-mooney sys 0m9.505s
13:26:00 sean-k-mooney that said before i got that working yesterday i would agree way to slow
13:28:30 gibi hehe this rtox python package implements what I have in bash for years. I independently named that bash script 'rtox'
13:29:39 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Support old & new versions of svm and avx traits https://review.opendev.org/660515
13:29:57 efried iiuc, ^ stands a chance of passing functional and tempest, but will still fail unit.
13:30:40 efried which does kind of defeat the purpose, as it's unit tests I've changed.
13:30:56 sean-k-mooney gibi: ya im proably goign to stop using it. the python version takes way to long to actully output to stdout
13:31:07 efried The tests I changed succeed locally though. shrug.
13:31:55 sean-k-mooney efried: wait why are we reported avx and svm in a vendor speficic way
13:32:54 sean-k-mooney oh we are not at least for AVX
13:32:56 efried because they're vendor-specific flags
13:33:12 sean-k-mooney efried: you ment VMX not AVX
13:33:14 efried point is, we weren't, but we should have been, so now we are.
13:33:27 efried oh, did I misspell something? :P
13:33:42 sean-k-mooney :)
13:34:10 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Support old & new versions of svm and vmx traits https://review.opendev.org/660515
13:34:10 efried thanks sean-k-mooney, nice catch
13:34:19 sean-k-mooney are the only spelling mistakes i can spot instruction sets?
13:34:29 efried I'll take it.
13:34:49 efried we have plenty of other anal-retentive people to catch the rest.
13:35:04 efried community development ftw
13:35:11 sean-k-mooney that was more a self commentry on how deep the intel brainwashing must have penetrated :P
13:36:01 sean-k-mooney it bypassed my in ability to spell and inserted an instruction set filter into my brain lol
13:39:09 efried aarents: I feel like I must be forgetting some background on https://review.opendev.org/#/c/659054/
13:39:26 efried pieces of this look vaguely familiar. Have we talked about this before?
13:39:38 efried I can see I've not been on the review itself...
13:40:37 stephenfin efried: I assume you got sorted? I'm knee deep in crappy unit tests atm :)
13:40:54 efried stephenfin: Yes thanks.
13:41:06 efried got an answer anyway, if not sorted how I would like.
13:54:59 dansmith mriedem: do we still have a job that does a full archive after tempest?
13:56:37 dansmith I see mention in nova-next, but not sure where it's called
13:56:48 sean-k-mooney as in one that runs https://github.com/openstack/nova/blob/master/gate/post_test_hook.sh
13:57:09 dansmith ah yep
13:57:11 dansmith thanks
13:57:27 sean-k-mooney it looks like 3 https://github.com/openstack/nova/search?q=post_test_hook.sh&unscoped_q=post_test_hook.sh
13:58:47 mriedem nova-next is the one you're looking for
13:59:29 sean-k-mooney its done the live migration job too and the grenade version
13:59:41 mriedem not the same scripts dan is looking for
14:00:18 sean-k-mooney oh right they run a different post test hook
14:00:29 dansmith mriedem: okay the --before patch is too stale to have test output, but wanted to see what that run looks like
14:00:32 sean-k-mooney so ya nova-nextg
14:18:19 openstackgerrit Surya Seetharaman proposed openstack/python-novaclient master: Allow passing negative values for the locked search_opt in cs.servers.list https://review.opendev.org/659783
14:25:53 aarents efried: I think you where about to read this 2 days ago, but you saw that it was no more mergeable. never mind I need a core having a look on this since Matthew has "+1" do you know who can help me on this ? ( https://review.opendev.org/#/c/659054/ )
14:27:03 efried aarents: I'm reviewing it. I have some issues with the test. Working on formulating a solution...
14:28:20 aarents ok
14:28:22 aarents thanks
14:43:40 mriedem https://www.youtube.com/watch?v=SGZqDzb__bw
14:43:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method https://review.opendev.org/634832
14:43:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova.compute.utils.delete_image https://review.opendev.org/637605
14:43:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Add PrepResizeAtSourceTask https://review.opendev.org/627891
14:43:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor ComputeManager.remove_volume_connection https://review.opendev.org/642183
14:43:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Add power_on kwarg to ComputeDriver.spawn() method https://review.opendev.org/642590
14:43:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Add finish_snapshot_based_resize_at_dest compute method https://review.opendev.org/635080
14:43:50 openstackgerrit Matt Riedemann proposed openstack/nova master: Add FinishResizeAtDestTask https://review.opendev.org/635646
14:43:50 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field https://review.opendev.org/614035
14:43:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute CrossCellMigrationTask from MigrationTask https://review.opendev.org/635668
14:43:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Plumb allow_cross_cell_resize into compute API resize() https://review.opendev.org/635684
14:43:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Filter duplicates from compute API get_migrations_sorted() https://review.opendev.org/636224
14:43:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.opendev.org/614037
14:43:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Start functional testing for cross-cell resize https://review.opendev.org/636253
14:43:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle target host cross-cell cold migration in conductor https://review.opendev.org/642591
14:43:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Validate image/create during cross-cell resize functional testing https://review.opendev.org/642592
14:43:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Add zones wrinkle to TestMultiCellMigrate https://review.opendev.org/643450
14:43:55 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.opendev.org/643451
14:43:55 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add confirm_snapshot_based_resize_at_source https://review.opendev.org/637058
14:44:12 edleafe mriedem: oh, are you out of /dev/jail?
14:44:20 mriedem let's say yes
14:45:39 prometheanfire failures for the new fasteners fasteners===0.15 http://logs.openstack.org/54/660254/2/check/cross-nova-functional/a0617a9/testr_results.html.gz
14:45:59 openstackgerrit Merged openstack/nova stable/stein: Exclude fake marker instance when listing servers https://review.opendev.org/657421
14:47:07 hansmoleman wth is fasteners and what's using it?
14:47:28 openstackgerrit Boxiang Zhu proposed openstack/nova-specs master: Update "add host and hypervisor_hostname flag to create server" spec https://review.opendev.org/660560
14:49:43 hansmoleman bah oslo.concurrency http://codesearch.openstack.org/?q=import%20fasteners&i=nope&files=&repos=
14:50:02 hansmoleman prometheanfire: open a bug please
14:52:03 hansmoleman tssurya: i can address the small issues in https://review.opendev.org/#/c/659783/ if you're busy and then +2 it
14:53:29 prometheanfire hansmoleman: ok, so far only nova has had issue with it
15:07:09 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Allow passing negative values for the locked search_opt in cs.servers.list https://review.opendev.org/659783
15:15:34 tssurya hansmoleman: thanks! yea I am snowflake
15:22:10 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311
15:22:19 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop passing 'kwargs' to 'rebuild_instance' https://review.opendev.org/651312
15:22:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop passing 'delete_type' to 'terminate_instance' https://review.opendev.org/651313
15:23:11 openstackgerrit Stephen Finucane proposed openstack/nova master: filters: Stop handling cells v1 https://review.opendev.org/651314
15:24:51 efried stephenfin: I was considering reworking https://review.opendev.org/#/c/660527/ (get rid of nova.compute.*API()) to just change most of the imports from
15:24:51 efried from nova import compute
15:24:51 efried to
15:24:51 efried from nova.compute import api as compute
15:24:51 efried so that the code itself wouldn't change at all.
15:25:13 efried Again "most of" - some other minor tweaks will be necessary, like where there was already an import of nova.compute.api as compute_api
15:25:21 efried anyway, what do you think of that idea?
15:25:58 stephenfin I must admit I kind of like it how it is atm, with the few exceptions noted
15:26:15 efried The code lines would stay kind of intuitive: compute.API() is nice and clean, clearer than api.API() and less redundant than compute_api.API()
15:26:19 efried okay.
15:26:23 stephenfin Yeah, fair point
15:26:31 efried It's less work to leave it like it is, so I'm not going to fight for this :)
15:26:44 stephenfin I'd be happy with either in that case, actually

Earlier   Later