Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-03-04
20:59:43 openstackgerrit Kashyap Chamarthy proposed openstack/nova-specs master: cpu-model-selection: s/cpu_models/cpu_model_list/ https://review.openstack.org/640866
21:31:01 mriedem cfriesen: some questions in https://review.openstack.org/#/c/620706/
21:31:07 mriedem cfriesen: happy to discuss here
21:31:15 cfriesen checking
21:31:49 openstackgerrit guang-yee proposed openstack/nova master: pass endpoint interface to Ironic client https://review.openstack.org/640879
21:33:37 openstackgerrit guang-yee proposed openstack/nova master: pass endpoint interface to Ironic client https://review.openstack.org/640879
21:34:22 cfriesen mriedem: I've got a change queued up for get_pci_requests_from_flavor. can add to existing or do as followup. Happy to split out PciInvalidAlias.
21:35:13 cfriesen rest of comments look reasonable, will address later today.
21:35:19 mriedem cfriesen: i'd split as much as possible for review sanity
21:36:35 cfriesen the get_pci_requests_from_flavor change is pretty straightforward, just hadn't pushed it yet since I was expecting to respin this.
21:43:23 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: cleanup pci.alias references https://review.openstack.org/640882
21:58:24 openstackgerrit Merged openstack/nova master: Add nits from Id2beaa7c4e5780199298f8e58fb6c7005e420a69 https://review.openstack.org/640729
22:14:44 sean-k-mooney from the description of https://review.openstack.org/#/c/603844/ im not sure if its needed due to https://review.openstack.org/#/c/634276/
22:15:08 sean-k-mooney it look like we are fixign the same or related bugs in different ways on both the nova and neutron side.
22:15:23 sean-k-mooney that said i need to review https://review.openstack.org/#/c/603844/ properly
22:31:40 openstackgerrit Eric Fried proposed openstack/nova master: Stop using PlacementDirect https://review.openstack.org/640887
22:31:55 efried mriedem, cdent: A different solution ^
22:38:59 sean-k-mooney efried: ignore whitespacce helps alot
22:39:18 efried sean-k-mooney: Indeed.
22:39:32 efried Are you telling me that having seen my comment already?
22:40:22 sean-k-mooney initally i tought you rewrote most of the file. and yes i was about to ask if they were fucntuionally the same after then i made that change looked and commented on irc
22:43:27 sean-k-mooney so basically you removed the _interceptor context manager which used placement direct and instead repace teh client with the versionCheckingReportClient after enabling the placemetnFixture
22:51:35 sean-k-mooney efried: im more or less trusting that you ran the fucntional test locally with master placement and it worked but ya droping placement direct form the fuctional test makes sense.
22:52:06 sean-k-mooney is there a similar followup to remove placement direct form placement itself or is there a usecase to keep it in placement
22:52:17 efried sean-k-mooney: I ran those two suites locally. I didn't run the whole world.
22:52:21 efried letting zuul do that.
22:52:36 efried We don't know whether there's a real use case, cdent and I were talking about that (again) earlier.
22:52:50 sean-k-mooney hehe ya thats resonable :) zull is rather slow currently
22:52:53 efried It was deemed a hard requirement for something-or-other (we can't remember exactly what) for extraction, so we did it.
22:53:06 efried but at this point it's not clear that it's needed at all.
22:53:17 efried I want to say maybe it was going to be a prerequisite for cinder using it in certain cases
22:53:31 efried but then in denver they decided they weren't going to use placement for those cases
22:53:41 efried so it's possible it will never be needed.
22:54:04 efried cdent did point out that it would probably be expected to work better from "real deployment" than from test
22:54:43 sean-k-mooney placement direct just allows you to bypass the placement api is thtat correct?
22:54:45 efried anyway, at this point I think the plan is to let it languish, see if anyone decides they want/need it, and if/when they do, find out if it's broken for whatever problem they're trying to solve.
22:55:09 efried sean-k-mooney: You're not bypassing it. It behaves as a real placement service. It just doesn't go "over the wire" and uses an in-memory database (I think).
22:55:21 efried sean-k-mooney: It really does effectively the same thing as PlacementFixture, just in a slightly different way.
22:55:43 sean-k-mooney oh i tougth you could use placemetn direct to talk to a real database too
22:55:44 efried I don't understand all the "wsgi intercept" magic.
22:55:51 efried oh, yeah, I think you can.
22:56:03 efried in this case we didn't want to do that, because we wanted to use a fixture.
22:56:51 sean-k-mooney so addmitly flawed understading was it was a way to run a placement api service in process to directly acess and maniupluate placement without needing to go over the wire
22:57:13 sean-k-mooney e.g. just skip the http transprot part
22:57:31 efried yes
22:57:51 efried sorry, I was focused in on the change I just made, where I believe we were faking the database as well.
22:58:35 sean-k-mooney oh ya in the fuctional test i woudl expect us to be using a fake deb or a per test sqlite db
22:58:43 efried sean-k-mooney: Yeah, https://review.openstack.org/#/c/640887/1/nova/tests/functional/test_report_client.py@a95
22:58:47 efried mocking the db
22:59:44 sean-k-mooney right makes sense.
23:45:20 mriedem efried: i'm +2 on cdent's gate fixes https://review.openstack.org/#/q/topic:bug/1818560+(status:open+OR+status:merged) - i see you have an alternative but i can't grok that right now
23:45:36 mriedem and we could do your alternative later yeah?
23:46:57 sean-k-mooney mriedem: looking at cdents change yes there woudl be no real conflcit with merging that quick and then efried change
23:48:56 sean-k-mooney efried: change looks resoable to me as it just drops teh use of placement direct an swaps to useing the palcement fixture instead but i also dont review the report clinet stuff often so im +1 but i also am not that familar those tests
23:49:02 sean-k-mooney mriedem: ^
23:51:24 sean-k-mooney mriedem: for what its work like at the live ci log it just passed all the functional test and should report back in about 3-4 mins
23:51:50 sean-k-mooney http://logs.openstack.org/87/640887/1/check/nova-tox-functional/1ba43e5/testr_results.html.gz
23:52:08 sean-k-mooney that ^ is the result of efried alternitve
23:52:19 mriedem the gate is broken and it's 6pm and i literally do not want to spend the time reviewing a new change
23:52:52 mriedem i only logged back on to +2 those changes
23:52:55 mriedem and now i'm dropping of
23:52:56 mriedem *off
23:52:57 mriedem o/
23:53:09 sean-k-mooney mriedem: sure night o/
#openstack-nova - 2019-03-05
00:54:59 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support delete_on_termination in volume attach api https://review.openstack.org/612949
00:58:03 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support for changing deleted_on_termination after boot https://review.openstack.org/580336
00:59:32 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support for changing deleted_on_termination after boot https://review.openstack.org/580336
01:01:10 openstackgerrit Merged openstack/nova stable/rocky: Avoid BadRequest error log on volume attachment https://review.openstack.org/640110
01:05:46 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support delete_on_termination in volume attach api https://review.openstack.org/612949
01:20:58 openstackgerrit Brin Zhang proposed openstack/nova master: Remove the string check of the flavor attribute 'swap' https://review.openstack.org/639012
01:28:04 openstackgerrit Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.openstack.org/621476
01:38:28 yonglihe everyone seems tired, take care, just update the patch. jaypipe, forgive my bad English. ping alex_xu, you must be energized.
02:05:00 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Add pmem_namespaces field to the NUMACell obj https://review.openstack.org/634547
02:05:01 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configures PMEM namespace when initiating libvirt driver https://review.openstack.org/634549
02:05:01 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Add virtual_pmems fields to the InstanceNUMACell obj https://review.openstack.org/634548
02:05:02 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report pmem namespaces resources by provider tree https://review.openstack.org/634551
02:05:02 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Update PMEM namespaces info and usage https://review.openstack.org/634550
02:05:03 openstackgerrit Luyao Zhong proposed openstack/nova master: scheduler: translate virtual pmems request to placement request group https://review.openstack.org/634553
02:05:03 openstackgerrit Luyao Zhong proposed openstack/nova master: API: parse pmem related flavor extra spec https://review.openstack.org/634552
02:05:04 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: boot VM with vpmems and cleanup vpmems when destroying https://review.openstack.org/634555
02:05:04 openstackgerrit Luyao Zhong proposed openstack/nova master: update _numa_fit_instance_cell for the support of virtual_pmems https://review.openstack.org/634554
02:05:05 openstackgerrit Luyao Zhong proposed openstack/nova master: support VM resizing with vpmem data migration https://review.openstack.org/634556
02:27:14 openstackgerrit Merged openstack/nova master: Use a placement conf when testing report client https://review.openstack.org/640853
03:23:34 openstackgerrit Fan Zhang proposed openstack/nova master: Retry after hitting libvirt error VIR_ERR_OPERATION_INVALID in live migration. https://review.openstack.org/612272
05:55:19 openstackgerrit Lajos Katona proposed openstack/python-novaclient master: Add support for microversion v2.72 https://review.openstack.org/637234
06:29:59 masber good afternoon, I am trying to setup OVS offload but nova fails getting representor port for PF. The question is: How is the representor created?
07:08:40 yumapath Hi team
07:08:57 yumapath while running the CI we are seeing one issue in the master devstack installation
07:08:58 yumapath 2019-03-04 23:54:21.262 | + functions:wait_for_compute:465 : openstack --os-cloud devstack-admin --os-region RegionOne compute service list
07:09:07 yumapath this command is timing out
07:09:25 yumapath : echo 'Didn'\''t find service registered by hostname after 60 seconds' 2019-03-04 23:54:21.257 | Didn't find service registered by hostname after 60 seconds
07:09:29 yumapath with this error
07:09:36 yumapath the stable releases are all passing
07:09:41 yumapath the issue is seen only with master
07:09:49 yumapath did anything change in the master to cause this issue
07:30:17 openstackgerrit Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.openstack.org/621476
07:53:40 openstackgerrit melanie witt proposed openstack/nova master: Add user_id field to InstanceMapping https://review.openstack.org/633350
07:53:40 openstackgerrit melanie witt proposed openstack/nova master: Add user_id column to the instance_mappings table https://review.openstack.org/633349
07:53:41 openstackgerrit melanie witt proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351
07:53:41 openstackgerrit melanie witt proposed openstack/nova master: Populate InstanceMapping.user_id during migrations and schedules https://review.openstack.org/638574

Earlier   Later