Thursday, 2020-10-29

*** tosky has quit IRC00:06
openstackgerritMerged openstack/openstack-ansible-repo_server master: Updated the git source  https://review.opendev.org/65735000:07
openstackgerritMerged openstack/openstack-ansible-os_nova master: Support spicehtml5proxy on baremetal with haproxy  https://review.opendev.org/75906800:14
*** cshen has quit IRC00:30
*** macz_ has joined #openstack-ansible00:38
*** MickyMan77 has joined #openstack-ansible00:40
*** macz_ has quit IRC00:43
*** MickyMan77 has quit IRC00:49
*** macz_ has joined #openstack-ansible01:36
*** macz_ has quit IRC01:41
*** MickyMan77 has joined #openstack-ansible01:45
*** MickyMan77 has quit IRC01:54
*** johnsom has quit IRC02:00
*** rpittau|afk has quit IRC02:00
*** rpittau|afk has joined #openstack-ansible02:00
*** johnsom has joined #openstack-ansible02:01
*** gyee has quit IRC02:02
*** MickyMan77 has joined #openstack-ansible02:23
*** MickyMan77 has quit IRC02:31
*** MickyMan77 has joined #openstack-ansible03:08
*** MickyMan77 has quit IRC03:16
*** MickyMan77 has joined #openstack-ansible04:03
*** MickyMan77 has quit IRC04:11
*** dave-mccowan has quit IRC04:12
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-ansible05:33
openstackgerritPerToft proposed openstack/openstack-ansible master: Added Netplan examples for a Openstack Infrastructure node.  https://review.opendev.org/76009006:27
*** miloa has joined #openstack-ansible07:01
*** sshnaidm|afk is now known as sshnaidm|rover07:01
noonedeadpunkpto: can I adjust your patch not to torture you?:)07:29
*** andrewbonney has joined #openstack-ansible08:09
*** pto has quit IRC08:14
*** pto has joined #openstack-ansible08:14
ptonoonedeadpunk: I would appreciate that. Sorry for me spamming, but I have not worked with CI/CD for a long time08:17
ptonoonedeadpunk: Please let me know what I did wrong :-)08:18
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Added Netplan examples for a Openstack Infrastructure node.  https://review.opendev.org/76009008:18
noonedeadpunkhttps://review.opendev.org/#/c/760090/7..8/etc/netplan/README.md08:19
*** cshen has joined #openstack-ansible08:19
ptoI would suggest to update the OpenStack Ansible contribution guide with the actual commands to run, like git --amend, git reveiw -d etc.08:21
*** yolanda has quit IRC08:24
*** yolanda has joined #openstack-ansible08:25
*** macz_ has joined #openstack-ansible08:31
*** Adri2000 has quit IRC08:36
*** macz_ has quit IRC08:36
*** rpittau|afk is now known as rpittau08:40
*** pto has quit IRC08:46
*** pto_ has joined #openstack-ansible08:46
*** tosky has joined #openstack-ansible08:51
openstackgerritAndrew Bonney proposed openstack/openstack-ansible-os_zun master: Fix linter errors  https://review.opendev.org/76030409:08
*** gshippey has joined #openstack-ansible09:18
*** pto has joined #openstack-ansible09:20
*** pto_ has quit IRC09:20
*** Adri2000 has joined #openstack-ansible09:24
andrewbonneyLooks like Galaxy is broken again :( https://github.com/ansible/galaxy/issues/253209:30
noonedeadpunkuh....09:32
noonedeadpunkpto: I think it's covered in general openstack contribution guide https://docs.openstack.org/contributors/code-and-documentation/index.html09:34
noonedeadpunkspecificly https://docs.openstack.org/contributors/code-and-documentation/using-gerrit.html09:35
ptonoonedeadpunk: I was looking for that part. Thanks"09:43
*** sum12 has quit IRC09:50
openstackgerritAndrew Bonney proposed openstack/openstack-ansible-os_zun master: Updated from OpenStack Ansible Tests  https://review.opendev.org/75418510:09
*** macz_ has joined #openstack-ansible10:19
*** macz_ has quit IRC10:24
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Bump SHAs for master  https://review.opendev.org/75597311:32
*** dave-mccowan has joined #openstack-ansible11:44
*** dave-mccowan has quit IRC11:48
*** rh-jelabarre has joined #openstack-ansible11:52
*** spatel has joined #openstack-ansible11:56
*** spatel has quit IRC12:01
*** rfolco has joined #openstack-ansible12:08
*** nurdie has joined #openstack-ansible12:33
*** nurdie has quit IRC12:34
*** ianychoi__ is now known as ianychoi12:34
*** nurdie has joined #openstack-ansible12:34
*** nurdie has quit IRC12:40
jamesdentonmornin13:13
spotzpto yeah we were just discussing that yesterday as a review of the OpenStack Upstream Training last night. Some projects do have things within their own docs but there is the main contributor guide that we keep updated from First COntact SiG and the OUI mentors13:16
spotzmorning jamesdenton13:16
noonedeadpunko/13:16
jamesdentonhello hello13:17
jamesdentonnoonedeadpunk i'll look into that mtu issue for you13:18
noonedeadpunkyou mean ovn one? I think it just needs tempest patch to be merged13:18
noonedeadpunkas depend-son doesn't really work with fucntional tests13:18
jamesdentonmaybe, but the error was different this time. "Attribute 'mtu' not allowed in POST""13:19
noonedeadpunkuh....13:19
mgariepyhttps://github.com/openstack/openstack-ansible/commit/f4946d3aceab1d1a56422d7a907e46c59e897a63#diff-f879ec2769daf7a1bca3987370534ac0330cbfd14dd13142cd573790e305213fR27513:21
noonedeadpunkyeah, didn't saw that13:21
mgariepynoonedeadpunk, any reason why ?13:21
noonedeadpunkmgariepy: yep, they've backported breaking change to13:31
noonedeadpunkto stable-5.0 branch13:31
openstackgerritamolkahat proposed openstack/openstack-ansible-os_tempest master: Migrate to content provider jobs/templates  https://review.opendev.org/75928713:31
noonedeadpunkso in order for bump tool not to update SHA trackbranch was set to None13:32
noonedeadpunkeventually they've dropped variables we were using by default13:32
mgariepyok13:33
noonedeadpunkWe've fixed it with https://opendev.org/openstack/openstack-ansible/commit/ec3233f9f8948bc3354121e5336ff5cbda0c41cb13:36
noonedeadpunkas they've dropped osd_pool_default_pg_num and some of the variables with https://github.com/ceph/ceph-ansible/commit/80879df44df46c9d73d515fedbba2bb5c3412f3313:39
*** jbadiapa has joined #openstack-ansible13:45
noonedeadpunkjrosser: I was recalling mitogen discussion that is being held for a while, and we had some descision on previous ptg. I really have vague memories of that but it was not profitable in terms of execution speed ?13:50
Adri2000noonedeadpunk: hello, would you be ok if I backported https://review.opendev.org/#/c/758207/ ? thinking of backporting it up to train13:53
jrossernoonedeadpunk: i'm not totally convinced we would see an improvement in CI using mitogen tbh, but for a real deployment it could be huge13:54
jrosserhowever, i've never had an LXC AIO run properly with it due to bugs related to delegation13:55
noonedeadpunkAdri2000: yes, totally13:55
*** macz_ has joined #openstack-ansible13:55
jrosseralso i expect collections will break it too13:56
Adri2000noonedeadpunk: ok, will do that13:58
noonedeadpunkjrosser: yeah, it seems they do not fully support 2.813:58
noonedeadpunk`Ansible 2.8 interpreter discovery and become plugins are not yet supported.`13:59
jrosserthe main contributor has their own repo and i see some stuff for collections in there13:59
jrosserhttps://github.com/s1113950/mitogen/commits/collectionsSupport14:00
*** macz_ has quit IRC14:00
noonedeadpunkwondering what has went wrong with upstream....14:01
noonedeadpunkbut whatever14:02
jrosseryeah, i think just let the patch sit there, when i get some time i can rebase it and maybe make some bug reports14:02
noonedeadpunkyeah, totally, was just trying to recall where we've stopped there:)14:08
openstackgerritAdrien Cunin proposed openstack/openstack-ansible stable/ussuri: Actually use ANSIBLE_FORKS in openstack-ansible.rc  https://review.opendev.org/76036614:29
openstackgerritAdrien Cunin proposed openstack/openstack-ansible stable/train: Actually use ANSIBLE_FORKS in openstack-ansible.rc  https://review.opendev.org/76036714:32
*** evrardjp has quit IRC14:38
*** evrardjp has joined #openstack-ansible14:40
*** macz_ has joined #openstack-ansible14:52
mgariepyjamesdenton, https://paste.ubuntu.com/p/gQZHhKbJ3j/14:52
*** spatel has joined #openstack-ansible14:52
jamesdentonnice!14:53
mgariepyi think that would be a bit cleaner and if you want do do DVR with ovn just add the gateway group to the compute !14:53
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Fix upgrade jobs for bind-to-mgmt  https://review.opendev.org/75846114:54
mgariepyhttps://paste.ubuntu.com/p/xyGs7fP9wX/14:54
*** cshen has quit IRC14:58
openstackgerritMarc GariĆ©py (mgariepy) proposed openstack/openstack-ansible master: [DNM] split the gateway service.  https://review.opendev.org/76038415:02
*** smcginnis has joined #openstack-ansible15:38
*** mmercer has quit IRC15:38
*** rfolco is now known as rfolco|bbl15:41
*** NewJorg has quit IRC15:46
*** NewJorg has joined #openstack-ansible15:48
jrosserspotz: http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-operators15:58
jrosser"no such list" :(15:58
jrosserfrom the link on here https://wiki.openstack.org/wiki/Operations15:59
spotzat heppened16:00
spotzOk lost the whole front half of that:(16:00
spotzIt was combined into the discuss list when the list down sizing happened16:01
jrosserah ok, makes sense16:06
jrosserfrom what was just said earlier i was convinced that there was some ML i was not subscribed to16:07
spotzNo it didn't get a ton of traffic and there's less ops discussions on discuss. I forward ops things that come into mentoring or community lists16:19
*** spatel has quit IRC16:23
jamesdentonnoonedeadpunk think i found the mtu issue. overrides in the neutron test ought to be enough to make it go w/o changes on the tempest side16:25
*** spatel has joined #openstack-ansible16:27
spateljrosser: quick question, i am planning to build 6 node controller where 3 node for api services and other 3 nodes for infra services like (rabbit,mysql,memcache etc). does this looks good to you?16:29
jrosseri have one setup just like that16:30
spatelnice, do you use same hostname for them like os-infra-XX or do you use os-api-XX and os-infra-XX (i am confused what should i do) :)16:31
spatelits not big deal but trying to see what other people doing in their setup16:31
jrosseri called them shared<N> for db/mq/memcache and infra<N> for API16:33
spatelhmm, that is also make sense16:33
*** ThiagoCMC has joined #openstack-ansible16:48
spotzjrosser noonedeadpunk jamesdenton - if someone is installing OSA master are the projects pinned or installing master versions? A question in the openinfralabs session16:48
ThiagoCMCHey guys! Is "stable/victoria" good to go?  :-P16:48
jrosserspotz: is it worth joining?16:48
spotzThiagoCMC: I do not believe OSA has released yet16:48
spotzjrosser: sure16:48
jrosseris the the same link for the previous session?16:49
ThiagoCMCspotz, oh, ok... Thanks! Not even for testing?16:49
spotzThiagoCMC: it's master branch still I think16:49
spotzjrosser: yes16:49
ThiagoCMCOk, cool16:49
jamesdentonspotz i believe it's master branch, master version.16:50
spotzjamesdenton: That's what I thought but wanted to be sure16:51
*** mwhahaha has quit IRC16:55
*** mwhahaha has joined #openstack-ansible16:55
*** rpittau is now known as rpittau|afk16:57
*** rfolco|bbl is now known as rfolco17:01
*** elduderino80 has joined #openstack-ansible17:07
*** elduderino80 has quit IRC17:25
*** cshen has joined #openstack-ansible17:44
*** cshen has quit IRC17:48
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Move keystone sp_idp setup to it's own tasks file  https://review.opendev.org/76042917:49
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Setup keystone sp/idp after db contract  https://review.opendev.org/76043117:55
jrossernoonedeadpunk: ^ i make an attempt there to fix the trouble with federation setup that pto was having, not sure if it is actually valid though17:56
mgariepyjrosser, what is the issue with sso ? i've done some tests in the past i can maybe check if i have something about it.18:00
mgariepytested is in the summer of 2019.18:00
jrosserthe tasks I moved there failed for pto if you try to setup the federation as part of the first deploy18:02
jrosserif you deploy then add federation later it was ok because the db contract was already done18:02
jrosseractually there is probably a bug on LP for this18:03
mgariepyok18:05
mgariepydid saw that back then but i might have deployed it first without the sso config.18:07
mgariepybut i fixed a couple other issue for it.18:07
*** cshen has joined #openstack-ansible18:10
*** andrewbonney has quit IRC18:13
*** rh-jelabarre has quit IRC18:28
*** tosky has quit IRC18:29
*** rh-jelabarre has joined #openstack-ansible18:34
*** cshen has quit IRC18:37
*** cshen has joined #openstack-ansible18:46
*** recyclehero has quit IRC18:50
*** recyclehero has joined #openstack-ansible18:57
noonedeadpunkjrosser: you with gshippey are experts here, i have no idea if it's valid or not.... I will rely on your descision here19:05
jrosserwell maybe more the ansible - role: and tasks_from:19:06
jrosserif it passes CI we should see if pto can test it i think19:06
noonedeadpunkI think it should be import_role?19:07
jrosseryeah, i was really unsure about that19:07
jrosseri don't know if roles: in a play is implicitly that, or not19:08
noonedeadpunkI don't think it's valid attribute for role but will see19:08
noonedeadpunkactually here it doesn't matter much if we do tasks: import_role or roles: role19:09
jrosserno, it just needs to respect the tasks_from19:09
jrosseror maybe theres another way to do this entirely.....19:10
jrosserthose tasks are just a whole set of os_<> modules delegated to the service setup host19:10
noonedeadpunkeventually then we can add it as the last task of previous block19:11
jrossermove them out of the role? i guess we lose access to os_keystone defaults/main.yml19:11
jrosseroh sorry i see19:13
noonedeadpunkno, I meant if we do import_tasks, then we can move it right after `Disable the need for any further db sync` without need for separate iteration. but whatever19:13
jrosseruse import_role in the existing play19:13
noonedeadpunkyeah19:13
noonedeadpunkI think we have contract not in role for upgrades? when we upgrade and restart all instances but do not launch it untill everything is up?19:14
jrosserthats what the git history suggests, yes19:15
noonedeadpunkum, what's that then.... https://opendev.org/openstack/openstack-ansible-os_keystone/src/branch/master/tasks/keystone_db_sync.yml#L9119:15
jrosser /o\19:16
noonedeadpunkmaybe we just need to flush handlers somewhwere? or move that task right after handlers flush?19:16
noonedeadpunkas currently it's right before them19:17
noonedeadpunkas I was thinking that we do like every other service - upgrade single instance, migrate db and do the rest of them19:18
noonedeadpunkbut have no idea tbh.... was using keystone pretty basically - max ldap integration at the moment...19:19
jrosseri am nervous to change this too much tbh19:19
noonedeadpunkwell yeah... this worth some extra testing, but I have no proper user scenario I guess... but what I see tells me we have some mess here19:21
jrosserthis looks important https://github.com/openstack/openstack-ansible-os_keystone/blob/master/tasks/keystone_db_sync.yml#L9519:21
jrosserit could be that whole task is redundant and it's actually the one in the playbook which does the db contract19:21
noonedeadpunktha funny fact is https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/os-keystone-install.yml#L14019:22
*** jbadiapa has quit IRC19:22
noonedeadpunkwell and the only place where it is set - right above19:22
noonedeadpunknot sure why we need fact setting... for readability maybe...19:23
noonedeadpunk`Run keystone-manage db_sync --contract to remove the old schema and all data migration triggers. When this process completes, the database will no longer be able to support the previous release.`19:24
noonedeadpunkwell, it feels that we can move this all to the role and I think it should have the same effect there19:25
noonedeadpunkI mean defenition of keystone_all_software_versions, keystone_all_software_deployed, keystone_all_software_updated but yeah, a bit cautios about that19:27
*** vesper11 has quit IRC19:35
*** cshen has quit IRC19:36
*** cshen has joined #openstack-ansible19:42
mgariepyjamesdenton, https://paste.ubuntu.com/p/xtgThXSqYh/19:54
mgariepynooo. lol19:55
jamesdentonhmm, that seems familiar19:55
mgariepyi needed to add the networking-ovn pip pkg on the neutron server19:56
jamesdentonprob too old19:56
jamesdentonhttps://review.opendev.org/#/c/697039/19:56
*** rh-jelabarre has quit IRC19:57
*** gyee has joined #openstack-ansible19:57
mgariepy11 months ago.19:58
jamesdentonbut actually networking-ovn has been deprecated, all ovn code is in neutron now19:58
mgariepyouf.19:58
mgariepyok19:58
mgariepyhttps://paste.ubuntu.com/p/qGcBFPfyJh/19:59
mgariepythat's on U.19:59
jamesdentoncould be the neutron_plugin_base path is wrong in the docs20:03
mgariepynon-sense dov not correct ? haha20:04
jamesdentonlol20:04
jamesdentonone sec20:04
mgariepyi'ts probably that haha20:05
jamesdentontry neutron.services.ovn_l3.plugin.OVNL3RouterPlugin20:06
*** gshippey has quit IRC20:07
jamesdentoni fixed it in the overrides but not the docs20:07
jamesdentonhttps://github.com/openstack/openstack-ansible-os_neutron/commit/5b059649f49d7d10dd69d7479938db3d3f1b631e20:07
jamesdentonand it is wrong here, too: https://docs.openstack.org/networking-ovn/latest/install/manual.html20:07
mgariepywhat have i tought following the doc ahha20:08
mgariepyi'll never do that again lol20:08
mgariepylet's see.20:09
jamesdentonso there's an alias now, i guess: https://docs.openstack.org/neutron/latest/install/ovn/manual_install.html20:09
jamesdentonservice_plugins = ovn-router20:09
jamesdentonyou mind adding that to your patch?20:16
mgariepyno i'll20:16
mgariepyjamesdenton, https://pastebin.pl/view/f2e5a56820:21
mgariepyle'ts go in prod it works i can list the agent ! \20:21
mgariepysome agents20:22
mgariepylol20:22
jamesdentonwoohoo!20:26
mgariepyyep.20:26
mgariepyenough for today, i'll patch a few things tomorrow.20:27
jamesdentonsounds good20:27
*** jamesdenton has quit IRC20:47
*** jamesdenton has joined #openstack-ansible20:54
*** jamesdenton has quit IRC20:54
*** spatel has quit IRC20:59
*** vesper11 has joined #openstack-ansible21:14
*** nsmeds has joined #openstack-ansible21:16
*** cshen has quit IRC21:20
*** vesper has joined #openstack-ansible21:35
*** vesper11 has quit IRC21:36
*** rfolco has quit IRC21:36
*** cshen has joined #openstack-ansible21:51
*** tosky has joined #openstack-ansible21:57
*** ChosSimbaOne has quit IRC23:03
*** ChosSimbaOne has joined #openstack-ansible23:04
*** miloa has quit IRC23:09
*** spatel has joined #openstack-ansible23:47

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!