Earlier  
Later
New Search
Posted Nick Remark
#openstack-cyborg - 2020-04-02
03:22:19 Sundar chenke: Interesting, what's the bug?
03:22:48 chenke That's because the foriegn key constraint.
03:22:50 Sundar The original idea was to allow the device profile deletion when the VM is running, but we don't allow it today
03:23:03 Sundar chenke: Yes, the ARQs refer to the device profile entry
03:23:29 Sundar But they don't need to -- we copy the device profile groups into the ARQs, so we can remove that link
03:23:36 chenke So, is it reasonable to throw this error?
03:23:55 chenke No. Just copy the device profile groups is not enough.
03:24:24 chenke the error like this: `cyborg`.`extended_accelerator_requests`, CONSTRAINT `extended_accelerator_requests_ibfk_1` FOREIGN KEY (`device_profile_id`)
03:25:22 Sundar chenke: For now, the shortest improvement would be to block it at the API level rather than throw an error
03:26:03 Sundar For the future, we may need more changes to use the device profile groups copied into the ARQ
03:26:13 Sundar However, there is a fundamental problem.
03:26:47 Sundar On the Nova side, the operator may do a hard reboot or other ops for the VM anytime, and those ops will delete the old ARQs, and create new ones from the device profile name.
03:26:59 Sundar So, if the devic eprofile has been deleted, that is a problem.
03:29:30 Sundar Any thoughts?
03:29:46 chenke Ok.
03:30:04 chenke So we should keep the device profile.
03:30:17 Sundar So, let us block the deletion at the device profile API level rather than throw an error
03:30:27 chenke Agree.
03:31:00 Sundar Cool.
03:31:34 Sundar BTW, I will be proposing a patch shortly for CYborg's highlights for the U release. I'll share that link.
03:31:42 Yumeng yes ,agree now we just blocl the API level. but later maybe we can do like nova flavor. create a copy in nova for ops like reboot
03:32:54 Yumeng but that needs nova approval.. :-
03:33:13 Sundar Yumeng: The instance object has a copy of the flavor inside it, but does not have the ARQs. It has to call Cyborg with the device profile name to create the ARQ for some instance ops.
03:33:32 Sundar Yes, it needs more changes in Nova
03:33:57 Sundar We may need to make ARQs into an OVO, so it can be stored in the instance object.
03:34:10 Sundar Plus, db changes in Nova
03:34:26 Yumeng yes.
03:35:10 chenke actually that's hard.
03:36:04 Sundar Yup
03:36:19 Sundar That's why I didn;t do it :)
03:37:42 chenke Ok. I have no other needed to be discussed.
03:38:14 Sundar Anything else, folks?
03:38:35 brinzhang_ one patch
03:39:26 brinzhang_ Sundar, chenke, s_shogo: I think this is why py37 timeout previous https://review.opendev.org/#/c/716796/1/tox.ini
03:39:59 brinzhang_ the ignore_basepython_conflict flag introduced in 3.1.1, before we are in 2.0
03:40:32 brinzhang_ that I upgrade this to 3.1.1 to adopt python3
03:41:18 chenke good catch.
03:41:29 Sundar Yea, makes sense to me.
03:41:46 s_shogo good
03:42:18 Sundar #topic AoB
03:42:39 Sundar Anything else?
03:43:20 brinzhang_ Nothing else, many thing talked in wechat, and have done of mine ^^
03:43:56 Sundar Thanks a lot, folks. Have a good day, and stay safe away from the virus!
03:44:00 Sundar #endmeeting
03:44:02 openstack Meeting ended Thu Apr 2 03:44:00 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
03:44:03 openstack Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-02-03.00.html
03:44:04 openstack Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-02-03.00.txt
03:44:06 openstack Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-02-03.00.log.html
03:44:09 s_shogo Me too. I'll update the deployable programming patch, please review it. > folks
03:44:26 chenke Thanks all. Bye. will do it.
03:45:19 s_shogo bye
03:47:09 Yumeng bye
04:48:02 openstackgerrit Merged openstack/cyborg master: Upgrade minversion to match python3 https://review.opendev.org/716796
04:53:56 openstackgerrit Merged openstack/cyborg master: Add Cyborg API Reference to Cyborg index page https://review.opendev.org/711869
06:35:22 openstackgerrit Brin Zhang proposed openstack/cyborg master: Optimize device profiles APIs https://review.opendev.org/712049
06:58:39 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: Add hostname to FakeDevice name to support multi-host deployment https://review.opendev.org/693784
07:20:47 openstackgerrit Merged openstack/cyborg master: Add comments for deletion of ARQs by instance or ARQ UUIDs. https://review.opendev.org/714782
07:52:07 openstackgerrit Merged openstack/cyborg master: Optimize device profiles APIs https://review.opendev.org/712049
10:05:49 openstackgerrit Brin Zhang proposed openstack/cyborg master: Remove future imports https://review.opendev.org/716910
10:21:01 openstackgerrit Brin Zhang proposed openstack/cyborg master: Use unittest.mock instead of third party mock https://review.opendev.org/716916
10:28:10 openstackgerrit Brin Zhang proposed openstack/cyborg master: hacking: force explicit import of python's mock https://review.opendev.org/716920
17:31:06 openstackgerrit Merged openstack/cyborg-tempest-plugin master: Remove py27 from tox env list https://review.opendev.org/704076
#openstack-cyborg - 2020-04-03
10:06:06 openstackgerrit Brin Zhang proposed openstack/cyborg master: Use unittest.mock instead of third party mock https://review.opendev.org/716916
10:06:06 openstackgerrit Brin Zhang proposed openstack/cyborg master: hacking: force explicit import of python's mock https://review.opendev.org/716920
#openstack-cyborg - 2020-04-04
14:38:14 openstackgerrit Andreas Jaeger proposed openstack/python-cyborgclient master: Cleanup py27 support https://review.opendev.org/717459
#openstack-cyborg - 2020-04-05
01:06:59 openstackgerrit Brin Zhang proposed openstack/cyborg-tempest-plugin master: Update to hacking 3.0 https://review.opendev.org/717519
01:22:04 openstackgerrit Brin Zhang proposed openstack/cyborg-tempest-plugin master: Cleanup py27 support https://review.opendev.org/717520
01:34:26 openstackgerrit Brin Zhang proposed openstack/cyborg master: Cleanup py27 support https://review.opendev.org/717521
01:40:16 openstackgerrit Brin Zhang proposed openstack/cyborg master: Cleanup py27 support https://review.opendev.org/717521
01:49:19 openstackgerrit Merged openstack/python-cyborgclient master: Update home-page https://review.opendev.org/630104
05:23:24 openstackgerrit Andreas Jaeger proposed openstack/python-cyborgclient master: Cleanup py27 support https://review.opendev.org/717459
05:24:54 openstackgerrit Andreas Jaeger proposed openstack/python-cyborgclient master: Cleanup py27 support https://review.opendev.org/717459
06:05:04 openstackgerrit Brin Zhang proposed openstack/cyborg master: Introduce functional tests to the Cyborg https://review.opendev.org/702863
06:16:20 openstackgerrit Brin Zhang proposed openstack/cyborg master: WIP: add api_samples_test_base to functional https://review.opendev.org/709992
07:11:58 brinzhang gmann: while you are free, pls review https://review.opendev.org/#/c/702863/6 patch, I would like to introduce the functional test for Cybrog, but it does not enabled in zuul, I donot know what I missed
07:12:43 brinzhang gmann: thanks if you can review that
#openstack-cyborg - 2020-04-06
06:29:52 openstackgerrit Shogo Saito proposed openstack/cyborg master: Programming support (v2 Deployable API) https://review.opendev.org/698190
12:58:14 openstackgerrit Merged openstack/cyborg master: Update Nova API microversion in nova_client.py. https://review.opendev.org/716493
12:58:15 openstackgerrit Merged openstack/cyborg master: Update contributing.rst per Ussuri goal and guidelines. https://review.opendev.org/712246
14:43:18 openstackgerrit Xinran WANG proposed openstack/cyborg master: Introduce microversion https://review.opendev.org/696860
15:36:18 openstackgerrit Xinran WANG proposed openstack/cyborg master: Finish v2 API doc https://review.opendev.org/715980
23:26:46 openstackgerrit Merged openstack/cyborg master: Check during ARQ bind that the target instance has no other ARQs. https://review.opendev.org/712231
#openstack-cyborg - 2020-04-07
01:27:39 openstackgerrit Merged openstack/cyborg-specs master: Update tox py version conflict flag https://review.opendev.org/698824
03:28:56 openstackgerrit Brin Zhang proposed openstack/cyborg-tempest-plugin master: Cleanup py27 support https://review.opendev.org/717520
03:38:11 openstackgerrit Xinran WANG proposed openstack/cyborg master: Introduce microversion https://review.opendev.org/696860
03:47:17 openstackgerrit Xinran WANG proposed openstack/cyborg master: Introduce microversion https://review.opendev.org/696860
04:04:23 openstackgerrit Xinran WANG proposed openstack/cyborg master: Introduce microversion https://review.opendev.org/696860
05:01:48 openstackgerrit Xinran WANG proposed openstack/cyborg master: Introduce microversion https://review.opendev.org/696860
06:18:59 openstackgerrit Xinran WANG proposed openstack/cyborg master: Introduce microversion https://review.opendev.org/696860
08:55:27 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: support program for fake driver https://review.opendev.org/697011
09:30:12 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: support program for fake driver https://review.opendev.org/697011
09:30:13 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: support program for all drivers. https://review.opendev.org/697639
10:11:20 openstackgerrit Li Liu proposed openstack/cyborg master: Address the hostname problem mentioned https://review.opendev.org/696014
11:40:10 openstackgerrit Li Liu proposed openstack/cyborg master: Address the hostname problem mentioned https://review.opendev.org/696014
11:43:39 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: support program for fake driver https://review.opendev.org/697011
11:43:40 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: support program for all drivers. https://review.opendev.org/697639
12:14:17 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: bug fix for cyborg run on multi-host https://review.opendev.org/693784
12:24:42 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: bug fix for cyborg run on multi-host https://review.opendev.org/693784
13:29:11 gmann brinzhang: ACK, i will check sometime later
16:48:09 openstackgerrit Merged openstack/cyborg master: Introduce microversion https://review.opendev.org/696860

Earlier   Later