Tuesday, 2017-03-14

openstackgerritAttila Darazs proposed openstack/tripleo-quickstart-extras master: GATE CHECK  https://review.openstack.org/43027700:00
*** marst_ has quit IRC00:02
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo stable/mitaka: Update puppet-midonet in Puppetfile  https://review.openstack.org/44524700:04
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo stable/newton: Stop the chronyd service  https://review.openstack.org/44333300:06
*** dmacpher has joined #tripleo00:09
*** dsneddon has quit IRC00:10
openstackgerritMerged openstack/puppet-tripleo stable/newton: Add bindep support  https://review.openstack.org/44522100:11
*** karimb has joined #tripleo00:14
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Update README for Glance coverage  https://review.openstack.org/44524900:16
*** marst has joined #tripleo00:20
*** bfournie has joined #tripleo00:20
*** bfournie has quit IRC00:21
*** dmacpher has quit IRC00:22
*** karimb has quit IRC00:22
*** bfournie has joined #tripleo00:23
*** tzumainn has quit IRC00:28
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Use sphinx warning-is-error  https://review.openstack.org/44525400:35
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Use sphinx warning-is-error  https://review.openstack.org/44525400:40
*** funzo has joined #tripleo00:46
*** funzo has quit IRC00:50
*** saneax is now known as saneax-_-|AFK00:54
*** limao has joined #tripleo00:55
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Use sphinx warning-is-error  https://review.openstack.org/44525400:57
*** rhallisey has quit IRC00:59
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-heat-templates master: Updated from global requirements  https://review.openstack.org/44012401:00
*** ipsecguy_ has joined #tripleo01:21
openstackgerritMerged openstack/diskimage-builder master: blockdevice.py: python3 fixes  https://review.openstack.org/44475801:22
*** ipsecguy has quit IRC01:23
*** eck` is now known as eck`gone01:25
*** mburned is now known as mburned_out01:25
openstackgerritFeng Pan proposed openstack/os-net-config master: Add support for VPP interface  https://review.openstack.org/42442601:27
openstackgerritMerged openstack/diskimage-builder master: Make our virtualenv source py3 safe  https://review.openstack.org/44471501:35
openstackgerritMerged openstack/diskimage-builder master: squash-package-install to use the correct python  https://review.openstack.org/44333601:35
*** dsneddon has joined #tripleo01:36
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Install packages with roles in multinode  https://review.openstack.org/44368701:39
*** saneax-_-|AFK is now known as saneax01:42
*** funzo has joined #tripleo01:47
*** funzo has quit IRC01:51
openstackgerritMerged openstack/puppet-tripleo stable/ocata: Add bindep support  https://review.openstack.org/44522001:52
openstackgerritMerged openstack/tripleo-heat-templates master: congress/tacker: switch auth_uri to use uri_no_suffix  https://review.openstack.org/44445301:52
openstackgerritMerged openstack/tripleo-heat-templates master: Add bindep support  https://review.openstack.org/44496801:52
*** cwolferh has quit IRC01:54
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-heat-templates master: Updated from global requirements  https://review.openstack.org/44012401:55
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: DONT REVIEW: testing new ovb oooq on branches  https://review.openstack.org/44522701:56
*** sshnaidm is now known as sshnaidm|off01:56
*** oaken_chris has quit IRC01:57
openstackgerritVincent S. Cojot proposed openstack/tripleo-heat-templates master: Fixes multiple issues with retry function in rhel-registration.  https://review.openstack.org/44504502:06
*** fragatina has quit IRC02:19
*** fragatina has joined #tripleo02:20
*** fragatina has quit IRC02:21
*** fragatina has joined #tripleo02:22
openstackgerritAlex Schultz proposed openstack/puppet-tripleo stable/mitaka: Align Stars to fix mitaka CI  https://review.openstack.org/44522202:25
*** fragatina has quit IRC02:26
*** fragatina has joined #tripleo02:29
*** zzzeek has quit IRC02:30
*** zzzeek has joined #tripleo02:31
openstackgerritVincent S. Cojot proposed openstack/tripleo-heat-templates master: Fixes multiple issues with retry function in rhel-registration.  https://review.openstack.org/44504502:31
*** fragatina has quit IRC02:33
*** dhill_ has quit IRC02:37
*** fzdarsky_ has joined #tripleo02:39
*** yamahata has quit IRC02:42
*** fzdarsky has quit IRC02:43
*** saneax is now known as saneax-_-|AFK02:46
*** liverpooler has quit IRC02:48
*** dsneddon has quit IRC02:53
*** sudipto has joined #tripleo02:54
*** sudipto has quit IRC02:56
*** dsneddon has joined #tripleo02:57
openstackgerritMerged openstack-infra/tripleo-ci master: Quickstart transition: replace toci_gate_test.sh  https://review.openstack.org/43102903:20
*** psahoo has joined #tripleo03:37
*** gkadam has joined #tripleo03:43
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Use sphinx warning-is-error  https://review.openstack.org/44525403:45
*** funzo has joined #tripleo03:48
openstackgerritMerged openstack/puppet-tripleo master: Fixes issues with raising mysql file limit  https://review.openstack.org/44375603:49
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Use sphinx warning-is-error  https://review.openstack.org/44525403:50
*** funzo has quit IRC03:53
*** nmathew has joined #tripleo03:53
*** fragatina has joined #tripleo03:58
*** fragatina has quit IRC04:03
*** rlandy has quit IRC04:04
*** fragatina has joined #tripleo04:08
*** fragatina has quit IRC04:08
*** fragatina has joined #tripleo04:09
*** links has joined #tripleo04:34
openstackgerritMerged openstack/puppet-tripleo stable/newton: Stop the chronyd service  https://review.openstack.org/44333304:36
*** funzo has joined #tripleo04:49
openstackgerritChandan Kumar proposed openstack/tripleo-quickstart-extras master: Added tempest_version in validate-tempest role  https://review.openstack.org/44418104:52
*** dmacpher has joined #tripleo04:53
openstackgerritChandan Kumar proposed openstack/tripleo-quickstart-extras master: [DONOT MERGE] test tempest_version patch  https://review.openstack.org/44421104:53
*** funzo has quit IRC04:54
*** udesale has joined #tripleo04:59
openstackgerritChandan Kumar proposed openstack-infra/tripleo-ci master: DON'T MERGE: test tempest all branches  https://review.openstack.org/43692305:01
chandankumarsshnaidm|off: hello05:02
chandankumarsshnaidm|off: we need to fix this review https://review.openstack.org/436923 for tempest all branches05:02
openstackgerritSaravanan KR proposed openstack/tripleo-heat-templates master: WIP: Add RoleName as a new parameter for Services  https://review.openstack.org/43795605:32
*** prateek has joined #tripleo05:33
*** fragatina has quit IRC05:33
*** skramaja has joined #tripleo05:42
*** pgadiya has joined #tripleo05:42
*** fragatina has joined #tripleo05:49
*** funzo has joined #tripleo05:50
*** funzo has quit IRC05:55
*** yprokule has joined #tripleo05:55
*** ukalifon has joined #tripleo06:03
*** george_goh has joined #tripleo06:03
*** masco has joined #tripleo06:06
*** yamahata has joined #tripleo06:14
*** bkopilov has joined #tripleo06:14
*** prateek has quit IRC06:15
*** prateek has joined #tripleo06:16
*** dsneddon has quit IRC06:23
*** iranzo has joined #tripleo06:33
*** iranzo has joined #tripleo06:33
*** saneax-_-|AFK is now known as saneax06:34
*** dsariel has quit IRC06:49
*** funzo has joined #tripleo06:50
*** dparkes has joined #tripleo06:51
*** fragatina has quit IRC06:52
*** funzo has quit IRC06:55
*** lmiccini has joined #tripleo06:58
*** ealcaniz has joined #tripleo07:00
*** ccamacho has quit IRC07:05
*** dmacpher has quit IRC07:09
*** tesseract has joined #tripleo07:20
*** abehl has joined #tripleo07:20
*** cwolferh has joined #tripleo07:22
*** zoli|gone is now known as zoli07:22
*** jlinkes has joined #tripleo07:44
*** jaganathan has joined #tripleo07:45
*** george_goh has quit IRC07:48
*** george_goh has joined #tripleo07:48
openstackgerritAndreas Florath proposed openstack/diskimage-builder master: Refactor: block-device filesystem creation, mount and fstab  https://review.openstack.org/44458607:49
*** pcaruana has joined #tripleo07:50
*** george_goh is now known as george_goh|gone07:58
*** zzzeek has quit IRC08:00
*** ealcaniz has quit IRC08:00
*** zzzeek has joined #tripleo08:00
mandrepradk: I'm now pushing a new image for gnocchi-metricd that has python-redis08:02
*** shardy has joined #tripleo08:04
*** aufi has joined #tripleo08:05
*** cylopez has joined #tripleo08:07
*** florianf has joined #tripleo08:19
*** ccamacho has joined #tripleo08:19
*** jaosorior has joined #tripleo08:24
*** udesale has quit IRC08:24
marios908:24
*** athomas has joined #tripleo08:27
*** lmiccini has quit IRC08:27
*** lmiccini has joined #tripleo08:27
jaosoriormarios: 1008:28
mariosjaosorior: yo uwin08:28
*** lmiccini has quit IRC08:28
*** pmannidi has quit IRC08:29
bandinilol08:29
*** mcornea has joined #tripleo08:29
*** lmiccini has joined #tripleo08:30
*** suuuper has joined #tripleo08:34
*** lmiccini has quit IRC08:34
*** aufi has quit IRC08:34
*** lmiccini has joined #tripleo08:34
*** karimb has joined #tripleo08:36
*** udesale has joined #tripleo08:37
*** yolanda has quit IRC08:37
*** amoralej|off is now known as amoralej08:39
*** yolanda has joined #tripleo08:42
*** jpena|off is now known as jpena08:43
*** aufi has joined #tripleo08:50
*** lmiccini has quit IRC08:50
*** lmiccini has joined #tripleo08:51
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-ui master: Imported Translations from Zanata  https://review.openstack.org/44536408:51
*** sshnaidm|off is now known as sshnaidm08:52
*** funzo has joined #tripleo08:52
*** bogdando has joined #tripleo08:54
*** funzo has quit IRC08:56
*** jpena is now known as jpena|off08:56
*** jpich has joined #tripleo08:58
*** pgadiya has quit IRC08:59
*** udesale has quit IRC09:00
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Create profile to request certificates for the services in the node  https://review.openstack.org/44487309:00
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Remove certificate request bits from service profiles  https://review.openstack.org/44489109:00
*** udesale has joined #tripleo09:01
*** lmiccini has quit IRC09:01
*** lmiccini has joined #tripleo09:01
*** gchamoul|afk is now known as gchamoul09:01
jaosoriorsshnaidm: got time to check this out https://github.com/sshnaidm/sova/pull/12 ?09:02
*** jpena|off is now known as jpena09:03
sshnaidmjaosorior, done, thanks09:04
jaosoriorsshnaidm: thanks dude09:04
chandankumarsshnaidm: hello09:04
sshnaidmchandankumar, hi09:05
chandankumarsshnaidm: we need to fix this review https://review.openstack.org/#/c/436923/ for all tempest branches09:05
*** hewbrocca_afk is now known as hewbrocca09:05
*** lmiccini has quit IRC09:05
chandankumarsshnaidm: for master https://review.openstack.org/444211 this review is working fine09:05
openstackgerritGael Chamoulaud proposed openstack/tripleo-quickstart-extras master: [overcloud-prep-images] Update deprecated openstack command  https://review.openstack.org/44051809:06
*** lmiccini has joined #tripleo09:06
*** gkadam_ has joined #tripleo09:07
*** gkadam has quit IRC09:07
*** gkadam_ is now known as gkadam09:08
*** gkadam is now known as Guest5807609:09
*** snecklifter has joined #tripleo09:09
*** psahoo has quit IRC09:09
*** Guest58076 is now known as gkadam09:09
openstackgerritGael Chamoulaud proposed openstack/tripleo-quickstart master: Fix ansible-lint.sh script for roles and playbooks  https://review.openstack.org/43145609:12
sshnaidmchandankumar, and what is problem with review?09:12
*** lmiccini has quit IRC09:12
*** lmiccini has joined #tripleo09:12
chandankumarsshnaidm: https://review.openstack.org/#/c/436923/  toci_gate_test.sh file got written so got confused that i have done something wrong while rebasing so asked for updating the patch09:13
sshnaidmchandankumar, uh, right, I'll fix it09:14
chandankumarsshnaidm: cool, Thanks!09:14
*** pgadiya has joined #tripleo09:14
*** lucas-afk is now known as lucasagomes09:15
*** aufi has quit IRC09:16
*** lmiccini has quit IRC09:18
openstackgerritArx Cruz proposed openstack/tripleo-quickstart-extras master: Fix stackviz report generation  https://review.openstack.org/44426709:18
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: DONT MERGE: test tempest all branches  https://review.openstack.org/43692309:19
sshnaidmchandankumar, ^09:19
chandankumarsshnaidm: Thanks :-)09:20
jaosoriorjistr, mandre: for a containerized deployment, do you mount the /etc/keystone directory? or do you fetch stuff specifically from it?09:22
*** ealcaniz has joined #tripleo09:24
mandrejaosorior: we use the kolla config mechanism to copy files to the container09:24
mandrejaosorior: https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/keystone.yaml#L70-L10609:24
*** psahoo has joined #tripleo09:26
jaosoriormandre: what if a file isn't available?09:26
*** limao has quit IRC09:27
mandrejaosorior: is the file you need in /var/lib/config-data/keystone/ on the host?09:27
jaosoriormandre: nope, it's the fernet keys09:27
*** aufi has joined #tripleo09:27
mandrejaosorior: how is it created?09:28
jaosoriormandre: if fernet is enabled as a provider09:28
jaosoriorthen the hash containing the fernet keys specification is read by puppet09:28
jaosoriorand it creates the files09:28
mandrejaosorior: yeah, so /var/lib/config-data/keystone/ contains the files that are generated by puppet, and we bind mount this dir inside the container09:29
mandrejaosorior: check that you have what you need in /var/lib/config-data/keystone/ on the host09:30
jaosoriormandre: I'm not sure if I follow09:30
jaosoriormandre: what exactly is put into /var/lib/config-data/keystone/ ?09:30
*** stendulker has joined #tripleo09:30
*** pblaho has quit IRC09:31
mandrejaosorior: /var/lib/config-data/ is where we store the config files for the containers, all the stuff that is generated by puppet09:32
mandrejaosorior: for the containerized tripleo, we're still using puppet to generate the config files09:32
mandrejaosorior: we restrict the catalog to just a few tags (that we can customize in our templates)09:33
*** lmiccini has joined #tripleo09:33
*** tosky has joined #tripleo09:34
jaosoriormandre: ok, so the fernet keys are created with just the file resource from puppet09:35
mandrejaosorior: the way it works, for each service we start a container that runs puppet (restricted to only generate config files with the --tags option), then we copy /etc/ from the container to the host09:35
mandrejaosorior: good, then you shouldn't need any additional puppet tag09:37
mandrejaosorior: if that's the case you should see the fernet keys in /var/lib/config-data/keystone/ on your host09:37
jaosorioroh09:38
jaosoriorThese are the tags that are used09:38
jaosorior          puppet_tags: 'keystone_config,keystone_domain_config,keystone_endpoint,keystone_identity_provider,keystone_paste_ini,keystone_role,keystone_service,keystone_tenant,keystone_user,keystone_user_role,keystone_domain'09:38
jaosoriorso no, that file is not created, if only those tags run09:38
mandrejaosorior: we also use some pre-defined tags that run for all config containers09:39
mandremoment09:39
mandrejaosorior: https://github.com/openstack/tripleo-heat-templates/blob/master/docker/docker-puppet.py#L248-L25109:40
jaosoriorgreat!09:40
jaosoriorso it should work then09:41
*** derekh has joined #tripleo09:42
mandrejaosorior: not necessarily, we need to make sure the containers find the files at the right place09:42
jaosoriormandre, sshnaidm: Any idea why this https://review.openstack.org/#/c/432420/ failed the containers job?  http://logs.openstack.org/20/432420/5/check-tripleo/gate-tripleo-ci-centos-7-ovb-containers-oooq-nv/b52ffb2/ I'm not too versed in parsing that output yet :/09:42
*** Vijayendra has joined #tripleo09:43
*** dbecker has joined #tripleo09:43
sshnaidmjaosorior, http://logs.openstack.org/20/432420/5/check-tripleo/gate-tripleo-ci-centos-7-ovb-containers-oooq-nv/b52ffb2/logs/oooq/undercloud/home/jenkins/overcloud_deploy.log.txt.gz09:43
sshnaidm /etc/keystone/fernet-keys does not exist09:43
*** ckyriakidou has joined #tripleo09:44
sshnaidmjaosorior, http://logs.openstack.org/20/432420/5/check-tripleo/gate-tripleo-ci-centos-7-ovb-containers-oooq-nv/b52ffb2/logs/oooq/undercloud/home/jenkins/failed_deployments.log.txt.gz09:44
mandrejaosorior: so the keys may be in the container under /var/lib/kolla/config_files/src/etc/keystone/, you need to copy them to /etc/keystone using the kolla_config mechanism09:45
*** pblaho has joined #tripleo09:46
*** dsariel has joined #tripleo09:47
*** funzo has joined #tripleo09:53
*** salmankhan has joined #tripleo09:54
jaosoriormandre: got it09:54
openstackgerritChristian Schwede proposed openstack/tripleo-quickstart master: Add customization option to convert_image script  https://review.openstack.org/44538909:56
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: keystone/containers: Add support for fernet keys  https://review.openstack.org/44539109:56
jaosoriormandre: ^^09:56
*** funzo has quit IRC09:57
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo master: Add syncdb to bgpvpn  https://review.openstack.org/44474609:59
*** dtantsur|afk is now known as dtantsur09:59
jistreasy review ready for merging https://review.openstack.org/#/c/443093/10:01
mandrejaosorior: the experimental undercloud containers job runs much quicker (and failed for the same reason in the parent patch)10:02
openstackgerritLuke Hinds proposed openstack/tripleo-heat-templates master: Extends audit serivce  https://review.openstack.org/44480410:06
skramajashardy: for role-specifc parameters, discussed in ptg, https://etherpad.openstack.org/p/tripleo-ptg-nfv-updates, i have thought of 2 options, i have added in the same etherpad line 77.. is option 2 way of providing paramters is OK or any better way to do it?10:08
openstackgerritMerged openstack/tripleo-ui master: Imported Translations from Zanata  https://review.openstack.org/44536410:08
*** pblaho has quit IRC10:10
*** gkadam_ has joined #tripleo10:10
*** gkadam has quit IRC10:13
jaosoriorjistr: done10:14
jistrjaosorior: thanks :)10:15
jaosoriormarios: thanks for the review.10:17
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Create profile to request certificates for the services in the node  https://review.openstack.org/44487310:17
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo master: Remove certificate request bits from service profiles  https://review.openstack.org/44489110:17
*** psahoo has quit IRC10:18
mariosjaosorior: np, i actually started at the tht and am back there now10:18
mandrejaosorior: if you get a minute, can you also review https://review.openstack.org/#/c/442193/?10:20
jaosoriormandre: that commits fine, but I would prefer you didn't use the version-specific plugins...10:21
jaosorioruhm...10:21
jaosoriorI'll upload a patch on top of that one10:21
jaosoriorand merge it10:21
*** b00tcat has joined #tripleo10:21
*** pblaho has joined #tripleo10:23
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-validations master: Use generic keystone plugins instead of version-specific  https://review.openstack.org/44540810:24
jaosoriormandre: ^^10:24
mandrejaosorior: thanks, it was new territory for me10:25
jaosoriormandre: no biggie10:25
*** gfidente has joined #tripleo10:25
*** gfidente has quit IRC10:25
*** gfidente has joined #tripleo10:25
*** zoli is now known as zoli|lunch10:26
openstackgerritMerged openstack/tripleo-validations master: Migrate from keystoneclient to keystoneauth  https://review.openstack.org/44219310:28
*** psahoo has joined #tripleo10:31
jaosoriormandre: where is docker/docker-distribution installed? is that in the undercloud install?10:33
*** athomas has quit IRC10:34
*** dciabrin has quit IRC10:34
jaosoriorjistr: ^^10:36
rnoriegashardy, ping10:38
*** jkilpatr has quit IRC10:39
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Install tripleo-admin ssh keys on CI nodes  https://review.openstack.org/36363010:44
*** dciabrin has joined #tripleo10:46
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Close testenv-client before zuul finishes  https://review.openstack.org/41047010:47
shardyrnoriega: hi10:49
rnoriegashardy, hey! do you have a minute for a quick question?10:49
shardyskramaja: Hi, thanks for the etherpad updates - I was thinking of the Option 1 approach, as it'll work well with map_replace, but option 2 looks OK too10:49
shardyskramaja: I suspect Option 1 may be easier to implement?10:50
shardyrnoriega: sure10:50
*** fzdarsky_ has quit IRC10:50
rnoriegashardy, I'm trying to deploy a baremetal All-in-One with TripleO. Taking a look at how the templates and services are composed... I think it's possible, but don't know yet.10:50
rnoriegashardy, have you tried something similar?? Basically, I modified the roles yaml putting some services from the compute role into the controller section10:51
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: TEST: DONT RECHECK or REVIEW: periodic jobs  https://review.openstack.org/35921510:51
shardyrnoriega: yes it is possible, that's exactly what we test in CI for the multinode jobs10:51
rnoriegashardy, but I'm getting some issues. Do you think it's something doable??10:51
*** fzdarsky has joined #tripleo10:51
shardyrnoriega: sec let me get the environment file link10:51
rnoriegashardy, nice! any doc?? or which services should I move?10:52
rnoriegashardy, perfect10:52
rnoriegashardy, thanks man!!10:52
shardyrnoriega: https://github.com/openstack/tripleo-heat-templates/blob/master/ci/environments/multinode.yaml#L1510:52
*** fzdarsky has quit IRC10:52
shardythat's probably a good starting point10:52
*** fzdarsky has joined #tripleo10:52
*** fzdarsky has quit IRC10:53
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Add undercloud user to docker group  https://review.openstack.org/44542810:53
jaosoriormandre: ^^10:53
rnoriegashardy, looks good! thank youuuuuu! :-D10:53
*** fzdarsky has joined #tripleo10:53
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Close testenv-client before zuul finishes  https://review.openstack.org/41047010:53
*** funzo has joined #tripleo10:54
shardyIIRC you can also just add the NovaCompute/NovaLibvirt services to a normal ControllerServices list, but note there are a couple of ControllerExtraConfig overrides there too10:54
*** akrivoka has joined #tripleo10:54
shardyrnoriega: for baremetal I assume you won't want the qemu ones, but not sure about the rest10:54
skramajashardy: yes.. option1 is easy to implement and it also group all the role specific parameters of a feature together, easy to use... one overhead i thought was we need to pass all the parameters under the role to the service, even though not using.. probably i will write the sample code to implement option 1, then lets conclude.10:54
rnoriegashardy, alright. let me try10:54
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Install tripleo-admin ssh keys on CI nodes  https://review.openstack.org/36363010:55
shardyskramaja: yeah I was thinking we'd add a RoleParameters interface to the service templates, but open to other ideas10:56
*** funzo has quit IRC10:58
*** athomas has joined #tripleo10:59
skramajashardy: exactly..  only overhead is we need to pass all role-specific paramaeters to all services.. let me put this inplace, may be we will get better ideas..10:59
shardyskramaja: ack - I added some notes to the etherpad11:00
shardyskramaja: but I think there's enough here to work on a patch, and perhaps we'll have ideas to refine the interface during the code review11:00
*** oaken_chris has joined #tripleo11:01
skramajashardy: sure. thanks.11:02
openstackgerritMerged openstack/tripleo-heat-templates master: Tasks hook for preparing BM host for deploying containerized services  https://review.openstack.org/44375111:02
EmilienMhello11:03
openstackgerritGael Chamoulaud proposed openstack/tripleo-quickstart-extras master: [gate-quickstart] Fix some wild typos  https://review.openstack.org/44543111:03
*** ckyriakidou has quit IRC11:03
openstackgerritMerged openstack/tripleo-heat-templates master: Update README for Glance coverage  https://review.openstack.org/44524911:04
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Compress logs to tar.gz archive  https://review.openstack.org/43414311:07
jaosoriormandre: this is copied to the container, right? https://review.openstack.org/#/c/445391/1/docker/services/keystone.yaml  not bind-mounted11:09
openstackgerritFlorian Fuchs proposed openstack/tripleo-ui master: Fix blank overcloud credentials  https://review.openstack.org/40813811:10
*** iranzo has quit IRC11:13
openstackgerritLuke Hinds proposed openstack/tripleo-heat-templates master: Extends audit serivce  https://review.openstack.org/44480411:14
jaosoriorlhinds: wow, you've been busy11:15
lhindsjaosorior: :) - want to try and get it all in for Pike11:16
jaosoriorlhinds: added myself as a reviewer11:16
lhindsthanks jaosorior , needs a little work still, but do please comment on anything you see still11:16
bogdandojaosorior: hi. wrt https://review.openstack.org/#/c/445428 , my -1 is just a fromal. Could you help me with finding docs about undercloud_user security implications in oooq?11:17
bogdandotried to google but...11:17
bogdandobtw, which repo oooq docs leaves?11:18
jaosoriorlhinds: so far it looks fine. But then again, it only sets parameters in the auditd manifest. Would need to read those.11:18
bogdandolive*11:18
openstackgerritMerged openstack/tripleo-common master: Add heat-api-cloudwatch container  https://review.openstack.org/44309311:18
jaosoriorbogdando: the existing docs are in the tripleo-quickstart/tripleo-quickstart-extras repo. that's it.11:18
jaosoriorbogdando: I only found the existance of that variable by searching code :/11:19
bogdandothanks I'll read carefully then and reevaluate11:19
bogdandoyeah, but code doesn't explain security contracts :(11:19
*** jkilpatr has joined #tripleo11:19
jaosoriormarios, EmilienM, jistr: got time to check this out https://review.openstack.org/#/c/444850/ ?11:20
jaosoriorbogdando: might want to ask in #oooq11:21
bogdandojaosorior: docker allows users in its group to escalate to root in the host's file system and read 0400 files. That's not a bug, just how it works perhaps11:21
EmilienMjaosorior: what is #oooq?11:21
bogdandoI was pleased to know as well! EmilienM :)11:21
jaosoriorEmilienM: a channel for tripleo-quickstart11:22
EmilienMthat's weird, I thought we agreed to use #tripleo11:22
jaosoriorEmilienM: don't ask me11:22
lhindsbogdando: so is your concern someone in a container, could escalate on the host using the stack / undercloud_user?11:23
mariosjaosorior: will have a look/added11:23
*** udesale__ has joined #tripleo11:24
*** udesale has quit IRC11:24
bogdandolhinds: Yes. And I hoped to get some details from the oooq docs wrt what is allowed to the undercloud_user11:24
jaosoriormarios: that is the base of the ones you were reviewing.11:24
bogdandoon undercloud node11:24
mariosjaosorior: ah cool thanks11:25
bogdandothat's why it is a bad idea to add users into docker group11:25
bogdandoAFAIK, the recommended way is to always run docke cli with sudo11:26
lhindsbogdando: so on the undercloud, the user can already do $sudo su , so they have full control already11:26
bogdandook, thanks. Then we11:26
bogdandoare ok11:26
*** jkilpatr has quit IRC11:27
lhindswell its not ideal, but there are plans to address this..I am looking at some PAM.d management to allow some granualar controls, but at some point, someone will need an account with elevated some sort of sudo wheel leverage11:27
jaosoriorbogdando: https://github.com/openstack/tripleo-quickstart/blob/master/roles/convert-image/templates/convert_image.sh.j2#L3011:28
EmilienMjaosorior: done11:28
EmilienMjaosorior: can you look https://review.openstack.org/#/c/442795/ please?11:28
jaosoriorsure11:28
jaosoriorEmilienM: scenario001 is failing :/ is it related?11:28
jaosoriorah11:28
jaosoriorit's scenario001 from upgrades11:28
jaosoriornevermind11:28
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-common master: Updated from global requirements  https://review.openstack.org/44515411:29
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-heat-templates master: Updated from global requirements  https://review.openstack.org/44012411:29
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-validations master: Updated from global requirements  https://review.openstack.org/44515511:29
*** adarazs is now known as adarazs_lunch11:30
*** gkadam_ is now known as gkadam11:34
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: WIP: Conntainers with multi-nic and net-iso  https://review.openstack.org/42244811:35
lhindsanyone know if its possible to view the build logs for the master overcloud image?11:35
*** mburned_out is now known as mburned11:35
sshnaidmlhinds, in which job?11:35
*** links has quit IRC11:36
lhindsexternal puppet modules set within dib-elements11:36
lhindssshnaidm: ^11:36
mandrejaosorior: we're installing docker-distribution with https://github.com/openstack/instack-undercloud/blob/master/elements/puppet-stack-config/puppet-stack-config.pp#L508-L51011:36
mandrehttps://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/docker_registry.pp11:36
lhindssshnaidm: sorry, wrong project.. tripleo-puppet-elements, not dib-elements11:37
*** gkadam_ has joined #tripleo11:37
sshnaidmlhinds, not sure I understand.. do you mean logs of building image which is used for CI currently?11:38
lhindssshnaidm: yes, the following puppet-module is missing from the image:11:38
lhindshttps://github.com/openstack/tripleo-puppet-elements/blob/910a88f1ef16b63a9428f813b3ce2da918ea3aef/elements/puppet-modules/environment.d/02-puppet-modules-install-types.sh#L811:38
*** fzdarsky is now known as fzdarsky|lunch11:38
mandrejaosorior: the undercloud containers job was successful on https://review.openstack.org/#/c/445391/, looks like you fixed it11:39
jaosorioryay11:39
lhindssshnaidm: its on RDO trunk and is working ok: https://trunk.rdoproject.org/centos7-master/current/11:39
*** jkilpatr has joined #tripleo11:39
*** gkadam has quit IRC11:39
jaosoriormandre: if you have some time, can you review that change then? hopefully we can merge that one and the base one. To keep the containers jobs working.11:40
*** zoli|lunch is now known as zoli11:40
sshnaidmlhinds, http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-ha/b2e6692/logs/undercloud.tar.xz11:41
jaosorioruhm...11:41
sshnaidmlhinds, in this archive, look for /home/jenkins directory, there are images logs11:41
jaosoriorjistr, mandre: Getting this failure when trying to do a containerized overcloud deployment: Error: /Stage[main]/Panko::Db::Sync/Exec[panko-db-sync]: panko-dbsync --config-file /etc/panko/panko.conf  returned 1 instead of one of [0]11:41
*** rhallisey has joined #tripleo11:41
lhindsthanks sshnaidm , hopefully this might tell me whats wrong11:41
*** stendulker has quit IRC11:41
jaosoriorjistr, mandre: Oh... actualle these http://paste.openstack.org/show/602669/11:42
EmilienMsshnaidm: have you looked at promotion failures today yet?11:44
*** pgadiya has quit IRC11:45
sshnaidmEmilienM, failures?11:45
sshnaidmEmilienM, I see it passed..11:45
EmilienMsshnaidm: ah, right. I was wondering why it says 1 day on the dashboard11:45
EmilienMsshnaidm: but your rechecked the patch, so let's see :à)11:46
lhindssshnaidm: "Installed: puppet-tripleo-7.0.0-0.20170314050155.c9acf8a.el7.centos.noarch" - wierd, its completely missing from /etc/puppet/modules, but everything looks ok.11:46
sshnaidmEmilienM, it's slow scripts11:46
sshnaidmEmilienM, actually I rebased it, so we could continue to run it manually11:46
*** nmathew has quit IRC11:46
EmilienMsshnaidm: ok11:46
sshnaidmEmilienM, we merged yesterday big change about transition which changed toci_gate_test.sh, so now every patch that touches it should be rebased11:47
* sshnaidm rebases the whole day :)11:47
*** links has joined #tripleo11:48
jaosoriorsshnaidm: what about jobs that are not yet running over quickstart? how are those run?11:48
sshnaidmjaosorior, right now everything runs as usual11:48
sshnaidmjaosorior, we'll move multinode and scenarios only to oooq today (maybe)11:49
sshnaidmEmilienM, btw, I wanted to raise a point about manual promotion triggering, when we add scenario jobs to promotion pipeline we'll lose option to run the promotion manually11:50
sshnaidmEmilienM, we'll need to change regexps maybe for them11:50
jaosoriorsshnaidm: where are featuresets files set up?11:50
*** liverpooler has joined #tripleo11:50
sshnaidmjaosorior, https://github.com/openstack/tripleo-quickstart/tree/master/config/general_config11:51
dtantsuris it a known breakage? http://logs.openstack.org/86/430886/3/check/gate-instack-undercloud-pep8-ubuntu-xenial/6fc53ce/console.html#_2017-03-14_10_30_38_75531211:52
* dtantsur suspects it's caused by DIB 2.011:52
jaosoriorsshnaidm: and I see that it comes from TOCI_JOBTYPE. is that being set in project-config?11:52
*** zoli is now known as zoli|mtg11:52
*** zoli|mtg is now known as zoli|mtg|afk11:53
jaosoriordtantsur: I've seen that happen for other patches in instack-undercloud11:53
jaosoriordtantsur: at first I thought it was an issue with my patch... but yeah, seems it's affecting others as well11:53
dtantsurDIB 2.0 was a big refactoring.. we may have to fix a few things :)11:54
sshnaidmEmilienM, or to stop promote manually and use a special pipeline in zuul, which is blocked right now by pabelanger: https://review.openstack.org/#/c/443964/11:54
sshnaidmjaosorior, right11:54
jaosoriorshardy: at what point does the image deletion failure happen? after the upgrade?11:54
sshnaidmjaosorior, https://review.openstack.org/#/c/441186/11:54
*** funzo has joined #tripleo11:54
shardyjaosorior: yeah, so I'm suspcious the new configuration isn't completely working11:55
shardyhttp://logs.openstack.org/98/442798/5/check/gate-tripleo-ci-centos-7-multinode-upgrades-nv/9b55866/logs/subnode-2/var/log/glance/api.txt.gz#_2017-03-14_05_19_45_81311:55
shardyglance breaks trying to talk to keystone AFAICS11:55
jaosoriorsshnaidm: thanks, makes sense now11:55
*** jaganathan has quit IRC11:56
jaosoriorshardy: right, that should be fixed by this https://review.openstack.org/#/c/443104/411:57
jaosoriorshardy: that commit is in the depends-on: But is not being taken into account for some reason :/11:57
*** ckyriakidou has joined #tripleo11:57
shardyugh, probably more n-v-r related excitement :(11:57
*** liupe__ has joined #tripleo11:58
EmilienMsshnaidm: I see, interesting11:58
*** liupe__ has quit IRC11:58
jaosoriordtantsur: dammit :/11:58
*** pkovar has joined #tripleo11:58
*** funzo has quit IRC11:58
jaosoriorshardy: either way, that t-h-t commit is blocked by the two commits it depends-on. So no biggie. We can just wait until those merge, then wait for a promotion, and finally recheck it11:59
EmilienMjaosorior: if your patch merges without us getting a promotion is it going to break CI?11:59
EmilienMjaosorior: we might wait a bit and do recheck without the depends-on once the dependencies land12:00
jaosoriorEmilienM: which one?12:00
EmilienMto make sure we don't break12:00
jaosoriorEmilienM: my patches for glance and swift are safe12:00
shardyjaosorior: yeah, unless we still install the wrong version until swiftclient tags a new release12:00
EmilienMjaosorior: https://review.openstack.org/#/c/442798/12:00
shardyI guess we'll see12:00
*** gkadam__ has joined #tripleo12:01
jaosoriorEmilienM: yeah, it'll break CI if no promotion happens and that's merged.12:01
jaosoriorEmilienM: I added a -W12:01
EmilienMjaosorior: thanks12:01
*** ealcaniz has quit IRC12:01
jaosoriorEmilienM: let me know when you can talk about key rotation12:02
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates master: Containerize panko api service  https://review.openstack.org/43913812:02
*** gkadam_ has quit IRC12:03
*** liverpooler has quit IRC12:04
*** liverpooler has joined #tripleo12:04
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo master: Add syncdb to bgpvpn  https://review.openstack.org/44474612:04
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Add undercloud user to docker group  https://review.openstack.org/44542812:04
*** morazi has joined #tripleo12:05
*** oaken_chris has quit IRC12:09
*** saneax is now known as saneax-_-|AFK12:10
jaosoriormandre: did I have to add the docker-network.yaml environment to my overcloud deploy?12:10
*** jcoufal has joined #tripleo12:11
*** trown|outtypewww is now known as trown12:15
openstackgerritGabriele Cerami proposed openstack-infra/tripleo-ci master: [WIP] Quickstart transition: fallback to old method on stable branches  https://review.openstack.org/44546112:16
trownEmilienM: can you look at https://review.openstack.org/441186 ... I doubt infra will merge something like that without PTL vote (rightfully so)12:17
*** [1]cdearborn has joined #tripleo12:17
*** ckyriakidou has quit IRC12:17
*** oaken_chris has joined #tripleo12:18
*** ckyriakidou has joined #tripleo12:22
openstackgerritMerged openstack/puppet-tripleo master: Correct haproxy's stat unix socket path  https://review.openstack.org/44314812:23
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Install packages with roles in multinode  https://review.openstack.org/44368712:23
*** adarazs_lunch is now known as adarazs12:26
jaosoriorjistr: this addresses the fernet issue https://review.openstack.org/#/c/445391/12:28
openstackgerritGabriele Cerami proposed openstack-infra/tripleo-ci master: [WIP] quickstart transition: add promotion logic  https://review.openstack.org/44546712:29
jaosoriorjistr: for containers12:32
jistrthanks! looking12:32
EmilienMquick reminder for our friends in US / Canada (and all who passed to DST last week-end): tripleo meeting is in 1h30 and NOT in 30 min12:33
*** ansmith has joined #tripleo12:33
openstackgerritBrad P. Crochet proposed openstack/tripleo-common master: Implement verify profiles as a custom action  https://review.openstack.org/42968412:35
openstackgerritBrad P. Crochet proposed openstack/tripleo-common master: Implement check ironic boot configuration as a custom action  https://review.openstack.org/42822512:35
openstackgerritBrad P. Crochet proposed openstack/tripleo-common master: Implement check flavors as a custom action  https://review.openstack.org/42817812:35
openstackgerritBrad P. Crochet proposed openstack/tripleo-common master: Implement check node counts as a custom action  https://review.openstack.org/42975712:35
openstackgerritBrad P. Crochet proposed openstack/tripleo-common master: Implement check boot images as a custom action  https://review.openstack.org/42817712:35
jaosoriorjistr: and that commit is on top of the one that sets fernet as the default provider. So it's testing it as well.12:35
*** oaken_chris has quit IRC12:36
jistrjaosorior: yea i was just going to say that the patches would need to be in reverse order to fully pass CI on both12:36
jaosoriorjistr: or merge at the same time ;)12:36
*** rlandy has joined #tripleo12:36
jaosoriorjistr: dude, I tried to deploy a containerized overcloud. But it seems that the non-containerized services are having issues connecting to the database. which, from what I understood, is containerized as well.12:37
jistri guess... :D idk personally i'm not too fond of doing this kind of merging unless we have to12:38
jistrhmm it runs in host networking mode so it should be reachable12:38
EmilienMtrown: looking12:38
jaosoriorjistr: did I need to add the docker-network.yaml environment?12:39
jaosorioronly added the docker.yaml one12:39
jistrjaosorior: i don't think so, i haven't been using docker-network.yaml either12:39
*** salmankhan has quit IRC12:39
*** salmankhan has joined #tripleo12:39
jistrjaosorior: is there a difference between the IPs that the containerized and non-containerized services try connecting to?12:40
*** jaganathan has joined #tripleo12:40
EmilienMtrown: I'm not sure multinode-oooq jobs are really stable: https://review.openstack.org/#/c/445391/12:40
EmilienMtrown: are they really ready to vote?12:42
arxcruzchandankumar: hey, around ?12:42
jaosoriorjistr: where do I check for the containerzed noes?12:42
chandankumararxcruz: yes12:42
panda|zZEmilienM: those jobs are using the old scripts12:42
jistrjaosorior: you can enter the container context by `docker exec -ti <container name> bash`12:43
EmilienMpanda|zZ: ah ok12:43
*** eck`gone is now known as eck`12:43
jistrjaosorior: and then you'll find configs etc. in usual places12:43
EmilienMpanda|zZ: are you confident for https://review.openstack.org/#/c/441186 ?12:43
EmilienMpanda|zZ: where can I see the results with new scripts?12:43
trownEmilienM: I thought so12:44
jaosoriorjistr: they have the same IP12:45
jaosoriorjistr: I tried connecting manually and it seems mariadb is not responding12:45
arxcruzchandankumar: cinder volume api v1 is deprecated since kilo, is there a way in configure-tempest to put it as False by default?12:45
arxcruzor should I change the configure-tempest.sh in oooq to add that as false?12:46
chandankumararxcruz: you need to change in configure-tempest.sh and i need to fix it in python-tempestconf12:47
*** masco has quit IRC12:48
chandankumararxcruz: that reminds me i need to use tempest verify-config in ci once tempest conf is generatted12:49
*** dprince has joined #tripleo12:49
arxcruzchandankumar: okay, so just add volume_feature_enable.api_v1=false right ?12:49
openstackgerritPradeep Kilambi proposed openstack/tripleo-common master: Add redis images to overcloud containers  https://review.openstack.org/44217912:49
EmilienMtrown: I would +1 once I see the results of the jobs somewhere12:50
trownpanda|zZ: ^12:50
panda|zZEmilienM: it's complicated, test history on the new script is all here https://review.openstack.org/431029, patchset 102 is the iteration from where all started working with master, then we started testing with stable branches. the tested job are renamed and repurposed gate-tripleo-ci-centos-7-nonha-multinode and gate-tripleo-ci-centos-7-undercloud are correclty named,12:50
panda|zZgate-tripleo-ci-centos-7-nonha-multinode-updates-nv tests scenario001 and gate-tripleo-ci-centos-7-multinode-upgrades-nv tests scenario002, which is working. THe rest of the iteration have mixed results, but what I did was polling zuul status page, and upload new patchsets when something failed, so there's no recorded history of how much each job really passed.12:50
saishardy: dprince bnemec trying to push triple limits by deploying 128 nodes at once seeing, Networking client is experiencing an unauth12:51
jaosoriorjistr: should I be able to see the IPAddress in the NetworkSettings for the mysql container?12:51
saiorized exception. (HTTP 400)12:51
saiwhen trying to build some computes12:51
jaosoriordprince: ^^12:51
saihttps://gist.github.com/smalleni/ebcd22ae344c397a8a50f79649cae6b412:51
*** panda|zZ is now known as panda12:51
jistrjaosorior: what do you mean NetworkSettings? Something that `docker inspect` spits out?12:52
*** links has quit IRC12:52
jaosoriorjistr: yeah12:52
jistrjaosorior: i don't think so as we're not assigning any IPs to the containers, they reuse host network12:52
sailooks reelated but not sure https://bugs.launchpad.net/nova/+bug/165605612:52
openstackLaunchpad bug 1656056 in OpenStack Compute (nova) "NeutronAdminCredentialConfigurationInvalid in n-cpu logs" [High,Confirmed]12:52
EmilienMpanda: ok, so please vote on https://review.openstack.org/#/c/441186/612:53
*** links has joined #tripleo12:53
jaosoriorjistr: how do I get the logs for mariadb? can't access them via the container :/12:53
*** amoralej is now known as amoralej|lunch12:53
jaosoriorjistr: nevermind, EBKAC12:54
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo stable/ocata: Correct haproxy's stat unix socket path  https://review.openstack.org/44547912:54
chandankumararxcruz: you will be passing as a argument volume_feature_enable.api_v1=false to python-tempestconf or are you doing sed in tempest.conf?12:54
jistrok :) just for the sake of general info -- i don't recall how the logging is set up for mariadb in particular, but it's either `docker logs <container name>` or you can do the `docker exec` i wrote above and navigate to /var/log12:54
*** tzumainn has joined #tripleo12:54
arxcruzchandankumar: jusg a sec12:55
dprincejaosorior: we use net=host. Other than that what network IP are you looking for with MySQL?12:55
jaosoriorjistr: docker logs shows journalctl, however, I did need to go into /var/l;og12:55
arxcruzjust*12:55
*** lucasagomes is now known as lucas-hungry12:55
jaosoriorjistr, dprince: Anyway, for some reason both the containerized services and the non-containerized services are pointing to the wrong IP for mariadb12:55
dprincesai: is the undercloud keystone server failing over?12:55
*** funzo has joined #tripleo12:55
*** Goneri has joined #tripleo12:55
openstackgerritwes hayutin proposed openstack/tripleo-quickstart master: update localhost instructions to indicate 127.0.0.2 is required  https://review.openstack.org/44521312:55
jistrFYI the gate is broken on pep8 related to container templates (different issue than dib)12:56
dprincejaosorior: is this with an undercloud? or overcloud setup?12:56
jistrhttps://bugs.launchpad.net/tripleo/+bug/167270812:56
openstackLaunchpad bug 1672708 in tripleo "pep8 job broken on docker/heat-api-cloudwatch yaml" [Critical,Triaged] - Assigned to Jiří Stránský (jistr)12:56
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates master: Add ceilometer ipmi agent  https://review.openstack.org/43044712:56
jaosoriordprince: overcloud.12:56
jistrshould be a low hanging fruit, will send a patch12:56
openstackgerritMerged openstack/tripleo-quickstart-extras master: Clone tripleo-quickstart dependant change to tripleo-quickstart-gate-repo  https://review.openstack.org/43176012:56
dprincejaosorior: did something change in the endpoint map?12:57
*** jayg|g0n3 is now known as jayg12:57
*** fzdarsky|lunch is now known as fzdarsky12:57
dprincejaosorior: the VIP is wrong then? or the local bind IP?12:57
jaosoriordprince: nothing has changed AFAIK12:58
jaosoriorI'll try to debug it further.12:59
*** funzo has quit IRC12:59
saidprince: keystone just using 2 cores, doesnt look like abottleneck13:00
jaosoriordprince, jistr: nevermind, the endpoints are right. Just got confused with the VIP... however, the services still can't communicate with mariadb. When I attempt to log into it I get the following error:13:00
jaosoriorERROR 2013 (HY000): Lost connection to MySQL server at 'reading initial communication packet', system error: 0 "Internal error/check (Not system error)"13:00
jistrmeh forget what i posted about gate broken, it's an unmerged patch ;) there was more patches related to heat-api-cloudwatch and some of them got merged, the broken one didn't though13:00
*** pkovar has quit IRC13:00
*** psahoo has quit IRC13:01
openstackgerritwes hayutin proposed openstack/tripleo-quickstart master: update localhost instructions to indicate 127.0.0.2 is required  https://review.openstack.org/44521313:01
*** lblanchard has joined #tripleo13:01
dprincejaosorior: firewall rules?13:01
jaosoriordprince: no idea man. I just deployed my environment and haven'13:01
jaosoriorhaven't changed a thing13:01
jaosoriorso this is just a standard tripleo-quickstart deploymeny, using the latest images, and I changed nothing :/13:02
*** tzumainn has quit IRC13:03
*** ealcaniz has joined #tripleo13:03
*** chlong has joined #tripleo13:04
openstackgerritAttila Darazs proposed openstack/tripleo-quickstart master: Update ci-script to enable TripleO gate dependencies  https://review.openstack.org/43176413:05
jaosoriordprince: not a firewall issue. Just flushed all my iptables rules, and set it to accept everything (to make sure) and it still has the same issue :/13:06
*** jmelvin has joined #tripleo13:06
jaosoriordprince: anyway, removed the stack and gonna try it again, maybe it's transient... But hey, thanks for checking out the fernet patch. Got into it and here's the fernet support for the containers profile https://review.openstack.org/#/c/445391/13:08
dprincejaosorior: oh cool, I'll try and cherry-pick that guy and give him a spin too13:09
jaosoriorgreat!13:09
*** fultonj has joined #tripleo13:11
*** gbarros has joined #tripleo13:12
*** pkovar has joined #tripleo13:15
*** tzumainn has joined #tripleo13:15
jistrmandre: whoops i didn't know you were editing the patch at the same time13:17
mandrejistr: hmm, interesting... we managed to push a patch at the same time and confused gerrit :)13:17
jistrmandre: i'll revert to what you pushed13:17
pandaEmilienM: voted13:18
*** zoli|mtg|afk is now known as zoli13:19
EmilienMpanda: thx13:19
EmilienMdprince: would it be possible to have you or someone from container squad looking at https://review.openstack.org/#/c/387631 before we approve this spec?13:19
dprinceEmilienM: I don't think it impacts containers. By doing that we are basically saying TripleO's deployment architecture (containers or not) is no longer responsible for setting up Ceph13:21
*** trown is now known as trown|brb13:21
EmilienMdprince: ok, please bless the spec then13:21
jistrmandre: indeed i think i broke gerrit :D it gives me13:21
jistrCouldn't find remote ref refs/changes/95/443095/213:21
dprinceEmilienM: the minor point is that Ceph-Ansible also uses containers no... which is cool as far as isolation goes13:21
jistrmandre: do you still have your version locally?13:21
jistrmandre: if you do, i think you could just do `git commit --amend` so that it gets a newer timestamp (and a different hash), and then run `git review` again, and we'd get a correct patch set 4 hopefully13:22
dprinceEmilienM: by ignoring it I am blessing it. I don't like this idea :)13:23
mandrejistr: I do, let me try to fix it13:23
dprinceEmilienM: if you ask me to look at it closely you might be asking for trouble :/13:23
*** skramaja has quit IRC13:23
* dprince doesn't like stringing together installers much13:24
mandrejistr: FWIW, my `git review` command for PS2 did push something but returned a failure13:24
EmilienMdprince: I'm asking you to look at it :)13:24
EmilienMdprince: please13:24
jaosoriordprince: are the containers steps run at the same time as the puppet steps for regular services?13:25
*** prateek has quit IRC13:25
jistrmandre: yea that's maybe why the git ref refs/changes/95/443095/2 doesn't exist... i can see ps2 in the UI though, interesting13:25
fultonjon line 322 of the spec it describes the containerization of conf files to be passed up and I want it to coexist happily with docker-puppet.py13:25
dprincejaosorior: the containers for a given step number always run after the baremetal step13:25
dprincejaosorior: http://git.openstack.org/cgit/openstack/tripleo-heat-templates/tree/docker/services/README.rst#n10913:26
mandrejistr: the problem might be on my side, 'git review' is taking forever13:26
jistrmandre: whohoo there's PS413:27
mandreit has pushed a new changeset but it's still going on13:27
mandreand no irc notification13:27
*** trown|brb is now known as trown13:27
jaosoriordprince: at what point do these files in the containers config_files (in kolla_config) get copied into the container? is that after the steps?13:27
mandrejistr: internal server error13:27
mandre:/13:27
dprincejaosorior: that happens when each container is started13:28
dprincejaosorior: the config files are generated (via puppet running in the containers) before any of the stepwise deployment even starts though13:28
jaosoriorgreat13:28
jaosoriordprince: thanks13:29
dprinceEmilienM: I will review it then13:29
jistrmandre: someone just mentioned problems with gerrit on #openstack-infra13:29
EmilienMdprince: thank you13:29
fultonjdprince: thank you13:29
jistrmandre: so i wonder if we hit a breakage or *were* the breakage :D13:30
mandrejistr: ah!13:30
mandrejistr: the review disapeared :)13:30
*** zoli is now known as zoli|FBCI13:30
jistrmandre: and now gerrit disappeared13:31
mandreboom13:31
* mandre quickly hides13:32
*** artom has quit IRC13:32
jistrmandre: parallel gerit editing => <insert a few more steps> => end of the world13:32
beaglesoh gerrit, don't you fail me now!!!!13:33
beaglesfigures, when you finally get around to reviewing a spec13:33
*** zoli|FBCI is now known as zoli13:37
mariosshardy: hey i spent some time thinking about l3 agent and batch upgrade tasks today (jarda mentioned you spoke with him about it).. the trouble is we don't run puppet at that point so we have to manage the service start as well.13:38
mariosanyone else having issues with gerrit btw?13:38
trownmarios: yep13:38
trownmarios: it decided to take a nap13:39
mariosshardy: i just poked at the l3 agent manifest especially and have a WIP to get out for discussion, as soon as gerrit lets me13:39
mariosshardy: is simple to start with (only l3 agent, not sure if that is enough yet, i need to test it) and i still don't know if we can do the service starts from ansible without problems (you mentinoed you already tried this in early iterations and its one of the reasons we decided to let puppet do it)13:39
mariosthanks trown13:40
shardymarios: hey, yeah I looked at it, but currently /usr/bin/neutron-l3-agent is owned by the openstack-neutron package, which contains scriptlet stuff to restart services13:40
dtrainormarios, i am13:40
shardymarios: and yeah, if any deprecated parameters are removed etc from neutron, we then need puppet to reconfigure things before the service start will work13:41
mariosshardy: hmm (yeah my review has the yum update for openstack-neutron services) ... so restarting services is OK13:41
mariosshardy: i mean, as long as we do batch upgrade tasks.13:41
openstackgerritMerged openstack/tripleo-common master: Handle unprovisioned Ironic nodes in fencing parameter generator.  https://review.openstack.org/44441913:41
mariosshardy: the idea is even if we stop the l3 agent on one node, the floating ips can fail over to another l3 agent on another controller13:41
openstackgerritMerged openstack/puppet-tripleo stable/mitaka: Update puppet-midonet in Puppetfile  https://review.openstack.org/44524713:41
shardymarios: but we'll restart all neutron services, which as you say won't always work until after puppet configures things13:41
shardymarios: I was hoping we could do a cleaner update of only the l3 agent, but it doesn't look that simple unfortunately13:42
mariosshardy: right this is one of the things i call out in the commit... ok thanks i'll get that out for the discussion anyway and tkae it from there? if nothing else we can point at it to say this is why we can't do it13:42
-openstackstatus- NOTICE: Gerrit is going to be restarted due to performance problems13:42
*** ChanServ changes topic to "Gerrit is going to be restarted due to performance problems"13:42
mariosthanks dtrainor13:42
*** jpena is now known as jpena|lunch13:43
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates master: WIP - Add upgrade_batch_tasks to neutron-l3-agent  https://review.openstack.org/44549413:43
shardymarios: ack, sounds good13:44
b00tcatHi, maybe it's too broad of a question but: if I want my services to be pacemaker-compatible, is there anything I have to do?13:44
mandrejistr: everything seems in order now13:44
b00tcatthe chapter in tripleo.org about HA is a little bit short13:44
shardymarios: I did wonder if we could just special-case the l3 agent here, e.g do an rpm update disabling the scriptlets, and bounce only that service13:44
mariosshardy: lots of unkowns really but it would be great if we could13:45
shardymarios: that's not super-clean, but it might be a stepping stone we can use until containers land for all-the-things13:45
jistrmandre: the git repo that backs gerrit is still broken probably13:45
jistrmandre: i can't fetch the change13:45
mandrejistr: lemme try to push an new patch13:45
shardyb00tcat: hey, in general we're moving away from pacemaker mangaging most things, what's the use-case?13:46
mariosshardy: besides the n/o config issue potentially, the other major thing right now is can we run l3 agent with /for existing routes/ips by itself without the server (maybe this means you can't create new resources). or we cannot in which case we need to revisit that ...13:46
bandinimarios, matbu: ever seen this one http://logs.openstack.org/28/444928/1/check/gate-tripleo-ci-centos-7-multinode-upgrades/5f0c98a/logs/postci.txt.gz#_2017-03-14_12_42_58_00013:47
openstackgerritMartin André proposed openstack/tripleo-heat-templates master: Docker service for heat-api-cloudwatch  https://review.openstack.org/44309513:47
bandiniseems httpd restart is failing on upgrades (this is stable/ocata and it happened twice to me already)13:47
EmilienMmeeting in 15 min, feel free to add topics https://etherpad.openstack.org/p/tripleo-meeting-items13:47
mariosbandini: no haven't (httpd can't restart?)13:48
b00tcatshardy: I'm going to deploy a several-controllers, and after reading https://docs.openstack.org/developer/tripleo-docs/advanced_deployment/high_availability.html I assumed pacemaker was the only option to achieve HA13:48
-openstackstatus- NOTICE: Gerrit has been successfully restarted13:49
*** ChanServ changes topic to "Gerrit has been successfully restarted"13:49
jaosoriorbandini: I've seen it... and it's been transient :/13:49
bandinijaosorior: ah thanks. I will dig some more then13:49
EmilienMdo we know why it fails?13:49
bandinino, not yet. will investigate now13:49
shardyb00tcat: in recent releases only a few services are actually managed by pacemaker, mostly we now rely on systemd to restart services and pacemaker only manages the vips13:49
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: WIP:Parameterize OVB stack flavors  https://review.openstack.org/44516313:50
EmilienMbandini: have you seen it in scenarios upgrades too?13:50
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: WIP:Parameterize OVB stack flavors  https://review.openstack.org/44516313:50
bandiniEmilienM: I don't think so, no13:50
bandinilemme dig some more13:50
openstackgerritVincent S. Cojot proposed openstack/tripleo-heat-templates master: Fixes multiple issues with retry function in rhel-registration.  https://review.openstack.org/44504513:50
EmilienMbandini: I just want to see if it's horizon or something else13:51
bandiniEmilienM: yeah I could not tell from a quick look at the httpd logs13:51
*** yprokule_ has joined #tripleo13:52
EmilienMack13:52
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates master: Keep existing data when moving to containerized MariaDB  https://review.openstack.org/44114613:52
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates master: Keep existing data for containerized MongoDB  https://review.openstack.org/44246913:53
*** lucas-hungry is now known as lucasagomes13:53
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates master: Keep existing data for containerized RabbitMQ  https://review.openstack.org/44255713:53
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates master: WIP - Add upgrade_batch_tasks to neutron-l3-agent  https://review.openstack.org/44549413:53
*** yprokule has quit IRC13:54
*** yprokule_ is now known as yprokule13:54
*** ChanServ changes topic to "Gerrit is going to be restarted due to performance problems"13:55
-openstackstatus- NOTICE: Gerrit has been successfully restarted13:55
*** funzo has joined #tripleo13:56
EmilienMmeeting in 5 min13:56
b00tcatshardy: so how does pacemaker know which services need to be managed through the VIP?13:56
mandrewho wants to do an easy review? https://review.openstack.org/#/c/442194/13:57
trowni like easy reviews :P13:57
*** links has quit IRC13:57
EmilienMmandre: can you update RDO packaging too?13:58
mandreEmilienM: will do13:58
mwhahahai haven't looked yet but do we have a bug for the instack-undercloud pep8 failures yet (something about dib)13:58
EmilienMmwhahaha: the bindep patch?13:59
mwhahahano dib-lint is broken too13:59
mwhahahahttp://logs.openstack.org/76/441776/2/gate/gate-instack-undercloud-pep8-ubuntu-xenial/47c9b5e/console.html#_2017-03-14_12_43_15_32436013:59
EmilienMoops13:59
* mwhahaha goes and creates an LP bug14:00
openstackgerritMerged openstack/tripleo-validations master: Add missing mistralclient requirement  https://review.openstack.org/44219414:00
*** funzo has quit IRC14:00
*** jaganathan has quit IRC14:01
*** gkadam__ has quit IRC14:01
shardyb00tcat: in general the pattern is we create a vip per network, and you assign services to a network via ServiceNetMap:14:02
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/network/service_net_map.j2.yaml14:02
shardyb00tcat: so, for most services, you just need a ServiceNetMap or ServiceNetMapDefaults entry mapping service name to network name14:02
b00tcatshardy: I have that, but my VIP is not listening in some of my custom services ports14:04
b00tcatthat's why I think I'm missing something14:04
*** aufi has quit IRC14:04
*** dmacpher has joined #tripleo14:04
openstackgerritmathieu bultel proposed openstack/tripleo-quickstart-extras master: add missing skip case for extra environment for composable scenario  https://review.openstack.org/44550814:04
*** gbarros has quit IRC14:06
shardyb00tcat: Have you seen the pattern for specifying the service bind IP https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/glance-api.yaml#L17314:07
shardywe replace the network name with the IP for the appropriate network on each node14:07
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/glance-api.yaml#L15814:07
shardyand that shows how firewall rules are managed14:07
*** amoralej|lunch is now known as amoralej14:08
b00tcatshardy: ok I was missing the service bind IP part, thanks a lot! :-)14:09
*** ooolpbot has joined #tripleo14:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION14:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273014:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,Confirmed]14:10
*** ooolpbot has quit IRC14:10
sshnaidmdtantsur, hi, can you please take a look? https://review.openstack.org/#/c/441706/14:11
*** toure|gone is now known as toure14:13
jaosoriorjistr, dprince: So, I redeployed my undercloud, and the issue connecting to mariadb appears again. I can actually connect by logging into the container, and also by using the specific ip address it actually binds to. However, I can't connect via haproxy for some reason14:14
*** saneax-_-|AFK is now known as saneax14:17
jistrah interesting... so maybe haproxy is misconfigured? But i wonder how come all of a sudden. As far as i've seen, the overcloud CI job seemed to be passing very recently. I'll try to redeploy locally too14:17
jistrjaosorior: fwiw this is how i deploy:  openstack overcloud deploy     --libvirt-type qemu     --ntp-server clock.redhat.com     --control-scale 1     --templates $THT     -e $THT/environments/low-memory-usage.yaml     -e $THT/environments/docker.yaml14:18
mariosmcornea: around? https://bugs.launchpad.net/tripleo/+bug/1661347 is this still a problem?14:18
openstackLaunchpad bug 1661347 in tripleo "During Newton->Ocata upgrade pattern is ignored in the 'Stop Ceph Mon' task in ceph-mon upgrade" [Critical,Triaged]14:18
mcorneamarios: hey, I'm not sure but I haven't seen any trouble caused by it so it can be considered low prio14:19
jaosoriorjistr: that looks quite similar to what I do :/14:19
mariosmcornea: nm... thanks it just came u pin weekly irc and shardy found it at https://review.openstack.org/#/q/I0a0e60d277240130c6bd76a74ccc13354b87a30a14:20
jistrjaosorior: including deployment with just 1 controller? What we have currently doesn't have pacemaker, so services like mariadb don't do HA14:20
jaosoriorjistr: yep, just one controller14:20
*** aufi has joined #tripleo14:20
jaosoriorah14:21
jaosoriorjistr: wait up14:21
jaosoriorjistr: fuck, so I was enabling pacemaker there14:21
jistrjaosorior: yea that would be it :)14:21
jaosoriorjistr: but network isolation should work, right?14:23
jistrjaosorior: hmm.. i think so, i'm not sure if we're CIing containers on it presently14:23
jaosoriordprince: ^^14:24
dprincejaosorior: I've not tried that yet. mandre set up the overcloud CI job and has done more local testing w/ overcloud contaienrs at this point. you might ask him14:25
jistrso i went to check it and boom https://github.com/openstack-infra/tripleo-ci/blob/master/toci_gate_test.sh14:25
jistr:) gotta explore the new CI14:25
Nghmm, am I right in remembering that there is a way to log into a Triple CI node while it's working, to debug weird failures?14:26
Ngtrying to help bandini with a very odd CI failure that seems to involve apache failing to be restarted by puppet, but the logs don't seem to actually have any apache errors, just normal looking output14:27
d0ugalthrash: does the general approach in https://review.openstack.org/#/c/406178 seem okay?14:27
d0ugalthrash: it is kinda messy, ideally we would re-write it to be non-blocking but that would be much more work.14:27
*** gbarros has joined #tripleo14:28
thrashd0ugal: yes... I agree with the general approach... -1 is about lack of tests for workflows/base.py14:29
openstackgerritArx Cruz proposed openstack/tripleo-quickstart-extras master: Disable cinder v1 api from tempest  https://review.openstack.org/44551614:30
d0ugalthrash: cool thanks, I'll work on adding some.14:30
jistrjaosorior: fwiw CI is not using net iso -- http://logs.openstack.org/29/431029/109/check-tripleo/gate-tripleo-ci-centos-7-ovb-containers-oooq-nv/0efc74e/logs/oooq/undercloud/home/jenkins/overcloud-deploy.sh14:31
*** dhill_ has joined #tripleo14:31
*** b00tcat has quit IRC14:31
*** b00tcat has joined #tripleo14:32
jaosoriorjistr, dprince, mandre: Have you guys tried out the containers work with net-iso?14:32
openstackgerritAlexander Chuzhoy proposed openstack/tripleo-docs master: Booting BM instance with cell_v2  https://review.openstack.org/44262714:33
jistri haven't tested that myself14:33
*** dparkes has quit IRC14:33
dprincejaosorior: no14:35
dprincemandre: hi, we were just talking about the need to backport this to Mitaka? I think weshay had a question about this in the #openstack-meeting this week14:36
dprincemandre: https://review.openstack.org/#/q/I89e14cc2a27299ce4c191d2a823deb0424693831,n,z14:36
dprinceweshay: do you know why this is needed again? Mitaka seems quite old at this point to add this change14:37
*** shardy has quit IRC14:37
*** shardy has joined #tripleo14:37
mandredprince: yeah, I volunteered to backport, or at least do the necessary to fix stable/mitaka14:37
*** dparkes has joined #tripleo14:37
weshaydprince, I think we're one month or so from dropping support.  The RDO guys, aka.. apevec and crew are pushing to get it fixed14:37
mandrejaosorior: yep, net-iso works for me14:38
jaosorioryay14:38
weshayamoralej, has some patches for it as well14:38
mandreweshay: so no need to backport anything?14:39
*** aufi has quit IRC14:40
weshayif it's the same root cause imho we might as well fix it if apevec and crew need it14:41
weshayhttp://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-ha-mitaka/2cdca52/console.html14:41
weshayI didn't think it was the same root cause, so I'm looking14:41
mandreweshay: an easy fix would be to change the default value for enable_docker_registry from true to false14:41
mandrehttps://github.com/openstack/instack-undercloud/blob/stable/mitaka/elements/puppet-stack-config/puppet-stack-config.pp#L46614:41
*** pblaho has quit IRC14:43
jristhonza: so with webpack2 everything needs to be -loader ?14:43
honzajrist: yes14:44
jristhonza-loader14:44
honzajrist: https://webpack.js.org/guides/migrating/#automatic-loader-module-name-extension-removed14:44
honzajrist: or, if you've had your coffee, hozna-loader14:44
jristLOL14:45
jristah there you go14:45
jristnice find, hozna14:45
jrist:)14:45
arxcruzchandankumar: https://review.openstack.org/#/c/445516/ I'm not being able to add you as reviewer14:46
jristhonza: I've had 3 cups so it might be hhhonzzzzaaa14:46
chandankumararxcruz: added myself14:47
honzajrist: I'm looking into why the build is failing today.  Probably just the missing dependency.14:47
arxcruzchandankumar: thanks :)14:47
jristhonza: the webpack2 build?14:47
honzajrist: CI14:47
jristCI period14:47
jristok14:47
weshaymandre, indeed14:47
* jrist yells at cloud14:47
trownmandre: weshay, I think that is actually most sane for mitaka14:48
trowngiven the support for containers in mitaka was pretty slim anyways and it is EOL in a month14:48
weshaycool w/ me.. that was the original workaround posted in the report14:48
mandretrown, weshay: alright, patch incoming14:48
openstackgerritMerged openstack/tripleo-quickstart-extras master: Make repo file visible in browser  https://review.openstack.org/44517014:49
arxcruzadarazs: sshnaidm https://review.openstack.org/#/c/445516/ when you guys have time :)14:50
*** yprokule_ has joined #tripleo14:50
adarazswe should really have a gate specifically for tempest changes that run full or at least smoke tempest :/14:51
*** jpena|lunch is now known as jpena14:51
openstackgerritMartin André proposed openstack/instack-undercloud stable/mitaka: Disable docker-registry by default  https://review.openstack.org/44552214:52
*** athomas has quit IRC14:53
*** yprokule has quit IRC14:53
*** yprokule_ is now known as yprokule14:53
openstackgerritDan Prince proposed openstack/python-tripleoclient master: Mark 'undercloud deploy' as experimental  https://review.openstack.org/44552314:53
dprincemwhahaha: I think that should help clarify the state of things ^^^14:53
adarazsarxcruz: can you indicate in the patches about tempest with a comment if you tested them?14:54
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates master: Add BGPVPN services to scenario004  https://review.openstack.org/44355314:54
arxcruzadarazs: sure, I always test my patches before submit :)14:54
mwhahahadprince: ok, i think the concept should  just get wrapped into install at some pointif it gets agreed upon.14:54
adarazswell, I don't because gates. but you don't have that luxury. :)14:55
*** marst has quit IRC14:55
*** artom has joined #tripleo14:55
dprincemwhahaha: we can debate that I guess. Undercloud install is a TripleO specific thing14:56
openstackgerritwes hayutin proposed openstack/tripleo-quickstart master: update localhost instructions to indicate 127.0.0.2 is required  https://review.openstack.org/44521314:56
dprincemwhahaha: I did this with an eye towards other projects like Packstack as well... which would probalby not share the 'undercloud install' interface/config file14:56
dprincemwhahaha: but that is all for the future I think14:56
*** funzo has joined #tripleo14:57
mwhahahadprince: the problem is you still need some sort of prep step and i like the yum install oooclient/undercloud install process as it's cleaner than most14:57
mwhahahadprince: we can chat about it another time14:57
openstackgerritArx Cruz proposed openstack/tripleo-quickstart-extras master: Fix stackviz report generation  https://review.openstack.org/44426714:57
dprincemwhahaha: 'undercloud install' --> calling 'undercloud deploy' gives you that prep step14:58
*** aufi has joined #tripleo14:58
mwhahahadprince: it doesn't, see my argument on oooq14:58
dprincemwhahaha: patch?14:58
mwhahahadprince: https://review.openstack.org/#/c/419040/14:58
mwhahahadprince: specifically, https://review.openstack.org/#/c/419040/25/roles/undercloud-deploy/templates/undercloud-deploy.sh.j214:59
*** skramaja has joined #tripleo14:59
*** athomas has joined #tripleo14:59
*** prateek has joined #tripleo15:00
dprincemwhahaha: The feature exists today as a useful developer tool15:00
dprincemwhahaha: if you don't want that in quickstart I guess whatever15:00
dprincemwhahaha: but some do, and I see no harm in adding it15:01
mwhahahadprince: no, my issue is with the large amount of extra prep items that could be wrapped into the install15:01
dprincemwhahaha: we aren't ready to do that yet15:01
mwhahahadprince: right that's my argument15:01
*** funzo has quit IRC15:01
mwhahahathat's not a productized install process, that's a bunch of hacks15:01
dprincemwhahaha: I could create a feature branch and do it. That would be my preferrred approach15:01
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Add redux-form dependency  https://review.openstack.org/44552515:02
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Introduce nodes assignment using NodePicker component  https://review.openstack.org/44552615:02
mwhahahadprince: right, so i'd like to see all of the extra bits currently wrapped in that undercloud-deploy.sh captured in a single undercloud install/deploy process15:02
mwhahahadprince: my concern about throwing things like that into oooq is that people just assume it's  ready to be consumed by the end user and it is not15:03
mwhahahadprince: i'm not against support for it, but i'd like to see an improvement on the process prior to supporting it at a larger scale15:03
dprincemwhahaha: if we tag the feature as 'experimental' I think it should be fine to let things proceed though right?15:03
mwhahahadprince: i'd rather not because to me it's just hiding the ugliness in oooq that the end user cannot use15:04
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates master: Keep existing data for containerized Swift  https://review.openstack.org/44311215:05
mwhahahadprince: don't get me wrong, i like the concept. i don't think the implementation is userfriendly enough past a PoC15:06
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates master: Keep existing data for containerized Libvirt  https://review.openstack.org/44317215:06
mwhahahadprince: i think it's probably a trival change to add a something like --containerized flag to install that captures the contents of that install script and simplifies the process15:07
*** pblaho has joined #tripleo15:08
dprincemwhahaha: no, there is *much* more. We also need a peice that translates the config file from undercloud.conf into the relevant heat templates and parameters15:08
mwhahahadprince: no, we don't15:08
mwhahahadprince: if --containerized support -e and call deploy15:08
dprincemwhahaha: then there is no upgrade path!?15:08
dprincemwhahaha: a user would expect the same undercloud.conf to work.... and it wouldn't15:09
mwhahahadprince: like there's an upgrade path now?15:09
mwhahahadprince: parsing the undercloud.conf and spitting out heat template is pretty trivial15:09
dprincemwhahaha: that is my point. We shouldn't pretend 'undercloud install' supports containers until it really does with undercloud.conf support15:09
mwhahahadprince: but it sounds like you don't want it to ever support it15:09
dprincemwhahaha: you are arguing that we simply install some RPMs and call it a day. I think there is much more to this15:10
*** marst has joined #tripleo15:10
dprincemwhahaha: I do want to support it, but there is work to be done15:10
*** ooolpbot has joined #tripleo15:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION15:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273015:10
*** ooolpbot has quit IRC15:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,Confirmed]15:10
mwhahahadprince: then that needs to be communicated and worked on15:10
mwhahahadprince: i don't think it's as much as you think it is but that's probably because i'm not aware of what might be missing15:10
dprincemwhahaha: you asked, so I'm communicating with you now :)15:10
openstackgerritMerged openstack/tripleo-common master: Updated from global requirements  https://review.openstack.org/44515415:11
*** dmarlin has joined #tripleo15:11
dprincemwhahaha: like I said, I've got a local patch which starts to do this... until that is a priority though I think marking 'undercloud deploy' as experimental is wise15:11
chandankumarsshnaidm: arxcruz feel free to have your feedback on my blog post https://github.com/redhat-openstack/website/pull/86215:11
mwhahahadprince: once again that's fine, i just don't want to see experimental made un-experimental without a bunch of effort behind it15:12
mwhahahadprince: i think we're in general agreement but it's the messaging15:12
dprincemwhahaha: yes, I think so.15:12
*** abehl has quit IRC15:14
*** saibarspeis has joined #tripleo15:14
*** cdearborn has joined #tripleo15:17
openstackgerritMerged openstack/tripleo-heat-templates master: Update properties being set for octavia rabbit properties  https://review.openstack.org/44163715:18
openstackgerritMerged openstack/tripleo-heat-templates stable/newton: Remove the openvswitch special case upgrade code  https://review.openstack.org/44119215:18
jtomasekflorianf, it is... It is all that formsy-react isn't15:22
*** yprokule has quit IRC15:23
jtomasekflorianf: I've hit a lot of problems to solve regarding forms and nodes assignment and redux-form had all the answers15:23
florianfjtomasek: will it replace formy-react?15:23
florianf*formsy15:23
jtomasekflorianf, I'd like to, gradually15:23
jtomasekflorianf, I know having two form libraries at once is not ideal, but I think the transition is necessary and right now seems to me as a good moment to start with it15:24
*** udesale__ has quit IRC15:24
florianfjtomasek: "Gradually", as in: It's a potential complete replacement, we only have to do the work?15:24
jtomasekflorianf, yes15:24
florianfSo having both libraries is just a matter of migration.15:25
jtomasekflorianf, yes15:25
florianfCool15:25
jtomasekflorianf, there are many good things about redux-form: 1. it has great user base and it is frequently updated, has stable API, does not re-render all inputs when one of them is changed (this is a big performance improvement)15:26
jtomaseketc.15:27
florianfjtomasek: sounds great!15:27
florianfjtomasek: Looks pretty healthy.15:27
shardyjtomasek: hey can you pls point me to the code which uploads user environments in the UI?15:27
shardyjtomasek: I'd like to ensure the naming used in my tripleoclient patch aligns, e.g we use user-files atm for user files, but what about user environments?15:28
jaosoriorEmilienM: can you review this one https://review.openstack.org/#/c/444873/8 ?15:28
*** dsariel has quit IRC15:28
jtomasekshardy, so the UI does not make difference between user added files and those coming from THT. User can provide own files by updating the plan (uploads new files to swift). environment files which are not in capabilities-map.yaml are then filed under 'Other' in UI15:30
shardyjtomasek: Ok, so there's no naming convention15:30
jtomasekshardy, yes, only convention is that environment file needs to be in environments directory15:30
jaosoriordprince, jistr: containerized overcloud deploy worked :D15:30
jaosoriormandre: ^^15:30
jaosoriorthanks folks!@15:31
shardyjtomasek: I need to transform e.g -e /home/shardy/foo_env.yaml into <container>/user-files/home/shardy/foo_env.yaml15:31
*** [1]cdearborn has quit IRC15:31
mandrejaosorior: woot!15:31
rookshardy if stack delete fails to nuder the env, (ie leaves ports behind) what could we look at to determine why?15:32
shardyjtomasek: Ok, what environments directory?  We only have user-files in the CLI atm15:32
shardyand a merged user-environment.yaml15:32
shardyjtomasek: I'm thinking of adding user-environments/ directory15:32
jtomasekshardy, https://github.com/openstack/tripleo-heat-templates/tree/master/environments15:32
shardyor they could just go in user-files15:32
jtomasekshardy, triple-common searches the files there afaik15:32
shardyjtomasek: Ok, I don't think that works for the CLI, as it's possible (although perhaps unlikely) filenames could collide15:33
jtomasekshardy, compares them with capabilities map and the files from there which are not in capabilities map get put in 'Other' group15:33
jtomasekshardy, I see15:33
jtomasekshardy, so you'd need to update the tripleo-common action which lists environments to also check the user-files/environments directory15:34
shardyjtomasek: Ok, I'll go with adding a user-environments directory, and yeah we can have tripleo-common updated to search it15:34
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates master: WIP - Add upgrade_batch_tasks to neutron-l3-agent  https://review.openstack.org/44549415:34
jtomasekshardy, are the user files only environments? they can be also templates, right?15:34
jtomasekshardy, nice!15:34
shardyjtomasek: we already upload the templates (and any other files) to (container)/user-files15:34
*** saibarsp_ has joined #tripleo15:35
shardyjtomasek: but we don't upload the individual environment files, only the merged one15:35
jtomasekshardy, cool15:35
jtomasekshardy, so as part of your work, you're getting rid of merged environment, right?15:35
shardyjtomasek: yes, that's my plan15:35
jtomasekshardy, ok, this is awesome15:35
jtomasek:)15:36
shardywe'll probably still need to calculate it in the client, but I'm going to stop uploading it to swift15:36
shardyand later I'll look at ways to stop calculating it at all in the client ref https://bugs.launchpad.net/heat/+bug/164568615:36
openstackLaunchpad bug 1645686 in python-heatclient "Server side environment merging requires client-side workarounds" [Undecided,New]15:36
shardybut for now I'm looking for the simplest way to fix https://bugs.launchpad.net/tripleo/+bug/163540915:37
openstackLaunchpad bug 1635409 in tripleo "parameter_merge_strategies: merge behaves like overwrite" [High,In progress] - Assigned to Steven Hardy (shardy)15:37
shardy(and hopefully align CLI/UI better in the process)15:37
*** saibarspeis has quit IRC15:37
jtomasekshardy, very nice15:38
mwhahahaianw: you around?15:39
*** zoli is now known as zoli|brb15:39
*** oaken_chris has joined #tripleo15:41
*** saibarsp_ has quit IRC15:41
*** tremble has joined #tripleo15:41
*** dsariel has joined #tripleo15:42
*** tremble has quit IRC15:46
openstackgerritIan Main proposed openstack/tripleo-heat-templates master: Remove kolla config from heat services.  https://review.openstack.org/44397415:48
Slowerdprince: what do you think of that? ^15:48
*** Erming__ has quit IRC15:48
*** fultonj has quit IRC15:49
*** tremble has joined #tripleo15:49
*** tremble has quit IRC15:49
*** tremble has joined #tripleo15:49
jaosoriorSlower: heat is now running over httpd though15:51
mandrejaosorior: yeah and it's breaking my deployment15:52
mandrejaosorior: https://github.com/openstack/puppet-heat/commit/fe394add6f2a8e1ef87dc4288aefa899b437a82b requires the httpd service15:53
mandreand openstack-heat-api rpm doesn't depends on httpd -> problem15:53
openstackgerritAlex Schultz proposed openstack/instack-undercloud master: Align stars to fix CI  https://review.openstack.org/44522615:53
*** tremble has quit IRC15:55
openstackgerritAlex Schultz proposed openstack/diskimage-builder master: Allow ELEMENTS_DIR to be configurable  https://review.openstack.org/44554515:57
openstackgerritAlex Schultz proposed openstack/instack-undercloud master: Align stars to fix CI  https://review.openstack.org/44522615:57
*** funzo has joined #tripleo15:57
*** fultonj has joined #tripleo15:57
jaosoriormandre: so... how does one add the dependency for that case? should it be added to the image?15:57
*** tremble has joined #tripleo15:58
*** tremble has quit IRC15:58
*** tremble has joined #tripleo15:58
EmilienMmwhahaha: depends-on is missing?15:58
mwhahahaEmilienM: going to follow on with that15:58
mwhahahaEmilienM: since a dib change is required and who knows how long that'll take15:58
mandrejaosorior: if httpd is now a hard requirement for heat-api then the fix should go in RDO15:59
openstackgerritAlex Schultz proposed openstack/instack-undercloud master: Reenable dib-lint  https://review.openstack.org/44554715:59
openstackgerritAttila Darazs proposed openstack/tripleo-quickstart-extras master: GATE CHECK  https://review.openstack.org/43027716:00
jaosoriormandre: well, it isn't a hard-requirement. One can still run and deploy heat without httpd.16:00
*** zoli|brb is now known as zoli16:01
mandrejaosorior: in that case we can customize our heat images16:01
*** abehl has joined #tripleo16:01
mandrejaosorior: is that only for heat-api* services?16:01
jaosoriormandre: are those the kolla images or is there a definition for those in t-h-t?16:01
jaosoriormandre: that has been the most recent one, yes16:01
mandrejaosorior: we'll add httpd for heat here https://github.com/openstack/tripleo-common/blob/master/contrib/tripleo_kolla_template_overrides.j216:02
*** funzo has quit IRC16:02
mandrejaosorior: this file represents our tripleo flavor of kolla images16:03
jaosoriorI see16:04
*** dparkes has quit IRC16:04
*** oaken_chris has quit IRC16:08
*** ealcaniz has quit IRC16:08
mandrejaosorior: the patch would actually make more sense in kolla16:10
*** ooolpbot has joined #tripleo16:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION16:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273016:10
*** ooolpbot has quit IRC16:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)16:10
*** jkilpatr has quit IRC16:10
dprinceSlower: in general looks good. one comment on the 'command:' arrays vs. using strings there...16:10
jaosoriormandre: will you do it or should I try it out?16:11
mandrejaosorior: I'll do it16:12
adarazsgrr, we still don't have this merged: https://review.openstack.org/441186 -- anybody knows another friendly infra core?16:12
jaosoriormandre: can you link me to the review when you have it?16:12
trownadarazs: I will ping on openstack-infra now that it has a +216:12
trownadarazs: oh just saw you did that already :P16:13
adarazstrown: yeah, but it was a "long time ago" :P :)16:13
adarazsyou can try again :D16:13
jaosoriordprince: yep, both need to get merged... so, either they could merge together, or I need to reverse the order of dependencies. Right now the containers patch depends on the one that sets fernet as the default16:14
trownadarazs: I will ping this afternoon if nothing happens with it16:14
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: WIP: Conntainers with multi-nic and net-iso  https://review.openstack.org/42244816:15
adarazstrown: all right. I'll send the announcement email tomorrow morning if it happens. I have to get going soon.16:15
trownadarazs: cool, thanks16:17
*** aufi has quit IRC16:18
*** dparkes has joined #tripleo16:18
mandrejaosorior: https://review.openstack.org/44556316:22
jaosoriormandre: great!16:23
jaosoriormandre: by the way, how does that list work? will those be ALL the packages available in that image?16:23
openstackgerritChris Jones proposed openstack/python-tripleoclient master: Make fencing action parameter optional.  https://review.openstack.org/44249716:23
*** jkilpatr has joined #tripleo16:23
mandrejaosorior: not exactly, the images are layered, this one (heat-base) depends on openstack-base that has a ton of packages16:25
mandrejaosorior: heat-base serves as the base image for all the other heat images16:25
mandreyou never use heat-base directly16:26
jaosoriormandre: right, cause I noticed there is no mentione of mod_ssl in any image :/16:26
mandrejaosorior: that should be added16:26
mandrefeel free to submit a patch, but I suspect you'll need to add it to quite a few places16:27
mandrejaosorior: everywhere we install httpd16:27
*** trown is now known as trown|lunch16:29
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates master: Skip stopping neutron cleanup on upgrade  https://review.openstack.org/42629916:30
*** yamahata has quit IRC16:32
openstackgerritDan Prince proposed openstack/python-tripleoclient master: Mark 'undercloud deploy' as experimental  https://review.openstack.org/44552316:32
*** tremble has quit IRC16:33
*** dparkes has quit IRC16:34
*** prateek has quit IRC16:35
*** gchamoul is now known as gchamoul|afk16:36
*** dsariel has quit IRC16:37
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart-extras master: Add undercloud user to docker group  https://review.openstack.org/44542816:38
*** prateek has joined #tripleo16:39
*** panda is now known as panda|bbl16:39
openstackgerritChris Jones proposed openstack/tripleo-common stable/ocata: Handle unprovisioned Ironic nodes in fencing parameter generator.  https://review.openstack.org/44557016:39
*** dmacpher is now known as dmacpher-afk16:39
jaosoriormandre: what's the channel for kolla?16:41
mandrejaosorior: it's #openstack-kolla16:42
*** ccamacho has quit IRC16:42
jaosoriormandre: thanks16:42
jaosoriordprince: so what do you think? Should we attempt to merge those two commits for fernet at the same time?16:44
*** pblaho has quit IRC16:45
sshnaidmbnemec, ping16:46
dprincejaosorior: you mean merge the patches?16:47
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci master: Use custom network-isolation-absolute.yaml in ipv4 net-iso jobs  https://review.openstack.org/42229716:47
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci master: Test with bonded public interface  https://review.openstack.org/38556216:47
dprincejaosorior: that would be fine...16:47
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci master: Add storage-mgmt network to compute nodes  https://review.openstack.org/41904916:47
bnemecsshnaidm: ack16:48
*** fzdarsky is now known as fzdarsky|afk16:48
*** rbrady is now known as rbrady-afk16:48
sshnaidmbnemec, I think we need to add dev env requirements for RDO cloud too, like we have them on rh1 now16:48
sshnaidmbnemec, I'm talking about: https://etherpad.openstack.org/p/tripleo-ci-devenvs16:48
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates master: Add BGPVPN services to scenario004  https://review.openstack.org/44355316:48
jaosoriordprince: I mean merge into master, or did you mean merge as in squash the patches?16:49
*** dsariel has joined #tripleo16:49
sshnaidmbnemec, good to have it in rdo cloud too, and more than 4 ovb envs in parallel like now we have16:50
bnemecsshnaidm: Eventually all users should be able to have their own creds on RDO cloud though.16:50
sshnaidmbnemec, dev envs are not under user accounts16:50
bnemecsshnaidm: Not right now, but they will be.16:51
*** dtantsur is now known as dtantsur|brb16:51
bnemecOr should be, IMHO.16:51
dprincejaosorior: I would've squashed the patches16:51
dprincejaosorior: otherwise, if we were actually gating there would be no way they would make it in16:51
bnemecThere's no sense having common accounts in a cloud where we're supposed to be able to request user quota.16:51
*** zoli is now known as zoli|gone16:51
bnemecWe did it in rh1 because there was no process for people to ask for an account.16:52
sshnaidmbnemec, the point in dev env it's to reproduce exactly CI environment to test CI specific issue, not just to use OVB16:52
jaosoriordprince: why not just approve both patches at the same time?16:52
*** udesale has joined #tripleo16:53
sshnaidmbnemec, I mean, it involves nodepool image for example, etc etc16:53
bnemecsshnaidm: What difference does it make whether you using the tripleo-user account (which is not what we use for CI) or bnemec/sshnaidm/whomever?16:53
dprincejaosorior: because that doesn't guarantee us that zuul would merge them at the same time16:53
dprincejaosorior: unless you change the order so it doesn't break containers16:53
*** udesale has quit IRC16:53
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: keystone/containers: Add support for fernet keys  https://review.openstack.org/44539116:54
dprincejaosorior: I don't really care. change the order so the containers patch lands first I think. Or squash them16:54
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates master: Switch keystone default provider to fernet  https://review.openstack.org/43242016:54
jaosoriordprince: done16:54
sshnaidmbnemec, actually if nodepool image will be available for booting it, then maybe no difference..16:55
*** jaganathan has joined #tripleo16:55
*** d0ugal has quit IRC16:55
bnemecsshnaidm: That's how it has to work.  We can't give people the actual nodepool credentials, so we need to make sure the user accounts have access to what they need.16:56
*** fragatina has joined #tripleo16:56
*** fragatina has quit IRC16:57
sshnaidmbnemec, ok16:57
jaosoriordprince, mandre: If I need to mount a file from the node that's not available in /var/lib/kolla/config_files/src/ in the container... how would I do it?16:57
*** fragatina has joined #tripleo16:57
*** funzo has joined #tripleo16:58
dprincejaosorior: we are moving away from kolla config soon too fwiw. See Slower's patch here: https://review.openstack.org/#/c/443974/16:59
mandrejaosorior: you would bind mount it using a volume, like in https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/keystone.yaml#L122-L12816:59
dprincejaosorior: per what you've been doing you might consider just doing that now yourself for Keystone...16:59
dprincejaosorior: kolla config is just getting in our way I'm afraid when we could just bind mount things directly16:59
dprincejaosorior: but yea, bind mounting is the general mechanism used to expose things to the container.17:00
jaosoriordprince, mandre alright, thanks17:01
*** fzdarsky|afk is now known as fzdarsky17:01
*** funzo has quit IRC17:03
*** fragatina has quit IRC17:03
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates master: DNM testing plan_environment tripleo-common changes  https://review.openstack.org/44558017:05
*** jlinkes has quit IRC17:09
*** arxcruz has quit IRC17:09
*** ooolpbot has joined #tripleo17:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION17:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273017:10
*** ooolpbot has quit IRC17:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)17:10
*** ukalifon has quit IRC17:10
*** arxcruz has joined #tripleo17:12
*** yamahata has joined #tripleo17:13
*** fzdarsky has quit IRC17:14
*** pblaho has joined #tripleo17:22
*** fzdarsky has joined #tripleo17:22
openstackgerritEmilien Macchi proposed openstack/tripleo-specs master: specs/pike: propose a solution for keystone fernet tokens rotation  https://review.openstack.org/44559217:22
*** hjensas has quit IRC17:22
*** fragatina has joined #tripleo17:23
*** lucasagomes is now known as lucas-afk17:26
*** bkopilov has quit IRC17:27
*** agurenko has joined #tripleo17:27
*** trown|lunch is now known as trown17:28
*** jaganathan has quit IRC17:29
*** lmiccini has quit IRC17:29
openstackgerritBrent Eagles proposed openstack/os-net-config stable/newton: Allow setting NM_CONTROLLED=yes in os-net-config ifcfg files  https://review.openstack.org/44559517:35
*** hewbrocca is now known as hewbrocca_afk17:37
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart master: Use buildlogs mirror for opstools repo  https://review.openstack.org/44559717:38
sshnaidmtrown, ^^17:38
*** bkopilov has joined #tripleo17:38
sshnaidmtrown, the second question, why is it enabled in 000 featureset, it should be disabled there imho..17:38
sshnaidmopstools is enabled in scenario001 only17:39
openstackgerritArx Cruz proposed openstack/tripleo-specs master: Tool to send tempest report by mail  https://review.openstack.org/42087817:39
trownsshnaidm: hmm.. the repo-setup role does not really have a clean way to add only a single repo though17:40
trownsshnaidm: ie. take all repos from release config, but just add one extra17:40
sshnaidmtrown, what do you mean?17:40
*** cylopez has quit IRC17:40
trownthere is not config setting we can add to featureset that would do that17:41
trownsshnaidm: also, it should not hurt anything (unless the repo is broken) to have it enabled all the time17:41
sshnaidmtrown, why no config settings?17:42
trownsshnaidm: how would you add a single repo (without overwriting repos from release config) via config setting?17:42
sshnaidmtrown, I dunno, maybe there are different version of some package which will break something17:42
*** dbecker has quit IRC17:43
sshnaidmtrown, repos: {{ repos  + my_lovely_repo }} ?17:43
trownsshnaidm: I dont think that works in ansible17:43
sshnaidmtrown, it's yaml, not related to ansible17:44
trownsshnaidm: well it matters how ansible loads all the yamls17:44
jaosoriordprince, jistr: I changed the order for https://review.openstack.org/#/q/status:open+project:openstack/tripleo-heat-templates+branch:master+topic:fernet so the gate should be passing at all times. If you guys still think that patches should have a -1 I can change them.17:44
jaosorioranyway, I'm off. Have a good day folks!17:44
trownsshnaidm: I am pretty sure "repos: {{ repos  + my_lovely_repo }}" will result in "repos is not defined" error17:44
sshnaidmtrown, I did uglier things: https://github.com/openstack-infra/tripleo-ci/blob/master/scripts/quickstart/ovb-settings.yml#L5217:45
*** jaosorior has quit IRC17:45
sshnaidmtrown, if "repos" is defined, no reason to fail on "undefined variable"17:45
trownsshnaidm: ya but there regular_interfaces and netiso_interface are defined in the same file17:45
*** jpich has quit IRC17:45
*** ckyriakidou has quit IRC17:45
trownsshnaidm: in any case I am cool with only enabling on featureset that uses it (if you can make it work) :p17:46
sshnaidmtrown, but I don't want to do it :)17:46
sshnaidmtrown, when I started to talk about nested configs, you all recalled khaleesi..17:47
sshnaidmtrown, I think repo are pretty important and no so many of them, so it's better to be explicit there17:47
*** b00tcat has quit IRC17:48
trownsshnaidm: I am confused what you are proposing now17:48
trownhttps://review.openstack.org/445597 will fix the imediate issue with CBS being down from planned CentOS outage17:49
*** tesseract has quit IRC17:49
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates master: Add upgrade tasks for aodh containers  https://review.openstack.org/44560017:49
trownsshnaidm: for me that is good and we can just move on :P17:49
sshnaidmtrown, ok, maybe I misunderstood what you mean by repos adding..17:50
*** snecklifter has quit IRC17:52
openstackgerritPaul Belanger proposed openstack/diskimage-builder master: Split debootstrap into debootstrap-minimal  https://review.openstack.org/44560117:52
openstackgerritPaul Belanger proposed openstack/diskimage-builder master: Split debootstrap into debootstrap-minimal  https://review.openstack.org/41310317:53
*** derekh has quit IRC17:55
*** athomas has quit IRC17:56
*** panda|bbl is now known as panda17:58
openstackgerritPaul Belanger proposed openstack/diskimage-builder master: Add ubuntu-rootfs operating-system element  https://review.openstack.org/41311517:58
sshnaidmpanda, take a look pls: https://review.openstack.org/#/c/445597/ we have problem with downloading opstools repo17:59
*** funzo has joined #tripleo17:59
*** d0ugal has joined #tripleo17:59
sshnaidmtrown, can you please review multinode-build patch: https://review.openstack.org/#/c/443687/17:59
pandasshnaidm: this is a temporary workaround ?18:00
*** jpena is now known as jpena|away18:00
openstackgerritPaul Belanger proposed openstack/diskimage-builder master: Clean up cache for ubuntu-minimal  https://review.openstack.org/41472818:02
sshnaidmpanda, no18:02
*** dtantsur|brb is now known as dtantsur18:03
pandasshnaidm: ok, +1W18:03
trownpanda: it is permanent18:03
*** funzo has quit IRC18:03
trownpanda: points to CDN location of repo rather than directly to the koji18:03
trownsshnaidm: sure18:03
*** pcaruana has quit IRC18:03
*** d0ugal has quit IRC18:04
EmilienMcan someone approve https://review.openstack.org/#/c/442795/ please?18:05
*** mgagne_ is now known as mgagne18:05
EmilienMoh wait no18:05
EmilienMI want to see it passing CI without Depends-On18:05
*** snecklifter has joined #tripleo18:05
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: scenario001/pingtest: enable Gnocchi resource again  https://review.openstack.org/44279518:05
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: scenario001/pingtest: enable Gnocchi resource again  https://review.openstack.org/44279518:05
chandankumarEmilienM: how we are going to test a tripleo deployment done using containers? currently all deployments are tested or verified using tempest18:06
EmilienMtrown: we'll need to update http://tripleo.org/cistatus.html since we switched the jobs18:08
*** rbrady-afk is now known as rbrady18:09
EmilienMtrown: add the columns or replace the existing ones18:09
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart-extras master: DONT MERGE: patched repo-setup  https://review.openstack.org/44560818:09
trownEmilienM: indeed... where is the source for that?18:09
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Introduce nodes assignment using NodePicker component  https://review.openstack.org/44552618:09
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Add redux-form dependency  https://review.openstack.org/44552518:09
EmilienMchandankumar: I'm not sure all deployments are tested by tempest now, in CI we still use the pingtest18:09
EmilienMtrown: openstack-infra/tripleo-ci/scripts/tripleo-jobs-gerrit.py18:10
*** ooolpbot has joined #tripleo18:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION18:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273018:10
*** ooolpbot has quit IRC18:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)18:10
trownEmilienM: cool, will submit a patch18:11
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo stable/ocata: panko: Do db_sync in api manifest  https://review.openstack.org/44433818:11
EmilienMtrown: i'm in favor of adding columns, but not sure it's the best thing for everyone18:11
EmilienMdprince, bnemec ^ thoughts?18:12
*** jkilpatr has quit IRC18:12
trownEmilienM: I think we have to18:12
trownEmilienM: since "classic" multinode jobs still run on non-TripleO projects18:12
EmilienMtrown: or maybe hire a webdev and do some web design, so we have a selector that pick the jobs and show history for one job18:13
*** jkilpatr has joined #tripleo18:13
bnemecEmilienM: I'm fine either way.  Note that I have a PR open against the site theme to get rid of the giant left margin: https://github.com/dprince/tripleosphinx/pull/218:13
trownEmilienM: I like the idea of hiring a webdev :P18:13
bnemecThat would make the large number of columns slightly less obnoxious.18:13
EmilienMdprince: any change to look at the PR ?18:14
chandankumarsshnaidm: anything more to update on this https://review.openstack.org/#/c/444181/18:14
*** florianf has quit IRC18:15
openstackgerritJiri Tomasek proposed openstack/tripleo-ui master: Introduce nodes assignment using NodePicker component  https://review.openstack.org/44552618:16
EmilienMshardy: I think we should run tripleo upgrade jobs in tripleo-common18:17
EmilienMshardy: https://review.openstack.org/#/c/414169/ was the signal18:17
*** Vijayendra has quit IRC18:17
*** gbarros has quit IRC18:17
*** gbarros has joined #tripleo18:18
shardyEmilienM: yeah agreed18:19
*** gbarros has quit IRC18:21
shardyEmilienM: I pushed https://review.openstack.org/#/c/445580/ so we can see if my comment is correct18:21
sshnaidmchandankumar, no, seems good for me18:21
*** salmankhan has quit IRC18:21
EmilienMshardy: right18:21
chandankumarsshnaidm: so all my oooq-extras patches are good to go, :-)18:22
chandankumarsshnaidm: only one tempest related bug i need to fix, then we will be in good shape for tempest18:22
EmilienMshardy: https://review.openstack.org/44561018:23
sshnaidmchandankumar, great work, thanks!18:23
*** gbarros has joined #tripleo18:25
*** suuuper has quit IRC18:26
*** Vijayendra has joined #tripleo18:29
*** gfidente is now known as gfidente|afk18:31
*** bogdando has quit IRC18:33
*** gfidente|afk is now known as gfidente18:34
*** dprince has quit IRC18:38
*** rbrady is now known as rbrady-afk18:38
*** tosky has quit IRC18:40
*** panda is now known as panda|dinner18:40
panda|dinnerWe're doing some polling, but please if you see multinode jobs failing, ping us18:41
*** fragatina has quit IRC18:44
*** fragatina has joined #tripleo18:44
*** shardy is now known as shardy_afk18:45
*** gfidente is now known as gfidente|afk18:48
openstackgerritGabriele Cerami proposed openstack-infra/tripleo-ci master: multinode settings: add multinode_mode variable  https://review.openstack.org/44561818:50
panda|dinnersshnaidmm trown: ^ this variable is needed, but only in tripleo-ci playbooks, added to the environemnt specific config18:51
openstackgerritIan Main proposed openstack/tripleo-heat-templates master: Remove kolla start from ironic.  https://review.openstack.org/44562018:53
*** pkovar has quit IRC18:54
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: Add scenario005 job (4nodes)  https://review.openstack.org/41809819:00
*** dtantsur is now known as dtantsur|afk19:02
sshnaidmEmilienM, trown can we merge this multinode build patch? https://review.openstack.org/#/c/443687/19:04
trownsshnaidm: I commented on that patch19:05
sshnaidmit would solve this issue with multinode_mode too actually19:05
trownhttps://review.openstack.org/445618 solves the multinode_mode issue in a much more targeted way19:05
sshnaidmtrown, no, it add a redundant hack, which is not needed there19:05
sshnaidmtrown, it was tested on CI already, I just added it to active multinode playbook19:06
sshnaidmtrown, when this patch is merged, multinode_mode won't be used anywhere19:06
trownsshnaidm: your patch happens to fix the current CI issue... but that is not its purpose19:07
trownsshnaidm: you can remove the variable in your patch then... I already merged the CI fix19:07
sshnaidmtrown, and wait another week to merge it.. no, I'd prefer to remove it in followup patch19:08
trownsshnaidm: there is nonzero risk with your patch (it has red CI)19:08
trownsshnaidm: with the actual CI fix there is zero risk19:08
trownsshnaidm: follow up is fine... I want to see green CI on https://review.openstack.org/#/c/443687/ before merging ... it actually wont even merge without it at this point since we switched jobs over19:10
*** ooolpbot has joined #tripleo19:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION19:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273019:10
*** ooolpbot has quit IRC19:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)19:10
trownsshnaidm: and now we are blocking the fix for ^ because of the multinode_mode thing19:10
sshnaidmtrown, yeah.. that's why I was asking to merge it before transition19:10
* trown does not understand why CI was passing before19:11
trownsshnaidm: without the CI breakage (which your patch is a very non-obvious fix for) the repo stuff seemed lower priority to me... I looked at that patch a few times, but didnt think it needed to go in before we had actual CI voting on it rather than dependent do not merge patch to test it19:13
sshnaidmtrown, because we didn't build anything in CI checks, this stuff fails only if we build19:14
sshnaidmmultinode_mode is used only when building tasks start..19:15
trownsshnaidm: ah that makes sense... I hope that is the only thing that breaks if we build... if something else is broken we may need to revert the infra patch and try again next week19:15
sshnaidmtrown, my patch should have been solved this, and I checked  it with real building19:16
sshnaidmit's the same problem we had in rdo ci, need to check real build if we change build part19:16
trownsshnaidm: that seems like a pretty big hole for the merge queue... we could merge a patch in quickstart-extras/tripleo-ci that breaks every other gate19:18
*** eck` is now known as eck`gone19:18
*** mcornea has quit IRC19:18
trownsshnaidm: even with merge gate on extras/tripleo-ci19:18
*** dsneddon_ has joined #tripleo19:18
*** dsneddon_ is now known as dsneddon19:19
openstackgerritMerged openstack/diskimage-builder master: Allow ELEMENTS_DIR to be configurable  https://review.openstack.org/44554519:19
sshnaidmtrown, or patch in build-test role in extras, that will break tripleo-ci, the same19:20
sshnaidmtrown, that's why I always do a following rebased patch with some dependency on tht or tripleo-common and see if it succeeds, if I change the build part19:21
*** artom has quit IRC19:22
trownsshnaidm: crap... I think https://review.openstack.org/445597 will probably fail in the merge queue on that... it is just finishing check19:23
sshnaidmtrown, maybe it's good to have kind of build-gating job in tripleo-ci, like we have in rdo-ci, it will run with dependency on some patch19:23
trownsshnaidm: will depends-on work to merge them together?19:24
*** eggmaster has joined #tripleo19:25
*** dsariel has quit IRC19:25
trownsshnaidm: or did we just destroy the gate in the first couple hours of switching to quickstart?19:25
sshnaidmtrown, I think it passes19:25
trownsshnaidm: ah right because it is not building19:25
*** leanderthal|afk has quit IRC19:25
trownphew19:25
sshnaidmtrown, it collects log according to zuul page19:25
dsneddonEmilienM, beagles wasn't able to +2 this branch for some reason, but he wanted to get this backport merged: https://review.openstack.org/44506019:26
sshnaidmtrown, hmm.. do we still run quickstart.sh with debug19:26
trownsshnaidm: ya in check... wasnt sure if that job started before we switched over and we would get different behavior in merge19:26
trownsshnaidm: seems like it19:27
trownsshnaidm: wrt debug19:27
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates master: Add upgrade tasks for gnocchi container services  https://review.openstack.org/44562719:27
panda|dinnerthe debug  was very helpful while testing the scripts19:28
panda|dinnerwithout it I wouldn't be able to discover at least two bugs19:28
*** jmelvin has quit IRC19:28
panda|dinnerI'd prefer to keep it until things stabilize19:28
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Clean up oooq jobs  https://review.openstack.org/44562819:30
sshnaidmpanda|dinner, trown ^^19:31
sshnaidmpanda|dinner, when testing - yes, but in production it makes output less readable19:32
trownsshnaidm: looks like even https://review.openstack.org/445618 will fail CI until https://review.openstack.org/445597 merges19:33
trownsshnaidm: it failed in repo-setup19:33
sshnaidmtrown, cool, so we can abandon it and merge my patch \o/ :D19:33
trownbad timing to transition CI jobs the day of CentOS outage :P19:34
*** panda|dinner is now known as panda19:34
openstackgerritPradeep Kilambi proposed openstack/tripleo-heat-templates master: Containerize Redis service  https://review.openstack.org/44215119:34
pandaso,  are we in testing or in production ? It's like we're testing in production :P19:35
trownpanda: are you ok with merging https://review.openstack.org/443687 instead? it has to pass CI to merge anyways which gets rid of my only concern19:35
stevebakermorning19:35
pandatrown: instead of what ?19:35
trownpanda: we need to wait for https://review.openstack.org/445597 to merge before we can proceed with either fix19:35
trownpanda: instead of your fix https://review.openstack.org/44561819:36
sshnaidmyeah, centos could wait with their outage until we complete transition..19:37
trownsshnaidm: lol19:37
pandahow were we installing packages in nodes without https://review.openstack.org/443687 ?19:38
trownthe tripleo.sh boostrap?19:39
pandathis outage is what I intendend with "unpredictable" problems19:39
sshnaidmpanda, using build-install-playbook.yml19:39
pandatrown sshnaidm : ok with me to use this instead then19:40
pandatrown: I don't think it's the bootstrap, and if it is maybe we should remove it then ..19:40
*** salmankhan has joined #tripleo19:42
*** paramite has quit IRC19:42
*** amoralej is now known as amoralej|off19:43
pandaif we remove the debug, and we later see that we need it to solve a failure, what's the best way ? change in toci with debug mode required by the change we want to test ?19:45
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates master: Add BGPVPN services to scenario004  https://review.openstack.org/44355319:45
sshnaidmpanda, yeah, or just set debug on in your change that fails while testing it19:46
sshnaidmpanda, it's more ssh debug, I wonder if it's so helpful now, when we solved all these custom users problems19:46
trownya, hopefully that is pretty rarely needed19:46
pandahopefully, it was immensely helpful with a collect logs bug19:47
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates master: Add BGPVPN services to scenario004  https://review.openstack.org/44355319:47
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: WIP:Parameterize OVB stack flavors  https://review.openstack.org/44516319:50
pandatrown: it worked before the transition, because the jobs were not actually gating anything, so all the repo building part was skipped.19:50
trownpanda: ya sshnaidm explained that part to me19:51
trownpanda: does seem like a hole in CI... but one that pre-existed quickstart19:51
sshnaidmtrown, yeah, it's not something new..19:51
sshnaidmpanda, trown we may need strategy: debug https://docs.ansible.com/ansible/playbooks_debugger.html19:51
*** eck`gone is now known as eck`19:52
trownsshnaidm: dont think we want to start an interactive debug shell in CI :P19:52
trownsshnaidm: though it would be nice if that instead provided only debug output of a failed task19:53
sshnaidmtrown, I think it does, why interactive shell?19:53
pandatrown: why not, we may pair this solution with an expect script in pseudo interactive shell19:54
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: Parameterize OVB stack flavors  https://review.openstack.org/44516319:54
trownsshnaidm: according the doc you linked it invokes a debugger19:54
pandasshnaidm: looks pretty interactive to me19:54
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: Parameterize OVB stack flavors  https://review.openstack.org/44516319:55
sshnaidmwhoops.. I hoped it's debug only on failure19:58
trownwould be sweet19:59
pandathat's why we're never going to get it :)19:59
pandaanyway, even with this it wouldn't have helped with the bug I found in collect logs.20:00
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: Add scenario005 job (4nodes)  https://review.openstack.org/41809820:00
*** funzo_ has joined #tripleo20:00
*** funzo_ has quit IRC20:05
*** gfidente|afk has quit IRC20:07
*** gbarros has quit IRC20:09
*** pkovar has joined #tripleo20:09
*** ooolpbot has joined #tripleo20:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION20:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273020:10
*** ooolpbot has quit IRC20:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)20:10
openstackgerritLeehom Li proposed openstack/diskimage-builder master: Fix problem unable disable iptables service  https://review.openstack.org/41970320:11
trownhmm zuul keeps trying to merge https://review.openstack.org/445597 with other changes (which fail because of the other CI issue) ... do we just have to wait until it tries to merge it by itself or is there some magic?20:19
*** artom has joined #tripleo20:25
*** dparkes has joined #tripleo20:26
*** dprince has joined #tripleo20:35
pandatrown: no idea, worth asking in #openstack-infra ?20:35
openstackgerritMerged openstack/tripleo-quickstart master: Use buildlogs mirror for opstools repo  https://review.openstack.org/44559720:36
*** achadha has joined #tripleo20:36
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: Parameterize OVB stack flavors  https://review.openstack.org/44516320:37
trownpanda: just merged ^20:37
trown:)20:37
*** jkilpatr has quit IRC20:37
trownEmilienM: can we go ahead and merge https://review.openstack.org/443687 to unscrew the gates?20:37
EmilienMok20:38
EmilienMgate is broken?20:38
EmilienMtrown: how gate can be broken if we tested everything before?20:38
*** prateek has quit IRC20:38
trownit will pass the job it failed there (or it wont merge if not anyways)20:38
EmilienMalso, a lot of jobs aren't passing in this patch20:39
trownEmilienM: gap of not testing actually building an external change20:39
*** dsariel has joined #tripleo20:39
EmilienMshould we ignore it?20:39
trownEmilienM: ya it will pass in merge queue... it was because of https://review.openstack.org/445597 which just merged20:39
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Add scenario005 scenario  https://review.openstack.org/41809920:39
EmilienMmhh ok... approving it20:39
trownhttps://review.openstack.org/445597 was from CentOS outage today... kind of bad timing to transition to new CI20:40
trownbut there is always something :P20:40
*** jmelvin has joined #tripleo20:40
*** gbarros has joined #tripleo20:41
trownEmilienM: that is what is blocking https://review.openstack.org/445226 btw... so once https://review.openstack.org/443687 merges we can recheck 44522620:41
bkeroThere's always something that break you during release day20:41
EmilienMtrown: k thx20:42
dmsimardEmilienM: fyi topic didn't get changed back in #tripleo (and a couple other places), discussing the issue in #openstack-infra but you probably have to change it back manually here20:42
EmilienMNg is the only admin here20:43
EmilienMand is probably afk at this time20:43
NgI am here20:43
Ngwhat do you need? :)20:44
dmsimardNg: just need to put the original topic back which was uuuuuhhhh20:45
EmilienMNg: nice20:45
dmsimardTripleO : http://tripleo.org/ | https://wiki.openstack.org/wiki/TripleO | Meetings On Tuesdays at 14:00 UTC in #openstack-meeting-alt20:45
EmilienMand maybe give to some of us +O20:45
EmilienMso i can kick folks who troll tripleo at deman20:45
EmilienMjoking20:45
*** Goneri has quit IRC20:46
bkeroDon't do it, he'll just kick me.20:46
EmilienMbkero: I will next time oooq breaks our CI :P20:46
bkeroEmilienM: present time excluded?20:46
EmilienMlol20:46
*** ChanServ sets mode: +v EmilienM20:47
NgEmilienM: you now have the same things I do (which, btw, I have no precious feelings about, if anyone wants to strip them from me, I would not feel hurt)20:48
*** Ng changes topic to "TripleO : http://tripleo.org/ | https://wiki.openstack.org/wiki/TripleO | Meetings On Tuesdays at 14:00 UTC in #openstack-meeting-alt"20:48
EmilienMthat's fine, as long as we have someone with operator here, in case we got spam20:49
*** bogdando has joined #tripleo20:49
Ngseems crazy for the PTL to not have superpowers :)20:49
EmilienMall: let me know who wants to be operator on this channel20:49
openstackgerritPaul Belanger proposed openstack/diskimage-builder master: Clean up cache for ubuntu-minimal  https://review.openstack.org/41472820:52
openstackgerritIan Main proposed openstack/tripleo-heat-templates master: Remove kolla config from heat services.  https://review.openstack.org/44397420:55
*** jkilpatr has joined #tripleo20:58
*** athomas has joined #tripleo20:59
*** trown is now known as trown|outtypewww20:59
*** toure is now known as toure|gone21:01
*** funzo has joined #tripleo21:01
*** funzo has quit IRC21:05
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: Parameterize OVB stack flavors  https://review.openstack.org/44516321:07
*** jpena|away is now known as jpena|off21:07
*** ooolpbot has joined #tripleo21:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION21:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273021:10
*** ooolpbot has quit IRC21:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)21:10
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: WIP: Conntainers with multi-nic and net-iso  https://review.openstack.org/42244821:12
*** ukalifon has joined #tripleo21:17
*** fultonj has quit IRC21:17
*** ElCoyote_ has quit IRC21:20
EmilienMtherve: quick stupid question21:24
EmilienMhttps://github.com/openstack/tripleo-heat-templates/blob/master/environments/hyperconverged-ceph.yaml#L721:24
EmilienMwhen changing ComputeServices with a template is it merging with the existing ComputeServices in the registry or is it replacing?21:24
EmilienMI think it's replacing but I'm not 100% sure21:24
therveEmilienM, It's replacing21:25
mwhahahatherve: is there a way to do a merge?21:25
thervemwhahaha, https://docs.openstack.org/developer/heat/template_guide/environment.html#environment-merging21:25
EmilienMtherve: merci21:25
mwhahahacool21:25
therveIt is a tricky feature (written for tripleo) but it (can) work21:26
mwhahahawe should be merging instead of replacing :D21:26
mwhahahaespecially for roles21:26
mwhahahait's on my list of things to look at this cycle with defining example roles21:26
EmilienMit works (tm)21:26
mwhahahaa likely story21:27
*** pradk has quit IRC21:30
*** lblanchard has quit IRC21:32
*** abehl has quit IRC21:33
*** jcoufal has quit IRC21:36
stevebakerbogdando: this is my nick :)21:38
dsneddonEmilienM, These instructions need to be updated with the stable/ocata branch, correct? https://docs.openstack.org/developer/tripleo-docs/basic_deployment/basic_deployment_cli.html21:38
dsneddonEmilienM, If so, should I file a launchpad bug?21:38
EmilienMdsneddon: most probably need to be updated, not sure about the need of a bug report21:39
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates master: Add BGPVPN services to scenario004  https://review.openstack.org/44355321:42
*** akrivoka has quit IRC21:42
*** dmacpher-afk has quit IRC21:46
dsneddonEmilienM, I'd submit a patch myself, but I don't know what the repo is for Otaca Ceph21:46
*** percevalbot has quit IRC21:47
EmilienMdsneddon: the version of Ceph in Ocata is Jewel21:47
*** ukalifon has quit IRC21:47
dsneddonEmilienM, That's what's currently listed for Newton. Should it be the same for Ocata?21:48
openstackgerritMerged openstack-infra/tripleo-ci master: Install packages with roles in multinode  https://review.openstack.org/44368721:48
EmilienMdsneddon: yep!21:48
EmilienMwe didn't upgrade Ceph AFIK21:48
dsneddonEmilienM, OK, I'll submit an update, and others can double check it21:49
EmilienMdsneddon: thanks a lot21:49
sshnaidmpanda, ok, so now everything should be ok?21:51
weshaybogdando, I'm rebasing that undercloud containers patch and testing it now21:53
dtrainorshoot, I can't bring up docs.openstack.org21:53
mwhahahado we have an ETA on a fix for the multinode jobs? http://logs.openstack.org/26/445226/3/check/gate-tripleo-ci-centos-7-nonha-multinode-oooq/2923d1a/console.html#_2017-03-14_20_44_14_25513821:56
*** dprince has quit IRC21:56
sshnaidmmwhahaha, it should work now21:56
*** pradk has joined #tripleo21:57
sshnaidmmwhahaha, fix was merged just a few minutes before21:57
*** bogdando has quit IRC21:57
* mwhahaha recheck ︵ /(.□. \)21:57
sshnaidmpanda, if this job passes, we can celebrate: telnet://146.20.103.102:1988521:57
bkeromwhahaha: That error is actually here: http://logs.openstack.org/26/445226/3/check/gate-tripleo-ci-centos-7-nonha-multinode-oooq/2923d1a/console.html#_2017-03-14_20_42_34_03990121:57
bkerogcerami fixed that a few hours ago21:58
bkeroin this review: https://review.openstack.org/#/c/443687/21:58
pandasshnaidm: we can celebrate until the next problem21:59
pandaand we still need stable branches to work right ?21:59
sshnaidmpanda, yeah, need to check them now again btw22:01
*** ukalifon has joined #tripleo22:01
*** funzo has joined #tripleo22:02
*** jmelvin has quit IRC22:02
*** kjw3 has quit IRC22:06
*** jayg is now known as jayg|g0n322:06
*** funzo has quit IRC22:06
*** ooolpbot has joined #tripleo22:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION22:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273022:10
*** ooolpbot has quit IRC22:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)22:10
*** ukalifon has quit IRC22:10
*** thrash is now known as thrash|g0ne22:13
*** salmankhan has quit IRC22:15
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: Parameterize OVB stack flavors  https://review.openstack.org/44516322:18
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: Parameterize OVB stack flavors  https://review.openstack.org/44516322:19
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: Parameterize OVB stack flavors  https://review.openstack.org/44516322:20
*** gbarros has quit IRC22:22
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: DO NOT MERGE - Adjust post command on repo install  https://review.openstack.org/44566622:23
*** jcoufal has joined #tripleo22:24
*** Goneri has joined #tripleo22:25
*** achadha has quit IRC22:27
openstackgerritMerged openstack/instack-undercloud stable/mitaka: Disable docker-registry by default  https://review.openstack.org/44552222:28
*** achadha has joined #tripleo22:28
*** rhallisey has quit IRC22:38
EmilienMtrown|outtypewww: did you submit a patch for the dashboard or not yet? I don't see it22:42
*** pradk has quit IRC22:44
*** dsariel has quit IRC22:46
openstackgerritSagi Shnaidman proposed openstack/tripleo-heat-templates stable/ocata: DONT MERGE: testing ocata with scenarios  https://review.openstack.org/44467622:50
openstackgerritSagi Shnaidman proposed openstack/tripleo-heat-templates stable/newton: DONT REVIEW: test scnearios on newton  https://review.openstack.org/44467422:51
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: Add scenario005 job (4nodes)  https://review.openstack.org/41809822:52
*** dmarlin has quit IRC22:56
*** hjensas has joined #tripleo23:00
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo master: Deploy WSGI apps at the same step (3)  https://review.openstack.org/43401623:00
openstackgerritDan Sneddon proposed openstack/tripleo-docs master: Update installation instructions to include stable/ocata branch  https://review.openstack.org/44567623:00
*** hjensas has quit IRC23:00
*** hjensas has joined #tripleo23:00
*** hjensas has quit IRC23:01
EmilienMdsneddon: thanks23:01
*** hjensas has joined #tripleo23:01
dsneddonEmilienM, Sure, no problem. I probably use those instructions more often than nearly anyone else.23:01
EmilienM:D23:02
EmilienMDOC DOC DOC repeat after me23:02
*** achadha has quit IRC23:02
*** funzo has joined #tripleo23:03
*** rhallisey has joined #tripleo23:04
*** saneax is now known as saneax-_-|AFK23:05
*** pkovar has quit IRC23:06
*** funzo has quit IRC23:08
*** ooolpbot has joined #tripleo23:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION23:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/167273023:10
*** ooolpbot has quit IRC23:10
openstackLaunchpad bug 1672730 in tripleo "instack-undercloud pep8 failures due to broken dib-lint" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)23:10
*** Goneri has quit IRC23:12
bkero"overcloud_deploy_result": "passed"23:17
pandayep23:18
bkero\o/ successful quickstart run23:19
dtrainorI'm having some serious difficulties connecting to review.openstack.org and a handful of tripleo.org sites today.  Anyone else experiencing anything similar?  I suspect my ISP, though.23:24
bkerodtrainor: a pastebin of curl -v might help23:27
*** karimb has quit IRC23:30
dtrainorit's a boring Network is Unreachable error, and since i can't hit paste.openstack.org, here's a pastebin http://pastebin.com/1hxEtZr3 nothing too exciting.  a traceroute stops inside of what I can see as Cox's network23:33
*** oaken_chris has joined #tripleo23:35
*** pmannidi has joined #tripleo23:41
*** panda is now known as panda|zZ23:42
*** dhill_ has quit IRC23:43
*** dhill_ has joined #tripleo23:45
weshayhrm.. containerized undercloud..23:47
weshayError: Package: 1:openstack-heat-monolith-8.0.0-0.20170309072146.990f484.el7.centos.noarch (delorean-tripleo)23:47
weshay           Requires: openstack-heat-common = 1:8.0.0-0.20170309072146.990f484.el7.centos23:47
weshay           Installed: 1:openstack-heat-common-8.0.0-0.20170314053434.148e969.el7.centos.noarch (@delorean)23:47
weshay               openstack-heat-common = 1:8.0.0-0.20170314053434.148e969.el7.centos23:47
weshay           Available: 1:openstack-heat-common-8.0.0-0.20170309072146.990f484.el7.centos.noarch (delorean-tripleo)23:47
weshay               openstack-heat-common = 1:8.0.0-0.20170309072146.990f484.el7.centos23:47
openstackgerritwes hayutin proposed openstack/tripleo-quickstart-extras master: Use openstack deploy to install the undercloud  https://review.openstack.org/41904023:47
*** marst has quit IRC23:50
*** jcoufal has quit IRC23:51

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