Earlier  
Later
New Search
Posted Nick Remark
#openstack-cyborg - 2019-09-23
22:54:32 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: conductor writes device_profile update to db https://review.opendev.org/679406
#openstack-cyborg - 2019-09-24
01:01:42 openstackgerrit YumengBao proposed openstack/cyborg master: conductor writes device_profile update to db https://review.opendev.org/679406
01:56:02 openstackgerrit chenker proposed openstack/cyborg master: Fix the hardcoding of user role using sdk_adapter approach https://review.opendev.org/682565
03:57:40 Sundar Hi chenke
03:57:47 chenke Hi ~
03:57:59 Sundar Not sure what is going wrong
03:58:18 chenke we chat in this.
03:59:01 chenke Maybe we can let xinran test this in her env.
03:59:08 chenke If she has free time.
03:59:16 chenke In my env, This patch is ok.
03:59:46 chenke I have to launch now. Reply for you after 20min.
03:59:53 Sundar Xinran is out of office today
04:00:01 Sundar Here's my cyborg.conf : http://paste.openstack.org/show/779148/
04:00:35 Sundar Oh sure. Enjoy your lunch
04:21:41 chenke Hi I am back.
04:22:37 chenke The same as my cyborg.conf
04:24:04 chenke sees the conf is ok.
04:27:25 chenke Is your environment reporting to placement is ok?
04:28:16 chenke Use the code before this patch.
04:29:45 chenke IMO, First of all, to verify that the placement is ok, and then add my changes, the placement report is also ok, and then test the nova part.
04:55:06 Sundar Hi chenke
04:55:21 Sundar Yes, the placement client is fine. Nova cl]lient alone fails.
04:55:44 Sundar However, the [nova] section is a copy-paste of [placement] section
04:58:29 Sundar I am sure your code is good as cyborg-tempest passed
05:23:00 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Fix v1 API. https://review.opendev.org/684193
06:49:29 openstackgerrit YumengBao proposed openstack/cyborg master: conductor writes device_profile update to db https://review.opendev.org/679406
06:57:17 openstackgerrit YumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg. https://review.opendev.org/673957
07:48:52 openstackgerrit YumengBao proposed openstack/cyborg master: remove rootwrap in cyborg https://review.opendev.org/684213
08:56:16 openstackgerrit wangzhh proposed openstack/cyborg master: Enable api v2 policy check https://review.opendev.org/678177
#openstack-cyborg - 2019-09-25
02:20:17 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Fix v1 API. https://review.opendev.org/684193
02:22:24 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Fix v1 API. https://review.opendev.org/684193
02:35:53 openstackgerrit wangzhh proposed openstack/cyborg master: Enable api v2 policy check https://review.opendev.org/678177
04:10:48 openstackgerrit wangzhh proposed openstack/cyborg master: Enable api v2 policy check https://review.opendev.org/678177
04:12:52 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Install OPAE tools library. https://review.opendev.org/684456
04:57:59 openstackgerrit Merged openstack/cyborg master: [train][goal] Run 'cyborg-tempest-ipv6-only' job in gate https://review.opendev.org/682517
05:58:01 openstackgerrit chenker proposed openstack/cyborg master: Fix the hardcoding of user role using sdk_adapter approach https://review.opendev.org/682565
06:00:14 openstackgerrit chenker proposed openstack/cyborg master: Fix the hardcoding of user role using sdk_adapter approach https://review.opendev.org/682565
06:19:01 openstackgerrit chenker proposed openstack/cyborg master: Fix the hardcoding of user role using sdk_adapter approach https://review.opendev.org/682565
06:50:28 openstackgerrit Merged openstack/cyborg master: remove rootwrap in cyborg https://review.opendev.org/684213
06:55:21 openstackgerrit Xinran WANG proposed openstack/cyborg master: Program FPGAs with oslo.privsep. https://review.opendev.org/683733
07:11:03 chenke Sundar Hi~
07:28:36 Sundar Hi chenke
07:29:30 chenke We can chat on this.
07:29:56 Sundar Sure. I think we all agreed.
07:29:56 chenke I am replying faster here than the phone.
07:30:03 chenke Ok.
07:30:30 chenke But you need to solve your merge confilct patches.
07:30:33 Sundar I will resolve merge conflicts in the Nova patches, update Yumeng's 670999 and put a dependency from 670999 to your patch
07:30:40 Sundar So you can remove the dependency
07:30:48 chenke Ok.
07:31:41 openstackgerrit chenker proposed openstack/cyborg master: Fix the hardcoding of user role using sdk_adapter approach https://review.opendev.org/682565
07:36:36 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: async job for bind https://review.opendev.org/681005
07:45:24 Sundar Done with Nova rebasing
09:26:22 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: async job for bind https://review.opendev.org/681005
09:31:36 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: async job for bind https://review.opendev.org/681005
11:27:17 openstackgerrit zhurong proposed openstack/cyborg master: Enable fake_driver by default https://review.opendev.org/684712
11:29:45 openstackgerrit Merged openstack/cyborg master: Enable api v2 policy check https://review.opendev.org/678177
18:20:14 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Program FPGAs with oslo.privsep. https://review.opendev.org/683733
#openstack-cyborg - 2019-09-26
02:15:08 openstackgerrit YumengBao proposed openstack/cyborg master: conductor writes device_profile update to db https://review.opendev.org/679406
02:22:26 openstackgerrit YumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg. https://review.opendev.org/673957
02:50:42 openstackgerrit chenker proposed openstack/cyborg master: Fix the hardcoding of user role using sdk_adapter approach https://review.opendev.org/682565
03:01:34 Sundar #startmeeting openstack-cyborg
03:01:35 openstack Meeting started Thu Sep 26 03:01:33 2019 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot.
03:01:36 openstack Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
03:01:38 openstack The meeting name has been set to 'openstack_cyborg'
03:01:41 Sundar Hi all
03:01:50 Sundar #topic Who's here
03:01:51 shaohe_feng good evening
03:01:57 Sundar o/
03:02:01 s_shogo Hi all
03:02:04 shaohe_feng o/
03:02:18 chenke o/
03:02:22 chenke Hi all.
03:02:50 Yumeng #info Yumeng
03:02:53 Yumeng hi all
03:03:10 xinranwang Hi all
03:03:16 xinranwang #info xinranwang
03:03:25 s_shogo #info s_shogo
03:03:39 Sundar Let's start with some good news. In Chenke's patch https://review.opendev.org/#/c/682565/, Eric asked to try 'strict_proxies=False'
03:03:47 Sundar I tried and it works for me at least :)
03:03:53 chenke Wow
03:03:59 chenke Sound great.
03:04:20 Sundar If chenke can resubmit with that change, I'll trigger the tempest check
03:04:39 chenke I had done it.
03:04:57 chenke And triggered the tempest check.
03:05:36 Sundar https://review.opendev.org/#/c/682565/15/cyborg/common/utils.py@149 still has i, right?
03:05:39 Sundar *it
03:06:00 Sundar I set that to False
03:06:44 chenke The eric's advice is to see it to False, pls see Line 130 the default value.
03:07:02 xinranwang I discussed with colleague who has experience on CI, there is no other solution except add "depend on" in each patch.
03:07:30 xinranwang it is better to merge nova patches as soon as possible.
03:08:12 Sundar chenke: I see.
03:08:31 Sundar Ok. Hope that does the trick.
03:08:39 chenke Hope so.
03:08:45 Yumeng agree with Xinran
03:08:57 Yumeng we have this to test related patches :https://review.opendev.org/#/c/670999/
03:09:17 Yumeng to test nova-cyborg-integration related patches
03:09:44 chenke Agree with Xinran. Now Yumengs' patch is a better solution so far.
03:09:44 Sundar It is not up to us to get Nova patches merged. There are challenge sin getting reviewer attention, not only for Cyborg, but for other subjects too.
03:10:59 Sundar Are we saying we should put Depends-On for a Nova patch in every Cyborg patch?

Earlier   Later