| *** otter768 has joined #openstack-chef | 00:15 | |
| *** otter768 has quit IRC | 00:20 | |
| *** stanchan has quit IRC | 00:40 | |
| *** chlong has quit IRC | 00:52 | |
| *** chlong has joined #openstack-chef | 00:54 | |
| *** zhiwei has joined #openstack-chef | 01:13 | |
| *** otter768 has joined #openstack-chef | 02:16 | |
| *** otter768 has quit IRC | 02:21 | |
| *** stevemar has quit IRC | 02:40 | |
| openstackgerrit | Song Li proposed stackforge/cookbook-openstack-network: Auth_url changes following auth_plugin in nova section https://review.openstack.org/186347 | 03:30 |
|---|---|---|
| *** otter768 has joined #openstack-chef | 04:17 | |
| *** pradipta has joined #openstack-chef | 04:22 | |
| *** otter768 has quit IRC | 04:22 | |
| *** wojdev has quit IRC | 05:08 | |
| openstackgerrit | Ma Wen Cheng proposed stackforge/cookbook-openstack-bare-metal: Cleanup minor rubocop offenses https://review.openstack.org/186519 | 05:12 |
| openstackgerrit | Merged stackforge/cookbook-openstack-bare-metal: Cleanup minor rubocop offenses https://review.openstack.org/186519 | 05:20 |
| *** ChrissKO has quit IRC | 05:23 | |
| *** mitz has quit IRC | 05:41 | |
| *** ChrissKO has joined #openstack-chef | 05:48 | |
| *** ogny has joined #openstack-chef | 05:55 | |
| *** otter768 has joined #openstack-chef | 06:18 | |
| *** ChrissKO has quit IRC | 06:20 | |
| *** otter768 has quit IRC | 06:23 | |
| *** epcim_ has joined #openstack-chef | 06:26 | |
| *** wojdev has joined #openstack-chef | 06:35 | |
| openstackgerrit | YangLei proposed stackforge/cookbook-openstack-common: Correct registry_host in glance-api.conf https://review.openstack.org/185914 | 07:36 |
| *** ChrissKO has joined #openstack-chef | 07:58 | |
| jklare | zhiwei you here? | 08:10 |
| zhiwei | yes | 08:10 |
| jklare | zhiwei not sure if someone already pinged you about +1 workflowing the patches? | 08:10 |
| jklare | zhiwei i just saw some patches in the last time where you gave the second +2 but forgot to +1 Workflow it | 08:11 |
| zhiwei | I made a +2 without +1 workflow for some patches indicates that the patch can be optimized. | 08:12 |
| jklare | zhiwei ah ok, but i think that is not the usual workflow and mostly misunderstood by the other cores | 08:12 |
| zhiwei | I will make +2 and +1 if there already a +2 on the patch in next patches. | 08:13 |
| zhiwei | so, it is the right workflow, right? | 08:13 |
| jklare | zhiwei if you think the patch is good and could go in, but there is potential for optimization, you could just go ahead and leave a comment with 'lgtm' , no + or minus but a hint what could be optimized | 08:14 |
| zhiwei | And Christopther sent an email to said: The person +2'ing the change and +1 Workflow should never be the same. | 08:15 |
| jklare | huh? | 08:15 |
| zhiwei | April 20 | 08:15 |
| jklare | let me check | 08:15 |
| zhiwei | But I did not agree on this. | 08:15 |
| zhiwei | :) | 08:16 |
| zhiwei | the email subject is: Chef Openstack core code review | 08:16 |
| zhiwei | jklare: | 08:16 |
| jklare | zhiwei ahh, is see | 08:18 |
| jklare | zhiwei what he probably meant is, that you should not +2 and +1 a change without somebody else +2ing it | 08:18 |
| jklare | so if there is already a initial +2, you are good | 08:18 |
| zhiwei | There is already an official git workflow. | 08:19 |
| *** otter768 has joined #openstack-chef | 08:19 | |
| jklare | and i think we are following it | 08:19 |
| jklare | the mail was mainly about the testing i guess | 08:19 |
| zhiwei | :) | 08:19 |
| zhiwei | ok, I will +1 workflow if there already had a +2. | 08:20 |
| jklare | zhiwei i did not want to blame you or anything, just to make sure that we have no patches laying around which have already been reviewed by 2 cores | 08:20 |
| jklare | :) | 08:20 |
| zhiwei | I see. | 08:20 |
| *** otter768 has quit IRC | 08:24 | |
| *** chlong has quit IRC | 08:34 | |
| openstackgerrit | Jan Klare proposed stackforge/cookbook-openstack-ops-database: switched from db_create_with_user to lwrp openstack_common_database https://review.openstack.org/181430 | 09:19 |
| *** wojdev has quit IRC | 09:29 | |
| *** wojdev has joined #openstack-chef | 09:53 | |
| *** wojdev has quit IRC | 10:03 | |
| *** otter768 has joined #openstack-chef | 10:20 | |
| *** otter768 has quit IRC | 10:24 | |
| *** wojdev has joined #openstack-chef | 10:24 | |
| *** pradipta has quit IRC | 10:43 | |
| *** jaycee has quit IRC | 10:49 | |
| *** wojdev has quit IRC | 11:06 | |
| *** os-chef-bot1 has quit IRC | 11:23 | |
| *** os-chef-bot has joined #openstack-chef | 11:23 | |
| *** zhiwei has quit IRC | 11:53 | |
| *** webguynow has quit IRC | 12:01 | |
| *** otter768 has joined #openstack-chef | 12:21 | |
| *** otter768 has quit IRC | 12:25 | |
| *** mattray has joined #openstack-chef | 12:45 | |
| *** ChanServ sets mode: +o mattray | 12:45 | |
| *** wojdev has joined #openstack-chef | 12:47 | |
| *** mattray has quit IRC | 12:48 | |
| *** mattray has joined #openstack-chef | 12:51 | |
| *** ChanServ sets mode: +o mattray | 12:51 | |
| *** mattray has quit IRC | 12:56 | |
| *** rtheis has joined #openstack-chef | 12:56 | |
| *** rtheis_ has quit IRC | 12:58 | |
| *** rtheis_ has joined #openstack-chef | 13:11 | |
| *** rtheis has quit IRC | 13:13 | |
| *** wojdev has quit IRC | 13:15 | |
| *** bdemers has joined #openstack-chef | 13:26 | |
| openstackgerrit | Jan Klare proposed stackforge/cookbook-openstack-common: update the apt cookbook version https://review.openstack.org/188776 | 13:48 |
| openstackgerrit | Jan Klare proposed stackforge/cookbook-openstack-common: add support for percona-cluster as db backend https://review.openstack.org/179816 | 14:03 |
| *** wojdev has joined #openstack-chef | 14:12 | |
| *** otter768 has joined #openstack-chef | 14:21 | |
| *** otter768 has quit IRC | 14:26 | |
| openstackgerrit | Jan Klare proposed stackforge/cookbook-openstack-common: add support for percona-cluster as db backend https://review.openstack.org/179816 | 14:38 |
| *** stevemar has joined #openstack-chef | 14:44 | |
| *** wojdev has quit IRC | 14:49 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-image: Use correct endpoint scheme for api to registry connection https://review.openstack.org/188576 | 14:59 |
| *** mattray has joined #openstack-chef | 15:11 | |
| *** ChanServ sets mode: +o mattray | 15:11 | |
| *** wojdev has joined #openstack-chef | 15:18 | |
| openstackgerrit | Jan Klare proposed stackforge/openstack-chef-repo: update apt cookbook version https://review.openstack.org/188818 | 15:18 |
| *** ogny has quit IRC | 15:18 | |
| *** wojdev has quit IRC | 15:54 | |
| *** wojdev has joined #openstack-chef | 15:54 | |
| markvan | @core See my note on https://review.openstack.org/#/c/181429/ about CI testing cross cookbook patches. Probably a good topic for discussion | 16:05 |
| os-chef-bot | @j^2 @markvan @mattray @wenchma @jklare @cmluciano @zhiwei See my note on https://review.openstack.org/#/c/181429/ about CI testing cross cookbook patches. Probably a good topic for discussion | 16:05 |
| *** otter768 has joined #openstack-chef | 16:22 | |
| *** otter768 has quit IRC | 16:27 | |
| *** wojdev has quit IRC | 16:32 | |
| *** epcim_ has quit IRC | 16:35 | |
| sc` | markvan: was thinking about the prospect of maintaining "stable" repos. today, for centos, there exists a few repos that one has to add in, in order to get ironic. it doesn't translate well to the cookbooks, so we'd need to aggregate all of those packages into a single repo that could be leveraged by openstack-common | 16:40 |
| *** ChrissKO has quit IRC | 16:41 | |
| sc` | alternately, for the master branch, add in those additional repos and nowhere else, but that seems rather anti-patternish to me | 16:41 |
| sc` | perhaps it's worth collaborating with the puppet folks since they have the same goal | 16:43 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-common: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188850 | 16:51 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-bare-metal: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188851 | 16:52 |
| *** aspiers has quit IRC | 16:53 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-block-storage: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188852 | 16:56 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-compute: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188855 | 16:59 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-identity: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188858 | 17:02 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-image: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188861 | 17:05 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-network: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188865 | 17:09 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-orchestration: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188867 | 17:14 |
| *** famanfortune has joined #openstack-chef | 17:23 | |
| *** famanfortune has left #openstack-chef | 17:23 | |
| *** wojdev has joined #openstack-chef | 17:31 | |
| *** lastops has joined #openstack-chef | 17:33 | |
| *** wojdev has quit IRC | 17:48 | |
| *** jmickle has joined #openstack-chef | 18:04 | |
| jmickle | has anyone use openstack + chef + nexus switches? | 18:04 |
| *** otter768 has joined #openstack-chef | 18:23 | |
| *** otter768 has quit IRC | 18:28 | |
| *** nacer has quit IRC | 18:30 | |
| *** nacer has joined #openstack-chef | 18:31 | |
| *** botang has quit IRC | 18:33 | |
| *** botang_ has joined #openstack-chef | 18:34 | |
| *** Y4Rv1K has joined #openstack-chef | 18:35 | |
| *** Y4Rv1K has left #openstack-chef | 18:35 | |
| *** ctracey has quit IRC | 18:54 | |
| *** victor_lowther has quit IRC | 18:54 | |
| *** botang_ has quit IRC | 18:55 | |
| *** mattray has quit IRC | 18:58 | |
| *** lastops has quit IRC | 19:00 | |
| j^2 | not i | 19:02 |
| *** dclain has quit IRC | 19:03 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-bare-metal: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188851 | 19:09 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-block-storage: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188852 | 19:10 |
| *** lastops has joined #openstack-chef | 19:10 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-compute: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188855 | 19:11 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-identity: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188858 | 19:12 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-image: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188861 | 19:12 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-network: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188865 | 19:13 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-orchestration: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188867 | 19:14 |
| *** lastops has quit IRC | 19:15 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-telemetry: Allow oslo rabbit heartbeat configuration https://review.openstack.org/188894 | 19:17 |
| *** pradipta has joined #openstack-chef | 19:19 | |
| *** lastops has joined #openstack-chef | 19:22 | |
| *** lastops has quit IRC | 19:25 | |
| *** jmickle has quit IRC | 19:28 | |
| markvan | sc`: is there some reason for the hold up with ironic with centos/redhat? | 19:33 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/openstack-chef-repo: CI TEST - ALPHA https://review.openstack.org/185085 | 19:37 |
| sc` | markvan: there was a regression with neutron a few weeks ago that impacted ironic. i'm manually patching in the rdo-manager repos to see what happens, but running into unrelated mariadb regressions | 19:42 |
| markvan | sc`: thx for update | 19:42 |
| os-chef-bot | markvan say thanks to sc` for update | 19:42 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/openstack-chef-repo: CI TEST - ALPHA https://review.openstack.org/185085 | 19:43 |
| *** botang has joined #openstack-chef | 19:44 | |
| *** botang has quit IRC | 19:53 | |
| *** botang has joined #openstack-chef | 19:55 | |
| *** botang has quit IRC | 19:59 | |
| *** cmluciano has quit IRC | 20:02 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/openstack-chef-repo: CI TEST - ALPHA https://review.openstack.org/185085 | 20:17 |
| j^2 | who thanks me | 20:20 |
| j^2 | show ranking | 20:21 |
| *** otter768 has joined #openstack-chef | 20:24 | |
| *** otter768 has quit IRC | 20:29 | |
| *** victor_lowther has joined #openstack-chef | 20:38 | |
| *** cmluciano has joined #openstack-chef | 20:46 | |
| *** ctracey has joined #openstack-chef | 20:47 | |
| *** pradipta has quit IRC | 20:49 | |
| *** botang has joined #openstack-chef | 20:55 | |
| *** epcim_ has joined #openstack-chef | 21:00 | |
| *** rtheis_ has quit IRC | 21:11 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 21:11 |
| epcim_ | markvan: fyi: see the issues section - may get handy for multi node CI -> https://www.rdoproject.org/Networking (like use virtio driver if os host is virtualized) | 21:14 |
| markvan | epcim_: will take a look | 21:16 |
| *** lastops has joined #openstack-chef | 21:18 | |
| epcim_ | markvan: re boostrap.sh - you may install chefdk this way: 'curl -L https://www.opscode.com/chef/install.sh | sudo bash -s -- -P chefdk' | 21:19 |
| markvan | epcim_: yeah infra does not like the direct curl bash installs, so we did the formal package | 21:20 |
| epcim_ | also you should then do eval "$(chef shell-init bash)" (or/and redirect that to bashrc) | 21:20 |
| *** lastops has quit IRC | 21:23 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 21:26 |
| markvan | jklare: I think were going to need to add a log publishers like: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml#L650 to the CI gate job, so folks can review them | 21:34 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 21:36 |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 21:40 |
| *** openstack has joined #openstack-chef | 21:43 | |
| *** ChanServ sets mode: +o openstack | 21:43 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 21:50 |
| *** stevemar has quit IRC | 22:12 | |
| *** otter768 has joined #openstack-chef | 22:25 | |
| *** otter768 has quit IRC | 22:30 | |
| *** openstackgerrit has quit IRC | 22:37 | |
| *** openstackgerrit has joined #openstack-chef | 22:37 | |
| *** epcim_ has quit IRC | 22:46 | |
| *** ozialien has joined #openstack-chef | 22:49 | |
| openstackgerrit | JJ Asghar proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 22:57 |
| openstackgerrit | JJ Asghar proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 22:58 |
| *** ozialien has quit IRC | 23:02 | |
| *** ozialien has joined #openstack-chef | 23:05 | |
| j^2 | anyone around? | 23:14 |
| sc` | i am | 23:19 |
| j^2 | so it looks like we’re gonna need to put no host-key checking on: https://review.openstack.org/#/c/188924/ | 23:19 |
| j^2 | https://github.com/stackforge/openstack-chef-repo/blob/master/tools/test_patch.rb#L46 | 23:19 |
| j^2 | unless i’m not reading that correctly | 23:20 |
| sc` | did the key get regenerated somewhere? | 23:21 |
| sc` | that seems like a known_hosts issue | 23:21 |
| j^2 | mabye it’s getting sucked in via the internal network connection? | 23:21 |
| j^2 | yeah we could blow away the known_hosts file i guess | 23:22 |
| j^2 | i’ll try that, but i’m betting it’ll fail because you have to say “yes" | 23:22 |
| openstackgerrit | JJ Asghar proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 23:23 |
| j^2 | same problem it seems | 23:25 |
| sc` | might be worth running it by infra | 23:25 |
| j^2 | yeah probably | 23:26 |
| j^2 | i thought i might give it a shot without bugging em | 23:26 |
| j^2 | interesting | 23:26 |
| j^2 | + rm /home/jenkins/.ssh/known_hosts | 23:27 |
| j^2 | 2015-06-05 23:23:43.417 | rm: cannot remove ‘/home/jenkins/.ssh/known_hosts’: No such file or directory | 23:27 |
| *** ozialien has quit IRC | 23:33 | |
| openstackgerrit | JJ Asghar proposed stackforge/openstack-chef-repo: Added check for jenkins and ci https://review.openstack.org/188972 | 23:40 |
| sc` | disabling strict host key checking makes things more succeptible to mitm :( | 23:41 |
| j^2 | ye | 23:41 |
| j^2 | p | 23:41 |
| j^2 | but this is being ran as jenkins | 23:41 |
| j^2 | and only jenkins | 23:41 |
| openstackgerrit | JJ Asghar proposed stackforge/cookbook-openstack-common: CI TEST - ALPHA https://review.openstack.org/188924 | 23:42 |
| j^2 | i hate fucking linters | 23:43 |
| openstackgerrit | JJ Asghar proposed stackforge/openstack-chef-repo: Added check for jenkins and ci https://review.openstack.org/188972 | 23:45 |
| sc` | got a centos 7 convergence with manually patching in the rdo-manager repos and overriding some defaults for mariadb | 23:49 |
| sc` | not sure why /var/log/mysql doesn't have the correct ownership | 23:49 |
| j^2 | grrr | 23:50 |
| j^2 | but at least progress right? | 23:50 |
| sc` | yup. ironic installs just fine after adding in the extra repos | 23:51 |
| sc` | but mariadb on c7 still has issues | 23:51 |
| j^2 | https://review.openstack.org/#/c/188972 | 23:51 |
| j^2 | just succeeded | 23:51 |
| j^2 | can you +2 it? | 23:51 |
| j^2 | you know you want to | 23:52 |
| sc` | ha. just clicked :) | 23:52 |
| j^2 | nice, i just need the WF now :p | 23:52 |
| j^2 | i could be bad…. | 23:52 |
| j^2 | this is stopping us from the CI work... | 23:52 |
| j^2 | it’s so blue... | 23:53 |
| j^2 | and clickable | 23:53 |
| openstackgerrit | JJ Asghar proposed stackforge/cookbook-openstack-compute: rake -T is useful https://review.openstack.org/188446 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!