Thursday, 2020-01-09

« Wednesday, 2020-01-08 Index Friday, 2020-01-10 »
*** jawad_axd has joined #openstack-sdks00:31
*** jawad_axd has quit IRC00:35
*** slaweq has joined #openstack-sdks01:10
*** slaweq has quit IRC01:17
*** jangutter has quit IRC01:42
*** jangutter has joined #openstack-sdks01:42
*** tonyb has quit IRC02:10
*** goldyfruit has joined #openstack-sdks02:14
*** jangutter_ has joined #openstack-sdks02:28
*** jangutter has quit IRC02:30
*** goldyfruit has quit IRC02:41
*** goldyfruit has joined #openstack-sdks02:42
*** goldyfruit has quit IRC03:01
*** goldyfruit has joined #openstack-sdks03:01
*** ricolin has quit IRC03:07
*** slaweq has joined #openstack-sdks03:11
*** slaweq has quit IRC03:16
*** ricolin has joined #openstack-sdks03:28
*** goldyfruit has quit IRC03:46
*** goldyfruit has joined #openstack-sdks03:46
*** ricolin_ has joined #openstack-sdks03:57
*** ricolin_ has quit IRC03:57
*** ricolin has quit IRC04:02
*** ricolin has joined #openstack-sdks04:03
*** __ministry has quit IRC04:03
*** __ministry has joined #openstack-sdks04:05
*** slaweq has joined #openstack-sdks05:11
*** slaweq has quit IRC05:16
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-sdks05:34
*** goldyfruit has quit IRC05:34
*** slaweq has joined #openstack-sdks06:29
*** slaweq has quit IRC06:33
*** gtema has joined #openstack-sdks07:13
*** jawad_axd has joined #openstack-sdks07:19
*** gtema has quit IRC07:25
*** slaweq has joined #openstack-sdks07:52
*** sshnaidm|afk is now known as sshnaidm07:53
*** jpena|off is now known as jpena07:56
*** gtema has joined #openstack-sdks07:56
*** tkajinam has quit IRC08:18
*** tosky has joined #openstack-sdks08:25
*** slaweq_ has joined #openstack-sdks08:29
*** slaweq has quit IRC08:31
*** ralonsoh has joined #openstack-sdks09:03
*** jpich has joined #openstack-sdks09:11
openstackgerritBence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route  https://review.opendev.org/67432509:34
*** slaweq_ has quit IRC09:38
openstackgerritAkihiro Motoki proposed openstack/python-openstackclient master: Replace six.iteritems() with .items()  https://review.opendev.org/48140709:42
*** slaweq_ has joined #openstack-sdks09:50
openstackgerritAkihiro Motoki proposed openstack/openstacksdk master: Bump min version of decorator  https://review.opendev.org/70170610:54
*** dtantsur|afk is now known as dtantsur10:55
*** jangutter_ is now known as jangutter11:49
*** jpena is now known as jpena|lunch11:58
*** gtema has quit IRC12:05
*** enriquetaso has joined #openstack-sdks12:08
*** mgariepy has joined #openstack-sdks12:11
*** gtema has joined #openstack-sdks12:15
*** gtema has quit IRC12:18
openstackgerritDaniel Bengtsson proposed openstack/python-openstackclient master: Stop configuring install_command in tox and stop use pip.  https://review.opendev.org/69448412:23
*** jawad_axd has quit IRC12:50
*** jpich has quit IRC12:57
*** gtema has joined #openstack-sdks12:58
*** jpich has joined #openstack-sdks12:58
*** jawad_axd has joined #openstack-sdks12:59
*** gtema has quit IRC13:03
*** jpena|lunch is now known as jpena13:11
*** smcginnis has joined #openstack-sdks13:23
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Replace six.iteritems() with .items()  https://review.opendev.org/70172913:50
*** gtema has joined #openstack-sdks13:54
*** gtema has quit IRC13:54
*** gtema has joined #openstack-sdks13:57
mordredsmcginnis: wow. you found an old osc patch14:22
*** slaweq_ is now known as slaweq14:26
smcginnismordred: I like to scroll to the oldest once and awhile to see what's been getting really stale. ;)14:31
mordredsmcginnis: I tink it's neat that that change has a +1 from both Jenkins and Zuul14:32
smcginnisI actually abandoned at least one from 2016 that wasn't really relevant anymore but was still sitting out there. Probably a lot that could get cleaned up by now.14:32
smcginnisHaha, yeah!14:32
mordredsmcginnis: that made me curious - my oldest still-open change: https://review.opendev.org/#/c/119920/14:43
smcginnisNice! Any open patch that starts with a 1 is impressive. :)14:44
*** jawad_axd has quit IRC14:55
*** jawad_axd has joined #openstack-sdks14:56
*** jawad_axd has quit IRC14:58
*** jawad_ax_ has joined #openstack-sdks14:58
*** jawad_ax_ has quit IRC15:02
openstackgerritMerged openstack/openstacksdk master: Bump min version of decorator  https://review.opendev.org/70170615:17
openstackgerritMerged openstack/python-openstackclient master: Replace six.iteritems() with .items()  https://review.opendev.org/48140715:23
*** dave-mccowan has quit IRC15:25
sshnaidmhi all15:57
elmikohi15:57
sshnaidmanybody available for openstack ansible modules meeting in 2 minutes?15:58
dtantsuro/15:58
sshnaidmmnaser, mordred, gtema15:59
gtemayupp15:59
sshnaidmodyssey4me16:00
sshnaidmShrews,16:00
elmiko2020 API SIG office hours start \o/16:00
sshnaidmsorry if forgot someone :)16:00
mordredo/16:00
sshnaidmdtantsur, can you start the bot please?16:00
dtantsur#startmeeting api-sig16:00
openstackMeeting started Thu Jan  9 16:00:54 2020 UTC and is due to finish in 60 minutes.  The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: api-sig)"16:00
openstackThe meeting name has been set to 'api_sig'16:01
dtantsur#chair sshnaidm elmiko16:01
openstackCurrent chairs: dtantsur elmiko sshnaidm16:01
sshnaidmyay16:01
sshnaidmhappy holidays everyone16:01
sshnaidmwelcome back to 2020 meetings..16:01
dtantsurwhat, work again??16:01
dtantsurI'm sorry, I already worked last year..16:01
sshnaidmdtantsur, no, to talk :)16:01
elmikohehe, ++ dtantsur16:01
sshnaidm#link https://etherpad.openstack.org/p/openstack-ansible-modules16:02
sshnaidmour agenda ^^16:02
sshnaidm#topic Need to decide finally about the strategy16:02
*** openstack changes topic to "Need to decide finally about the strategy (Meeting topic: api-sig)"16:02
sshnaidmoops16:02
sshnaidmdidn't mean it16:02
dtantsurthere is always #undo16:02
sshnaidm#undo topic16:02
openstackRemoving item from minutes: #topic Need to decide finally about the strategy16:02
sshnaidmit's too complicated16:03
sshnaidmwe talked and sent mails about how we need to move modules16:03
sshnaidmI just want to be sure we're all on the same page16:03
sshnaidmI think we stopped on moving modules to ansible SIG repo, linking with new names in Ansible repo16:04
sshnaidmand redesigning if need with kind deprecation notices16:04
gtemaright16:04
sshnaidmis it correct ^16:04
sshnaidmany comments, objections ?16:05
sshnaidmok, let's move on16:05
sshnaidmabout freezing current modules in Ansible github repo16:06
sshnaidmI notices we still have patches merging:  https://github.com/ansible/ansible/commits/devel/lib/ansible/modules/cloud/openstack16:06
*** jpena is now known as jpena|off16:06
*** yoctozepto has joined #openstack-sdks16:06
*** jawad_axd has joined #openstack-sdks16:07
sshnaidmso, do we have a good way to make sure we really freeze them after a moving?16:07
sshnaidmand we move them next week when gundalow is back from PTO16:07
gtemadrop permissions of cloudnull16:07
sshnaidmmordred, stephenfin ^^16:07
sshnaidmgtema, :)16:07
gtemahe was approving lots of PRs16:08
sshnaidmI'll talk with cloudnull16:08
sshnaidmbut I think we need to send something to Ansible core team16:08
sshnaidmlike make it official16:08
gtemayeah, perhaps a good idea16:09
dtantsuryep16:09
sshnaidmand kindly to ask to ignore patches or make their bot to print a message there16:09
*** jawad_ax_ has joined #openstack-sdks16:09
sshnaidmlike "from now please use gerrit in .. blabla"16:09
*** jawad_axd has quit IRC16:09
sshnaidm#action sshnaidm to send message to Ansible core team about freezing OS modules16:10
mnaser(sorry, i am in and out but following along)16:10
sshnaidmmnaser, sure16:10
sshnaidmok, let's move on if no objections16:10
sshnaidmShould we move modules with keeping history?16:10
Shrewsi think you'll probably need some process (and an actual collection) in place before they'll actually freeze them (if they even do)16:10
gtemanope - ansible github is history16:11
mordredyeah- I think moving with history is too much work at this point16:11
sshnaidmShrews, we will do it after a move I think16:11
sshnaidmI think dtantsur had some experience with keeping history, was is complicated ?16:11
openstackgerritDmitry Tantsur proposed openstack/os-client-config master: Add a release note for Python 2 support removal  https://review.opendev.org/70176116:12
yoctozeptohi (just listening today)16:12
dtantsursplitting a subtree in git is not hard16:12
dtantsurbut submitting all the resulting patches to gerrit is going to kill our infra16:12
sshnaidmdtantsur, ack16:12
dtantsurso somebody will have to (force-)push them to gerrit16:12
dtantsurthat's what we did for ironic-tempest-plugin in the end16:12
dtantsur(actually, we also merged two git trees from two repos, that was fun)16:12
mordredif someone wants do to the subtree split, I can do the gerrit goo16:13
sshnaidmany volunteers? ^16:13
dtantsurc'mon, it's fun!16:14
gtemanot me. I do not see any benefit16:14
sshnaidmif no volunteers - not keeping history then16:14
* dtantsur assumes gtema has something to hide in `git blame` :D16:14
dtantsursshnaidm: I can do it if people find it useful16:14
dtantsurbut I don't insist if nobody does16:14
dtantsurtime for a poll? :)16:15
sshnaidmI think it might be useful, but not deal-breaker16:15
gtemadtantsur - actually only very few fixes were really landed, so - nope ;-)16:15
* dtantsur hasn't used the poll functionality in a while16:15
sshnaidmdtantsur, does it have a poll?16:15
dtantsur#poll Should we keep the git history of the moved modules? Yes, No16:15
dtantsurgiven that it doesn't react, I might have done something wrong16:15
dtantsur#startpoll Should we keep the git history of the moved modules? Yes, No16:16
sshnaidmYes, No16:16
sshnaidmall right, let's make it simple.. no keeping history16:16
gtema+116:16
sshnaidmwe have things to do beside that..16:17
sshnaidmlet's move on16:17
*** slaweq has quit IRC16:17
sshnaidmnext question - python 2/3 compatibility16:17
sshnaidmAnsible supports from 2.6(?), but Openstack is moving to 3 completely. Should we stick to Ansible and support from 2.6/2.7?16:17
* dtantsur finally remembered it was #startvote16:18
dtantsurif openstacksdk drops python 2, we cannot support it16:18
sshnaidmdtantsur, we can use it for this topic ^ )16:18
dtantsurhaha, let's try16:18
dtantsur#startvote Should we let Python 2 live? Yes, No, OMG16:18
openstackBegin voting on: Should we let Python 2 live? Valid vote options are Yes, No, OMG.16:18
openstackVote using '#vote OPTION'. Only your last vote counts.16:18
sshnaidmOMG16:18
sshnaidm#vote OMG16:19
dtantsur#vote No16:19
gtema#vote hehe16:19
openstackgtema: hehe is not a valid option. Valid options are Yes, No, OMG.16:19
sshnaidminteresting, what gundalow thinks about it, but we'll know that only next week16:19
dtantsurTo elaborate on my vote: we're planning something forward-looking, and Python 2 is dead today16:19
gtema# vote no16:19
gtema#vote no16:19
dtantsuranyone else wants to play with the vote functionality? :)16:20
gtema#vote again - no16:20
openstackgtema: again - no is not a valid option. Valid options are Yes, No, OMG.16:20
sshnaidmgtema, dtantsur made it case sensitive :o16:20
dtantsurit's not case sensitive, but it requires a valid option16:20
gtemadoesn't looke like that16:21
dtantsur#endvote16:21
openstackVoted on "Should we let Python 2 live?" Results are16:21
openstackOMG (1): sshnaidm16:21
gtemasimly "no" was accepted16:21
openstackNo (2): gtema, dtantsur16:21
dtantsuras you see ^^16:21
sshnaidmah, worked16:21
sshnaidmok, I hope Ansible people won't be mad we break their 2.7 support..16:21
mordredsorry - juggling two meetings - #vote no :)16:22
gtematoo late16:22
sshnaidmtoo late, we decided no :)16:22
mordredlooks like there were no yes votes anyway :)16:22
dtantsur#agreed Unless gundalow objects next week, we won't keep Python 2 support16:22
mordred\o/16:22
sshnaidmmordred, you just want to join the winning part!16:22
mordredI mean - sdk isn't keeping python 2 support anyway16:22
mordredso, you know, don't know that these modules have much choice :)16:22
dtantsur:D16:23
gtemayeah, but so far we haven't stripped it off16:23
mordredthis is true16:23
mordredwe *have* removed testing16:23
gtemayes, that's true. But I explicitely was not starting to drop "six"16:23
gtemahehe16:23
dtantsurnot tested == broken16:23
sshnaidmvery true ^16:23
gtemayou forgot "if" in a comparison16:24
sshnaidmok, I think we finished all topics from agenda, and now - open discussions16:24
dtantsurit's a boolean expression16:24
gtemathen it should be a single "="16:24
sshnaidmgtema, in boolean?16:25
dtantsurgtema: I'm thinking in python :)16:25
gtemaopen discussion - what is with jobs for proper testing and releasing?16:25
sshnaidmgtema, good question16:26
sshnaidmI configured a job that doesn't test anything for now16:26
sshnaidmhttps://review.opendev.org/#/c/698085/16:26
dtantsuron opendev writing jobs should be easier16:26
sshnaidmit's a job that always was running in ansible patches16:27
gtemaeverything is easy until you start doing that16:27
sshnaidmso I'll configure it to run on current repo and it will be a start16:27
sshnaidmwe can run some tripleo jobs as well16:28
gtemashould we move func tests from SDK here, or link to SDK?16:28
dtantsurI have a feeling that the new repo is a better place for tests16:28
gtemaand basically those SDK func tests must install collection first16:28
gtemame too16:28
gtema(not this "me too")16:29
sshnaidmgtema, and apply a patch I suppose..16:29
gtemawell, you can install collection from local16:29
sshnaidmgtema, yeah, I think you make a tarball and then install it, iirc16:29
gtemaso it's basically just "ansible-galaxy collection build && ansible-galaxy collection install"16:29
gtemaexactly16:29
sshnaidmyep16:30
dtantsurzuul applies patches for you16:30
sshnaidmwell, I think it will be more clear when we move it finally16:30
sshnaidmI'm not familiar with SDK jobs16:31
sshnaidmbut do we have there something16:31
sshnaidmthat can be helpful for modules as well?16:31
gtemathere are tests for modules16:31
sshnaidmI suppose we need installed Openstack to run modules that change it16:31
sshnaidmdevstack, tripleo, whatever16:32
gtemahttps://opendev.org/openstack/openstacksdk/src/branch/master/openstack/tests/ansible16:32
mordredyah - there is a module function test job that runs against devstack16:32
mordredI totally agree, those should move into the collection repo once we've got it16:32
mordredand we can still also trigger it from sdk for cross-testing16:32
gtemasure16:32
mordredessentially, it runs devstack and then runs the module tests against the api endpoint from the devstack16:33
mordredit's not COMPREHENSIVE16:33
mordredbut it's a solid starting place16:33
*** slaweq has joined #openstack-sdks16:34
sshnaidmas I see there are test actions, but not verifications..?  like in https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/tests/ansible/roles/keystone_domain/tasks/main.yml16:34
dtantsurironic coverage can be provided by bifrost16:34
gtemaright16:34
mordredsshnaidm: well, the verifications are a little implicit for now16:34
mordredthat update isn't going to work if the create didn't, etc ... but we can *definitely* improve these and make them better16:35
mordredand should16:35
mordredit's been harder to improve that with the modules in ansible/ansible and the tests in openstacksdk16:35
sshnaidmyeah, maybe using molecule will be helpful too16:36
mordredyah - maybe so - I hear you know something about that :)16:36
sshnaidmfor testing on various platforms for example16:36
gtemanot definitely - you want to mess with devstack and molecule?16:36
sshnaidmmordred, yeah, we use it heavily now16:36
sshnaidmdevstack is providing openstack cloud16:37
sshnaidmbut all tests can be wrapped in molecule16:37
sshnaidmfor last triple module I just used keystone container for testing16:38
sshnaidmand running os_keystone_* tasks with it16:38
sshnaidmnot even whole openstack16:38
sshnaidmthe same tried with ironic dev container16:38
mordredcool16:39
mordredlike - my brain doesn't necessarily know how that all hangs together, but I could imagine seeing some patches would bea . good learning experience all around16:40
sshnaidmOK, I think we agreed to design a good testing for modules after a move16:40
mordred++16:40
*** slaweq has quit IRC16:40
sshnaidmanything else in your minds?16:40
sshnaidmOK, I think we're good for today16:41
sshnaidmand welcome back everyone16:41
mordredhappy new year!16:41
sshnaidm+116:42
sshnaidm#endmeeting16:42
*** openstack changes topic to "Bug tracker for SDK and OSC is now at https://storyboard.openstack.org"16:42
openstackMeeting ended Thu Jan  9 16:42:09 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:42
openstackMinutes:        http://eavesdrop.openstack.org/meetings/api_sig/2020/api_sig.2020-01-09-16.00.html16:42
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/api_sig/2020/api_sig.2020-01-09-16.00.txt16:42
openstackLog:            http://eavesdrop.openstack.org/meetings/api_sig/2020/api_sig.2020-01-09-16.00.log.html16:42
*** sshnaidm is now known as sshnaidm|bbl16:48
*** gtema has quit IRC17:02
*** dtantsur is now known as dtantsur|afk17:03
*** ricolin has quit IRC17:03
*** gtema has joined #openstack-sdks17:04
*** tosky has quit IRC17:05
*** tosky has joined #openstack-sdks17:06
openstackgerritSean McGinnis proposed openstack/os-client-config master: Add a release note for Python 2 support removal  https://review.opendev.org/70176117:10
openstackgerritSean McGinnis proposed openstack/os-client-config master: Fix pep8 issue in o_c_c/__init__  https://review.opendev.org/70177317:10
smcginnismordred and/or amotoki: Could you push through that stack? ^17:19
smcginnisPretty trivial, then we can get a release request for milestone 1 updated and get that out.17:20
amotokismcginnis: I have no +2 right on os-client-config. we need to wait mordred17:21
smcginnisamotoki: OK, thanks for reviewing!17:21
mordredsmcginnis: +A17:21
smcginnisAwesome, thanks17:22
openstackgerritRodolfo Alonso Hernandez proposed openstack/openstacksdk master: "qos_network_policy_id" attribute added to port resource  https://review.opendev.org/69324717:27
*** gtema has quit IRC17:29
*** gtema has joined #openstack-sdks17:31
*** jpena|off is now known as jpena17:31
*** evrardjp has quit IRC17:33
*** evrardjp has joined #openstack-sdks17:34
*** sshnaidm|bbl is now known as sshnaidm17:38
openstackgerritRodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add qos_network_policy_id to network port  https://review.opendev.org/70178617:54
KeithMnemonicany chance to please get these moving along? https://review.opendev.org/#/c/701293/  https://review.opendev.org/#/c/701378/ dtroyer, amotoki17:56
openstackgerritMerged openstack/openstacksdk master: Replace six.iteritems() with .items()  https://review.opendev.org/70172917:58
*** jawad_ax_ has quit IRC18:04
*** gtema has quit IRC18:08
openstackgerritColleen Murphy proposed openstack/python-openstackclient master: Add support for app cred access rules  https://review.opendev.org/67785718:22
*** gtema has joined #openstack-sdks18:29
*** gtema has quit IRC18:29
*** jpena is now known as jpena|off18:29
openstackgerritRodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add qos_network_policy_id to network port  https://review.opendev.org/70178618:32
*** jpich has quit IRC18:46
*** tosky has quit IRC18:48
KeithMnemonicthanks for all of the effort it seems python-openstackclient on train is working better now. reviews please https://review.opendev.org/#/c/701299/18:51
openstackgerritRodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add qos_network_policy_id to network port  https://review.opendev.org/70178618:58
*** mgariepy has quit IRC18:59
*** ralonsoh has quit IRC19:44
*** openstackgerrit has quit IRC19:59
*** openstackgerrit has joined #openstack-sdks20:05
openstackgerritMerged openstack/os-client-config master: Fix pep8 issue in o_c_c/__init__  https://review.opendev.org/70177320:05
openstackgerritMerged openstack/os-client-config master: Add a release note for Python 2 support removal  https://review.opendev.org/70176120:05
*** slaweq has joined #openstack-sdks20:07
*** slaweq has quit IRC22:19
mordredyoctozepto: https://review.opendev.org/70185422:27
*** enriquetaso has quit IRC22:33
*** slaweq has joined #openstack-sdks23:20
*** slaweq has quit IRC23:25
*** goldyfruit has joined #openstack-sdks23:43
« Wednesday, 2020-01-08 Index Friday, 2020-01-10 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!