Monday, 2019-11-25

*** k_mouza has quit IRC00:53
*** duksoo has joined #openstack-kolla02:59
*** duksoo has quit IRC03:26
*** vesper11 has quit IRC03:45
*** vesper11 has joined #openstack-kolla03:47
*** duksoo has joined #openstack-kolla04:17
osmanlicilegimorning05:14
*** duksoo has quit IRC05:16
*** msalo has joined #openstack-kolla05:16
*** msalo has quit IRC05:20
*** duksoo has joined #openstack-kolla05:22
*** sri_ has joined #openstack-kolla05:41
*** jistr has quit IRC05:51
*** jistr has joined #openstack-kolla05:56
*** jistr has quit IRC06:06
*** jistr has joined #openstack-kolla06:06
*** skramaja has joined #openstack-kolla06:18
*** duksoo has quit IRC06:22
*** numans has joined #openstack-kolla06:23
CosmicSound good day06:35
*** dpawlik has joined #openstack-kolla06:47
*** vmixor has joined #openstack-kolla06:58
*** cah_link has joined #openstack-kolla07:11
*** CosmicSound4 has joined #openstack-kolla07:14
*** CosmicSound has quit IRC07:15
osmanlicilegicloudnull: ping07:17
*** CosmicSound4 has quit IRC07:19
*** CosmicSound has joined #openstack-kolla07:24
mnasiadkamorning07:29
mnasiadkaosmanlicilegi: he's located in US I think, so you'll need to wait a ,,bit'' :)07:29
mnasiadkayoctozepto: which doesn't say a thing :)07:29
yoctozeptomorning07:32
yoctozeptomnasiadka: still better than nothing07:32
yoctozeptobut, yeah, better insight would be nice07:32
yoctozeptoespecially in our position07:32
yoctozeptoosmanlicilegi: you mean to talk about rmq?07:33
yoctozeptoI think as the change owner, it would be kind of you to propose a change to tripleo too07:33
yoctozepto(since it is blocking your patch)07:33
osmanlicilegiyoctozepto: got it07:34
*** CosmicSound is now known as cosmicsound07:35
mnasiadkayoctozepto: but osmanlicilegi might not have insight what tripleo wants:)07:46
openstackgerritMichal Nasiadka proposed openstack/kolla-ansible master: CI: Add ansible-lint to tox  https://review.opendev.org/69477907:47
yoctozeptomnasiadka: we already established the minimum change is to change the exact pin to the unexact one07:48
yoctozeptopropose, depend-on and be done07:48
yoctozeptomnasiadka: if you were kind enough to approve all my pending CI changes :-)07:48
*** cosmicsound has quit IRC07:49
mnasiadkayoctozepto: well, I think tripleo doesn't use packagecloud and epel (they remove it in base)07:49
mnasiadkayoctozepto: reviewing is on my list, first I want to attend my changes (yeah, I'm selfish bastard) ;)07:49
openstackgerritMichal Nasiadka proposed openstack/kolla-ansible master: CI: Add ansible-lint to tox  https://review.opendev.org/69477907:53
*** cosmicsound has joined #openstack-kolla07:54
yoctozeptomnasiadka spamming unmergables :-)07:56
openstackgerritMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Use Docker healthchecks  https://review.opendev.org/67638908:00
*** tumble has quit IRC08:01
hrwmorning08:02
hrwtripleo... which channel they use on irc?08:03
osmanlicilegihrw: #tripleo08:05
hrwthx08:05
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: WIP: Enable repos only when needed  https://review.opendev.org/69546508:11
hrwcan someone review before I think of any improvements?08:11
Blinkizyoctozepto: Hi. Am about to upload a new commit with the change you requested. Anything more I should fix? https://review.opendev.org/#/c/692179/08:11
patchbotpatch 692179 - kolla-ansible - Support for keystone scoped authorization - 22 patch sets08:11
openstackgerritDincer Celik proposed openstack/kolla-ansible stable/train: Fixes the compatibility issue at HAProxy role with Ansible 2.9  https://review.opendev.org/69585108:12
mnasiadkayoctozepto: so, orchestrator is an abstraction layer with API that consumes ceph-ansible as the deployer (or you can choose to use ssh backend)08:17
mnasiadkayoctozepto: but ceph-ansible will cease to exist in it's current form some time in future08:18
yoctozeptoBlinkiz: don't worry upfront, upload :-)08:19
Blinkizkk08:19
yoctozeptomnasiadka: yeah, that is the main message08:19
yoctozeptobut it lacks details08:19
yoctozeptomnasiadka: re ovn http://lists.openstack.org/pipermail/openstack-discuss/2019-November/011115.html08:19
yoctozeptointeresting08:19
openstackgerritNiklas Hagman proposed openstack/kolla-ansible master: Support for keystone scoped authorization  https://review.opendev.org/69217908:20
mnasiadkayoctozepto: it's the same with opencontrail/tungsten fabric, and others - I'm not surprised - but from other example there is migration path from ML2/OVS to OVN08:21
yoctozeptoall right08:22
mnasiadkayoctozepto: only for tripleo, but there is one :D08:22
*** rpittau|afk is now known as rpittau08:33
*** msalo has joined #openstack-kolla08:42
mnasiadkayoctozepto: I think I went through all changes - if there's something I missed, tell me ;)08:54
yoctozeptomnasiadka: thanks, almost all I wanted :-)08:58
yoctozeptohttps://review.opendev.org/69580408:58
patchbotpatch 695804 - kolla-ansible - [Train] Fix Tacker config for the local Glance store - 2 patch sets08:58
yoctozeptohttps://review.opendev.org/69519208:58
patchbotpatch 695192 - kolla-ansible - CI: Cleanups, refactors and testing of Qinling - 20 patch sets08:58
*** dougsz has joined #openstack-kolla09:05
yoctozepto"tacker... y u so fugly" - only in select cinemas09:06
mnasiadkawell, looking at the second change and number of scripts that we have and bash functions spread over those files, I'm starting to be worried09:07
openstackgerritMerged openstack/kolla-ansible stable/train: Fix missing environment variable for Qinling  https://review.opendev.org/69536009:08
yoctozeptomnasiadka: good timing09:09
yoctozeptobut in all seriousness, I think this is the best step now (not humble I know, but seriously)09:10
*** gfidente has joined #openstack-kolla09:10
yoctozeptobest that comes to mind is rewriting this to dedicated ansible modules09:11
yoctozeptowe could write it much more resilient09:11
yoctozeptofor now, though, nicer split to aptly named testing scripts sounds just like what we need09:12
openstackgerritMerged openstack/kolla-ansible stable/stein: Fix conditionals in CI playbook  https://review.opendev.org/69566209:15
openstackgerritMerged openstack/kolla-ansible stable/stein: CI: Add mariadb test  https://review.opendev.org/69302409:15
openstackgerritMerged openstack/kolla-ansible stable/train: CI: Add mariadb test  https://review.opendev.org/69553309:17
yoctozeptoit's raining merges, hallelujah ;D09:18
mnasiadkayoctozepto: so, with using orchestrator the problem is, that it runs as mgr service, so it needs mons deployed :D09:19
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: Move definition of OpenStack sources into YAML file  https://review.opendev.org/69339809:19
hrwnow it should not fail09:19
yoctozeptomnasiadka: :D09:19
yoctozeptodecisions, decisions09:19
yoctozeptowell, deploying mon is the cleanest09:20
yoctozeptowe could have a dedicated action for that09:20
yoctozeptobut all day-2 ops are out of scope for us09:20
yoctozeptono upgrades via kolla09:20
yoctozeptoetc.09:20
*** dciabrin has quit IRC09:20
mnasiadkayeah, it's not a problem, it's rather a change of approach09:20
yoctozeptothis was the biggest issue09:20
yoctozeptodoing upgrades, handling osds09:21
mnasiadkaand second thing is do we want to use mon/mgr deployment from ceph-ansible, or do we want to use our own? :D09:21
yoctozeptomon, mgr and only to deploy are fine09:21
*** dciabrin has joined #openstack-kolla09:21
yoctozeptoif ceph-ansible is going bye-bye then you have your answer09:21
mnasiadkain octopus we have this: https://docs.ceph.com/docs/master/bootstrap/09:22
mnasiadkaso probably that's the best idea we have09:22
mnasiadkawell, not really - better to write new role09:27
yoctozeptoI still am curious why ceph-ansible is going eol09:27
yoctozeptowere the performance issues?09:28
yoctozeptoor just redhat not wanting to use redhat stack ;o09:28
mnasiadkawell, ceph-ansible was a bit... split from the other ceph stuff09:29
*** msalo has quit IRC09:29
mnasiadkait's a way to merge it together and manage it more in an organised fashion09:30
*** msalo has joined #openstack-kolla09:30
*** jbadiapa has joined #openstack-kolla09:30
mnasiadkaI need to rethink it, but my current idea is to run ceph-daemon to deploy a minimal mon+mgr and move from there09:30
*** hamdyk has joined #openstack-kolla09:31
mnasiadkaquestion do we want to install ceph-daemon on the host level, or build a ceph-daemon image - containing just a bloody python script: https://raw.githubusercontent.com/ceph/ceph/master/src/ceph-daemon/ceph-daemon09:31
hrwmnasiadka: or add it to kolla-toolbox09:32
*** rgogunskiy has joined #openstack-kolla09:32
hamdykHi all, is it possible to add new image in kolla that would would be built for centos only (no ubuntu) ?09:32
mnasiadkahrw: yeah, that was my idea I wanted to write now :D09:32
hrwhamdyk: yes. we have such09:33
hrwhamdyk: cyborg one is centos/binary/x86-64 only iirc09:33
hamdykI see09:33
hamdykhrw: will take a look, thanks09:33
*** cah_link1 has joined #openstack-kolla09:34
mgoddardmorning09:34
hrwhamdyk: 1. add an image into docker/ 2. get it buildable 3. add it to UNBUILDABLE_IMAGES in kolla/image/build.py09:34
hrwhamdyk: 4. send for review09:34
mnasiadkamorning mgoddard09:34
hrwhi mgoddard09:34
yoctozeptohello mgoddard09:35
*** cah_link has quit IRC09:35
*** cah_link1 is now known as cah_link09:35
hrwfolks, what do you think of moving sources.yaml, repos.yaml to etc/ dir (or some dir other than subdirs of kolla/ one)?09:35
hrwI would like to move UNBUILDABLE_IMAGES to yaml file as well and then we can merge it with status-matrix09:36
yoctozeptohrw: nice09:38
yoctozeptoboth ideas09:38
*** k_mouza has joined #openstack-kolla09:42
hrwa question is: etc/ or data/ or some-other-name/09:43
*** k_mouza has quit IRC09:46
yoctozeptohrw: since it's kinda part of config, etc/ sounds fine imho09:48
osmanlicilegihrw: +1 for etc/09:49
openstackgerritMerged openstack/kolla-ansible stable/train: CI: Remove unused tests/ansible_setup_ceph_disks.yml  https://review.opendev.org/69553709:52
openstackgerritMerged openstack/kolla-ansible stable/train: CI: Remove playbook-setup-nodes.yml  https://review.opendev.org/69553609:52
openstackgerritMerged openstack/kolla-ansible stable/train: CI: Reconfigure deployment after checking health  https://review.opendev.org/69553409:52
openstackgerritMerged openstack/kolla-ansible stable/stein: Parse MariaDB log messages in different formats into Fluentd  https://review.opendev.org/69539309:52
openstackgerritMerged openstack/kolla-ansible stable/train: CI: Wait for Zun to delete the test container  https://review.opendev.org/69582509:52
openstackgerritMerged openstack/kolla-ansible stable/rocky: Parse MariaDB log messages in different formats into Fluentd  https://review.opendev.org/69539409:52
openstackgerritMerged openstack/kolla-ansible stable/train: Parse MariaDB log messages in different formats into Fluentd  https://review.opendev.org/69539209:52
*** k_mouza has joined #openstack-kolla09:54
openstackgerritDincer Celik proposed openstack/kolla-ansible stable/stein: Fixes the compatibility issue at HAProxy role with Ansible 2.9  https://review.opendev.org/69587009:54
*** jistr is now known as jistr|afk09:58
*** cah_link has quit IRC10:01
*** cah_link has joined #openstack-kolla10:01
*** vmixor has quit IRC10:17
openstackgerritMichal Nasiadka proposed openstack/kolla-ansible master: CI: Add ansible-lint to tox  https://review.opendev.org/69477910:18
*** jistr|afk is now known as jistr10:22
openstackgerritDoug Szumski proposed openstack/kolla master: Add Elasticsearch Curator Docker image  https://review.opendev.org/69434110:24
*** msalo has quit IRC10:26
*** msalo_ has joined #openstack-kolla10:27
openstackgerritMerged openstack/kolla-ansible master: Default to disabling haproxy for memcached  https://review.opendev.org/69086910:30
openstackgerritMerged openstack/kolla master: Neutron: apply db migration for BGP dynamic routing agent  https://review.opendev.org/68924410:30
hrwtripleo7 - will it stay at py2 or move?10:41
hrwas I try to keep my patches not compatible with py2 if possible so this job fails10:41
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: Get rid of Python 2 support  https://review.opendev.org/69131610:43
yoctozeptohrw: ask in #tripleo ?10:46
openstackgerritMarcin Juszkiewicz proposed openstack/kolla-ansible master: Get rid of Python 2 support  https://review.opendev.org/69516810:46
openstackgerritMarcin Juszkiewicz proposed openstack/kolla-ansible master: Get rid of Python 2 support  https://review.opendev.org/69516810:47
yoctozeptomgoddard, mnasiadka: https://review.opendev.org/695192 <- accepted/not? :D10:48
patchbotpatch 695192 - kolla-ansible - CI: Cleanups, refactors and testing of Qinling - 20 patch sets10:48
* yoctozepto planning to base more jobs on it...10:48
*** gfidente has quit IRC10:52
openstackgerritViktor Michalek proposed openstack/kolla stable/train: Neutron: apply db migration for BGP dynamic routing agent  https://review.opendev.org/69588010:54
*** gfidente has joined #openstack-kolla10:54
*** vmixor has joined #openstack-kolla10:55
openstackgerritDoug Szumski proposed openstack/kolla-ansible master: Support deploying Elasticsearch Curator  https://review.opendev.org/69453710:58
mnasiadkayoctozepto: I guess it's ok, but we should think of a way to make it all more simple11:02
mnasiadkayoctozepto, mgoddard, hrw: there's a comment on the ansible-lint change, that it would be better if we would use ansible-lint as pre-commit - but we don't have any pre-commits - so either we go that way and add linters to pre-commit, or leave as is. votes?11:04
hrwmnasiadka: CI job.11:04
openstackgerritMerged openstack/kolla master: test-requirements: bump pyflakes to get f-strings support  https://review.opendev.org/69579611:05
openstackgerritPierre Riteau proposed openstack/kolla-ansible master: Remove endpoint_type from neutron.conf [DEFAULT] group  https://review.opendev.org/69588211:05
hrwmnasiadka: if we treat ansible-lint as pre-commit then we should move pep8 to pre-commit too, right?11:06
hrwmnasiadka: we use two languages in k-a: python and ansible. so treat linters equally11:07
mnasiadkahrw: yeah, I think that makes it a bit heavy - and I don't know how many people have tox and python working on the machine they write code on :)11:07
zbrmarkmcclain: hrw: yeah. using pre-commit would simplify it, but lets make these changes one at a time?11:07
zbrfrom the dev point of view, they only need to do the same "tox -e linters".11:08
zbrwhat it happenign inside is that this installs pre-commit and runs "pre-commit run -a"11:09
openstackgerritMarcin Juszkiewicz proposed openstack/kolla-ansible master: Some ansible-lint fixes  https://review.opendev.org/69479711:09
zbrso is basically the same thing11:09
hrwzbr: tox: command not found11:09
hrwI can do patch work on pythonless machine even11:10
zbrhrw: yeah, you can. you can always ignore contributing guidelines, nobody stops you.11:10
zbradding pre-commit tool does not enable any hooks!11:10
zbrconsidering that git-review is python based I think you really like using advanced git stuff to create a CR on gerrit ;)11:12
vmixorHi, there, what ubuntu version is used in upstream CI checks11:12
zbrvmixor: depends, usually xenial11:12
zbrthe rule is LTS11:12
zbrvery unlikely to allow non LTS jobs, changes too often.11:13
vmixorah, I have patch witch enables Vitrage images building11:13
vmixoron Debian and Ubuntu11:14
hrwvmixor: we use bionic (18.04)11:14
*** pcaruana has joined #openstack-kolla11:14
openstackgerritMarcin Juszkiewicz proposed openstack/kolla-ansible master: Get rid of Python 2 support  https://review.opendev.org/69516811:15
vmixorok, have to rework that patch, Vitrage is available since disco11:15
vmixor** Vitrage packages FOr Ubuntu11:15
hrwzbr: due to how I distribute my ssh keys machine where I work on patch is not always the same I submit patch to review from11:16
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: WIP: move to CentOS 8 in containers  https://review.opendev.org/69236811:18
zbrhrw: sure. shortly using the tool does not add any extra requirements or even behavior changes. I can show a dozen repos where we already did it or even propose a minimal patch to enable it.11:18
hrwzbr: submit patch, we review, probably merge and live with it.11:19
zbrthat's pragmatic :D11:19
hrwzbr: I install tox only in virtualenv used for kolla. most of shell sessions where I work on patch does not use it.11:19
mnasiadkavmixor: btw, I don't think Vitrage has a bright future :)11:20
hrwvmixor: in one year from now we will use 20.0411:20
hrwzbr: and when linters are voting on CI they remind any submitter that stuff needs to be cleaned11:23
hrwzbr: but as you are from tripleo... are there plans to move from py2 in centos7 builds?11:23
zbrhrw: YES! we will move to centos-8 asap, that is not debatable.11:24
hrwzbr: ok, will then use f-strings like it is in https://review.opendev.org/695465 (which works everywhere outside of tripleo7 ci job)11:25
patchbotpatch 695465 - kolla - WIP: Enable repos only when needed - 10 patch sets11:25
zbri would personally not hurry to add py36+ features, not yet. f-string could create a lot of issue with backporting stuff to older branches.11:27
zbrthey are cool, but they do not come without side-effect costs.11:27
hrwzbr: this is new functionality which will not be backported11:27
zbrhrw: yeah.. you add the in one place, next day someone else adds them to another patch,.... later you need to backport one change and you discover that you have to rewrite it completely. anyway, i am not against them, just raising awareness.11:29
hrwzbr: sure thing.11:29
yoctozeptomnasiadka: ok, but can't see your +2 on https://review.opendev.org/695192 :-)11:30
patchbotpatch 695192 - kolla-ansible - CI: Cleanups, refactors and testing of Qinling - 20 patch sets11:30
*** openstack has joined #openstack-kolla11:54
*** ChanServ sets mode: +o openstack11:54
openstackgerritSorin Sbarnea proposed openstack/kolla-ansible master: wip: call bashate and yamllint via pre-commit  https://review.opendev.org/69588811:55
openstackgerritRafael WeingÀrtner proposed openstack/kolla-ansible master: Add feature to support managing dynamic pollsters  https://review.opendev.org/67999912:01
*** radek__ has joined #openstack-kolla12:07
hrwbinary/source/infra images...12:11
hrwcan we for start agree that everything which is not based on openstack-base is infra?12:11
*** BaptisteGer has quit IRC12:11
openstackgerritDincer Celik proposed openstack/kolla master: DNM: Add rhel8 support for early CentOS8 testing  https://review.opendev.org/66947112:12
hrwor 'kolla-toolbox and openstack-base (+ all children) are not infra'12:13
mgoddardhrw: we probably need to go through and check, but sounds close12:20
mgoddardhrw: I'd like to see a proposal for how it will work before we go into implementation12:21
hrw{% if install_type == 'infra' %}12:21
hrwRUN echo 'This is not {{ install_type }} image' && /bin/false12:21
hrw{% endif %}12:21
hrwmgoddard: sure12:21
hrwmgoddard: experimenting with something before even going with proposal12:21
mgoddardsounds good12:22
hrwmgoddard: as this part of kolla is something I did not messed with before12:22
mgoddardhrw: we also need to think about how these images are consumed, both tripleo and kolla12:22
openstackgerritSorin Sbarnea proposed openstack/kolla-ansible master: wip: call bashate and yamllint via pre-commit  https://review.opendev.org/69588812:22
*** kplant has joined #openstack-kolla12:24
hrwmgoddard: I see it as infra replacing binary/source non-openstack ones12:27
*** dave-mccowan has joined #openstack-kolla12:30
openstackgerritSorin Sbarnea proposed openstack/kolla-ansible master: Call bashate and yamllint via pre-commit  https://review.opendev.org/69588812:31
openstackgerritMerged openstack/kolla-ansible stable/train: Fixes the compatibility issue at HAProxy role with Ansible 2.9  https://review.opendev.org/69585112:33
*** dave-mccowan has quit IRC12:35
hrw69 infra, 207 source images12:37
openstackgerritMerged openstack/kolla-ansible master: Fix hard-coded admin project name and username in blazar task  https://review.opendev.org/69521412:38
openstackgerritMerged openstack/kolla-ansible stable/stein: Fixes the compatibility issue at HAProxy role with Ansible 2.9  https://review.opendev.org/69587012:43
mnasiadkamgoddard, hrw, yoctozepto: as we're in the -infra area - for example skydive has moved to docker images instead of releasing rpms, what do you say to create images using skydive's images as base?12:43
mnasiadka(to add kolla startup scripts)12:43
hrwmnasiadka: reusing 'project images' is something other imho.12:44
mnasiadkahrw: well, either that or I remove using rpms from skydive's Dockerfiles :)12:44
hrwlet me write an email to openstack-discuss and then we can discuss there12:44
hrwI do wonder how to easily sort images into types. without creating kind of images.yaml with unbuildable, status, infra/non-infra stuff there12:46
hrwmnasiadka: and for reusing project images - you can do it even today. just change FROM in dockerfile.j2 D:12:47
mnasiadkahrw: I know :D12:47
mnasiadkahrw: well, maybe we need subdir infra/ && openstack/? and unbuildable... well - I guess it's still a yaml12:48
*** sri_ has quit IRC12:48
hrwmnasiadka: is storm openstack?12:48
mnasiadkaApache Storm? no :)12:48
hrwmnasiadka: but is it infra?12:50
hrwSuccessfully tagged infra/debian-binary-libvirt:9.1.0 ;D12:51
mnasiadkawell, it's infra required to use Storm in Sahara12:51
mnasiadkaquestion is if we should be even provisioning that and building images for that12:51
mnasiadkaseems like an overkill12:51
mnasiadkado we build images for Spark also? :D12:52
mgoddardmnasiadka: it's required for monasca12:54
mnasiadkaok, I thought it's for Sahara12:54
mnasiadkamaybe we should have some... descriptions of the infra images12:55
mgoddardmnasiadka: re skydive base image, I don't think we really came to a conclusion on that at the PTG, but there were concerns around introducing new base OS images12:56
mnasiadkamgoddard: so, in this case (skydive case), if we use project delivered base image - we will have arm64 (and maybe ppc64le), if not - only x86_64 - that's the motivation12:56
*** rgogunskiy has quit IRC12:57
mnasiadkafor now, it's easy just to remove installation using rpm and default to fetching a binary from github12:57
andreinhi guys, I'm trying to upgrade our openstack deployment from stein to train and I'm getting the following warnings when running `kolla-ansible pull`: http://paste.openstack.org/show/786665/12:57
*** rgogunskiy has joined #openstack-kolla12:57
andreinit fails later down the road because it can't find the kolla_address filter12:58
hrwI like how UNBUILDABLE_IMAGES sort out infra thing. "'infra': { 'openstack-base' },"12:58
hrwandrein: and you use train kolla-ansible?12:59
andreinhrw: i've pulled stable/train from git and installed it using `pip install kolla-ansible`12:59
hrwandrein: which ansible version you use?13:00
mnasiadkaandrein: and have you installed it in the venv?13:00
andreinansible 2.8.4 and it looks like it's not installed in the venv... bet that's the reason why it fails13:01
andreinyep, no more warnings after installing it in the venv13:02
andreinis there a reason why kolla-ansible doesn't list ansible in requirements.txt?13:04
*** goldyfruit has quit IRC13:11
*** goldyfruit has joined #openstack-kolla13:14
*** vmixor has quit IRC13:18
*** zbr_ has joined #openstack-kolla13:19
*** goldyfruit has quit IRC13:19
*** msalo has joined #openstack-kolla13:19
*** zbr has quit IRC13:22
*** msalo_ has quit IRC13:23
*** vmixor has joined #openstack-kolla13:26
openstackgerritMark Goddard proposed openstack/kayobe master: Add documentation for Bifrost and IPA configuration  https://review.opendev.org/69351513:42
hrwsimple mail to openstack-discuss sent about infra images13:44
openstackgerritHamdy Khader proposed openstack/kolla master: Added Neutron Mellanox agent image  https://review.opendev.org/67202313:45
zbr_hrw: mnasiadka: https://review.opendev.org/#/c/695888/ is ready (failure due to infra, recheck will do)13:47
patchbotpatch 695888 - kolla-ansible - Call bashate and yamllint via pre-commit - 3 patch sets13:47
*** tonythomas has joined #openstack-kolla13:49
zbr_finishing the subject will need 2-3 follow-ups, but i will not open them until this merges to avoid rebases13:49
*** KeithMnemonic has joined #openstack-kolla13:54
openstackgerritMark Goddard proposed openstack/kayobe master: Improve docs on updating the Ansible control host  https://review.opendev.org/69370113:57
openstackgerritDoug Szumski proposed openstack/kayobe master: Document requirements for BIOS and RAID configuration  https://review.opendev.org/69353313:57
yoctozeptoandrein: flexibility13:59
yoctozeptothe error you had simply says you need to install kolla_ansible13:59
numansyoctozepto, mnasiadka Hey, can you please take a look at this one whenever you get time - https://review.opendev.org/#/c/694453/ it's  trrain backport and the CI passed14:00
patchbotpatch 694453 - kolla (stable/train) - Use ovn-ctl script to start OVN ovsdb-servers - 1 patch set14:00
openstackgerritSorin Sbarnea proposed openstack/kolla-ansible master: Call bashate and yamllint via pre-commit  https://review.opendev.org/69588814:01
openstackgerritMark Goddard proposed openstack/kayobe master: Use internal API endpoints in overcloud API interaction  https://review.opendev.org/69309014:09
hrwzbr_: n00b question: how pre-commit works? I fetched your change with git-review, commited some random stuff and do not see anything run extra.14:11
mgoddardhrw: I think you need to configure a hook locally14:12
openstackgerritMerged openstack/kayobe master: Improve docs on updating the Ansible control host  https://review.opendev.org/69370114:17
openstackgerritMerged openstack/kayobe master: Document requirements for BIOS and RAID configuration  https://review.opendev.org/69353314:17
zbr_mgoddard: no need to configure hook, it is optional, only if you want to automate took execution.14:21
zbr_i personally do not use that feature14:21
mgoddardzbr_: oh ok, so how do you run it?14:21
zbr_tox -e linters (or old tox -e pep8)14:22
zbr_what it does is runs: "pre-commit run -a"14:22
zbr_tool works similar to tox, installing dependencies in isolated virtualenvs, and uses them from there.14:22
zbr_mainly this change does not require any behavior change for the developer14:23
mgoddardzbr_: I'm not really seeing what this gets us above tox then14:24
mgoddardif it's just another way to do the same thing14:24
mgoddardare projects standardising on this pattern?14:24
zbr_mgoddard: that is only what they have in common, but after this you can easily scale it to lots of linters14:24
zbr_for example, bumping linter versions: you just run "pre-commit autoupdate" and it will try to bump all of them.14:25
andreinhello again, I've hit another error while upgrading from stein to train: http://paste.openstack.org/show/786674/ I don't understand why it's even going through nova-cell, since I haven't enabled the cells feature. I've checked the cephx keyring and it is present in `/etc/kolla/config/nova/ceph.client.nova.keyring`14:25
mgoddardah yeah, I remember reading about this https://pre-commit.com/14:25
zbr_tox did not scale well with lots of linters, it clutters the tox.ini file until it becomes a nightmare14:25
zbr_even tox itself is using pre-commit'14:26
mgoddardhi andrein, every deployment has at least one nova cell. We added support for multiple, so you'll see this new nova-cell role14:27
andreinah, I see14:27
mgoddardandrein: I'll take a look at the code in that area.14:27
openstackgerritMerged openstack/kolla-ansible master: Remove endpoint_type from neutron.conf [DEFAULT] group  https://review.opendev.org/69588214:27
mgoddardandrein: do you have kolla ceph or external ceph14:28
andreinmgoddard: i have external ceph14:29
andreindeployed with ceph-ansible, if it matters14:29
mgoddardandrein: ok14:30
hrwbrb14:30
*** hrw has quit IRC14:30
*** hrw has joined #openstack-kolla14:31
hrwre14:31
mgoddardandrein: what is your nova_backend?14:31
mgoddardandrein: actually doesn't matter14:32
andreinshould be RBD, I don't explicitly set it14:32
mgoddardandrein: can you share the ~10 tasks before that one?14:33
andreinsure14:33
*** KeithMnemonic has quit IRC14:33
andreinhttps://www.irccloud.com/pastebin/n4KqGJUY/14:34
*** hamdyk has quit IRC14:37
*** goldyfruit has joined #openstack-kolla14:38
andreinI think the regex in "Extract nova key from file" doesn't match my key14:40
*** baha has quit IRC14:41
andreinnevermind, it matches, escaped backslashes got me14:43
mgoddardandrein: my guess is the condition here is getting in the way: https://opendev.org/openstack/kolla-ansible/src/branch/master/ansible/roles/nova-cell/tasks/config.yml14:45
mgoddardandrein: L3314:46
mgoddardandrein: or maybe not actually14:47
* hrw off14:47
andreinI think nova_backend is default on my deployment (we only use ceph for volumes, so we haven't set nova_backend_ceph)14:47
mgoddardandrein: you could try reverting commit 44709f413246a05b891f250e88c43226e1f1b0c9, it looks related14:50
mgoddardandrein: actually I think I see the issue14:51
mgoddardandrein: do you set nova_backend_ceph?14:52
*** spiette has joined #openstack-kolla14:53
yoctozeptomgoddard, could you link to review (me on mobile)14:53
andreinmgoddard: I do not14:53
*** goldyfruit has quit IRC14:53
mgoddardandrein: ok, makes sense14:53
andreinmgoddard: Will setting that configure the VMs to run off ceph for their ephemeral storage?14:53
mgoddardyes. I'm not suggesting you set it, just interested in your config14:54
mgoddardyoctozepto: https://review.opendev.org/#/c/689753/14:54
patchbotpatch 689753 - kolla-ansible - Extract cephx keys from vault encrypted files (MERGED) - 2 patch sets14:54
*** goldyfruit has joined #openstack-kolla14:55
mgoddardandrein: can you try this: http://paste.openstack.org/show/786676/14:55
andreinmgoddard: running it now14:59
*** goldyfruit_ has joined #openstack-kolla14:59
mgoddardandrein: could you raise a bug for this issue on launchpad?14:59
andreinwill do!14:59
mgoddardthanks15:00
mgoddardandrein: if the fix works, feel free to commit & propose15:01
*** goldyfruit has quit IRC15:01
*** baha has joined #openstack-kolla15:01
andreinmgoddard: it worked, thanks!15:11
openstackgerritMark Goddard proposed openstack/kolla master: WIP: CI: Move to CentOS 8 nodepool images  https://review.opendev.org/69279415:12
andreinmgoddard: can I skip the launchpad ticket and just open a review for it?15:12
mgoddardandrein: it's useful to have a bug report to track the issue15:13
andreinok15:13
mgoddardthanks :)15:13
openstackgerritMark Goddard proposed openstack/kolla master: WIP: move to CentOS 8 in containers  https://review.opendev.org/69236815:16
openstackgerritMark Goddard proposed openstack/kolla master: Switch to python3 in bindep.txt  https://review.opendev.org/69592715:20
*** rgogunskiy has quit IRC15:21
openstackgerritViktor Michalek proposed openstack/kolla master: Vitrage: build images for Debian  https://review.opendev.org/69593115:31
openstackgerritAndrei Nistor proposed openstack/kolla-ansible master: Fix 'nova_cephx_raw_key' is undefined  https://review.opendev.org/69593315:45
*** msalo has quit IRC15:47
openstackgerritViktor Michalek proposed openstack/kolla master: Horizon: install vitrage dashboard  https://review.opendev.org/69593715:50
mgoddardmnasiadka, yoctozepto: you have some TODOs from the PTG: https://etherpad.openstack.org/p/kolla-ussuri-ptg15:55
mgoddardI listed them all at the end of the page15:56
mgoddardhrw: there are two unassigned TODOs. Perhaps you could help?15:56
*** msalo has joined #openstack-kolla16:04
yoctozeptomgoddard: so it was a different issue with external ceph but not for nova...16:05
yoctozeptoandrein: ^16:06
mgoddardyoctozepto: it was introduced by the patch I shared16:06
mgoddardregistered variable was previoulsy always defined. Switched to a set_fact and it is now sometimes defined.16:07
yoctozeptoI see whens on both sides though16:08
*** radek__ has quit IRC16:08
yoctozeptoOr is gerrit playing tricks on me16:08
andreinyoctozepto: not sure what you mean :-?16:10
*** openstackstatus has joined #openstack-kolla16:15
*** ChanServ sets mode: +v openstackstatus16:15
*** rgogunskiy has joined #openstack-kolla16:17
openstackgerritMark Goddard proposed openstack/kolla master: WIP: move to CentOS 8 in containers  https://review.opendev.org/69236816:19
yoctozeptoandrein: just discussing with mgoddard that that change must have been irrelevant because the variables are defined conditionally in either case16:21
*** rgogunskiy has quit IRC16:22
mgoddardyoctozepto: registered variables are defined even when the task is skipped. You can do 'var is skipped'16:25
yoctozeptomgoddard: ah, right, my mind is not clear today, thanks for reminding me16:25
mgoddardnp16:26
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: Fix 'nova_cephx_raw_key' is undefined  https://review.opendev.org/69593316:33
*** goldyfruit___ has joined #openstack-kolla16:33
*** schwicht has joined #openstack-kolla16:34
yoctozeptoI just added comment and better explanation in commit message16:35
yoctozeptomnasiadka: https://review.opendev.org/695933 nice, simple, short and ceph-related - what else to require for an ideal review for you :-)16:35
patchbotpatch 695933 - kolla-ansible - Fix 'nova_cephx_raw_key' is undefined - 2 patch sets16:35
*** goldyfruit_ has quit IRC16:36
mnasiadkayoctozepto: deprecate ceph!16:36
*** rpittau is now known as rpittau|afk16:40
yoctozeptomnasiadka: done 8-)16:40
yoctozepto(well, joking aside, ceph-ansible seems to be)16:41
*** goldyfruit_ has joined #openstack-kolla16:55
*** goldyfruit___ has quit IRC16:57
mgoddardI found a CentOs 7 to 8 upgrade procedure https://www.tecmint.com/upgrade-centos-7-to-centos-8/16:59
mgoddardI don't think it's supported though16:59
dougszSurely there must be some official procedure - Fedora has had automated upgrades for many cycles now17:02
mgoddardnot for major upgrades, centos claims not to support them17:02
*** ChanServ sets mode: +o mgoddard17:03
*** msalo_ has joined #openstack-kolla17:03
dougsz:( I wonder if RHEL does17:03
mgoddarddougsz: they have leapp, I don't think it supports every case yet17:03
*** ChanServ changes topic to "Testing 1 2 3"17:06
mgoddardit works!17:06
*** msalo has quit IRC17:06
dougsz👍17:07
openstackgerritMerged openstack/kolla-ansible master: CI/Debian: Push images after building  https://review.opendev.org/69424317:09
mgoddardyoctozepto, hrw, mnasiadka: I have given you +o on this channel. Use it wisely17:09
*** ChanServ changes topic to "Kolla IRC meetings on Wednesdays @ 15:00 UTC - see agenda @ https://goo.gl/OXB0DL | Whiteboard: https://etherpad.openstack.org/p/KollaWhiteBoard | IRC channel is *LOGGED* @ http://goo.gl/3mzZ7b"17:11
mgoddardhow's that for a topic?17:11
yoctozepto:D17:14
mnasiadkamgoddard: no op wars? Come on :)17:17
*** msalo has joined #openstack-kolla17:17
*** igordc has joined #openstack-kolla17:18
*** msalo_ has quit IRC17:20
*** schwicht has quit IRC17:20
yoctozeptomgoddard: I don't feel my op17:21
mgoddardyoctozepto: you have it17:21
*** yoctozepto has left #openstack-kolla17:24
*** yoctozepto has joined #openstack-kolla17:24
yoctozeptomgoddard: nah-ah17:24
mgoddardhave you done /msg chanserv op #openstack-kolla17:25
*** ChanServ sets mode: +o yoctozepto17:25
yoctozeptomgoddard: only now17:25
yoctozeptothanks for teaching irc17:25
mgoddardI'm still learning17:25
kplant:O17:25
mgoddardI don't know how to remove it!17:25
yoctozeptoxD17:26
kplantwhat services does freenode use?17:26
mgoddardI imagine it's considered bad form to keep your op on17:26
*** ChanServ sets mode: -o yoctozepto17:27
yoctozeptomgoddard: /cs deop #openstack-kolla17:27
yoctozeptoyears since I learnt irc really but it's coming back to me17:27
yoctozepto:D17:27
kplantdoes it have an in-channel trigger?17:28
kplant!help17:28
openstackkplant: (help [<plugin>] [<command>]) -- This command gives a useful description of what <command> does. <plugin> is only necessary if the command is in more than one plugin.17:28
kplant.help17:28
kplantwrong bot17:28
yoctozeptoif /cs does not work then obviosuly /msg ChanServ17:28
yoctozeptokplant: ChanServ is not a bot to answer in channels17:28
yoctozeptoyou can ask it with /msg ChanServ help17:28
kplantmost irc services do respond in channel17:29
kplant:-\17:29
yoctozeptoprivately :-)17:29
yoctozeptonope17:29
kplantthey don't even have to be in the channel to answer it since they're linked as a server and not a client17:29
yoctozeptoif it responds on the channel17:29
yoctozeptothen it is on the channel17:29
kplantnah17:29
openstackgerritMerged openstack/kayobe stable/rocky: Fix issue with image download when checksum url is none  https://review.opendev.org/69283117:29
*** FlorianFa has quit IRC17:29
kplantservers see all in-channel traffic17:29
kplantservices are linked as ircds17:30
yoctozeptonever heard of any such feature17:30
yoctozepto(answering directly on channel)17:30
yoctozeptoexcept for being on channel ;-)17:30
*** ChanServ sets mode: +o yoctozepto17:30
kplantfor instance that's how opserv (srvx/x3) works by default17:31
*** ChanServ sets mode: -o mgoddard17:31
kplanti _think_ Q (the old quakenet stuff) used to work that way17:31
*** ChanServ sets mode: -o yoctozepto17:31
kplantit's been a while though17:31
yoctozeptohad to deop mgoddard17:31
kplanti wrote a lot of C for irc services though, won't forget things like that17:31
yoctozeptoah, so it could be17:31
yoctozeptoI only ever used freenode17:31
yoctozepto;D17:31
yoctozeptoI believe it is technically possible due to the reasons you mentioned17:32
kplanti used gamesnet/gamesurge/quakenet a lot17:32
kplantand also efnet :)))17:32
yoctozeptowe hosted game chat on freenode :-)17:32
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible stable/train: CI/Debian: Push images after building  https://review.opendev.org/69596217:34
*** schwicht has joined #openstack-kolla17:35
*** primeministerp has quit IRC17:37
openstackgerritMerged openstack/kayobe stable/rocky: Fix validation of locally built IPA when ironic is disabled  https://review.opendev.org/69282817:43
openstackgerritMerged openstack/kayobe stable/stein: Fix issue with image download when checksum url is none  https://review.opendev.org/69283017:43
openstackgerritMerged openstack/kayobe stable/queens: Fix issue with image download when checksum url is none  https://review.opendev.org/69283217:43
openstackgerritMerged openstack/kayobe stable/queens: Fix validation of locally built IPA when ironic is disabled  https://review.opendev.org/69282917:43
openstackgerritMerged openstack/kayobe stable/stein: Fix validation of locally built IPA when ironic is disabled  https://review.opendev.org/69282717:43
*** k_mouza has quit IRC17:47
openstackgerritDoug Szumski proposed openstack/kolla master: Add Elasticsearch Curator Docker image  https://review.opendev.org/69434117:55
openstackgerritDoug Szumski proposed openstack/kolla master: Add Elasticsearch Curator Docker image  https://review.opendev.org/69434117:58
*** watersj has joined #openstack-kolla18:02
watersjhello, what do I need to extend vxlan across networks? use OVS? currently using linuxbridge18:06
*** cah_link has quit IRC18:09
vmixorMy I ask for review of https://review.opendev.org/695931 and https://review.opendev.org/69593718:21
patchbotpatch 695931 - kolla - Vitrage: build images for Debian - 1 patch set18:21
patchbotpatch 695937 - kolla - Horizon: install vitrage dashboard - 1 patch set18:21
openstackgerritDoug Szumski proposed openstack/kolla-ansible master: Support deploying Elasticsearch Curator  https://review.opendev.org/69453718:22
openstackgerritDoug Szumski proposed openstack/kolla-ansible master: Support deploying Elasticsearch Curator  https://review.opendev.org/69453718:24
*** k_mouza has joined #openstack-kolla18:28
openstackgerritDoug Szumski proposed openstack/kolla-ansible master: Support deploying Elasticsearch Curator  https://review.opendev.org/69453718:30
*** skramaja has quit IRC18:32
*** k_mouza has quit IRC18:33
*** cah_link has joined #openstack-kolla18:34
*** dougsz has quit IRC18:36
openstackgerritMichal Nasiadka proposed openstack/kolla-ansible master: CI: Add ansible-lint to tox  https://review.opendev.org/69477918:37
*** igordc has quit IRC18:39
*** msalo has quit IRC18:56
*** msalo has joined #openstack-kolla18:57
*** cah_link has quit IRC18:59
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: [Train] Fix Tacker config for the local Glance store  https://review.opendev.org/69580419:13
*** msalo_ has joined #openstack-kolla19:13
zbr_mnasiadka: you could base your ansible-lint patch on top of https://review.opendev.org/#/c/695888/19:13
patchbotpatch 695888 - kolla-ansible - Call bashate and yamllint via pre-commit - 4 patch sets19:13
*** k_mouza has joined #openstack-kolla19:15
*** k_mouza has quit IRC19:16
*** k_mouza has joined #openstack-kolla19:17
*** msalo has quit IRC19:18
*** rgogunskiy has joined #openstack-kolla19:18
*** vmixor has quit IRC19:19
*** rgogunskiy has quit IRC19:20
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: DNM: Test Tacker upgrade  https://review.opendev.org/69597319:23
*** k_mouza has quit IRC19:32
*** watersj has quit IRC19:45
*** igordc has joined #openstack-kolla19:55
mnasiadkazbr_:Yeah, will look into that tomorrow19:58
openstackgerritMerged openstack/kayobe master: Remove unused IPA config variables  https://review.opendev.org/69352320:07
*** kplant has quit IRC20:37
*** igordc has quit IRC20:47
*** igordc has joined #openstack-kolla20:53
*** schwicht has quit IRC21:24
*** k_mouza has joined #openstack-kolla21:24
*** k_mouza has quit IRC21:28
*** k_mouza has joined #openstack-kolla21:34
*** pcaruana has quit IRC21:38
*** k_mouza has quit IRC21:48
*** msalo has joined #openstack-kolla22:03
*** msalo_ has quit IRC22:07
*** KeithMnemonic has joined #openstack-kolla22:41
*** msalo has quit IRC22:48
*** tonythomas has quit IRC23:10
*** schwicht has joined #openstack-kolla23:45
*** cosmicsound has quit IRC23:45
*** cosmicsound has joined #openstack-kolla23:45
*** dciabrin has quit IRC23:47
*** k_mouza has joined #openstack-kolla23:49
*** k_mouza has quit IRC23:53
*** dciabrin has joined #openstack-kolla23:53
*** schwicht has quit IRC23:54
*** jbadiapa has quit IRC23:59

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