Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-10-06

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:35 openstackgerrit Mohammed Naser proposed openstack/puppet-modulesync-configs master: Update test-requirements reno release to 2.5.1  https://review.openstack.org/509665
01:16 openstackstatus joined #puppet-openstack
01:31 zigo joined #puppet-openstack
01:55 ilbot3 joined #puppet-openstack
01:55 Topic for #puppet-openstack is now CI Status: GREENish (Most jobs are fixed for Zuulv3, but we could still have edge cases) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/
02:07 afisher joined #puppet-openstack
02:10 jistr joined #puppet-openstack
02:10 mtreinish joined #puppet-openstack
02:11 higgins` joined #puppet-openstack
02:24 mgagne joined #puppet-openstack
02:58 openstackgerrit Merged openstack/puppet-openstack-integration master: Install python-cinderclient in integration  https://review.openstack.org/508169
02:58 openstackgerrit Merged openstack/puppet-nova master: Remove Class nova::rabbitmq  https://review.openstack.org/508672
03:09 openstackgerrit Merged openstack/puppet-oslo master: Add use_journal option for logging configuration  https://review.openstack.org/508497
03:24 markvoelker joined #puppet-openstack
03:38 udesale joined #puppet-openstack
03:42 udesale joined #puppet-openstack
04:02 ykarel joined #puppet-openstack
04:21 jaosorior joined #puppet-openstack
04:23 asilenkov joined #puppet-openstack
04:48 skramaja joined #puppet-openstack
04:54 markvoelker joined #puppet-openstack
05:05 cgfbee joined #puppet-openstack
05:16 ratailor joined #puppet-openstack
05:18 udesale joined #puppet-openstack
05:24 yprokule joined #puppet-openstack
05:27 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove revocation_cache_time parameter  https://review.openstack.org/509971
05:38 ykarel_ joined #puppet-openstack
05:54 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove useless method file_path  https://review.openstack.org/509975
05:56 spectr joined #puppet-openstack
05:57 jaganathan joined #puppet-openstack
06:01 openstackgerrit yatin proposed openstack/puppet-zaqar master: Set *_domain_name vars to 'Default' by default  https://review.openstack.org/509977
06:16 openstackgerrit yatin proposed openstack/puppet-openstack-integration master: Use --os-identity-api-version 3 explicitly in osc  https://review.openstack.org/509978
06:16 ykarel__ joined #puppet-openstack
06:17 rcernin joined #puppet-openstack
06:18 clayton joined #puppet-openstack
06:24 markvoelker joined #puppet-openstack
06:28 karthiks joined #puppet-openstack
06:33 udesale joined #puppet-openstack
06:40 dtantsur joined #puppet-openstack
06:43 rcernin joined #puppet-openstack
06:59 jtomasek joined #puppet-openstack
07:03 openstackgerrit Daniel Pawlik proposed openstack/puppet-nova stable/ocata: Ensure that policy file exist  https://review.openstack.org/509710
07:07 openstackgerrit yatin proposed openstack/puppet-ec2api master: Set *_domain_name vars to 'Default' by default  https://review.openstack.org/509988
07:14 agurenko joined #puppet-openstack
07:14 openstackgerrit Daniel Pawlik proposed openstack/puppet-nova stable/ocata: Ensure that policy file exist  https://review.openstack.org/509710
07:20 ccamacho joined #puppet-openstack
07:21 gfidente joined #puppet-openstack
07:25 tesseract joined #puppet-openstack
07:39 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/509998
07:54 markvoelker joined #puppet-openstack
08:46 derekh joined #puppet-openstack
08:52 salmankhan joined #puppet-openstack
09:01 openstackgerrit Wojciech Dec proposed openstack/puppet-neutron master: Adding Cisco VTS ML2 mechanism driver configuration manifest  https://review.openstack.org/509807
09:09 openstackgerrit Merged openstack/puppet-nova master: Accept empty strings for log_dir  https://review.openstack.org/509105
09:13 openstackgerrit Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3  https://review.openstack.org/510022
09:25 markvoelker joined #puppet-openstack
09:32 amoralej jaosorior, re https://review.openstack.org/#/c/510022/ i can move it to keystone::resources::authtoken if it's better
09:34 spectr joined #puppet-openstack
09:36 spectr joined #puppet-openstack
09:36 openstackgerrit Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3  https://review.openstack.org/510022
09:37 spectr joined #puppet-openstack
09:40 spectr joined #puppet-openstack
09:40 jaosorior amoralej: that would be good :)
09:41 spectr joined #puppet-openstack
09:43 spectr joined #puppet-openstack
09:47 yprokule_ joined #puppet-openstack
09:48 amoralej jaosorior, common pattern is to create a class <service>::keystone::authtoken to configure that and include it from init.pp
09:48 amoralej however i don't want to break backwards compatibility
09:49 amoralej so i'm calling keystone::resource::authtoken from init.pp directly, is that ok?
09:49 amoralej is there a better way?
09:51 chem joined #puppet-openstack
09:57 spectr joined #puppet-openstack
09:58 spectr joined #puppet-openstack
10:09 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-ceilometer master: Accept empty strings for log_dir  https://review.openstack.org/510043
10:09 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-cinder master: Accept empty strings for log_dir  https://review.openstack.org/510044
10:09 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-glance master: Accept empty strings for log_dir  https://review.openstack.org/510045
10:10 jaosorior amoralej: that's fine for me
10:10 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-heat master: Accept empty strings for log_dir  https://review.openstack.org/510046
10:10 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Accept empty strings for log_dir  https://review.openstack.org/510047
10:10 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-keystone master: Accept empty strings for log_dir  https://review.openstack.org/510048
10:10 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-manila master: Accept empty strings for log_dir  https://review.openstack.org/510049
10:10 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-mistral master: Accept empty strings for log_dir  https://review.openstack.org/510050
10:10 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-neutron master: Accept empty strings for log_dir  https://review.openstack.org/510051
10:16 ykarel_ joined #puppet-openstack
10:21 ykarel__ joined #puppet-openstack
10:28 openstackgerrit Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3  https://review.openstack.org/510022
10:56 markvoelker joined #puppet-openstack
10:57 openstackgerrit Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3  https://review.openstack.org/510022
11:10 salmankhan joined #puppet-openstack
11:14 openstackgerrit Alfredo Moralejo proposed openstack/puppet-murano master: Add parameters for keystone API v3  https://review.openstack.org/510022
11:15 openstackgerrit Wojciech Dec proposed openstack/puppet-neutron master: Adding Cisco VTS ML2 mechanism driver configuration manifest  https://review.openstack.org/509807
11:23 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-barbican master: Allow setting log files for apache vhost  https://review.openstack.org/510064
11:23 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-ceilometer master: Allow setting log files for apache vhost  https://review.openstack.org/510065
11:23 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-cinder master: Allow setting log files for apache vhost  https://review.openstack.org/510066
11:23 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-gnocchi master: Allow setting log files for apache vhost  https://review.openstack.org/510067
11:23 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-heat master: Allow setting log files for apache vhost  https://review.openstack.org/510068
11:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Allow setting log files for apache vhost  https://review.openstack.org/510069
11:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-mistral master: Allow setting log files for apache vhost  https://review.openstack.org/510070
11:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-neutron master: Allow setting log files for apache vhost  https://review.openstack.org/510071
11:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-panko master: Allow setting log files for apache vhost  https://review.openstack.org/510072
11:24 amoralej mwhahaha, mnaser iurygregory could you review https://review.openstack.org/#/q/topic:keystone_v3+status:open+project:%255Eopenstack/puppet.* when you have a chance?
11:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-zaqar master: Allow setting log files for apache vhost  https://review.openstack.org/510073
11:24 amoralej we need it to unblock promotions
11:33 morazi joined #puppet-openstack
12:03 spectr joined #puppet-openstack
12:03 dprince joined #puppet-openstack
12:05 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove Class nova::wsgi::apache  https://review.openstack.org/510081
12:06 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove Class nova::wsgi::apache  https://review.openstack.org/510081
12:26 markvoelker joined #puppet-openstack
12:31 markvoelker joined #puppet-openstack
12:39 spectr joined #puppet-openstack
12:44 iurygregory amoralej|away, sure
12:45 ansmith joined #puppet-openstack
12:46 ansmith_ joined #puppet-openstack
12:57 openstackgerrit Matthew J Black proposed openstack/puppet-horizon master: Add parameter to configure instance defaults  https://review.openstack.org/510098
12:57 openstackgerrit Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/509021
13:04 salmankhan joined #puppet-openstack
13:06 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-barbican master: Configure *_domain_name to Default by default  https://review.openstack.org/510102
13:12 bnemec joined #puppet-openstack
13:33 openstackgerrit Harry Rybacki proposed openstack/puppet-ceilometer master: Configure *_domain_name to Default by default  https://review.openstack.org/510114
13:34 pradk joined #puppet-openstack
13:36 spectr joined #puppet-openstack
13:38 jaosorior dmsimard: a release note for accepting empty string now?
13:38 jaosorior dtantsur: not dmsimard
13:38 dtantsur jaosorior: that's a feature right? now you can enable logging to stderr
13:39 jaosorior I don't really see it as a feature to be honest
13:39 jaosorior dtantsur: more like, it has always been possible, but it wasn't possible cause of the way the code was made
13:39 dtantsur jaosorior: then it's a bug fix, right? both deserve a release note
13:40 jaosorior ..
13:40 jaosorior alright
13:43 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Allow setting log files for apache vhost  https://review.openstack.org/510069
13:43 openstackgerrit Harry Rybacki proposed openstack/puppet-cinder master: Configure *_domain_name to Default by default  https://review.openstack.org/510119
13:45 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Accept empty strings for log_dir  https://review.openstack.org/510047
13:46 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-zaqar master: Allow setting log files for apache vhost  https://review.openstack.org/510073
13:47 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Allow setting log files for apache vhost  https://review.openstack.org/510069
13:50 openstackgerrit Harry Rybacki proposed openstack/puppet-glance master: Configure *_domain_name to Default by default  https://review.openstack.org/510121
13:56 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-ironic master: Accept empty strings for log_dir  https://review.openstack.org/510047
13:57 openstackgerrit Harry Rybacki proposed openstack/puppet-glance master: Configure *_domain_name to Default by default  https://review.openstack.org/510121
13:58 dtantsur jaosorior: +2 to both, thanks
13:59 openstackgerrit Bob Fournier proposed openstack/puppet-ironic stable/pike: Increase Ironic inspector dnsmasq lease time  https://review.openstack.org/510125
14:00 jaosorior dtantsur: thanks for the reviews
14:00 dtantsur np
14:00 ratailor joined #puppet-openstack
14:04 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Update reno to 2.5.1  https://review.openstack.org/510128
14:09 archit joined #puppet-openstack
14:14 openstackgerrit Merged openstack/puppet-murano master: Add parameters for keystone API v3  https://review.openstack.org/510022
14:14 mnaser jaosorior thanks for your patches for logging :D
14:15 jaosorior mnaser: thanks for your reviews
14:19 mnaser :)
14:21 amoralej mwhahaha, iurygregory thanks for all those reviews
14:21 iurygregory amoralej, yw, did i missed something?
14:21 iurygregory many tabs open here XD
14:21 amoralej no, i mean, the keysone_v3 reviews
14:23 openstackgerrit Harry Rybacki proposed openstack/puppet-ironic master: Configure *_domain_name to Default by default  https://review.openstack.org/510134
14:26 mnaser github has so many issues :(
14:26 mnaser i wonder if cloning over http might be more sane
14:26 mnaser err, i mean over ssh
14:29 mnaser reading online, folks seem to report that ssh is better
14:34 openstackgerrit Harry Rybacki proposed openstack/puppet-manila master: Configure *_domain_name to Default by default  https://review.openstack.org/510139
14:39 mjblack mwhahaha: got a question, what am I missing in my spec changes in https://review.openstack.org/#/c/510098 that is causing the errors
14:40 mwhahaha mjblack: not sure i'll see when tests come back but i noticed an issue with the verify_concat_fragment_contents function
14:40 mwhahaha mjblack: if one of the lines is duplicated in that array it won't match
14:41 mjblack oh
14:41 mwhahaha ie "]" only matches once
14:41 mwhahaha or "}" rather
14:41 mjblack ah ok, now I understand
14:42 bnemec joined #puppet-openstack
14:42 openstackgerrit Harry Rybacki proposed openstack/puppet-octavia master: Configure *_domain_name to Default by default  https://review.openstack.org/510141
14:45 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove some useless parameters  https://review.openstack.org/510143
14:48 jaosorior EmilienM: could you check this out https://review.openstack.org/#/c/509127/ ?
14:50 Topic for #puppet-openstack is now CI Status: YELLOWish - scenarios 1/4 failing (Most jobs are fixed for Zuulv3, but we could still have edge cases) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/
14:50 ntpttr_laptop joined #puppet-openstack
14:52 mjblack mwhahaha: not seeing any duplicates
14:53 mwhahaha yea that bit is so hard to debug
14:53 mwhahaha i'll take a look later when the results post
14:54 openstackgerrit Harry Rybacki proposed openstack/puppet-panko master: Configure *_domain_name to Default by default  https://review.openstack.org/510147
14:57 EmilienM jaosorior: done
14:58 openstackgerrit Harry Rybacki proposed openstack/puppet-sahara master: Configure *_domain_name to Default by default  https://review.openstack.org/510150
14:59 mjblack mwhahaha: I found it, forgot to set a parameter in the hash
14:59 mwhahaha oops
14:59 mjblack heh, well at least it wasnt something major
14:59 dgurtner joined #puppet-openstack
14:59 dgurtner joined #puppet-openstack
15:01 openstackgerrit Matthew J Black proposed openstack/puppet-horizon master: Add parameter to configure instance defaults  https://review.openstack.org/510098
15:04 spectr joined #puppet-openstack
15:06 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove parameter nova::compute::pci_passthrough  https://review.openstack.org/510151
15:08 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove parameter nova::compute::pci_passthrough  https://review.openstack.org/510151
15:09 bnemec joined #puppet-openstack
15:25 andreww joined #puppet-openstack
15:36 openstackgerrit Wojciech Dec proposed openstack/puppet-neutron master: Adding Cisco VTS ML2 mechanism driver configuration manifest  https://review.openstack.org/509807
15:59 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove some rabbit parameters  https://review.openstack.org/510167
16:00 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove some rabbit parameters  https://review.openstack.org/510167
16:01 openstackgerrit zhangyangyang proposed openstack/puppet-nova master: Remove some rabbit parameters  https://review.openstack.org/510167
16:04 EmilienM mnaser: can I get a review on https://review.openstack.org/#/c/509850/ please?
16:13 mnaser EmilienM sure one sec, just checking 1 thing
16:16 openstackgerrit Alex Schultz proposed openstack/puppet-nova master: Allow setting log files for nova-api apache vhost  https://review.openstack.org/509127
16:17 openstackgerrit Alex Schultz proposed openstack/puppet-nova master: Allow setting log files for nova-placement apache vhost  https://review.openstack.org/509151
16:18 openstackgerrit Merged openstack/puppet-zaqar master: Set *_domain_name vars to 'Default' by default  https://review.openstack.org/509977
16:19 mwhahaha murano appears broken
16:19 mwhahaha http://logs.openstack.org/71/510071/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/8172604/logs/puppet.txt.gz#_2017-10-06_14_41_18
16:19 mwhahaha fyi
16:22 EmilienM BLAH
16:22 mnaser isn't murano like
16:22 mnaser unmaintained? i think?
16:22 mnaser wait or did we break it with the keystone v3 changes
16:23 mwhahaha probably broke it with v3
16:23 mwhahaha but yea we might need to drop it from scenario testing
16:23 mnaser ooooh
16:23 mnaser mwhahaha i found it
16:23 mwhahaha k
16:23 mnaser we got rid of admin_user etc
16:24 mnaser instead its keystone_authtoken/username etc
16:24 mnaser https://github.com/openstack/puppet-murano/blob/master/lib/puppet/provider/murano.rb#L68-L76
16:24 mnaser so these are wrong
16:24 mnaser 1 sec ill push up a patch
16:29 openstackgerrit Merged openstack/puppet-ec2api master: Set *_domain_name vars to 'Default' by default  https://review.openstack.org/509988
16:33 mnaser lol
16:33 mnaser puppet-murano was nonvoting on debian
16:33 mnaser but it was only testing on debian
16:33 mnaser and it was voting on centos
16:33 mnaser and doing nothing
16:33 mnaser im refactoring it to use p-o-i
16:34 openstackgerrit Mohammed Naser proposed openstack/puppet-murano master: Fix murano base provider  https://review.openstack.org/510178
16:34 mnaser EmilienM mwhahaha ^
16:34 morazi joined #puppet-openstack
16:35 EmilienM woops
16:35 EmilienM mnaser: ok thx
16:42 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Add un-tested resources to Murano integration  https://review.openstack.org/510183
16:43 openstackgerrit Mohammed Naser proposed openstack/puppet-murano master: Refactor to use integration classes  https://review.openstack.org/510184
16:43 mnaser there, that should avoid us having that happen again
16:59 bnemec joined #puppet-openstack
17:03 openstackgerrit Alfredo Moralejo proposed openstack/puppet-murano master: Switch murano auth provider to new authtoken params  https://review.openstack.org/510187
17:05 openstackgerrit Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Use --os-identity-api-version 3 explicitly in osc  https://review.openstack.org/509978
17:07 openstackstatus joined #puppet-openstack
17:09 mnaser amoralej https://review.openstack.org/510178 :p
17:09 openstack joined #puppet-openstack
17:10 amoralej mnaser, oh, you catched it first
17:12 openstackgerrit Merged openstack/puppet-cinder master: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/508926
17:12 mwhahaha mnaser: https://review.openstack.org/#/c/510178/ does it need to be OS_PROJECT_NAME instead?
17:13 mnaser mwhahaha im waiting for xenial beaker to run, i think it will be okay (ill fix it in a follow up patch so we don't sit in queue for much longer)
17:13 mwhahaha k
17:13 mnaser if its not okay ill edit this patch but im about 90% sure itll beo kay
17:13 mwhahaha xenial will probably pass because they haven't gotten the v2 removal yet
17:14 mwhahaha which reminds me i think we still deploy pike
17:17 ntpttr_laptop joined #puppet-openstack
17:18 mnaser mwhahaha it got the v2 removal, thats why it failed
17:18 mnaser but because beaker was non voting (and failing for some other reason)
17:18 mnaser it was merged, even though it failed for the same "missing credentials" but i guess it went unnoticed
17:19 mnaser http://logs.openstack.org/22/510022/5/check/gate-puppet-murano-puppet-beaker-rspec-ubuntu-xenial-nv/036d362/console.html#_2017-10-06_11_26_17_689474
17:20 openstackgerrit Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Use --os-identity-api-version 3 explicitly in osc  https://review.openstack.org/509978
17:21 amoralej mnaser, could we run p-o-i scenario003 on puppet-murano gate?
17:21 amoralej that would have detected the issue in my previous patch
17:22 mnaser amoralej i think we will do that but project-config is frozen right now due to zuulv3 stuff so i think that is a good path to go towards
17:23 amoralej ahh, right
17:23 mnaser amoralej also, i refactored the codebase to use the integration code for beaker tests, so it will run the same stuff and hopefully (properly) fail both in beaker and integration for now
17:23 amoralej that'd be nice
17:24 amoralej mnaser, EmilienM, i'm running pipeline job to check if we find more hidden issues with api v3 https://ci.centos.org/view/rdo/view/weirdo-pipelines/view/weirdo-promote-puppet/
17:24 EmilienM amoralej: excellent
17:24 amoralej so far only murano and designate should fail
17:24 amoralej all other are fixed
17:30 mnaser amoralej awesome work
17:31 amoralej not until we got a new promotion
17:33 amoralej s/got/get/
17:33 openstackgerrit Merged openstack/puppet-ceilometer master: Accept empty strings for log_dir  https://review.openstack.org/510043
17:39 mnaser we should totally
17:39 mnaser run all of our jobs in centos7 (The ones we can at least)
17:39 mnaser way less jobs use them, which means we get vms much faster
17:39 mnaser murano job is been in queue for an hour, all centos jobs of it were done loooong ago
17:43 mwhahaha ehhhh we've had the opposite before
17:43 mwhahaha we already do run most of them on centos7 :D
17:45 ntpttr_laptop joined #puppet-openstack
17:58 amoralej https://ci.centos.org/view/rdo/view/weirdo-pipelines/view/weirdo-promote-puppet/ only scenario003 is failing because of the missing patches which are in the gates, so we should be ok soon
18:08 amoralej mnaser, there seems to still be some issue with murano, scenario003 has failed for https://review.openstack.org/#/c/509978/ with depends on https://review.openstack.org/#/c/510178/
18:08 amoralej it seems it's not connecting to keystone properly
18:08 amoralej i need to leave now
18:08 mnaser amoralej ok thanks for investigating ill have a look
18:08 amoralej it seems to try to connect using 127.0.0.1:35357 but it's only listening in ipv1 ::1:35357
18:09 amoralej ipv6 i meant
18:09 amoralej http://logs.openstack.org/78/509978/3/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/2a7d72c/
18:09 mnaser 2017-10-06 17:55:35 +0000 Puppet (err): Failed to apply catalog: Execution of '/bin/murano package-list' returned 1: 503 Service Unavailable: The server is currently unavailable. Please try again at a later time. (HTTP 503)
18:09 mnaser indeed
18:10 mnaser i think p-o-i might need a fix for that
18:10 mnaser 2017-10-06 17:46:14.319 31434 DEBUG oslo_service.service [-] keystone_authtoken.auth_uri    = https://[::1]:5000 log_opt_values /usr/lib/python2.7/site-packages/oslo_config/cfg.py:2887
18:11 amoralej mnaser, but authtoken config seems ok http://logs.openstack.org/78/509978/3/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/2a7d72c/logs/etc/murano/murano.conf.txt.gz
18:11 amoralej Unable to validate token: Unable to establish connection to https://127.0.0.1:35357: HTTPSConnectionPool(host='127.0.0.1', port=35357): Max retries exceeded with u
18:11 amoralej it's weird
18:11 mnaser 2017-10-06 17:46:14.218 31434 WARNING keystonemiddleware.auth_token [-] Configuring admin URI using auth fragments was deprecated in the Kilo release, and will be removed in the Newton release, use 'identity_uri\ instead.
18:11 mnaser 2017-10-06 17:46:14.322 31434 DEBUG oslo_service.service [-] keystone_authtoken.identity_uri = None log_opt_values /usr/lib/python2.7/site-packages/oslo_config/cfg.py:2887
18:12 amoralej ups, that's my fault, :(
18:12 amoralej sorry, i need to leave now
18:13 mnaser amoralej np, ill fix it
18:13 mnaser i think because murano is unmaintained, its relying on identity_uri and ignoring auth_uri
18:25 openstackgerrit Mohammed Naser proposed openstack/puppet-murano master: Fix murano base provider  https://review.openstack.org/510178
18:25 openstackgerrit Mohammed Naser proposed openstack/puppet-murano master: Refactor to use integration classes  https://review.openstack.org/510184
18:25 openstackgerrit Mohammed Naser proposed openstack/puppet-murano master: Add identity_uri back to keystone_authtoken options  https://review.openstack.org/510206
18:28 mnaser that should fix it
18:54 openstackgerrit Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/509021
19:05 mnaser uhoh
19:07 mnaser EmilienM mwhahaha so it looks like murano codebase has not been updated, therefore still lives on the admin_* variables http://logs.openstack.org/78/509978/3/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/1e2cbe6/logs/murano/murano-api.txt.gz#_2017-10-06_18_48_14_514
19:07 mwhahaha time to non-vote and exclude murano it would seem
19:08 mnaser https://github.com/openstack/murano/commit/bd76070b138477013d737e85cd427efa1f6e5b3d
19:08 mnaser it would seem that it was "fixed" but more like patched up
19:08 mnaser it looks like it's pinned to v2, regardless
19:08 mwhahaha https://review.openstack.org/#/c/504122/
19:09 mwhahaha was just dealing with sessions too
19:09 mnaser i'll disable it with a note explaining the issue
19:11 mwhahaha https://review.openstack.org/#/c/503598/
19:11 mwhahaha that'll fix it
19:11 mwhahaha i think
19:11 openstackgerrit Merged openstack/puppet-manila master: Accept empty strings for log_dir  https://review.openstack.org/510049
19:11 * mwhahaha doesn't know murano code but seems like it might
19:12 mwhahaha looks like CI ate the change
19:12 mwhahaha it should have been merged days ago
19:13 mnaser mwhahaha trying to understand what is the point of "murano_auth"
19:14 mwhahaha murano didn't properly use keystoneauth originally
19:14 pester joined #puppet-openstack
19:14 mwhahaha so they had their own murano_auth section
19:14 mwhahaha seems that it's been updated at some point to use keystoneauth
19:14 mwhahaha but they just got around to removing the murano_auth bits
19:14 mwhahaha we haven't really touched the murano module much since fuel died
19:14 mwhahaha fuel was where murano was being pushed
19:15 mnaser i think they are using murano_auth instead of keystone_authtoken (not the other way around)
19:15 mnaser "Since [murano_auth] section is introduced in Pike, no need to fallback to [keystone_authtoken] now."
19:15 mnaser seems like they depend on it
19:15 mnaser "_create_keystone_admin_client" uses those credentials
19:15 * mnaser looks at murano docs
19:16 mwhahaha oh you're right
19:16 mwhahaha they've moved out of keystone auth to murano_auth
19:16 mnaser <shrug ascii>
19:17 mnaser mwhahaha i vote revert amoralej|off patch which made that change (because it's technically broken), remove murano out of p-o-i (till they fix their stuff?)
19:17 mwhahaha i doubt it'll ever get fixed
19:17 mwhahaha but sure
19:18 mwhahaha might want to send an ML note about it being completely broken
19:19 mnaser mwhahaha http://logs.openstack.org/98/503598/1/check/gate-tempest-dsvm-murano-api-ubuntu-xenial/f9cbaa0/logs/etc/murano/murano.conf.txt.gz this config passed their ci
19:20 mwhahaha maybe we just need to s/keystoneauth/murano_auth?
19:20 * mwhahaha shrugs
19:20 mnaser wait wait
19:20 mnaser mwhahaha is it possible we are not testing against new packages because we havent been able to get the puppet packages promoted?
19:20 mwhahaha yes
19:20 mnaser i think this might be the cause
19:21 mnaser they have both murano_auth and keystone_authtoken configured
19:21 mwhahaha well that drop change didn't merge yet
19:21 mwhahaha it got lost
19:25 mnaser mwhahaha ill attempt configuring both keystone_authtoken and murano_auth with the same info and maybe
19:26 mnaser aw shucks, keystone::resource::authtoken is hardcoded with keystone_authtoken
19:31 openstackgerrit Mohammed Naser proposed openstack/puppet-murano master: Add murano_auth configuration  https://review.openstack.org/510206
19:31 openstackgerrit Mohammed Naser proposed openstack/puppet-murano master: Fix murano base provider  https://review.openstack.org/510178
19:31 openstackgerrit Mohammed Naser proposed openstack/puppet-murano master: Refactor to use integration classes  https://review.openstack.org/510184
20:34 spectr joined #puppet-openstack
21:11 mnaser sigh, it failed against, so frustrating
21:12 mnaser i'll look into it and try to take a decision later today
21:23 openstackgerrit Merged openstack/puppet-panko master: Allow setting log files for apache vhost  https://review.openstack.org/510072
21:23 openstackgerrit Merged openstack/puppet-ironic master: Allow setting log files for apache vhost  https://review.openstack.org/510069
21:24 openstackgerrit Merged openstack/puppet-ironic master: Accept empty strings for log_dir  https://review.openstack.org/510047
21:24 openstackgerrit Merged openstack/puppet-ironic stable/pike: Increase Ironic inspector dnsmasq lease time  https://review.openstack.org/510125
22:50 ntpttr_laptop joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary