Monday, 2021-06-07

*** spatel has joined #openstack-ansible01:26
*** spatel has quit IRC02:01
*** mgoddard has quit IRC03:33
*** mgoddard has joined #openstack-ansible03:33
*** miloa has joined #openstack-ansible05:39
*** halali_ has joined #openstack-ansible05:43
*** luksky has joined #openstack-ansible06:45
*** Adri2000 has joined #openstack-ansible06:53
*** andrewbonney has joined #openstack-ansible07:21
*** rpittau|afk is now known as rpittau07:26
*** hindret has joined #openstack-ansible08:08
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Re-add NoLogPasswords ansible-lint rule  https://review.opendev.org/c/openstack/openstack-ansible/+/79506208:30
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Re-add NoLogPasswords ansible-lint rule  https://review.opendev.org/c/openstack/openstack-ansible/+/79506208:36
*** sshnaidm|afk is now known as sshnaidm08:51
*** sshnaidm has quit IRC08:58
*** sshnaidm has joined #openstack-ansible08:59
jonherhttps://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/794199 should have addressed the comments09:17
noonedeadpunklgtm09:29
jonhergreat09:35
jonherio saw your additional comments, should we also remove extras/os-cloudkitty-install.yml since that is also in integrated repo? then there is no more extras folder. either way we updated the install.yml in integrated repo so it's not the same currently10:30
noonedeadpunkyeah, I think we don't need extras at all nowadays10:34
opendevreviewJonathan Herlin proposed openstack/openstack-ansible-os_cloudkitty master: Cloudkitty role cleanup and config updates  https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/79419910:34
noonedeadpunkbut we can drop it with follow=up as well10:34
noonedeadpunkjonher: also ther're trailing whitespaces in reno10:35
jonheroh, i'll fix that10:35
opendevreviewJonathan Herlin proposed openstack/openstack-ansible-os_cloudkitty master: Cloudkitty role cleanup and config updates  https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/79419910:36
jonherextras removed and whitespaces fixed10:36
*** ioni has quit IRC11:38
*** luksky has quit IRC11:42
*** luksky has joined #openstack-ansible11:42
jrosserlooking at the badly failing centos-8 jobs it looks like somehow the centos-8-stream packages are trying to install on a not-stream centos-811:52
*** odyssey4me has quit IRC11:52
jrosseroh well also CentOS 8.4.2105 seems to be a thing now11:53
jrosseransible_distribution_version: '8.4'11:56
*** ioni has joined #openstack-ansible11:58
*** ioni has quit IRC11:59
*** tosky has joined #openstack-ansible11:59
noonedeadpunkoh damn12:03
noonedeadpunkthat's what it about :(12:03
noonedeadpunkI actually thought it's some repo issue :(12:04
*** ioni has joined #openstack-ansible12:05
jrosseri'm not actually sure what it is now tbh12:12
noonedeadpunkas like iptables from i686 was trying to be installed12:15
noonedeadpunkwhich could not be any centos 812:15
noonedeadpunk*libselinux12:16
jrosserhmm well thats actually is libselinux-devel-2.9-4.el8_3.i68612:17
jrosser8_3 rather than 8_4 or something12:17
jrosserit should be wanting this instead http://rpmfind.net/linux/RPM/centos/8.4.2105/baseos/x86_64/Packages/libselinux-devel-2.9-5.el8.x86_64.html12:18
*** wonder has joined #openstack-ansible12:19
noonedeadpunkalso I clean forgot what I recently propesed that assumed 8.3 to be latest one....12:19
jrosseroh some talk in #opendev about mirror storage being full12:20
noonedeadpunkyeah, indeed...12:20
*** ioni has quit IRC12:25
*** wonder is now known as ioni12:32
admin1if i need to change port of any service, which file has the ports data ?12:33
*** odyssey4me has joined #openstack-ansible12:42
noonedeadpunkum, it's in defaults/main.yml12:44
noonedeadpunkbut yu will also need to change haproxy endpoints as well, as we don't use variables for ports data there12:44
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_panko stable/victoria: Fix wsgi_venv path for panko-api  https://review.opendev.org/c/openstack/openstack-ansible-os_panko/+/79503912:52
*** spatel has joined #openstack-ansible12:52
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_panko stable/ussuri: Fix wsgi_venv path for panko-api  https://review.opendev.org/c/openstack/openstack-ansible-os_panko/+/79504012:52
admin1noonedeadpunk, thanks12:53
*** odyssey4me has quit IRC12:54
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum stable/victoria: Define region for Magnum trust  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/79504312:54
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum stable/ussuri: Define region for Magnum trust  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/79504412:54
noonedeadpunkok, found patch where I messed up with 8,3 - https://opendev.org/openstack/openstack-ansible-os_nova/commit/e1e078350a5657be09c688430ad600a89977775d12:59
noonedeadpunknot sure though if fixed edk2-ovmf is part of 8.4 and stream though12:59
*** odyssey4me has joined #openstack-ansible13:00
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Add CentOS 8.4 support  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/79510213:05
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Add CentOS 8.4 support  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/79510313:06
*** mgariepy has quit IRC13:27
*** mgariepy has joined #openstack-ansible13:28
*** jnamdar has joined #openstack-ansible13:32
*** miloa has quit IRC13:48
spatelnoonedeadpunk any idea how to resolve this issue - https://zuul.opendev.org/t/openstack/build/3e57c803e81b42e999bae665e03fab2c14:34
opendevreviewSatish Patel proposed openstack/openstack-ansible-os_neutron master: Add ovn clustering support  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/79426614:39
spatelno able to pinpoint error from CI logs14:45
jrosserspatel: https://zuul.opendev.org/t/openstack/build/3e57c803e81b42e999bae665e03fab2c/log/job-output.txt#21080-2108514:53
spatelI already dig into massive logs file because of size hard to understand which section is broken14:55
spatelhttps://zuul.opendev.org/t/openstack/build/3e57c803e81b42e999bae665e03fab2c14:56
jrosseri'm giving you a link to line 21080 to 2108514:57
spatelhmm my browser didn't take me there..14:57
jrosserfind the place where things fail, i usually search for "RUN END" in the log and find the one with failed: 1 just before it14:58
jrosserno it won't14:58
jrosserbut those lines will be hightlighted yellow if you scroll down to them14:58
jrosseri think this came up last week - that the services are trying to be stopped, but they do not exist14:58
spatelFound it :)14:59
spatelLet me debug and see14:59
spateli can spin up in my lab14:59
spatelIt would be good if i can login into CI build vm to debug15:00
jrosseryou can request a hold on a job if you want to do that15:00
jrossernormally the node is deleted right after the logs are collected15:00
spatelno worry, let me debug it in my lab must be something silly going on here.15:01
noonedeadpunkjust to say that - hold can be asked in #opendev15:01
noonedeadpunkbut it's really more for exception when issue can't be reproduced locally15:02
jrosserspatel: also15:02
jrosserTASK [os_neutron : Install ovn-northd packages]15:02
jrosserskipping: [aio1_neutron_server_container-20dcbd39]15:02
jrosser^ like find the root cause of this i think15:02
spatelLet me build it in my lab first15:02
jrosserit should just need exactly the same config as CI with your patch applied to os_neutron after bootstrap_ansible.sh15:03
jrosserspatel: it skips installing ovn-northd on aio1_neutron_server_container-20dcbd39, which feels correct15:13
jrosserit then tries to restart it on aio1_neutron_server_container-20dcbd39 which fails15:14
jrosserbut we have a container aio1_neutron_ovn_northd_container-ef8e38fe which is where it should actually be15:14
noonedeadpunkSo, we're anyway in circle dependescy with centos. As we need to land https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/795103 https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/795102 and https://review.opendev.org/c/openstack/openstack-ansible/+/794795 but they all kind of depend on each other15:26
noonedeadpunkSo I'd push setting CentOS to NV now15:26
noonedeadpunkto land all required changes15:26
*** ThiagoCMC has quit IRC15:39
*** rpittau is now known as rpittau|afk16:01
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Move CentOS 8 jobs to NV  https://review.opendev.org/c/openstack/openstack-ansible/+/79515316:16
*** ThiagoCMC has joined #openstack-ansible16:38
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Gather extra facts for healthcheck playbooks  https://review.opendev.org/c/openstack/openstack-ansible/+/79479516:58
*** djhankb has quit IRC17:00
*** djhankb has joined #openstack-ansible17:01
*** andrewbonney has quit IRC17:04
noonedeadpunkaio on 8.4 passed relatively clean, except really some small nits17:06
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Add CentOS 8.4 support  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/79510317:18
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Gather extra facts for healthcheck playbooks  https://review.opendev.org/c/openstack/openstack-ansible/+/79479517:20
*** odyssey4me has quit IRC17:44
spateljrosser back now, sorry i got busy in company work18:00
spatelyou said, <jrosser> it should just need exactly the same config as CI with your patch applied to os_neutron after bootstrap_ansible.sh18:01
spateli thought CI and manual building should be close18:02
jrosserdid you use lxc locally?18:07
spatelwhat do you mean lxc locally?18:17
spateli didn't start yet to build aio18:17
spateljust kickstarted new ubuntu box18:22
spatelgoing to build aio on it with my patch18:22
jrosseri mean, when you were testing the OVN stuff did you export SCENARIO=aio_metal, i remember you found that to be much quicker, as theres no lxc18:52
jrosserthe metal deploy lets you get away with a ton of errors about ansible groups as all the components are collapsed onto the same node18:53
jrosserthose only show up with LXC deploys, which is why i wondered in the previous tests you did if it was with or without containers18:53
*** ioni has quit IRC18:54
spatelearlier i build AIO-OVN using metal but never build with LXC  (i did multi-node deployment using LXC though)18:58
spateljrosser ^18:58
spatellet me try building AIO using LXC in my lab to see what issue we encounter18:59
spatellast time i did this SCENARIO='aio_ovn_metal'19:14
spatelwhere is aio_ovn_metal defined ?19:14
spateljrosser ^^19:14
jrosserspatel: it’s kind of parsed, not defined anywhere19:15
spateloh19:15
spatelshould i be doing aio_ovn_lxc ?19:15
jrosseryes try that19:16
spatelok19:16
spatelwhy don't we use apply_security_hardening: True  for AIO deployment :)19:23
spatelwhy don't we use apply_security_hardening: False for AIO deployment :)19:23
spatelI meant false19:23
spatelEvery time i build aio and forgetting to set false and later its painful to work and doing testing because of lots of security stuff19:24
spatelalso it takes little time to add sec stuff19:25
noonedeadpunkok, awesome, centos8 seems "unblocked" with https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/79510219:57
noonedeadpunkit's just symlink to 8.3 just in case19:59
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/ussuri: Disable magnum zero nodes test.  https://review.opendev.org/c/openstack/openstack-ansible/+/79513720:02
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum stable/ussuri: Define region for Magnum trust  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/79504420:03
spatelnoonedeadpunk i thought centos is gone gone20:16
spateljrosser i hit the same issue what CI hit so its good20:48
spateldebugging started....20:48
spatelhmm why its looking for ovs services on neutron-server, they don't be there -http://paste.openstack.org/show/806436/20:51
spatelit should be on ovn container so look like something is messed up here20:51
spatelmy patch has some issue, it doesn't know where to run...20:54
spateli need to set when condition here20:54
spateljrosser i have added when condition here so why it still trying to run this command on neutron-server ?  - http://paste.openstack.org/show/806438/21:06
jrosserspatel: the services are defined here https://github.com/openstack/openstack-ansible-os_neutron/blob/master/vars/main.yml#L34621:07
spatelhttps://github.com/openstack/openstack-ansible-os_neutron/blob/master/vars/main.yml#L49021:08
spatelits there21:08
jrosseris this right? - neutron_services['neutron-ovn-northd']['group'] in group_names21:08
jrosserisnt group names all the groups21:09
spateli took example from other playbooks code21:09
jrosserfeels like saying "is this group name in all the group names" -> always true21:09
spateli took that example from here - https://opendev.org/openstack/openstack-ansible-os_neutron/src/branch/master/tasks/providers/ovn_config.yml#L2721:10
jrosseroh hold on "group_names List of groups the current host is part of"21:11
jrosserso i think you need to add debug here to look at what group_names contains21:12
spatelok let me try -vvvv21:12
spateldo i need when condition under block: code?21:13
spateli don't think so21:13
jrosserthis could be an inventory structure issue21:14
jrosserlook at which containers are in which groups, there should be neutron_ovn_northd group21:14
jrosserbut i would add a debug task to print group_names near your block:21:14
spatellook at this inventory output - http://paste.openstack.org/show/806439/21:16
spatelwe do have neutron_ovn_northd group21:16
jrosserok right so we need to debug21:18
jrosseri still say you need to print group_names to see what is there21:19
spatelhow do i print group_name in playbook ?21:19
jrosseradd a debug: task21:19
jrosservar: group_names21:19
spatelmsg:21:19
spatelok21:19
opendevreviewMerged openstack/openstack-ansible-tests stable/victoria: Workaround linting issue  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/79290621:20
spateljrosser here is the debug output - http://paste.openstack.org/show/806440/21:21
spateli am not seeing northd here21:22
spatelwhy this line saying service_en: False ? - https://github.com/openstack/openstack-ansible-os_neutron/blob/master/vars/main.yml#L49221:24
jrosserspatel: we need to work out why the task runs when we think it should not21:27
spatelhmm21:28
spatelalso why its not able to find group21:28
jrossernow try printing the other part of the expression, the left hand side21:28
jrosserbecause it’s running against neutron server host21:28
spatelneutron_services['neutron-ovn-northd']['group'] ?21:28
jrosserwhich, correctly, is not in the northd group21:28
jrosseryes21:28
spatelok21:28
spatelmay be this is meant to run for metal not lxc21:29
spateldebug output print - "neutron_services['neutron-ovn-northd']['group']": "neutron_ovn_northd"21:30
spatelthat is correct21:30
spatelbut that group isn't part of this list - http://paste.openstack.org/show/806440/21:31
spatelBRB21:31
*** luksky has quit IRC21:34
admin1doesn't ansible honor .ssh/config host key files ?21:34
admin1i have a host where i need to give another key for root .. its in config so ssh   host works fine21:35
admin1but when i run the ansible playbook, it does not use the config it seems21:35
admin1is there a way to tell osa to use a specific key when running playbooks ( doing ssh as root )21:35
jrosseransible pretty much always just leverages your local ssh config21:36
*** spatel has quit IRC21:42
*** tosky has quit IRC22:04
admin1thanks jrosser.. i had used  c1: ip: .. and ssh c1 worked as root .. but ansible was doing ssh root@Ip directly22:15
jrosseryeah, it uses whatever is in the inventory22:15
admin1basically ssh infra1 worked as root  ssh infra1's ip did not worked22:15
admin1all working now .. thanks22:15
*** Adri2000 has quit IRC22:51
*** Adri2000 has joined #openstack-ansible22:54
*** Adri2000 has quit IRC23:14

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