EmilienM | crinkle: I think we still need Exec['apt_update'] -> Package<||> | 00:02 |
---|---|---|
EmilienM | crinkle: I'm about to produce a patch, let me know if you think we can do better | 00:03 |
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:04 |
_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:06 |
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:07 |
* xarses kicks nibalizer | 00:08 | |
EmilienM | nibalizer: it's not Friday. | 00:08 |
*** xavpaice has quit IRC | 00:10 | |
_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:12 |
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:12 |
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 |
*** gildub has quit IRC | 00:13 | |
crinkle | maybe *unpinning* would fix it :P | 00:13 |
_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:14 |
crinkle | xarses: you can copy the spec/acceptance/nodesets/* from another module into puppet-ceph | 00:15 |
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:17 |
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:18 |
EmilienM | crinkle: oh yes... | 00:19 |
EmilienM | crinkle: it's inconsistent | 00:19 |
EmilienM | across our modules | 00:19 |
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:20 |
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:23 |
*** markvoelker has joined #puppet-openstack | 00:24 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 00:27 |
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:28 |
*** markvoelker has quit IRC | 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:30 |
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:31 |
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:32 |
*** linkedinyou has quit IRC | 00:36 | |
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:37 |
EmilienM | I'm testing now the openstack_extra patch | 00:38 |
EmilienM | with unpin | 00:38 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 00:40 |
*** gildub has joined #puppet-openstack | 00:41 | |
*** aimon has joined #puppet-openstack | 00:42 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 00:49 |
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:53 |
xarses | sweet, vagrant up failed | 00:54 |
*** xingchao_ has joined #puppet-openstack | 00:54 | |
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 | 00:58 |
*** xarses has quit IRC | 01:01 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec https://review.openstack.org/195801 | 01:11 |
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:12 |
*** bklei has joined #puppet-openstack | 01:17 | |
*** xarses has joined #puppet-openstack | 01:20 | |
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:23 |
*** xavpaice has 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:28 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec https://review.openstack.org/195801 | 01:33 |
*** jasondotstar has joined #puppet-openstack | 01:33 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 01:34 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-designate: Update README.md for project namespace change https://review.openstack.org/195827 | 01:35 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-ceilometer: Update README.md for project namespace change https://review.openstack.org/195828 | 01:41 |
*** bklei has quit IRC | 01:45 | |
*** saneax has quit IRC | 01:48 | |
openstackgerrit | Xingchao Yu proposed openstack/puppet-glance: Update README.md for project namespace change https://review.openstack.org/195831 | 01:50 |
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:52 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-heat: Update README.md for project namespace change https://review.openstack.org/195833 | 01:55 |
crinkle | i think she's out for the day | 01:57 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec https://review.openstack.org/195801 | 01:59 |
*** jasondotstar has quit IRC | 01:59 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 01:59 |
EmilienM | crinkle: no prob, I make a short break and I continue. I succeed to have something working on my laptop :) | 02:09 |
*** jasondotstar has joined #puppet-openstack | 02:10 | |
*** xingchao_ has quit IRC | 02:13 | |
*** markvoelker has joined #puppet-openstack | 02:14 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec https://review.openstack.org/195801 | 02:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 02:16 |
*** markvoelker has quit IRC | 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:18 |
EmilienM | dmsimard: good luck... is it worth? :) | 02:19 |
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:20 |
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:21 |
dmsimard | Well, I guess it's not so much that I suck at ruby that I don't like the language. | 02:22 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec https://review.openstack.org/195801 | 02:33 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 02:34 |
*** jasondotstar has quit IRC | 02:42 | |
*** EmilienM sets mode: +o crinkle | 02:43 | |
*** EmilienM sets mode: +o clayton | 02:43 | |
*** EmilienM sets mode: +o mfisch | 02:43 | |
*** EmilienM sets mode: +o mgagne | 02:43 | |
*** EmilienM sets mode: +o sbadia | 02:43 | |
*** EmilienM sets mode: +o xingchao | 02:43 | |
*** EmilienM sets mode: +o fc__ | 02:43 | |
*** EmilienM sets mode: +o bodepd | 02:43 | |
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_ has joined #puppet-openstack | 02:47 | |
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:52 |
EmilienM | crinkle: I've tested it against puppet-neutron, https://review.openstack.org/#/c/195803/ (it works now) | 02:54 |
*** xingchao_ has quit IRC | 03:06 | |
*** ducttape_ has joined #puppet-openstack | 03:16 | |
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:19 |
*** ducttape_ has quit IRC | 03:22 | |
xingchao | EmilienM: is that the reason all beaker-rspec-dsvm-trusty failed by now ? | 03:29 |
*** ducttape_ has joined #puppet-openstack | 03:29 | |
*** ducttape_ has quit IRC | 03:31 | |
*** ducttape_ has joined #puppet-openstack | 03:37 | |
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:47 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-openstack_extras: Use apt::key instead of keyring package https://review.openstack.org/195858 | 03:58 |
*** ducttape_ has quit IRC | 03:59 | |
*** markvoelker has joined #puppet-openstack | 04:02 | |
*** markvoelker has quit IRC | 04:07 | |
*** ducttape_ has joined #puppet-openstack | 04:08 | |
*** ducttape_ has quit IRC | 04:12 | |
*** rushiagr_away is now known as rushiagr | 04:13 | |
*** ducttape_ has joined #puppet-openstack | 04:13 | |
*** saneax has joined #puppet-openstack | 04:17 | |
*** ducttape_ has quit IRC | 04:20 | |
*** richm has quit IRC | 05:34 | |
*** rushiagr is now known as rushiagr_away | 05:43 | |
*** markvoelker has joined #puppet-openstack | 05:51 | |
*** jtomasek has joined #puppet-openstack | 05:54 | |
*** markvoelker has quit IRC | 05:56 | |
*** stamak has joined #puppet-openstack | 06:05 | |
*** rushiagr_away is now known as rushiagr | 06:15 | |
*** stamak has quit IRC | 06:18 | |
*** jpena has joined #puppet-openstack | 07:02 | |
*** strider|afk is now known as gchamoul | 07:14 | |
*** dgurtner has joined #puppet-openstack | 07:17 | |
*** rushiagr is now known as rushiagr_away | 07:36 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Fix acceptance tests https://review.openstack.org/195905 | 07:45 |
*** gildub has quit IRC | 07:49 | |
*** stamak has joined #puppet-openstack | 07:50 | |
*** arnaud_orange has joined #puppet-openstack | 07:51 | |
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:02 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 08:04 |
*** derekh has joined #puppet-openstack | 08:06 | |
*** sergmelikyan has joined #puppet-openstack | 08:11 | |
dgurtner | xarses: gchamoul ping re: getting puppet-ceph back on track | 08:17 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Fix acceptance tests https://review.openstack.org/195905 | 08:19 |
*** xingchao_ has joined #puppet-openstack | 08:21 | |
*** xingchao_ has quit IRC | 08:24 | |
*** xingchao_ has joined #puppet-openstack | 08:24 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Fix dependency version https://review.openstack.org/193869 | 08:25 |
*** xingcha__ has joined #puppet-openstack | 08:27 | |
*** xingchao_ has quit IRC | 08:27 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Fix acceptance tests https://review.openstack.org/195905 | 08:27 |
*** stamak has quit IRC | 08:29 | |
gchamoul | dgurtner: ack | 08:37 |
*** stamak has joined #puppet-openstack | 08:37 | |
*** xingcha__ has quit IRC | 08:40 | |
*** xingchao_ has joined #puppet-openstack | 08:42 | |
*** stamak has quit IRC | 08:43 | |
*** xingchao_ has quit IRC | 08:49 | |
*** xingchao_ has 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:50 |
*** paramite has 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__ has joined #puppet-openstack | 08:51 | |
dgurtner | gchamoul: to me this now misses xarses patch https://review.openstack.org/#/c/195760/2 | 08:51 |
dgurtner | gchamoul: and then we need to figure out what to do about centos7 | 08:52 |
*** xingchao has quit IRC | 08:53 | |
*** xingchao has joined #puppet-openstack | 08:53 | |
*** rcallawa has quit IRC | 08:55 | |
*** xingchao_ has quit IRC | 08:55 | |
*** stamak has joined #puppet-openstack | 08:58 | |
gchamoul | dgurtner: this one https://review.openstack.org/#/c/195760/2 should be merged first | 09:00 |
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:02 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-horizon: Update README.md for project namespace change https://review.openstack.org/195931 | 09:03 |
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:05 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-ironic: Update README.md for project namespace change https://review.openstack.org/195934 | 09:06 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-keystone: Update README.md for project namespace change https://review.openstack.org/195936 | 09:07 |
gchamoul | dgurtner: rspec-system is retired now and we should use beaker-rspec! | 09:09 |
*** husanu3 has joined #puppet-openstack | 09:09 | |
openstackgerrit | Gael Chamoulaud proposed stackforge/puppet-ceph: Make puppet-ceph unit tests compliant with Puppet4 https://review.openstack.org/179645 | 09:12 |
dgurtner | gchamoul: yes, I'm aware. The main blocker used to be that adding additional disks in beaker didn't work | 09:13 |
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:14 |
gchamoul | dgurtner: good question! | 09:15 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-manila: Update README.md for project namespace change https://review.openstack.org/195942 | 09:17 |
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:18 |
*** husanu3 has quit IRC | 09:18 | |
dgurtner | gchamoul: sure, feel free to | 09:19 |
gchamoul | dgurtner: ack doing | 09:20 |
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:22 |
*** husanu2 has joined #puppet-openstack | 09:23 | |
openstackgerrit | Xingchao Yu proposed openstack/puppet-neutron: Update README.md for project namespace change https://review.openstack.org/195944 | 09:24 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-openstack_extras: Update README.md for project namespace change https://review.openstack.org/195947 | 09:26 |
openstackgerrit | Gael Chamoulaud proposed stackforge/puppet-ceph: Make puppet-ceph unit tests compliant with Puppet4 https://review.openstack.org/179645 | 09:28 |
*** husanu2 has quit IRC | 09:28 | |
*** markvoelker has 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:29 |
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:30 |
*** markvoelker has quit IRC | 09:33 | |
openstackgerrit | Xingchao Yu proposed openstack/puppet-ceilometer: Update README.md for project namespace change https://review.openstack.org/195828 | 09:36 |
openstackgerrit | Merged openstack/puppet-gnocchi: acceptance: Added noop acceptance test https://review.openstack.org/195495 | 09:37 |
dgurtner | gchamoul: done | 09:38 |
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:39 |
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:40 |
*** gfidente has joined #puppet-openstack | 09:41 | |
*** gfidente has quit IRC | 09:41 | |
*** gfidente has 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:41 |
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:43 |
*** gildub has 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:44 |
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:45 |
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:46 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-neutron: Update README.md for project namespace change https://review.openstack.org/195944 | 09:47 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-nova: Update README.md for project namespace change https://review.openstack.org/195958 | 09:48 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-openstack_extras: Update README.md for project namespace change https://review.openstack.org/195947 | 09:49 |
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:50 |
openstackgerrit | Xingchao Yu proposed openstack/puppet-swift: Update README.md for project namespace change https://review.openstack.org/195961 | 09:52 |
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 | 09:56 |
*** richm has joined #puppet-openstack | 10:04 | |
*** xingcha__ has quit IRC | 10:04 | |
*** sergmelikyan has quit IRC | 10:19 | |
*** xingchao_ has joined #puppet-openstack | 10:22 | |
openstackgerrit | Gael Chamoulaud proposed stackforge/puppet-ceph: Add EL7 support https://review.openstack.org/195959 | 10:26 |
*** rcallawa has joined #puppet-openstack | 10:28 | |
*** rcallawa has quit IRC | 10:29 | |
*** rcallawa has joined #puppet-openstack | 10:29 | |
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:35 |
gchamoul | dgurtner: independently to the packaging issue | 10:36 |
*** sergmelikyan has joined #puppet-openstack | 10:48 | |
dgurtner | gchamoul: those forks are mine (I had to patch them further). I can update them (or you could send pull requests) | 10:49 |
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:51 |
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! | 10:52 |
*** gildub has quit IRC | 10:53 | |
*** rcallawa has quit IRC | 10:55 | |
*** xingchao_ has quit IRC | 10:58 | |
*** xingchao_ has joined #puppet-openstack | 11:02 | |
*** xingchao_ has quit IRC | 11:09 | |
*** xingchao_ has joined #puppet-openstack | 11:11 | |
*** xingchao has quit IRC | 11:11 | |
*** slagle_ is now known as slagle | 11:14 | |
*** xarses_ has joined #puppet-openstack | 11:14 | |
*** xingchao_ has quit IRC | 11:15 | |
*** kbyrne has quit IRC | 11:16 | |
*** sergmelikyan has quit IRC | 11:16 | |
*** sergmelikyan has joined #puppet-openstack | 11:16 | |
*** xarses has quit IRC | 11:17 | |
*** markvoelker has joined #puppet-openstack | 11:17 | |
*** kbyrne has joined #puppet-openstack | 11:20 | |
*** kbyrne has quit IRC | 11:21 | |
*** kbyrne has joined #puppet-openstack | 11:21 | |
*** markvoelker has quit IRC | 11:22 | |
*** delattec has quit IRC | 11:31 | |
*** prad has joined #puppet-openstack | 11:52 | |
*** markvoelker has joined #puppet-openstack | 12:00 | |
*** andreww has joined #puppet-openstack | 12:11 | |
*** xarses_ has quit IRC | 12:14 | |
*** jayg|g0n3 is now known as jayg | 12:27 | |
*** morazi has quit IRC | 12:33 | |
*** morazi has joined #puppet-openstack | 12:34 | |
*** morazi has quit IRC | 12:35 | |
*** iurygregory has joined #puppet-openstack | 12:39 | |
*** morazi has joined #puppet-openstack | 12:39 | |
openstackgerrit | Javier Peña proposed openstack/puppet-ceilometer: Simplify parameters for rpc_backend https://review.openstack.org/196036 | 12:42 |
EmilienM | fyi, CI jobs on trusty are still broken for some projects | 12:44 |
*** jpena is now known as jpena|lunch | 12:44 | |
EmilienM | sbadia, crinkle and I are defining how to fix it, it should be unblocked this morning | 12:44 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec https://review.openstack.org/195801 | 12:48 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: repo: install keyring with an exec https://review.openstack.org/195801 | 12:58 |
EmilienM | gchamoul: why did you not restore my patch for https://review.openstack.org/#/c/195959/ ? | 13:06 |
EmilienM | https://review.openstack.org/#/c/156458/ | 13:07 |
iurygregory | Morning people | 13:11 |
*** delattec has joined #puppet-openstack | 13:25 | |
*** fvollero_ has joined #puppet-openstack | 13:28 | |
*** fvollero has quit IRC | 13:29 | |
andreww | morning dgurtner gchamoul | 13:31 |
gchamoul | EmilienM: hmm .. how do you that when it's not your change? :/ | 13:33 |
*** fvollero_ has quit IRC | 13:33 | |
*** arnaud_orange has quit IRC | 13:33 | |
*** fvollero has joined #puppet-openstack | 13:33 | |
EmilienM | gchamoul: you ask me :-p ? | 13:33 |
*** arnaud_orange has 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 |
*** chem has quit IRC | 13:34 | |
*** dfisher has joined #puppet-openstack | 13:34 | |
dgurtner | andreww: hi | 13:40 |
dgurtner | andreww: gchamoul so I found the issue for centos7, but not sure about the solution | 13:42 |
dgurtner | gchamoul: andreww the ceph clients are now all part of centos base | 13:43 |
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:44 |
*** andreww is now known as xarses | 13:45 | |
dgurtner | xarses: correct | 13:45 |
*** iurygregory has quit IRC | 13:45 | |
dgurtner | but apparently they're now in centos base as well | 13:46 |
*** xingchao has joined #puppet-openstack | 13:46 | |
*** iurygregory has 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:46 |
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:47 |
dgurtner | xarses: then we need to figure out a way to blacklist the client libraries from centos base | 13:48 |
*** jpena|lunch is now known as jpena | 13:56 | |
EmilienM | jpena: fyi, puppet-ceilometer CI is broken now, not related to your patch | 14:02 |
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:03 |
*** saneax has quit IRC | 14:12 | |
*** morazi has quit IRC | 14:16 | |
*** morazi has joined #puppet-openstack | 14:19 | |
*** chem has joined #puppet-openstack | 14:25 | |
*** ducttape_ has joined #puppet-openstack | 14:43 | |
*** ducttape_ has quit IRC | 14:43 | |
*** ducttape_ has joined #puppet-openstack | 14:44 | |
*** ducttape_ has quit IRC | 14:44 | |
*** sergmelikyan has quit IRC | 14:49 | |
*** ducttape_ has joined #puppet-openstack | 15:04 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Add EL7 support https://review.openstack.org/195959 | 15:04 |
crinkle | EmilienM: sbadia ping re: ci | 15:06 |
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:07 |
sbadia | hey! o/ | 15:08 |
sbadia | morning | 15:08 |
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:09 |
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:10 |
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:12 |
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:13 |
EmilienM | dfisher: debian/ubuntu | 15:14 |
* dfisher phew | 15:14 | |
EmilienM | crinkle: ok I can change it | 15:14 |
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:15 |
crinkle | EmilienM: yes | 15:16 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: unpin APT Puppet module https://review.openstack.org/195803 | 15:16 |
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:17 |
*** mdorman has joined #puppet-openstack | 15:19 | |
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:21 |
EmilienM | crinkle: it works, cool | 15:27 |
sbadia | \o/ | 15:27 |
*** rcallawa has joined #puppet-openstack | 15:29 | |
*** rcallawa has quit IRC | 15:31 | |
EmilienM | sbadia, crinkle: thanks for this work. I think we've taken a thorn out of your side | 15:31 |
crinkle | :) | 15:32 |
*** rcallawa has joined #puppet-openstack | 15:32 | |
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:33 |
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:35 |
*** paramite has quit IRC | 15:36 | |
*** mdorman has quit IRC | 15:38 | |
openstackgerrit | Merged openstack/puppet-openstack_extras: Increase major bound on puppetlabs-apt https://review.openstack.org/195910 | 15:38 |
*** mdorman has joined #puppet-openstack | 15:38 | |
EmilienM | crinkle: infra merged the experimental job about upgrade \o/ | 15:38 |
dmsimard | xarses, dgurtner, dachary: I think it's time to move puppet-ceph to beaker :) | 15:58 |
*** arnaud_orange has quit IRC | 15:58 | |
xarses | I thought we where using beaker prior | 15:58 |
*** rcallawa_ has joined #puppet-openstack | 16:00 | |
*** rcallawa has quit IRC | 16:01 | |
openstackgerrit | Sebastien Badia proposed openstack/puppet-openstack-cookiecutter: Add .gitreview configuration https://review.openstack.org/196121 | 16:03 |
*** rcallawa_ has quit IRC | 16:03 | |
*** rcallawa has joined #puppet-openstack | 16:04 | |
dgurtner | dmsimard: xarses I've almost fixed it :) | 16:04 |
xarses | awesome | 16:06 |
dgurtner | last issue is that the centos7 beaker boxes already come with epel enabled, so need to circumvent that | 16:07 |
*** stamak has quit IRC | 16:08 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Add EL7 support https://review.openstack.org/195959 | 16:09 |
*** rushiagr_away is now known as rushiagr | 16:09 | |
openstackgerrit | Merged openstack/puppet-keystone: Custom file source for wsgi scripts https://review.openstack.org/178181 | 16:15 |
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:16 |
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:17 |
xarses | two LTS + current | 16:18 |
*** derekh has quit IRC | 16:18 | |
xarses | sounds correct | 16:18 |
xarses | 3 in total | 16:18 |
*** jpena has quit IRC | 16:18 | |
xarses | dgurtner: we'll have to add the test runners back if thats what we want | 16:18 |
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:19 |
*** imcsk8 is now known as imcsk8|doctor | 16:19 | |
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:20 |
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:21 |
dgurtner | xarses: also from my understanding it's currently single host, but I haven't had time to look at it lately | 16:22 |
xarses | dgurtner: yes, the switch over seemed to muck it up | 16:23 |
*** aimon has quit IRC | 16:23 | |
EmilienM | dgurtner: it's single host and only trusty & centos7 now | 16:23 |
*** vinsh has joined #puppet-openstack | 16:24 | |
*** mdorman has quit IRC | 16:25 | |
dgurtner | EmilienM: thanks for the info | 16:26 |
dgurtner | lol... Ifixed it for centos, but broke trusty... | 16:28 |
*** mdorman has joined #puppet-openstack | 16:29 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Add EL7 support https://review.openstack.org/195959 | 16:32 |
dmsimard | Yeah I agree on supporting hammer and firefly | 16:39 |
*** dtalmage_ has joined #puppet-openstack | 16:43 | |
dmsimard | dgurtner, xarses: What reviews you want me to look at ? Got limited bandwidth today and saw a sudden surge :) | 16:44 |
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:48 |
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:49 |
EmilienM | crinkle: the idea would be to pin in stable branches & not pin in master | 16:50 |
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:51 |
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:52 |
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:53 |
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 |
*** gfidente has quit IRC | 16:54 | |
dgurtner | EmilienM: I just earlier dropped the 2.2.X pin and ran with 5.1.0 | 16:54 |
xarses | hmm, Is that the infra side, or repo? | 16:55 |
EmilienM | repo | 16:55 |
xarses | dgurtner, sounds like we need to unpin then | 16:56 |
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 has joined #puppet-openstack | 16:57 | |
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:58 |
EmilienM | dgurtner: again, please let me know if our CI is blocking you. We don't want to slow down your dev process | 16:59 |
*** ddmitriev has quit IRC | 16:59 | |
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:00 |
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:01 |
crinkle | lol | 17:02 |
*** mdorman has quit IRC | 17:17 | |
dgurtner | EmilienM: http://logs.openstack.org/59/195959/6/check/gate-puppet-ceph-puppet-beaker-rspec-upgrade-dsvm-trusty/52c44ab/console.html | 17:22 |
*** xingchao has quit IRC | 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:24 |
EmilienM | dgurtner: I'm relocating, back in 20 min max | 17:25 |
*** EmilienM is now known as EmilienM|brb | 17:25 | |
dgurtner | gchamoul: ping | 17:30 |
*** xarses has quit IRC | 17:35 | |
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:53 |
*** xarses has joined #puppet-openstack | 17:54 | |
sbadia | EmilienM|brb: the last one about lint issues :) (params doc is not enable on puppet-heat) | 17:54 |
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:57 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-trove: keystone/auth: make service description configurable https://review.openstack.org/195159 | 17:58 |
sbadia | EmilienM|brb: gate-puppet-*-puppet-beaker-rspec-upgrade-dsvm-{trusty,centos} are voting ? | 18:01 |
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:02 |
*** EmilienM|brb is now known as EmilienM | 18:04 | |
EmilienM | sbadia: should not | 18:04 |
*** rushiagr is now known as rushiagr_away | 18:04 | |
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:05 |
*** hogepodge has quit IRC | 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:06 |
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:07 |
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:08 |
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:09 |
*** aimon has quit IRC | 18:10 | |
*** aimon has joined #puppet-openstack | 18:10 | |
iurygregory | anyone can say when is the deadline for spec approval? | 18:10 |
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:11 |
xarses | EmilienM: I'm not sure we have enough features for it yet | 18:12 |
xarses | or a firm release schedule | 18:12 |
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 |
*** dtalmage_ has quit IRC | 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:13 |
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 |
*** imcsk8|doctor is now known as imcsk8 | 18:15 | |
iurygregory | yes | 18:15 |
dgurtner | EmilienM: I actually had a small change, so just pushed instead of recheck | 18:15 |
EmilienM | dgurtner: cool, let me know if any trouble | 18:16 |
dgurtner | EmilienM: sure! btw. this is your original centos7 patch which is now finally ready :) | 18:17 |
*** rushiagr_away is now known as rushiagr | 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:17 |
dgurtner | EmilienM: yeah, packaging of ceph clients in both centos base and epel is seriously broken from my pov | 18:18 |
EmilienM | honestly, I spent too much time to figure that out | 18:19 |
EmilienM | I'm happy if someone cares about that now | 18:19 |
dgurtner | I only care about the result - if we finally get el7 support merged, I don't care much about anything else :) | 18:20 |
EmilienM | crinkle: can you merge https://review.openstack.org/#/c/195208 please so we can unblock Puppet Ceilometer gate | 18:23 |
*** xingchao has joined #puppet-openstack | 18:23 | |
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:25 |
dgurtner | xarses: sure... it's been a team effort | 18:26 |
crinkle | EmilienM: done | 18:28 |
*** xingchao has quit IRC | 18:28 | |
openstackgerrit | Mathieu Gagné proposed openstack/puppet-designate: Introduce public_url, internal_url and admin_url https://review.openstack.org/70393 | 18:28 |
EmilienM | crinkle: https://goo.gl/jTpYfs | 18:29 |
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:30 |
EmilienM | _ody: hey! can you help me to got a new release of https://github.com/puppetlabs/puppetlabs-inifile ? | 18:31 |
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:32 |
openstackgerrit | Merged openstack/puppet-ceilometer: Fix Beaker CI according recent puppetlabs-rabbitmq change. https://review.openstack.org/195208 | 18:36 |
openstackgerrit | Merged openstack/puppet-neutron: PLUMgrid metadata config. https://review.openstack.org/194634 | 18:37 |
nibalizer | EmilienM: i believe you want to talk to _morgan and Hunner | 18:38 |
EmilienM | nibalizer: yeah | 18:38 |
*** rushiagr is now known as rushiagr_away | 18:39 | |
_morgan | EmilienM: we should be able to release inifile july 7th | 18:40 |
_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 has joined #puppet-openstack | 18:41 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Add EL7 support https://review.openstack.org/195959 | 18:42 |
EmilienM | _morgan: http://goo.gl/F8bNtt | 18:43 |
_morgan | EmilienM: commented! | 18:44 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Add EL7 support https://review.openstack.org/195959 | 18:44 |
xarses | ^ EmilienM dmsimard dachary gchamoul | 18:46 |
*** sergmelikyan has joined #puppet-openstack | 18:46 | |
dmsimard | I can +1 once jenkins weighs in, no +2 since I'm not a RHEL guy | 18:46 |
*** ducttape_ has quit IRC | 18:47 | |
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:48 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Add EL7 support https://review.openstack.org/195959 | 18:54 |
*** stamak has joined #puppet-openstack | 18:56 | |
*** rushiagr_away is now known as rushiagr | 18:56 | |
*** vrao has quit IRC | 18:57 | |
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:00 |
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:10 |
EmilienM | Could not prefetch keystone_role provider 'openstack': undefined method `collect' for nil:NilClass | 19:11 |
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:12 |
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:13 |
EmilienM | crinkle: once it's merged, I send patches to all modules | 19:16 |
xarses | dgurtner: looks like rev 10 failed syntax | 19:17 |
*** jtomasek has quit IRC | 19:18 | |
xarses | https://jenkins03.openstack.org/job/gate-puppet-ceph-puppet-syntax/8/console | 19:18 |
*** rushiagr is now known as rushiagr_away | 19:19 | |
*** markvoelker has quit IRC | 19:20 | |
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:22 |
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:23 |
*** mdorman has joined #puppet-openstack | 19:24 | |
crinkle | EmilienM: ++ | 19:26 |
EmilienM | crinkle: cool | 19:26 |
*** markvoelker has joined #puppet-openstack | 19:26 | |
*** markvoelker has quit IRC | 19:32 | |
*** markvoelker has joined #puppet-openstack | 19:32 | |
*** markvoelker_ has joined #puppet-openstack | 19:38 | |
dgurtner | xarses: having a look - found a small mistake which needs fixing before we merge | 19:38 |
*** markvoelker has quit IRC | 19:40 | |
*** markvoelker has joined #puppet-openstack | 19:44 | |
*** markvoelker_ has quit IRC | 19:45 | |
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:48 |
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:49 |
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:50 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: acceptance: install RDO repo the right way https://review.openstack.org/196214 | 19:51 |
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:52 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-vswitch: acceptance: install RDO repo the right way https://review.openstack.org/196219 | 19:53 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: acceptance: install RDO repo the right way https://review.openstack.org/196220 | 19:54 |
*** markvoelker has quit IRC | 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:54 |
crinkle | EmilienM: will wait for jenkins | 19:55 |
*** jayg is now known as jayg|g0n3 | 19:56 | |
EmilienM | yep | 19:56 |
*** markvoelker has joined #puppet-openstack | 19:58 | |
*** stamak has quit IRC | 19:59 | |
*** markvoelker_ has joined #puppet-openstack | 20:09 | |
*** markvoelker has quit IRC | 20:11 | |
*** markvoelker_ has quit IRC | 20:12 | |
EmilienM | crinkle: except swift (I'm investigating now), everything looks fine (trove in progress) | 20:14 |
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:15 |
EmilienM | crinkle: looking at logs, it seems we need to redo the ssh thing after upgrade :/ | 20:17 |
EmilienM | nibalizer: any idea? | 20:23 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Add EL7 support https://review.openstack.org/195959 | 20:25 |
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:30 |
EmilienM | upgrade jobs is passing jenkins (on trusty only now) \o/ | 20:33 |
*** sergmelikyan has quit IRC | 20:38 | |
*** markvoelker has joined #puppet-openstack | 20:51 | |
xarses | woot woot | 20:55 |
openstackgerrit | Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance https://review.openstack.org/176150 | 20:58 |
*** chem has quit IRC | 21:02 | |
EmilienM | nibalizer: nevermind, Hunner helped me and we figured | 21:02 |
nibalizer | woot | 21:02 |
EmilienM | nibalizer: https://review.openstack.org/196246 | 21:05 |
nibalizer | haha, i guess or 'none' hypervisor stopped working? | 21:07 |
Hunner | nibalizer: Well, technically the hypervisor is fine | 21:07 |
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:08 |
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:10 |
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:11 |
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:12 |
EmilienM | Hunner: not today ;-) | 21:13 |
*** markvoelker has quit IRC | 21:19 | |
*** iurygregory has quit IRC | 21:21 | |
*** openstack has joined #puppet-openstack | 21:26 | |
*** markvoelker has joined #puppet-openstack | 21:28 | |
*** markvoelker has quit IRC | 21:30 | |
*** aimon has quit IRC | 21:45 | |
*** dgurtner has quit IRC | 21:46 | |
openstackgerrit | Merged openstack/puppet-cinder: acceptance: install RDO repo the right way https://review.openstack.org/196207 | 21:47 |
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:48 |
*** rwsu has quit IRC | 21:56 | |
openstackgerrit | Merged openstack/puppet-nova: acceptance: install RDO repo the right way https://review.openstack.org/196213 | 21:57 |
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 | 21:58 |
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:09 |
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:10 |
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:11 |
dmsimard | oh, wait, nevermind, I understand now | 22:12 |
dmsimard | I thought the dependency was the other way around | 22:12 |
xarses | =) | 22:13 |
*** dgurtner has joined #puppet-openstack | 22:14 | |
*** dgurtner has quit IRC | 22:14 | |
*** dgurtner has joined #puppet-openstack | 22:14 | |
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:16 |
*** prad has quit IRC | 22:21 | |
openstackgerrit | Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance https://review.openstack.org/176150 | 22:24 |
openstackgerrit | Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance https://review.openstack.org/176150 | 22:27 |
*** aimon has joined #puppet-openstack | 22:36 | |
*** saneax has joined #puppet-openstack | 22:47 | |
*** dgurtner has quit IRC | 22:48 | |
*** mdorman has quit IRC | 23:41 | |
*** mdorman has joined #puppet-openstack | 23:42 | |
*** mdorman has quit IRC | 23:44 | |
*** mdorman has joined #puppet-openstack | 23:45 | |
*** mdorman has quit IRC | 23:46 | |
*** xarses has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!