Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-06-26

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

All times shown according to UTC.

Time Nick Message
00:02 EmilienM crinkle: I think we still need Exec['apt_update'] -> Package<||>
00:03 EmilienM crinkle: I'm about to produce a patch, let me know if you think we can do better
00:04 crinkle EmilienM: _morgan ^ may be able to help if we can point her at some logs
00:04 EmilienM _morgan: http://logs.openstack.org/63/195163/1/check/gate-puppet-neutron-puppet-beaker-rspec-dsvm-trusty/6159af9/console.html
00:05 _morgan EmilienM: looking
00:05 EmilienM _morgan: my fix from this morning: https://github.com/openstack/puppet-openstack_extras/commit/6feda8b3e978a3b588d9f84d3e3776f0993c2623
00:05 EmilienM and I'm doing a patch now
00:05 crinkle EmilienM: i don't actually see where apt is failing there
00:05 EmilienM crinkle: it's not failing
00:06 EmilienM it's just a wrong orchestration here
00:06 crinkle oh i see
00:06 EmilienM yeah
00:06 EmilienM we need to run apt-get update before ANY package resoruce
00:06 EmilienM but we CANT
00:06 EmilienM because we install a package *before*
00:06 EmilienM so we need to replace the Package resource by an exec
00:06 nibalizer put apt-get update in a fact
00:06 EmilienM and change the conditional
00:07 EmilienM nibalizer: apt module suggest to use an exec
00:07 EmilienM but well, why not
00:07 _morgan nibalizer: oh please no
00:07 nibalizer EmilienM: im like 99% kidding
00:08 * xarses kicks nibalizer
00:08 EmilienM nibalizer: it's not Friday.
00:11 _morgan EmilienM: does it work if you use the deprecated required_packages parameter?
00:11 EmilienM no
00:11 EmilienM this thing looked broken this morning
00:11 EmilienM and we could not find why
00:11 xarses Does the trusty/centos7 error on https://review.openstack.org/#/c/179645/ make sense? "/home/jenkins/workspace/gate-puppet-ceph-puppet-beaker-rspec-dsvm-trusty/spec/acceptance/nodesets/nodepool-trusty.yml' does not exist"
00:12 _morgan :(
00:12 EmilienM and it's deprecated anyway so don't want to use it
00:12 EmilienM otherwise we will hit the issue in 2 months
00:12 _morgan yeah I was wondering if it needed to be added back, if things are weird with using a package resource
00:13 crinkle EmilienM: we're still using 1.8.0 http://logs.openstack.org/63/195163/1/check/gate-puppet-neutron-puppet-beaker-rspec-dsvm-trusty/6159af9/console.html#_2015-06-25_19_17_50_249
00:13 crinkle maybe *unpinning* would fix it :P
00:14 _morgan 2.1.0 should make things happier :D
00:14 * _morgan crosses fingers
00:14 crinkle xarses: does puppet-ceph have a spec/acceptance/nodesets/nodepool-trusty.yml file?
00:14 xarses Ahh, It looks like the defaults moved to trusty/cent7 but we don't have the nodeyaml for it
00:14 crinkle we might not have added it
00:14 xarses crinkle: no, It's missing
00:15 crinkle xarses: you can copy the spec/acceptance/nodesets/* from another module into puppet-ceph
00:17 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
00:17 EmilienM _morgan, crinkle ^
00:17 EmilienM crinkle: the pb is not in apt now, it's in my patch of this morning
00:17 EmilienM crinkle: I did a mistake
00:18 crinkle EmilienM: did you see my message above ^
00:18 crinkle EmilienM: we should try unpinning the apt module in spec_helper_acceptance.rb
00:19 EmilienM crinkle: oh yes...
00:19 EmilienM crinkle: it's inconsistent
00:19 EmilienM across our modules
00:20 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
00:20 EmilienM crinkle: let's see ^
00:23 EmilienM crinkle: I think, even with that patch, we still need to either revert my mistake of this morning or consider https://review.openstack.org/195801
00:24 markvoelker joined #puppet-openstack
00:27 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
00:28 EmilienM crinkle, _morgan: what if 1/ we make sure apt is unpin everywhere 2/ consider https://review.openstack.org/195801 as the way to go, since it respect the new way of using puppetlabs-apt ?
00:30 crinkle EmilienM: but we're waiting to see if 1/ works on its own
00:30 openstackgerrit Andrew Woodward proposed stackforge/puppet-ceph: [WIP] Add missing nodepool files  https://review.openstack.org/195811
00:31 crinkle EmilienM: i do not see how using an exec to apt-get install a package could be the intended usage of the apt module
00:32 EmilienM crinkle: they say, if you need to install a package before the apt::source resource, you need to declare the package yourself. The problem is, we want to have all packages installed *after* the Apt_update thing
00:32 EmilienM otherwise it will lead to packaging errors in openstack
00:37 EmilienM crinkle: the unpin patch does not work
00:37 EmilienM (I tested it on my laptop)
00:37 EmilienM we have the orchestration issue
00:38 EmilienM I'm testing now the openstack_extra patch
00:38 EmilienM with unpin
00:40 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
00:41 gildub joined #puppet-openstack
00:42 aimon joined #puppet-openstack
00:49 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
00:53 xarses crinkle: I copied the older 1204 and cent66 ones we had back in to check that CI still works. I'll bump ubuntu, but I think centos doesn't work still
00:53 xarses (centos7)
00:54 xarses sweet, vagrant up failed
00:54 xingchao_ joined #puppet-openstack
00:58 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
00:58 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
01:11 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
01:12 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
01:12 EmilienM _morgan: as you can see, I'm trying to find a fix
01:12 EmilienM _morgan: not too durty :)
01:17 bklei joined #puppet-openstack
01:20 xarses joined #puppet-openstack
01:23 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
01:23 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
01:28 xavpaice joined #puppet-openstack
01:28 openstackgerrit Xingchao Yu proposed openstack/puppet-trove: Update README.md for project namespace change  https://review.openstack.org/195822
01:33 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
01:33 jasondotstar joined #puppet-openstack
01:34 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
01:35 openstackgerrit Xingchao Yu proposed openstack/puppet-designate: Update README.md for project namespace change  https://review.openstack.org/195827
01:41 openstackgerrit Xingchao Yu proposed openstack/puppet-ceilometer: Update README.md for project namespace change  https://review.openstack.org/195828
01:50 openstackgerrit Xingchao Yu proposed openstack/puppet-glance: Update README.md for project namespace change  https://review.openstack.org/195831
01:52 EmilienM _morgan: I need help :)
01:52 openstackgerrit Xingchao Yu proposed openstack/puppet-gnocchi: Update README.md for project namespace change  https://review.openstack.org/195832
01:52 EmilienM can you look at https://review.openstack.org/#/c/195801/ and https://review.openstack.org/#/c/195803/ logs ?
01:55 openstackgerrit Xingchao Yu proposed openstack/puppet-heat: Update README.md for project namespace change  https://review.openstack.org/195833
01:57 crinkle i think she's out for the day
01:59 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
01:59 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
02:09 EmilienM crinkle: no prob, I make a short break and I continue. I succeed to have something working on my laptop :)
02:10 jasondotstar joined #puppet-openstack
02:14 markvoelker joined #puppet-openstack
02:15 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
02:16 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
02:18 dmsimard EmilienM: Doing some groundwork for cinder provider.. Starting by what's missing in openstackclient, looks like I have a lot of work ahead of me
02:19 EmilienM dmsimard: good luck... is it worth? :)
02:20 dmsimard EmilienM: Learning a lot. Trying to implement the qos commands in openstackclient (use case is automating qos configs for Ceph iops throttling with libvirt..)
02:20 dmsimard The reviewers are picky but it's all fair game. Openstackclient is actually fairly well built.
02:20 EmilienM nice
02:21 EmilienM dmsimard: keep us updated!
02:21 dmsimard I'm more versed in python than ruby.. I suck at ruby, so this will take a while :p
02:22 dmsimard Well, I guess it's not so much that I suck at ruby that I don't like the language.
02:33 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
02:34 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
02:47 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
02:47 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
02:47 xingchao_ joined #puppet-openstack
02:51 EmilienM crinkle: I've tested https://review.openstack.org/195801 and it solves the problem
02:51 EmilienM it's open for review
02:52 EmilienM and it should unblock our CI
02:54 EmilienM crinkle: I've tested it against puppet-neutron, https://review.openstack.org/#/c/195803/ (it works now)
03:16 ducttape_ joined #puppet-openstack
03:19 nibalizer EmilienM: crinkle these people seem to be in a similar boat as you https://github.com/puppet-community/puppet-unattended_upgrades/issues/16#issuecomment-115287556
03:29 xingchao EmilienM: is that the reason all beaker-rspec-dsvm-trusty failed by now ?
03:29 ducttape_ joined #puppet-openstack
03:37 ducttape_ joined #puppet-openstack
03:47 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Use apt::key instead of keyring package  https://review.openstack.org/195858
03:47 crinkle EmilienM: ^ this is how i would prefer to solve the issue
03:58 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Use apt::key instead of keyring package  https://review.openstack.org/195858
04:02 markvoelker joined #puppet-openstack
04:08 ducttape_ joined #puppet-openstack
04:13 ducttape_ joined #puppet-openstack
04:17 saneax joined #puppet-openstack
05:51 markvoelker joined #puppet-openstack
05:54 jtomasek joined #puppet-openstack
06:05 stamak joined #puppet-openstack
07:02 jpena joined #puppet-openstack
07:17 dgurtner joined #puppet-openstack
07:45 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix acceptance tests  https://review.openstack.org/195905
07:50 stamak joined #puppet-openstack
07:51 arnaud_orange joined #puppet-openstack
08:02 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Restore apt-get update relationship  https://review.openstack.org/195908
08:02 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Fix source_defaults unit tests  https://review.openstack.org/195909
08:02 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Run unit tests using latest puppetlabs-apt  https://review.openstack.org/195910
08:04 openstackgerrit Colleen Murphy proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
08:06 derekh joined #puppet-openstack
08:11 sergmelikyan joined #puppet-openstack
08:17 dgurtner xarses: gchamoul ping re: getting puppet-ceph back on track
08:19 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix acceptance tests  https://review.openstack.org/195905
08:21 xingchao_ joined #puppet-openstack
08:24 xingchao_ joined #puppet-openstack
08:25 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix dependency version  https://review.openstack.org/193869
08:27 xingcha__ joined #puppet-openstack
08:27 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Fix acceptance tests  https://review.openstack.org/195905
08:37 gchamoul dgurtner: ack
08:37 stamak joined #puppet-openstack
08:42 xingchao_ joined #puppet-openstack
08:50 xingchao_ joined #puppet-openstack
08:50 dgurtner gchamoul: I tried to write my understanding of the current issues in https://bugs.launchpad.net/puppet-ceph/+bug/1468865/comments/2
08:50 openstack Launchpad bug 1468865 in puppet-ceph "Default release should be bumped to hammer" [Undecided,New]
08:51 paramite joined #puppet-openstack
08:51 dgurtner gchamoul: your and my patches are combined via dependencies in https://review.openstack.org/#/c/195905/
08:51 xingcha__ joined #puppet-openstack
08:51 dgurtner gchamoul: to me this now misses xarses patch https://review.openstack.org/#/c/195760/2
08:52 dgurtner gchamoul: and then we need to figure out what to do about centos7
08:53 xingchao joined #puppet-openstack
08:58 stamak joined #puppet-openstack
09:00 gchamoul dgurtner: this one https://review.openstack.org/#/c/195760/2 should be merged first
09:02 dgurtner gchamoul: agreed - I'm just not sure if we need to flatten all patches into one because of the gate
09:02 gchamoul dgurtner: well so at least I could fix the lint issue in my patch
09:03 openstackgerrit Xingchao Yu proposed openstack/puppet-horizon: Update README.md for project namespace change  https://review.openstack.org/195931
09:04 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Make puppet-ceph unit tests compliant with Puppet4  https://review.openstack.org/179645
09:04 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Fix dependency version  https://review.openstack.org/193869
09:04 openstackgerrit Arnoud de Jonge proposed openstack/puppet-trove: Changed $::trove::rabbit_user to $::trove::rabbit_userid.  https://review.openstack.org/195933
09:05 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Fix acceptance tests  https://review.openstack.org/195905
09:06 openstackgerrit Xingchao Yu proposed openstack/puppet-ironic: Update README.md for project namespace change  https://review.openstack.org/195934
09:07 openstackgerrit Xingchao Yu proposed openstack/puppet-keystone: Update README.md for project namespace change  https://review.openstack.org/195936
09:09 gchamoul dgurtner: rspec-system is retired now and we should use beaker-rspec!
09:09 husanu3 joined #puppet-openstack
09:12 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Make puppet-ceph unit tests compliant with Puppet4  https://review.openstack.org/179645
09:13 dgurtner gchamoul: yes, I'm aware. The main blocker used to be that adding additional disks in beaker didn't work
09:14 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Fix dependency version  https://review.openstack.org/193869
09:14 dgurtner gchamoul: I think the whole scaffolding is there though, now it's just porting over all of the tests (or abandoning them?)
09:14 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Fix acceptance tests  https://review.openstack.org/195905
09:15 gchamoul dgurtner: good question!
09:17 openstackgerrit Xingchao Yu proposed openstack/puppet-manila: Update README.md for project namespace change  https://review.openstack.org/195942
09:18 gchamoul dgurtner: well, do you want to merge https://review.openstack.org/#/c/193869/5 and https://review.openstack.org/#/c/195905/5 into https://review.openstack.org/#/c/179645/3 ?
09:19 dgurtner gchamoul: sure, feel free to
09:20 gchamoul dgurtner: ack doing
09:22 dgurtner gchamoul: we probably need to talk to boris about centos7 / epel packaging https://bugzilla.redhat.com/show_bug.cgi?id=1183097
09:22 openstack bugzilla.redhat.com bug 1183097 in ceph "Sync-up ceph in epel 7" [Unspecified,Closed: errata] - Assigned to branto
09:23 husanu2 joined #puppet-openstack
09:24 openstackgerrit Xingchao Yu proposed openstack/puppet-neutron: Update README.md for project namespace change  https://review.openstack.org/195944
09:26 openstackgerrit Xingchao Yu proposed openstack/puppet-openstack_extras: Update README.md for project namespace change  https://review.openstack.org/195947
09:28 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Make puppet-ceph unit tests compliant with Puppet4  https://review.openstack.org/179645
09:28 markvoelker joined #puppet-openstack
09:29 openstackgerrit Xingchao Yu proposed openstack/puppet-sahara: Update README.md for project namespace change  https://review.openstack.org/195948
09:29 gchamoul dgurtner: we can abandoned the two patches now
09:29 sbadia hi
09:30 gchamoul dgurtner: I mean https://review.openstack.org/#/c/195905/5 and https://review.openstack.org/#/c/193869/5
09:30 sbadia xingchao: please use OpenStack (in order to avoid a future trivial patch :-) (ref: https://wiki.openstack.org/wiki/Documentation/Conventions)
09:36 openstackgerrit Xingchao Yu proposed openstack/puppet-ceilometer: Update README.md for project namespace change  https://review.openstack.org/195828
09:37 openstackgerrit Merged openstack/puppet-gnocchi: acceptance: Added noop acceptance test  https://review.openstack.org/195495
09:38 dgurtner gchamoul: done
09:39 openstackgerrit Maksym Yatsenko proposed openstack/puppet-horizon: Update "/dashboard" URI to "/horizon"  https://review.openstack.org/195954
09:39 dgurtner gchamoul: now we need to figure out what to do about centos7 though
09:39 openstackgerrit Sebastien Badia proposed openstack/puppet-gnocchi: keystone/auth: make service description configurable  https://review.openstack.org/195166
09:39 openstackgerrit Maksym Yatsenko proposed openstack/puppet-horizon: Update "/dashboard" URI to "/horizon"  https://review.openstack.org/195954
09:40 openstackgerrit Xingchao Yu proposed openstack/puppet-cinder: Update README.md for project namespace change  https://review.openstack.org/195955
09:40 xingcha__ sbadia: get it, I'm working on it now.
09:41 gfidente joined #puppet-openstack
09:41 openstackgerrit Xingchao Yu proposed openstack/puppet-designate: Update README.md for project namespace change  https://review.openstack.org/195827
09:41 dgurtner gchamoul: maybe revive https://review.openstack.org/#/c/156458/ from EmilienM and see if we get the packaging fixed?
09:43 openstackgerrit Xingchao Yu proposed openstack/puppet-glance: Update README.md for project namespace change  https://review.openstack.org/195831
09:43 openstackgerrit Xingchao Yu proposed openstack/puppet-gnocchi: Update README.md for project namespace change  https://review.openstack.org/195832
09:44 gildub joined #puppet-openstack
09:44 openstackgerrit Xingchao Yu proposed openstack/puppet-heat: Update README.md for project namespace change  https://review.openstack.org/195833
09:44 openstackgerrit Xingchao Yu proposed openstack/puppet-horizon: Update README.md for project namespace change  https://review.openstack.org/195931
09:45 openstackgerrit Xingchao Yu proposed openstack/puppet-ironic: Update README.md for project namespace change  https://review.openstack.org/195934
09:45 gchamoul dgurtner: we can give a try
09:45 sbadia xingcha__: thx!
09:46 openstackgerrit Xingchao Yu proposed openstack/puppet-keystone: Update README.md for project namespace change  https://review.openstack.org/195936
09:46 openstackgerrit Xingchao Yu proposed openstack/puppet-manila: Update README.md for project namespace change  https://review.openstack.org/195942
09:47 openstackgerrit Xingchao Yu proposed openstack/puppet-neutron: Update README.md for project namespace change  https://review.openstack.org/195944
09:48 openstackgerrit Xingchao Yu proposed openstack/puppet-nova: Update README.md for project namespace change  https://review.openstack.org/195958
09:49 openstackgerrit Xingchao Yu proposed openstack/puppet-openstack_extras: Update README.md for project namespace change  https://review.openstack.org/195947
09:50 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
09:50 openstackgerrit Xingchao Yu proposed openstack/puppet-sahara: Update README.md for project namespace change  https://review.openstack.org/195948
09:52 openstackgerrit Xingchao Yu proposed openstack/puppet-swift: Update README.md for project namespace change  https://review.openstack.org/195961
09:56 openstackgerrit Xingchao Yu proposed openstack/puppet-trove: Update README.md for project namespace change  https://review.openstack.org/195822
09:56 openstackgerrit Xingchao Yu proposed openstack/puppet-tuskar: Update README.md for project namespace change  https://review.openstack.org/195964
10:04 richm joined #puppet-openstack
10:22 xingchao_ joined #puppet-openstack
10:26 openstackgerrit Gael Chamoulaud proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
10:28 rcallawa joined #puppet-openstack
10:29 rcallawa joined #puppet-openstack
10:35 gchamoul dgurtner: we will still get the issue .. cause the forks for rspec-system/rspec-system-puppet are still pinning rspec-* to 2.99.x :/
10:36 gchamoul dgurtner: independently to the packaging issue
10:48 sergmelikyan joined #puppet-openstack
10:49 dgurtner gchamoul: those forks are mine (I had to patch them further). I can update them (or you could send pull requests)
10:51 dgurtner gchamoul: hm, ok that might be more work than I'm ready for :/
10:51 gchamoul dgurtner: yes because I am not that will support rspec 3.x :/
10:52 dgurtner gchamoul: ok, so the solution would be to give up on rspec-system and switch to beaker entirely??
10:52 gchamoul dgurtner: I think so!
11:02 xingchao_ joined #puppet-openstack
11:10 xingchao_ joined #puppet-openstack
11:14 xarses_ joined #puppet-openstack
11:16 sergmelikyan joined #puppet-openstack
11:17 markvoelker joined #puppet-openstack
11:20 kbyrne joined #puppet-openstack
11:21 kbyrne joined #puppet-openstack
11:52 prad joined #puppet-openstack
12:00 markvoelker joined #puppet-openstack
12:11 andreww joined #puppet-openstack
12:34 morazi joined #puppet-openstack
12:38 iurygregory joined #puppet-openstack
12:39 morazi joined #puppet-openstack
12:42 openstackgerrit Javier Peña proposed openstack/puppet-ceilometer: Simplify parameters for rpc_backend  https://review.openstack.org/196036
12:43 EmilienM fyi, CI jobs on trusty are still broken for some projects
12:44 EmilienM sbadia, crinkle and I are defining how to fix it, it should be unblocked this morning
12:47 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
12:58 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
13:06 EmilienM gchamoul: why did you not restore my patch for https://review.openstack.org/#/c/195959/ ?
13:07 EmilienM https://review.openstack.org/#/c/156458/
13:11 iurygregory Morning people
13:25 delattec joined #puppet-openstack
13:28 fvollero_ joined #puppet-openstack
13:31 andreww morning dgurtner gchamoul
13:33 gchamoul EmilienM: hmm .. how do you that when it's not your change? :/
13:33 fvollero joined #puppet-openstack
13:33 EmilienM gchamoul: you ask me :-p ?
13:33 arnaud_orange joined #puppet-openstack
13:33 gchamoul EmilienM: yes but I didn't want to wake up you :p
13:34 EmilienM no prob, I don't care
13:34 dfisher joined #puppet-openstack
13:40 dgurtner andreww: hi
13:42 dgurtner andreww: gchamoul so I found the issue for centos7, but not sure about the solution
13:43 dgurtner gchamoul: andreww the ceph clients are now all part of centos base
13:44 dgurtner gchamoul: andreww so if the same packages are blacklisted as they for epel as well, it works
13:44 andreww dgurtner: previously, it was due to the package conflict between epel7 and ceph.com
13:45 dgurtner xarses: correct
13:46 dgurtner but apparently they're now in centos base as well
13:46 xingchao joined #puppet-openstack
13:46 iurygregory joined #puppet-openstack
13:46 dgurtner xarses: gchamoul another solution is to just check hammer only (because hammer has newer packages then centos base)
13:47 dgurtner but afaik we agreed to support 3 LTS releases
13:47 xarses ok, I was trying to get the ci back to where it was with cent66 and 1204 before fixing going forward
13:48 dgurtner xarses: then we need to figure out a way to blacklist the client libraries from centos base
14:02 EmilienM jpena: fyi, puppet-ceilometer CI is broken now, not related to your patch
14:03 jpena EmilienM: thanks for the info :). puppet-trove CI has also been trolling me for a couple days, no problem
14:03 EmilienM jpena: we're working on this
14:19 morazi joined #puppet-openstack
14:25 chem joined #puppet-openstack
14:43 ducttape_ joined #puppet-openstack
14:44 ducttape_ joined #puppet-openstack
15:04 ducttape_ joined #puppet-openstack
15:04 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
15:06 crinkle EmilienM: sbadia ping re: ci
15:07 EmilienM crinkle: good morning!
15:07 EmilienM crinkle: so sbadia and I were also thinking about using apt::key but we also mentioned it was not the best idea for the reasons in comment
15:08 sbadia hey! o/
15:08 sbadia morning
15:09 crinkle EmilienM: ah okay
15:09 crinkle it just makes me cringe to see a package installed with an exec >.<
15:09 EmilienM crinkle: though we should combine our work if you're fine
15:09 EmilienM yes, me too
15:09 EmilienM crinkle: I've +1 one of your patches for unit tests refacto
15:10 crinkle EmilienM: i'll abandon that key ring patch and re-stack the unit test patches on yours
15:10 EmilienM I think we should 1/ merge my patch 2/ update tests to work with 2.1.0 and pin metadata.json + fixtures
15:10 EmilienM crinkle: does it make sense to you ?
15:11 crinkle EmilienM: yes
15:12 EmilienM crinkle, sbadia: if so, please review this patch: https://review.openstack.org/#/c/195801/ - I'll take care of doing 'recheck' things today
15:12 sbadia +1 for me, but indeed it's an heresy to install packages with a exec :p
15:13 crinkle EmilienM: should this be notify apt_update instead of before? https://review.openstack.org/#/c/195801/11/manifests/repo/debian/debian.pp
15:13 crinkle based on https://github.com/puppetlabs/puppetlabs-apt#update-the-list-of-packages
15:13 dfisher i assume this package installation via exec is only for a specific OS?
15:14 EmilienM dfisher: debian/ubuntu
15:14 * dfisher phew
15:14 EmilienM crinkle: ok I can change it
15:15 EmilienM crinkle: just to make sure but it will be run before, right?
15:15 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
15:16 crinkle EmilienM: yes
15:16 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module  https://review.openstack.org/195803
15:17 EmilienM crinkle: I'm testing it right now
15:17 openstackgerrit Merged openstack/puppet-tripleo: Enable support for loadbalancing Ironic  https://review.openstack.org/195331
15:19 mdorman joined #puppet-openstack
15:21 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Increase major bound on puppetlabs-apt  https://review.openstack.org/195910
15:21 openstackgerrit Colleen Murphy proposed openstack/puppet-openstack_extras: Fix source_defaults unit tests  https://review.openstack.org/195909
15:27 EmilienM crinkle: it works, cool
15:27 sbadia \o/
15:29 rcallawa joined #puppet-openstack
15:31 EmilienM sbadia, crinkle: thanks for this work. I think we've taken a thorn out of your side
15:32 crinkle :)
15:32 rcallawa joined #puppet-openstack
15:33 EmilienM crinkle: gate seems okay today, I'm about to run recheck again
15:33 sbadia héhé, cool it's friday and we see the end of the tunnel :)
15:33 EmilienM sbadia: yeah...
15:33 EmilienM _morgan: you might be interested to see how we did: https://review.openstack.org/195801
15:35 openstackgerrit Merged openstack/puppet-openstack_extras: repo: install keyring with an exec  https://review.openstack.org/195801
15:35 openstackgerrit Merged openstack/puppet-openstack_extras: Fix source_defaults unit tests  https://review.openstack.org/195909
15:38 openstackgerrit Merged openstack/puppet-openstack_extras: Increase major bound on puppetlabs-apt  https://review.openstack.org/195910
15:38 mdorman joined #puppet-openstack
15:38 EmilienM crinkle: infra merged the experimental job about upgrade \o/
15:58 dmsimard xarses, dgurtner, dachary: I think it's time to move puppet-ceph to beaker :)
15:58 xarses I thought we where using beaker prior
16:00 rcallawa_ joined #puppet-openstack
16:03 openstackgerrit Sebastien Badia proposed openstack/puppet-openstack-cookiecutter: Add .gitreview configuration  https://review.openstack.org/196121
16:04 rcallawa joined #puppet-openstack
16:04 dgurtner dmsimard: xarses I've almost fixed it :)
16:06 xarses awesome
16:07 dgurtner last issue is that the centos7 beaker boxes already come with epel enabled, so need to circumvent that
16:09 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
16:15 openstackgerrit Merged openstack/puppet-keystone: Custom file source for wsgi scripts  https://review.openstack.org/178181
16:16 dgurtner xarses: dmsimard dachary what is the set of releases we want to support? current + last n LTS? where n <= 3 (or <= 2) ?
16:16 xarses I thought 3, but If we can't keep one because it's not packed I think we can accept that, otherwise we have to run cent 66 longer
16:17 openstackgerrit Merged openstack/puppet-keystone: Pull in upstream stable/kilo wsgi script  https://review.openstack.org/191886
16:17 dgurtner or going forward is it going to be hammer firefly and dumpling or hammer, giant and firefly or hammer and firefly?
16:17 xarses ok, read that wrong
16:17 dgurtner xarses: afaik we'll continue to support cent66 and precise? or not?
16:18 xarses two LTS + current
16:18 xarses sounds correct
16:18 xarses 3 in total
16:18 xarses dgurtner: we'll have to add the test runners back if thats what we want
16:19 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: CI test - do not merge  https://review.openstack.org/196126
16:19 xarses looking at the fixtures, we where doing 12.04 and cent 66 not 1404
16:20 dgurtner xarses: it's just from a ceph point of view, only hammer and firefly are still supported http://ceph.com/docs/master/releases/
16:20 xarses dgurtner: sounds good to me =)
16:21 dgurtner xarses: and it vastly reduces runtime :)
16:21 xarses yes it does =)
16:21 dgurtner xarses: anyways, the jenkins gate jobs only test latest against 14.04 and centos7 so we'll somehow need to add everything else
16:22 dgurtner xarses: also from my understanding it's currently single host, but I haven't had time to look at it lately
16:23 xarses dgurtner: yes, the switch over seemed to muck it up
16:23 EmilienM dgurtner: it's single host and only trusty & centos7 now
16:24 vinsh joined #puppet-openstack
16:26 dgurtner EmilienM: thanks for the info
16:28 dgurtner lol... Ifixed it for centos, but broke trusty...
16:29 mdorman joined #puppet-openstack
16:32 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
16:39 dmsimard Yeah I agree on supporting hammer and firefly
16:43 dtalmage_ joined #puppet-openstack
16:44 dmsimard dgurtner, xarses: What reviews you want me to look at ? Got limited bandwidth today and saw a sudden surge :)
16:48 dgurtner dmsimard: we're just trying to fix things with puppet4, centos7, ... support
16:48 dgurtner dmsimard: it's basically https://review.openstack.org/#/c/179645/4 and https://review.openstack.org/#/c/195959/
16:49 EmilienM crinkle: re- pin or not pin: do you think it's a big deal to not pin stable/kilo (when will be cut), to avoid having master issues in stable branches?
16:49 dgurtner gchamoul: xarses it's working https://review.openstack.org/#/c/195959/ but now additional "upgrade" CI's have been added which break...
16:49 dmsimard Ok, will keep an eye on these today
16:50 EmilienM crinkle: the idea would be to pin in stable branches & not pin in master
16:51 xarses upgrade?
16:51 dgurtner dmsimard: basically they will have to be merged together because of the gate
16:51 dgurtner xarses: there is now 4 beaker-rspec jobs, 2 new ones called "upgrade" which are failing
16:51 dgurtner xarses: they've been added in the last hour
16:51 EmilienM dgurtner: don't worry
16:52 xarses =/
16:52 EmilienM https://review.openstack.org/#/c/196129/
16:52 EmilienM this is why it's failing
16:52 dmsimard dgurtner: Can't be merged if Jenkins won't let you :p Put them in the same commit if you need it
16:52 EmilienM dgurtner: I don't want our beaker jobs blocking you.
16:52 EmilienM dgurtner: feel free to patch infra
16:53 EmilienM dgurtner: but upgrade job will really help I think
16:53 xarses dmsimard: what ever is in the patch chain on https://review.openstack.org/#/c/195959/
16:53 EmilienM like is doing grenade jobs in openstack
16:54 dgurtner EmilienM: hm, what do I need to do to get it to pass? it's complaining because it can't find rspec ~> 2.2.4
16:54 dgurtner EmilienM: I just earlier dropped the 2.2.X pin and ran with 5.1.0
16:55 xarses hmm, Is that the infra side, or repo?
16:55 EmilienM repo
16:56 xarses dgurtner, sounds like we need to unpin then
16:57 dgurtner xarses: I already unpinned, it looks more like I need to repin?
16:57 dgurtner xarses: https://review.openstack.org/#/c/195959/5/Gemfile
16:57 xarses yep, blind
16:57 aimon joined #puppet-openstack
16:58 openstackgerrit Merged openstack/puppet-keystone: Update README.md for project namespace change  https://review.openstack.org/195936
16:58 xarses Well, maybe we just retry https://review.openstack.org/#/c/196129/ isn't compley merged yet
16:59 EmilienM dgurtner: again, please let me know if our CI is blocking you. We don't want to slow down your dev process
17:00 dgurtner EmilienM: no it's fine, I'm just not entirely sure what I need to do now :)
17:00 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
17:00 openstackgerrit Merged openstack/puppet-horizon: Update README.md for project namespace change  https://review.openstack.org/195931
17:00 crinkle EmilienM: to be clear, do you mean pin the external dependencies or the openstack dependencies?
17:00 EmilienM crinkle: external
17:00 crinkle EmilienM: when we move to using a puppetfile it will let us pin to stable branches of the external dependencies so i think that will help
17:01 crinkle so we can use stable branches instead of specific releases
17:01 EmilienM crinkle: awesome
17:01 EmilienM I like that
17:01 crinkle okay good compromise
17:01 EmilienM crinkle: http://goo.gl/CxVUpy
17:02 crinkle lol
17:22 dgurtner EmilienM: http://logs.openstack.org/59/195959/6/check/gate-puppet-ceph-puppet-beaker-rspec-upgrade-dsvm-trusty/52c44ab/console.html
17:23 EmilienM dgurtner: yes
17:23 EmilienM this is 'normal'
17:23 dgurtner EmilienM: isn't this what your patch should fix?
17:23 EmilienM https://review.openstack.org/#/c/196129/
17:23 EmilienM it should work now or in short time, time to JJB to update
17:24 EmilienM dgurtner: jenkins jobs update is not instantaneous
17:24 dgurtner EmilienM: ok, thanks
17:24 EmilienM Last reconfigured: Fri Jun 26 2015 12:11:35 GMT-0400 (EDT)
17:25 EmilienM dgurtner: I'm relocating, back in 20 min max
17:30 dgurtner gchamoul: ping
17:53 sbadia gchamoul: https://review.openstack.org/#/c/196119/
17:53 openstackgerrit Sebastien Badia proposed openstack/puppet-swift: lint: Fix lint issues (missing documentation for define types)  https://review.openstack.org/196168
17:53 sbadia EmilienM|brb: ↑
17:54 xarses joined #puppet-openstack
17:54 sbadia EmilienM|brb: the last one about lint issues :) (params doc is not enable on puppet-heat)
17:57 sbadia https://www.ripe.net/participate/policies/proposals/2014-03
17:57 sbadia humr sorry… wrong chan…
17:57 xarses dgurtner: I see its 196129 is merged now Any luck?
17:58 openstackgerrit Sebastien Badia proposed openstack/puppet-trove: keystone/auth: make service description configurable  https://review.openstack.org/195159
18:01 sbadia EmilienM|brb: gate-puppet-*-puppet-beaker-rspec-upgrade-dsvm-{trusty,centos} are voting ?
18:02 iurygregory hey mfisch , if the agenda for the next week meeting have space, absolute this can be a topic: https://review.openstack.org/#/c/190361/ =)
18:04 EmilienM sbadia: should not
18:05 EmilienM sbadia: zuul takes time to be reconfigured
18:05 EmilienM sbadia: I think now it's good, I'm checking
18:05 sbadia EmilienM: erf… so I don't understand the -1 of jenkins here https://review.openstack.org/#/c/195208/
18:05 sbadia EmilienM: ack
18:05 EmilienM sbadia: it's because JJB & Zuul have been puppetize after your recheck
18:06 sbadia oki
18:06 EmilienM when a patch is merged in project-config, it takes time to be applied
18:06 EmilienM sbadia: I confirm it's good, look in zuul, 196126
18:06 EmilienM (non-voting)
18:06 sbadia \o/
18:06 dgurtner xarses: well the problem is we wont get the update jobs to run
18:06 sbadia oki
18:06 sbadia EmilienM: thx!
18:07 xarses dgurtner: because we arn't ready for it?
18:07 dgurtner xarses: our current patchset are to finally be in a state that passes the gate again
18:07 dgurtner xarses: and the jobs are specifically to run the "previous" commit, but on the current gate
18:08 EmilienM sbadia: gate should be good now, I'm doing rechecks
18:08 dgurtner xarses: I think we need to disable those jobs, merge todays changes and then reenable them
18:08 sbadia EmilienM: ack
18:09 xarses Are all the CI required in the merge side? If not we can just push it through. Otherwise ya, we need to pull them out of infra and then put them back in
18:10 aimon joined #puppet-openstack
18:10 iurygregory anyone can say when is the deadline for spec approval?
18:11 EmilienM iurygregory: we don't have any deadline \o/
18:11 dgurtner xarses: don't know, we can ask EmilienM
18:11 iurygregory thanks Good
18:11 EmilienM should we have?
18:12 xarses EmilienM: I'm not sure we have enough features for it yet
18:12 xarses or a firm release schedule
18:13 dgurtner EmilienM: are the 'upgrade' jobs voting?
18:13 EmilienM yeah we don't need that
18:13 iurygregory my boss was asking if my spec will need a exception, thanks EmilienM ;D
18:13 EmilienM iurygregory: tell him it will need one, and you need to send me chocolate. A lot.
18:13 EmilienM dgurtner: no
18:13 EmilienM dgurtner: it was by accident
18:13 EmilienM dgurtner: and now, it's fixed: do a 'recheck'
18:15 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
18:15 iurygregory EmilienM, i'll give you a chocolate if i go to tokyo summit
18:15 EmilienM fair enough
18:15 iurygregory yes
18:15 dgurtner EmilienM: I actually had a small change, so just pushed instead of recheck
18:16 EmilienM dgurtner: cool, let me know if any trouble
18:17 dgurtner EmilienM: sure! btw. this is your original centos7 patch which is now finally ready :)
18:17 EmilienM dgurtner: w00t
18:17 EmilienM dgurtner: thx guys for working on it. I had to give up
18:17 dgurtner EmilienM: alas it will probably have to be merged with a bunch of other things so all credit will go to gchamoul or whomever merges, because it won't get through the gate otherwise ;)
18:18 dgurtner EmilienM: yeah, packaging of ceph clients in both centos base and epel is seriously broken from my pov
18:19 EmilienM honestly, I spent too much time to figure that out
18:19 EmilienM I'm happy if someone cares about that now
18:20 dgurtner I only care about the result - if we finally get el7 support merged, I don't care much about anything else :)
18:23 EmilienM crinkle: can you merge https://review.openstack.org/#/c/195208 please so we can unblock Puppet Ceilometer gate
18:23 xingchao joined #puppet-openstack
18:25 xarses dgurtner: should we add co-authors for every one else that attempted to get EL7 working?
18:25 openstackgerrit Merged openstack/puppet-ironic: lint: Fix lint issues (missing documentation for define types)  https://review.openstack.org/195481
18:25 xarses I feel bad closing their CR's
18:26 dgurtner xarses: sure... it's been a team effort
18:28 crinkle EmilienM: done
18:28 openstackgerrit Mathieu Gagné proposed openstack/puppet-designate: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70393
18:29 EmilienM crinkle: https://goo.gl/jTpYfs
18:30 crinkle EmilienM: you must have a whole high-five bookmark folder
18:30 xarses there is a bot for that
18:30 EmilienM crinkle: http://www.reactiongifs.com/tag/high-five/
18:31 EmilienM _ody: hey! can you help me to got a new release of https://github.com/puppetlabs/puppetlabs-inifile ?
18:32 xarses denied is my favorite =)
18:32 EmilienM _ody: context: https://review.openstack.org/#/c/195003/
18:32 EmilienM _ody: if you know the maintainer/release manager, I'll send him/her chocolate
18:36 openstackgerrit Merged openstack/puppet-ceilometer: Fix Beaker CI according recent puppetlabs-rabbitmq change.  https://review.openstack.org/195208
18:37 openstackgerrit Merged openstack/puppet-neutron: PLUMgrid metadata config.  https://review.openstack.org/194634
18:38 nibalizer EmilienM: i believe you want to talk to _morgan and Hunner
18:38 EmilienM nibalizer: yeah
18:40 _morgan EmilienM: we should be able to release inifile july 7th
18:41 _morgan i'll add a ticket
18:41 EmilienM _morgan: ok. Can you write this in the review please ?
18:41 EmilienM in https://review.openstack.org/#/c/195003/
18:41 _morgan EmilienM: sure
18:41 hogepodge joined #puppet-openstack
18:42 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
18:43 EmilienM _morgan: http://goo.gl/F8bNtt
18:44 _morgan EmilienM: commented!
18:44 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
18:46 xarses ^ EmilienM dmsimard dachary gchamoul
18:46 sergmelikyan joined #puppet-openstack
18:46 dmsimard I can +1 once jenkins weighs in, no +2 since I'm not a RHEL guy
18:48 openstackgerrit Merged openstack/puppet-designate: MySQL: change default MySQL collate to utf8_general_ci  https://review.openstack.org/187451
18:48 openstackgerrit Merged openstack/puppet-swift: lint: Fix lint issues (missing documentation for define types)  https://review.openstack.org/196168
18:54 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
18:56 stamak joined #puppet-openstack
19:00 xarses aparently if you ^v with a url in the buffer and have a trello card open it automaticly creates a link in it
19:10 EmilienM crinkle: I think https://github.com/openstack/puppet-glance/commit/93f79bb56d7e761c1728beb050b661760683dbd6 is making puppet-glance CI unstable on centos, i've noticed that
19:10 EmilienM http://logs.openstack.org/31/195831/2/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-centos7/acbcaa0/console.html
19:11 EmilienM Could not prefetch keystone_role provider 'openstack': undefined method `collect' for nil:NilClass
19:12 crinkle EmilienM: i do not think adding a test is making ci unstable
19:12 crinkle EmilienM: http://logs.openstack.org/31/195831/2/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-centos7/acbcaa0/console.html#_2015-06-26_19_01_39_931
19:12 EmilienM oh this test is excellent
19:12 crinkle EmilienM: the redhat repos break all over our modules, not just puppet-glance
19:12 EmilienM I just think there is a side issue
19:13 EmilienM ok, I should fix it
19:13 EmilienM I think we're using testing
19:13 EmilienM we should use the non-testing repo
19:13 crinkle EmilienM:  merge https://review.openstack.org/#/c/193761/
19:13 crinkle then use that
19:16 EmilienM crinkle: once it's merged, I send patches to all modules
19:17 xarses dgurtner: looks like rev 10 failed syntax
19:18 xarses https://jenkins03.openstack.org/job/gate-puppet-ceph-puppet-syntax/8/console
19:22 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: acceptance: install RDO repo the right way  https://review.openstack.org/196196
19:22 EmilienM crinkle: something like this ? ^
19:23 openstackgerrit Merged openstack/puppet-cinder: Remove POSIX users, groups, and file modes.  https://review.openstack.org/193189
19:23 openstackgerrit Merged openstack/puppet-openstack_extras: Fix kilo repo installation for RedHat  https://review.openstack.org/193761
19:24 mdorman joined #puppet-openstack
19:26 crinkle EmilienM: ++
19:26 EmilienM crinkle: cool
19:26 markvoelker joined #puppet-openstack
19:32 markvoelker joined #puppet-openstack
19:38 markvoelker_ joined #puppet-openstack
19:38 dgurtner xarses: having a look - found a small mistake which needs fixing before we merge
19:44 markvoelker joined #puppet-openstack
19:48 EmilienM crinkle: your patch works and solves the glance case
19:48 EmilienM and I guess other projects
19:48 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: install RDO repo the right way  https://review.openstack.org/196206
19:48 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: acceptance: install RDO repo the right way  https://review.openstack.org/196207
19:48 openstackgerrit Emilien Macchi proposed openstack/puppet-designate: acceptance: install RDO repo the right way  https://review.openstack.org/196208
19:49 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: acceptance: install RDO repo the right way  https://review.openstack.org/196209
19:49 dgurtner xarses: yup, misst an open brace
19:49 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: acceptance: install RDO repo the right way  https://review.openstack.org/196210
19:49 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: acceptance: install RDO repo the right way  https://review.openstack.org/196211
19:49 openstackgerrit Emilien Macchi proposed openstack/puppet-manila: acceptance: install RDO repo the right way  https://review.openstack.org/196212
19:49 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: acceptance: install RDO repo the right way  https://review.openstack.org/196213
19:50 crinkle EmilienM: i see those problems intermittently everywhere, i could not say for sure if actually works or if the test was lucky
19:50 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
19:50 EmilienM crinkle: yeah me too, but your patch is the way to go, I checked with internal packagers
19:50 EmilienM crinkle: I was using testing, back in time when we did not have anything else
19:51 openstackgerrit Emilien Macchi proposed openstack/puppet-horizon: acceptance: install RDO repo the right way  https://review.openstack.org/196214
19:52 openstackgerrit Emilien Macchi proposed openstack/puppet-sahara: acceptance: install RDO repo the right way  https://review.openstack.org/196217
19:52 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: acceptance: install RDO repo the right way  https://review.openstack.org/196218
19:53 openstackgerrit Emilien Macchi proposed openstack/puppet-vswitch: acceptance: install RDO repo the right way  https://review.openstack.org/196219
19:54 openstackgerrit Emilien Macchi proposed openstack/puppet-trove: acceptance: install RDO repo the right way  https://review.openstack.org/196220
19:54 EmilienM sorry for spam
19:54 EmilienM if any core around can review https://review.openstack.org/#/q/status:open+topic:193761,n,z please
19:55 crinkle EmilienM: will wait for jenkins
19:56 EmilienM yep
19:58 markvoelker joined #puppet-openstack
20:09 markvoelker_ joined #puppet-openstack
20:14 EmilienM crinkle: except swift (I'm investigating now), everything looks fine (trove in progress)
20:15 EmilienM upgrade job has been updated, it's even running: https://jenkins04.openstack.org/job/gate-puppet-trove-puppet-beaker-rspec-upgrade-dsvm-centos7/1/consoleFull
20:17 EmilienM crinkle: looking at logs, it seems we need to redo the ssh thing after upgrade :/
20:23 EmilienM nibalizer: any idea?
20:25 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/195959
20:30 nibalizer EmilienM: im a bit busy right now
20:30 nibalizer will try to circle around later
20:30 EmilienM nibalizer: cool np
20:30 EmilienM not urgent at all
20:33 EmilienM upgrade jobs is passing jenkins (on trusty only now) \o/
20:51 markvoelker joined #puppet-openstack
20:55 xarses woot woot
20:58 openstackgerrit Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance  https://review.openstack.org/176150
21:02 EmilienM nibalizer: nevermind, Hunner helped me and we figured
21:02 nibalizer woot
21:05 EmilienM nibalizer: https://review.openstack.org/196246
21:07 nibalizer haha, i guess or 'none' hypervisor stopped working?
21:07 Hunner nibalizer: Well, technically the hypervisor is fine
21:08 Hunner BEAKER_provision is the convention for disabling the "provision" step in the helper too, like https://github.com/puppetlabs/puppetlabs-mongodb/blob/master/spec/spec_helper_acceptance.rb#L6
21:08 Hunner Similar to how vagrant doesn't provision stuff if the machine is already created, but the helper has to be made aware of that too
21:10 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: beaker: do not install Puppet  if BEAKER_provision=no  https://review.openstack.org/196249
21:10 EmilienM Hunner: this is this king of patch I'm about to do
21:10 EmilienM ^
21:10 EmilienM Hunner: can you +1 and I'll do the rest
21:10 Hunner EmilienM: Mind if I push to that patch?
21:11 EmilienM Hunner: sure, but I'm curious to know what :)
21:11 Hunner I could just gist it. I'd move the install_puppet out of hosts.each and surround the whole install_puppet/distmoduledir with BEAKER_provision
21:11 openstackgerrit Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance  https://review.openstack.org/176150
21:12 EmilienM Hunner: cool, I'm off until tonight, I'll catch-up
21:12 Hunner Also, you really only need hosts.each if you plan on testing with more than one node, but you'll probably never do that...
21:13 EmilienM Hunner: not today ;-)
21:26 openstack joined #puppet-openstack
21:28 markvoelker joined #puppet-openstack
21:47 openstackgerrit Merged openstack/puppet-cinder: acceptance: install RDO repo the right way  https://review.openstack.org/196207
21:48 openstackgerrit Merged openstack/puppet-heat: acceptance: install RDO repo the right way  https://review.openstack.org/196209
21:48 openstackgerrit Merged openstack/puppet-sahara: acceptance: install RDO repo the right way  https://review.openstack.org/196217
21:57 openstackgerrit Merged openstack/puppet-nova: acceptance: install RDO repo the right way  https://review.openstack.org/196213
21:58 openstackgerrit Merged openstack/puppet-swift: acceptance: install RDO repo the right way  https://review.openstack.org/196218
21:58 openstackgerrit Merged openstack/puppet-trove: acceptance: install RDO repo the right way  https://review.openstack.org/196220
21:58 openstackgerrit Merged openstack/puppet-vswitch: acceptance: install RDO repo the right way  https://review.openstack.org/196219
22:09 xarses dmsimard: dachary dgurtner, I think if we get another +1 on https://review.openstack.org/#/c/195959/ we should merge it without two +2's since we are light on cores in this case, and it un-breaks CI
22:10 dmsimard xarses: yeah, I can merge it if everyone is fine with it.
22:10 xarses we also need to review / merge https://review.openstack.org/#/c/179645/ which it depends on
22:10 dmsimard yeah I was actually looking at that one now that jenkins had reported back from my recheck
22:11 xarses as expected, It can't pass CI for cent7
22:11 dmsimard Looks like giant issues
22:11 dmsimard did we say we were no longer testing giant but instead hammer and firefly ?
22:12 dmsimard oh, wait, nevermind, I understand now
22:12 dmsimard I thought the dependency was the other way around
22:13 xarses =)
22:14 dgurtner joined #puppet-openstack
22:16 xarses I think we are still testing giant for now, we where talking about dropping it since it's not LTS
22:16 xarses and not the current release
22:24 openstackgerrit Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance  https://review.openstack.org/176150
22:27 openstackgerrit Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance  https://review.openstack.org/176150
22:36 aimon joined #puppet-openstack
22:47 saneax joined #puppet-openstack
23:42 mdorman joined #puppet-openstack
23:45 mdorman joined #puppet-openstack

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