Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-22
18:30:36 mriedem well i suppose it always feels good to come back and say, "this isn't a nova problem, it's a tripleo problem"
18:31:03 melwitt heh, indeed
18:31:13 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_from_dict' functions https://review.opendev.org/537414
18:31:14 openstackgerrit Stephen Finucane proposed openstack/nova master: Add '[compute] cpu_dedicated_set' option https://review.opendev.org/671792
18:31:14 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793
18:31:15 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: '_get_(v|p)cpu_total' to '_get_(v|p)cpu_available' https://review.opendev.org/672693
18:31:15 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Rewrap definitions of 'NUMACell' https://review.opendev.org/674395
18:31:16 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800
18:31:16 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Rename 'fields' import to 'obj_fields' https://review.opendev.org/674103
18:31:17 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start reporting 'HW_CPU_HYPERTHREADING' trait https://review.opendev.org/675571
18:31:17 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'hw:cpu_policy', 'hw:mem_page_size' extra specs from API samples https://review.opendev.org/675338
18:31:18 openstackgerrit Stephen Finucane proposed openstack/nova master: Add support for translating CPU policy extra specs, image meta https://review.opendev.org/671801
18:31:18 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Fold in argument to '_update_provider_tree_for_vgpu' https://review.opendev.org/676729
18:31:19 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reshaper for PCPU https://review.opendev.org/674895
18:38:53 openstackgerrit Eric Fried proposed openstack/nova master: Add docstring to check_availability_zone function https://review.opendev.org/677858
18:40:55 mriedem anyone else have an opinion about nova migration-list with --migration-type or just --type in https://review.opendev.org/#/c/675117/ ?
18:59:12 efried mriedem: sans click, I would say --migration-type just because --type is so generic.
19:02:45 artom mriedem, make it --kind-of-migration and watch the comments pour in
19:04:19 mriedem --migration-flavour?
19:05:51 artom Points for the UK spelling :D
19:06:13 artom (aka the *correct* spelling. Yours truly, a Canadian)
19:06:48 artom (except when I feel like being Quebecois or Russian - yay absence of national identity!)
19:14:15 efried mriedem: I left helpful comments on https://review.opendev.org/#/c/678051/
19:20:07 mriedem artom: how would you do it in old russ?
19:20:38 artom Gather 'round a bottle of vodka (or two) and let the solution emerge by itself
19:22:42 mriedem efried: you might need:
19:22:44 mriedem become: yes
19:24:18 artom mriedem, FWIW, I'd say just --type, because 'migration' is already explicitly implied (ha!) in the command
19:24:22 mriedem the devstack tasks that use synchronize all have some form of "become: true" for sudo
19:24:56 artom In (server-)list and flavor-list, we don't repeat the thing we're list in the args
19:25:12 artom So it's just flavor-list --extra-specs, not flavor-list --flavor-extra-specs
19:25:24 melwitt I don't know if it's just me but I didn't understand which option dtroyer was for in his comment
19:25:38 artom But I agree that --type is trickier, because it's more vague on its own than something like --extra-specs
19:29:48 mriedem melwitt: dean was saying since nova uses --*-type as a convention be consistent in nova cli
19:29:54 mriedem osc uses --type for the most part
19:30:03 mriedem since context is derived from the command you're using
19:30:15 melwitt does a nova osc command count as 'nova cli'?
19:30:21 mriedem so since nova cli uses --key-type, --console-type and --backup-type, use --migration-type
19:30:26 mriedem no
19:30:28 mriedem osc is osc
19:30:39 melwitt haha ok, that's what I was thinking
19:37:30 melwitt oh, geez, I didn't realize that change was for python-novaclient. ugh
19:37:50 melwitt I thought it was for osc 'openstack server' command
19:38:48 melwitt yeah I agree stay consistent with other python-novaclient things
19:39:21 efried mriedem: it looks like logging is enabled by default?
19:41:29 openstackgerrit Eric Fried proposed openstack/nova master: DNM: Try to repro bug 1823251 with mysql logs https://review.opendev.org/678051
19:41:30 openstack bug 1823251 in OpenStack Compute (nova) "Spike in TestNovaMigrationsMySQL.test_walk_versions/test_innodb_tables failures since April 1 2019 on limestone-regionone" [High,Confirmed] https://launchpad.net/bugs/1823251
19:41:54 mnaser sean-k-mooney: do you happen to know if you're able to backport the os-vif bug to stable/stein ?
19:42:14 efried mriedem: if it's writing to "the console"... is that the same console that tests are being run in?
19:44:03 mriedem efried: no,
19:44:11 mriedem likely the journal?
19:44:19 mriedem i.e. mysql systemd journal log
19:44:34 mriedem if i had a devstack handy i'd poke at this but i don't
19:45:00 mriedem efried: if this were an IMM or VIOS, wwyd?!
19:45:14 efried mriedem: How destructive is the poking? I have one you can jump on
19:46:06 openstackgerrit Jonathan Rosser proposed openstack/os-vif stable/stein: only disable mac ageing for ovs hybrid plug https://review.opendev.org/678098
19:47:33 efried I don't know what IMM is. If it were VIOS I would phone a friend.
19:47:33 efried The problem is that I have basically no understanding of the chain of events that's leading to mysqld running and/or being used in this environment.
19:48:38 mnaser sean-k-mooney: jrosser has backported the os-vif issue which fixes deployments letting vms see other vms traffic :)
19:49:53 mriedem integrated management module
19:49:58 mriedem like the CMM in a bladecenter
19:50:00 mriedem DOY
19:50:25 mriedem where is dansmith when i need him for ibm jokes
19:51:16 mriedem efried: https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/test-setup is the role for hitting the test-setup script in the unit test jobs
19:51:23 efried by the time we run tools/test-setup.sh, mysqld is already running, right?
19:51:52 mriedem has to be
19:51:56 mriedem since we're issuing commands against it
19:52:05 efried so whatever we're doing we need to do well before that
19:52:29 efried unless it's possible to restart mysqld from that script, is that what you were suggesting above?
19:56:29 melwitt mnaser: isn't that a pretty serious security issue? looked in the bug and it didn't look treated as such
19:57:33 openstackgerrit Merged openstack/nova master: Remove deprecated [neutron]/url option https://review.opendev.org/677755
20:01:04 mriedem efried: the mysql docs seem to suggest you can change logging config at runtime and flush the settings
20:01:09 mriedem but w/o an env to play with idk
20:01:35 mriedem efried: otherwise yeah i'd think you could restart from within the script just by doing: sudo systemctl restart mysqld
20:05:26 efried I'm looking at a default-y devstack and /var/log/mysql has files in it
20:07:49 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Fail compute service delete if resource provider delete fails https://review.opendev.org/678100
20:08:52 mriedem efried: huh yeah i guess my devstack patch works
20:08:52 mriedem https://ccb62560c9b6fbee62ac-3bdb45e74ea7431452edb68e5fcaf6be.ssl.cf5.rackcdn.com/678053/1/check/neutron-tempest-linuxbridge/d0d8b63/controller/logs/mysql/
20:09:00 mriedem https://review.opendev.org/#/c/678053/
20:09:12 mriedem maybe the ubuntu packages changed where the log files go?
20:09:15 mriedem since...trusty?
20:11:27 efried mriedem: Presumably that ^ will just work for dsvm jobs anyway since by default tox jobs don't copy *any* logs?
20:13:33 openstackgerrit Merged openstack/nova master: Limit get_sdk_adapter to requested service type https://review.opendev.org/674678
20:13:41 openstackgerrit Merged openstack/nova master: Add a hacking rule for useless assertions https://review.opendev.org/677610
20:13:49 openstackgerrit Merged openstack/nova master: doc: remove confusing docs about aggregate allocation ratios https://review.opendev.org/677472
20:28:49 mnaser melwitt: it is a very serious secuity issue
20:28:55 mnaser with linuxbridge, a vm can see traffic to other vms
20:29:49 melwitt ok. argh, so by default we get no OSSA handling with it because of the way os-vif project is tagged? feels like we need to fix that (I don't know how)
20:29:52 melwitt I commented on the lp bug
20:36:39 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Find instance in another cell during floating IP re-association https://review.opendev.org/656594
20:36:45 mriedem efried: tox jobs copy logs, like the tox logs and testr results
20:42:33 openstackgerrit Michael Still proposed openstack/nova master: Remove zVM requirement from the top level requirements. https://review.opendev.org/677870
21:06:47 mriedem dansmith: i looked at mdbooths thing here https://review.opendev.org/#/c/677736/ and it seems sane, just some nits and such, and maybe some unnecessary test code
21:06:54 mriedem guess we're just waiting on a customer feedback
21:07:53 openstackgerrit Merged openstack/nova master: Update docstring of 'revert_resize' function https://review.opendev.org/665976
21:21:12 jrosser mnaser: melwitt i can already see that this is failing https://review.opendev.org/#/c/678098/
21:21:28 openstackgerrit Artom Lifshitz proposed openstack/nova master: Introduce live_migration_claim() https://review.opendev.org/635669
21:21:28 openstackgerrit Artom Lifshitz proposed openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827
21:21:29 openstackgerrit Artom Lifshitz proposed openstack/nova master: LM: add support for augmenting migrate_data with info from claims https://review.opendev.org/634828
21:21:29 openstackgerrit Artom Lifshitz proposed openstack/nova master: LM: add support for updating NUMA-related XML on the source https://review.opendev.org/635229

Earlier   Later