*** openstackgerrit has quit IRC | 00:01 | |
*** tiswanso has joined #openstack-infra | 00:01 | |
*** openstackgerrit has joined #openstack-infra | 00:01 | |
*** otter768 has quit IRC | 00:02 | |
*** openstackstatus has quit IRC | 00:02 | |
*** openstack has joined #openstack-infra | 00:05 | |
*** dingyichen has joined #openstack-infra | 00:05 | |
*** sflanigan has quit IRC | 00:05 | |
*** MarkAtwood_ has quit IRC | 00:06 | |
anteaya | Clint: any thoughts on perhaps shortening this file name a bit? https://review.openstack.org/#/c/228002/8/tools/rename_projects_in_projects_yaml_and_channels_yaml_and_layout_yaml.py | 00:07 |
---|---|---|
openstackgerrit | Skyler Berg proposed openstack-infra/project-config: Add ciwatch - CI monitoring tool https://review.openstack.org/216840 | 00:07 |
*** sflanigan has joined #openstack-infra | 00:07 | |
*** sflanigan has joined #openstack-infra | 00:07 | |
Clint | anteaya: got any suggestions? | 00:07 |
jeblair | rename_*.yaml.py | 00:08 |
* Clint claps. | 00:08 | |
anteaya | I like it | 00:08 |
Clint | i won't refuse | 00:08 |
*** julim has joined #openstack-infra | 00:08 | |
*** devlaps has quit IRC | 00:09 | |
*** Sukhdev has quit IRC | 00:09 | |
*** geoffarnold has quit IRC | 00:11 | |
*** signed8bit has quit IRC | 00:11 | |
*** abitha has quit IRC | 00:11 | |
openstackgerrit | Merged openstack-infra/project-config: Test formatting of gerritbot/channels.yaml https://review.openstack.org/228656 | 00:12 |
anteaya | Clint: were you going to offer a new patch? | 00:15 |
Clint | anteaya: i was waiting for someone to object strenuously to the asterisk first | 00:15 |
Clint | but i can do something potentially less offensive | 00:16 |
*** Swami has quit IRC | 00:17 | |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 00:17 |
anteaya | thank you | 00:17 |
anteaya | Clint: you're so creative | 00:17 |
*** amotoki has joined #openstack-infra | 00:18 | |
Clint | :/ | 00:18 |
lifeless | Clint: I was expecting ☃ or something | 00:19 |
*** Sukhdev has joined #openstack-infra | 00:19 | |
jhesketh | clarkb: hmm os-loganalyze is throwing a 500... I'll log in and take a poke around... (it's only 500 on indexes so it's not hurting) | 00:19 |
*** tsekiyam_ has joined #openstack-infra | 00:19 | |
Clint | lifeless: you could have said that minutes ago | 00:19 |
*** mtanino has quit IRC | 00:19 | |
clarkb | jhesketh: just generated indexes or all index.htmls? | 00:19 |
jhesketh | clarkb: just generated | 00:20 |
anteaya | clarkb: how do I tell it I am finished inputting names and to run the script? | 00:20 |
* clarkb assumes anteaya really wnts clint | 00:20 | |
anteaya | Clint: ^^ | 00:20 |
anteaya | clarkb: you assume correctly | 00:20 |
anteaya | the cl<tab> wrong mailbox | 00:21 |
jhesketh | clarkb: hmm, it can't load the template (TemplateNotFound: file_index.html) | 00:21 |
Clint | anteaya: ctrl-d on a blank line should work | 00:21 |
anteaya | Clint: apparently enter and control-C are both wrong | 00:21 |
anteaya | I will do that | 00:21 |
jhesketh | clarkb: I wonder if we need to list it as a static file in os-loganalyze | 00:21 |
clarkb | jhesketh: I wonder if pbr/pip/setuptools ya | 00:21 |
*** amotoki has quit IRC | 00:22 | |
*** otter768 has joined #openstack-infra | 00:22 | |
*** tiswanso has quit IRC | 00:22 | |
lifeless | clarkb: ? | 00:23 |
clarkb | lifeless: os-loganalyze added an html template that we need to install properly | 00:23 |
*** tsekiyama has quit IRC | 00:23 | |
*** tsekiyam_ has quit IRC | 00:23 | |
anteaya | Clint: okay that seems to be working, how do you feel about changing the script to accept a text file of names as an argument rather than inputting them on the command line? | 00:25 |
lifeless | its not in setup.cfg atm | 00:25 |
*** yamamoto has quit IRC | 00:26 | |
clarkb | ya I think we know what the problem is | 00:26 |
clarkb | just need to add it to package files | 00:26 |
clarkb | or is it data file | 00:26 |
*** alivigni has joined #openstack-infra | 00:26 | |
lifeless | env = jinja2.Environment( | 00:26 |
lifeless | loader=jinja2.PackageLoader('os_loganalyze', 'templates')) | 00:26 |
lifeless | template = env.get_template('file_index.html') | 00:26 |
lifeless | AIUI that should read from the package dir | 00:26 |
Clint | anteaya: i could do that, though you can do that right now with tools/mass_rename_projects.py </tmp/file-of-names | 00:26 |
*** alivigni has quit IRC | 00:26 | |
anteaya | Clint: I will do that | 00:27 |
*** alivigni has joined #openstack-infra | 00:28 | |
*** rfolco has quit IRC | 00:29 | |
anteaya | Clint: that works, thank you | 00:30 |
*** rfolco has joined #openstack-infra | 00:30 | |
Clint | anteaya: yay | 00:30 |
clarkb | jhesketh: so we traded a 404 for 500 I can live with that for now | 00:30 |
clarkb | but also happy to review fix to setup.cfg if you wnat ot push one | 00:31 |
jhesketh | lifeless: right, so where is the best place to include the template file with the package? | 00:31 |
jhesketh | setup.cfg I'm assuming | 00:31 |
lifeless | jhesketh: are we sure its not being included? | 00:31 |
clarkb | jhesketh: I think you do a package_data = stuff in setup.cfg or update manifest.in | 00:31 |
clarkb | thats easy to check /me looks | 00:31 |
*** pahuang has quit IRC | 00:31 | |
jhesketh | lifeless: I'll look | 00:31 |
jhesketh | or maybe clarkb will beat me | 00:32 |
clarkb | hard linking os_loganalyze/templates/file_index.html -> os_loganalyze-0.0.2.dev48/os_loganalyze/templates | 00:32 |
clarkb | its in the tar.gz | 00:32 |
lifeless | file_index.html is installed here | 00:32 |
lifeless | ls ~/.virtualenvs/scratch/lib/python2.7/site-packages/os_loganalyze/templates/ | 00:32 |
lifeless | file_index.html | 00:32 |
lifeless | and the loader code works | 00:33 |
clarkb | so maybe we need an apache restart? | 00:33 |
clarkb | hrm | 00:33 |
clarkb | we should've fixed that already so likely not it | 00:34 |
lifeless | I would check a) its on disk on the server | 00:34 |
anteaya | Clint: okay blazar also changes blazar-nova: http://paste.openstack.org/show/474763/ | 00:34 |
lifeless | if its not, we can work that path. If it is, we can look at why its not loading | 00:34 |
lifeless | jhesketh: how is it being installed ? | 00:34 |
*** ddieterly has joined #openstack-infra | 00:34 | |
anteaya | Clint: which is going to cause problems with all the puppet repos, as they are going different places | 00:34 |
clarkb | its not on disk on the server | 00:35 |
Clint | anteaya: ah, good catch | 00:35 |
*** pvaneck has quit IRC | 00:35 | |
clarkb | /usr/local/lib/python2.7/dist-packages/os_loganalyze has no templates dir | 00:35 |
anteaya | Clint: I'll add a link to that paste to teh patch for the other reviewers | 00:36 |
lifeless | hang on, dist-packages? | 00:36 |
Clint | anteaya: thanks | 00:36 |
clarkb | lifeless: its a global pip install | 00:36 |
jhesketh | lifeless: installed with 'pip install .' | 00:36 |
clarkb | lifeless: https://git.openstack.org/cgit/openstack-infra/puppet-openstackci/tree/manifests/logserver.pp#n110 | 00:36 |
anteaya | Clint: welcome | 00:37 |
*** bdemers has joined #openstack-infra | 00:37 | |
lifeless | clarkb: how odd, what version of pip? | 00:37 |
openstackgerrit | Mike Scherbakov proposed openstack-infra/project-config: Enabled Python checks for Fuel repos where applicable https://review.openstack.org/228204 | 00:38 |
clarkb | 7.1.2 | 00:38 |
lifeless | is wheel installed ? | 00:38 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 00:38 |
Clint | anteaya: that should fi it | 00:38 |
Clint | x | 00:38 |
*** sdake_ has joined #openstack-infra | 00:38 | |
clarkb | lifeless: yes wheel==0.26.0 | 00:39 |
anteaya | Clint: thanks, I'll check | 00:39 |
*** stevemar has joined #openstack-infra | 00:40 | |
openstackgerrit | Mike Scherbakov proposed openstack-infra/project-config: Enabled Python checks for Fuel repos where applicable https://review.openstack.org/228204 | 00:40 |
*** Daisy has joined #openstack-infra | 00:42 | |
anteaya | Clint: it does thank you | 00:42 |
lifeless | clarkb: one interesting thing to note is the dependency on pbr < 1 | 00:42 |
openstackgerrit | lifeless proposed openstack-infra/os-loganalyze: Update requirements https://review.openstack.org/229198 | 00:42 |
Clint | anteaya: \o/ | 00:42 |
anteaya | Clint: my only remaining complaint is the removal of the comments in gerritbot/channels.yaml | 00:42 |
clarkb | I also wonder if we need pip install -U to be more forceful | 00:42 |
*** sdake has quit IRC | 00:43 | |
anteaya | Clint: do we just want to add them manually after running the script? | 00:43 |
Clint | anteaya: i don't have a better solution right now, though i was hoping someone else might | 00:43 |
openstackgerrit | Merged openstack-infra/project-config: Make gate-swift-tox-func voting https://review.openstack.org/205723 | 00:43 |
anteaya | Clint: well it doesn't seem to be monkeying with comments in zuul/layout.yaml which was my big concern | 00:43 |
lifeless | clarkb: is it possible to get the output of pip installing it ? | 00:44 |
Clint | anteaya: yep, definitely not doing that | 00:44 |
lifeless | clarkb: pip install . should always do something | 00:44 |
clarkb | lifeless: let me check roots pip log | 00:44 |
anteaya | Clint: can we have an acknowledgement in the commit message that comments in gerritbot/channels.yaml need to be added back manually after the script runs | 00:44 |
anteaya | so folks don't forget? | 00:44 |
Clint | sure | 00:44 |
clarkb | lifeless: the only thing in the pip.log is from march | 00:44 |
anteaya | adding a comment to the file obviously won't work :) | 00:44 |
lifeless | clarkb: what does pip list show ? | 00:44 |
clarkb | os-loganalyze (0.0.1.2.g70d32d5) | 00:45 |
anteaya | Clint: and then I think you can offer another patch to gerritbot/channels.yaml since swift3 didn't get the whitespace changes | 00:45 |
clarkb | which is not what the dist version is | 00:45 |
lifeless | clarkb: indeed | 00:45 |
clarkb | so maybe this is an older pbr version break | 00:45 |
*** Sukhdev has quit IRC | 00:45 | |
lifeless | clarkb: well my first test was with the older pbr | 00:45 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 00:45 |
lifeless | clarkb: and that worked | 00:45 |
lifeless | clarkb: but | 00:46 |
Clint | anteaya: it didn't? | 00:46 |
lifeless | clarkb: if its not installed, then it's not going to run :) | 00:46 |
*** Sukhdev has joined #openstack-infra | 00:46 | |
anteaya | yeah run your script with an updated master | 00:46 |
anteaya | swift3 needs the whitespace changes in channels.yaml | 00:46 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 00:46 |
*** hdd has quit IRC | 00:46 | |
lifeless | clarkb: try a manual sudo pip install . and see what happens ? | 00:46 |
clarkb | jhesketh: ^ want to give that a go? | 00:46 |
anteaya | Clint: I think swift3 is a recent addition | 00:47 |
Clint | anteaya: https://review.openstack.org/#/c/228653/ should have fixed that | 00:48 |
*** pahuang has joined #openstack-infra | 00:48 | |
anteaya | ah I will test with your rebased change and see it that makes a difference | 00:49 |
*** bdemers has quit IRC | 00:49 | |
*** bdemers_ has joined #openstack-infra | 00:49 | |
jhesketh | clarkb: can do | 00:49 |
anteaya | Clint: yep that was it | 00:50 |
Clint | anteaya: cool | 00:50 |
anteaya | Clint: your rebase caught it | 00:50 |
* Clint nods. | 00:50 | |
jhesketh | Successfully uninstalled os-loganalyze-0.0.1.2.g70d32d5 | 00:51 |
jhesketh | Running setup.py install for os-loganalyze | 00:51 |
jhesketh | Successfully installed os-loganalyze-0.0.1.40.g9350510 | 00:51 |
jhesketh | clarkb, lifeless: ^ | 00:51 |
anteaya | Clint: okay so do you have any other patches you need to work on for the rename? | 00:51 |
jhesketh | but I still can't see the templates folder | 00:51 |
*** rossella_s has joined #openstack-infra | 00:51 | |
clarkb | jhesketh: ya thats not the version I expect | 00:52 |
clarkb | should be 0.0.2.dev48 | 00:52 |
clarkb | but thats because I have newer pbr I think | 00:52 |
jhesketh | oh okay | 00:52 |
jhesketh | clarkb: should I try with the -U flag? | 00:52 |
Clint | anteaya: no, the only issue i'm currently aware of is the comment one, so i'm relying on everyone else to catch anything that's been missed | 00:52 |
clarkb | jhesketh: is git log -1 te correct commit in /opt/os-loganalyze? | 00:52 |
jhesketh | yep | 00:53 |
anteaya | Clint: great, so tomorrow let's offer up a work in progress patch to project config | 00:53 |
*** sdake_ has quit IRC | 00:53 | |
clarkb | we also have a os_loganalyze-0.0.1.49.ge107939.egg-info | 00:53 |
anteaya | Clint: with a text file of all the projects that want to be renamed so far according to the wiki | 00:53 |
clarkb | so I think that we may have confused pip somehow | 00:53 |
anteaya | Clint: and generated patches so we can start going over what the generated files will look like | 00:53 |
clarkb | jhesketh: I think we want to rm all those egg infos then try reinstalling | 00:53 |
clarkb | and go from there | 00:53 |
*** lawrancejing has joined #openstack-infra | 00:54 | |
Clint | anteaya: sounds good | 00:54 |
anteaya | Clint: maybe we can start with a text file of 20 intented renamed projects to start | 00:54 |
lifeless | jhesketh: git status ? | 00:54 |
jhesketh | clarkb: rm them from /opt/os-lognalazye right? | 00:54 |
anteaya | to save my eyes off the start | 00:54 |
jhesketh | lifeless: clean | 00:54 |
Clint | heh | 00:54 |
anteaya | Clint: thank you | 00:54 |
lifeless | jhesketh: gnaar | 00:54 |
clarkb | jhesketh: no from /usr/local/lib/python2.7/disk-packages | 00:54 |
clarkb | I think there is enough old state there that pbr+pip are getting confused | 00:54 |
anteaya | Clint: and we can include a paste of the mv commands that are generated as well | 00:55 |
clarkb | we saw this a lot more often with older pip | 00:55 |
jhesketh | clarkb: maybe... should we try a pip install -U . first? | 00:55 |
lifeless | jhesketh: ok so its now installed a new version | 00:55 |
clarkb | jhesketh: ya probably worth trying -U first | 00:55 |
anteaya | Clint: or perhaps we can capture them in a text file too, that can also be offered up somewhere shareable | 00:55 |
lifeless | jhesketh: so, check if its installed now | 00:55 |
lifeless | clarkb: jhesketh: no need - jhesketh's log shows it upgraded | 00:55 |
clarkb | lifeless: but to the wrong version | 00:55 |
jhesketh | apparently not the version we expected | 00:55 |
anteaya | Clint: anyway for tomorrow, nice work here, we might actually survive the rename :) | 00:55 |
clarkb | lifeless: that git sha doesn't match HEAD | 00:55 |
lifeless | clarkb: what version are you expecting ? | 00:56 |
lifeless | jhesketh: what does git describe output ? | 00:56 |
Clint | anteaya: we could also have it actually run those commands instead of displaying.. i was previously worried that there might be a problem with one of the acl files that are shared between projects, but maybe that's not a real concern | 00:56 |
clarkb | lifeless: 0.0.2.dev48 is what I get locally | 00:56 |
lifeless | clarkb: yes, thats your pbr | 00:56 |
clarkb | and e1079394e376885abc44d042aee8e554f809ae97 is the sha1 | 00:56 |
clarkb | so I expect .gthatsha1 at the very least | 00:56 |
anteaya | Clint: well the env that generates the patch is highly unlikely to also be running the mv commands | 00:56 |
Clint | ah | 00:57 |
lifeless | clarkb: me too, I'm just asking that we gather data | 00:57 |
anteaya | so I think having the mv commands somewhere where they can be copy/pasted is useful | 00:57 |
jhesketh | clarkb, lifeless: http://paste.openstack.org/show/474766/ | 00:57 |
lifeless | clarkb: if there's a pip bug here, I want to be able to file it | 00:57 |
clarkb | lifeless: sure thats fair | 00:57 |
Clint | anteaya: okay, great | 00:57 |
anteaya | Clint: thank you | 00:57 |
Clint | sure, thanks for looking closely | 00:57 |
anteaya | yep | 00:57 |
anteaya | thanks for doing this work | 00:57 |
*** duobei has joined #openstack-infra | 00:57 | |
anteaya | very helpful | 00:57 |
jhesketh | hmm, now http://logs.openstack.org/51/226751/14/check/nodepool-coverage/d06565b/console.html is failing | 00:58 |
jhesketh | clarkb: I'm going to restart apache | 00:58 |
jhesketh | *reload | 00:58 |
lifeless | jhesketh: uhh... i was saying not to :/. | 00:58 |
lifeless | jhesketh: what does git describe show in the os-loganalyze dir ? | 00:58 |
jhesketh | lifeless: 0.0.1-49-ge107939 | 00:59 |
jhesketh | clarkb: hmm, getting http://paste.openstack.org/show/474767/ on general log loads now | 00:59 |
lifeless | jhesketh: btw IIRC you want sudo -i, not sudo, to fix that warning at the top of your pip run | 01:00 |
clarkb | jhesketh: I think we have a broken install right now because it installed the wrong version for who knows what reason | 01:00 |
clarkb | once we fix that I think it will just work | 01:00 |
*** bharathm has quit IRC | 01:01 | |
lifeless | jhesketh: that pbr error indicates the pkg-info metadata for python-swiftclient is not on the python path for some reason | 01:02 |
*** duobei has quit IRC | 01:02 | |
*** Sukhdev has quit IRC | 01:03 | |
lifeless | jhesketh: you should have a ...-packages/python_swiftclient-2.6.0.dist-info/METADATA | 01:03 |
lifeless | jhesketh: (and RECORD and WHEEL and so on) | 01:03 |
lifeless | jhesketh: do you ? | 01:03 |
clarkb | we do | 01:04 |
jhesketh | yeah those are there | 01:04 |
lifeless | python -c "import pbr.version;print(pbr.version.VersionInfo('python-swiftclient'))" | 01:05 |
lifeless | should work | 01:05 |
lifeless | if it does, its something to do with your wsgi environment | 01:05 |
lifeless | if it doesn't, its global | 01:05 |
clarkb | it does not work | 01:05 |
*** bpokorny_ has quit IRC | 01:05 | |
clarkb | this is pbr 0.11.o | 01:06 |
clarkb | er 0.11.0 | 01:06 |
lifeless | yeah, so you'll want to merge my patch to update the requirements | 01:07 |
lifeless | but as a workaround | 01:07 |
*** alexsyip has quit IRC | 01:07 | |
lifeless | "sudo -i pip install pbr>1 ." | 01:07 |
lifeless | should fix it | 01:07 |
jhesketh | clarkb: did you want to run that, or shall I? | 01:07 |
clarkb | jhesketh: ^ relative to os-loganalyze I think | 01:08 |
clarkb | jhesketh: go for it | 01:08 |
*** sghanekar_ has quit IRC | 01:08 | |
jhesketh | jhesketh@static:/opt/os-loganalyze$ sudo -i pip install "pbr>1" . | 01:09 |
jhesketh | Directory '.' is not installable. File 'setup.py' not found. | 01:09 |
jhesketh | setup.py is definiately there | 01:09 |
*** baoli has joined #openstack-infra | 01:10 | |
clarkb | does -i change the dir? | 01:10 |
lifeless | FWIW I think os-loganaylse probably wants to participate in req syncing since it needs to be sanely coinstallable with swiftclient and keystoneclient | 01:10 |
lifeless | clarkb: oh, perhaps. | 01:10 |
clarkb | I think oyu may want -H intead | 01:10 |
lifeless | ah yeah thats the one | 01:10 |
jhesketh | lifeless, clarkb: http://paste.openstack.org/show/474768/ | 01:11 |
clarkb | it just keeps downgrading and downgrading :) | 01:12 |
clarkb | pbr is now 1.8.0 though | 01:13 |
clarkb | and swiftclient version is still borked | 01:13 |
lifeless | ok, so remaining candidate is pkg_resources | 01:14 |
lifeless | pip list | grep setuptools ? | 01:14 |
clarkb | oh wow setuptools (0.6rc11) thats ancient | 01:14 |
mordred | oh my | 01:14 |
clarkb | so likely want to juts upgrade setuptools before anything else | 01:14 |
mordred | that will certainly break things | 01:14 |
clarkb | then try again | 01:14 |
jlvillal | Any known issue with the log server? | 01:14 |
*** sridhar_ram has joined #openstack-infra | 01:14 | |
lifeless | jlvillal: yes, we're working on it | 01:14 |
jlvillal | Thanks! | 01:14 |
mordred | btw | 01:15 |
mordred | "sudo -i pip install pbr>1 ." will not work | 01:15 |
clarkb | jhesketh: so I think we want pip install -U setuptools | 01:15 |
mordred | oh - I see the quote - nm | 01:15 |
clarkb | jhesketh: then rerun the loganalyze install | 01:15 |
* mordred has many files called 1 for that reason | 01:15 | |
clarkb | thtas an amazingly old setuptools thing | 01:15 |
jhesketh | clarkb: hmm, will that be safe do you think? | 01:15 |
clarkb | mordred: haha | 01:15 |
mordred | I'm guesisng that something pulled it in | 01:15 |
clarkb | jhesketh: ya I think that should be safe | 01:15 |
mordred | with an == and a -U somewhere | 01:15 |
lifeless | clarkb: its actually not that old, but it is that bad :) | 01:15 |
mordred | lifeless: ++ | 01:15 |
clarkb | lifeless: its like more than 9 months old | 01:16 |
clarkb | in setuptools terms that ancient | 01:16 |
jhesketh | okay upgraded setuptool | 01:16 |
jhesketh | should I reinstall os-loganalyze with -U? | 01:16 |
clarkb | and swiftclient version works now | 01:16 |
lifeless | jhesketh: no | 01:16 |
clarkb | jhesketh: just pip install no -U | 01:16 |
*** bltavares has joined #openstack-infra | 01:16 | |
lifeless | its already installed | 01:16 |
clarkb | unless lifeless has other ideas | 01:16 |
lifeless | I'd try the swiftclient version check first | 01:17 |
clarkb | lifeless: that works | 01:17 |
clarkb | 2.6.0 is reported correctly | 01:17 |
lifeless | great | 01:17 |
lifeless | start up the wsgi thingamy | 01:17 |
jhesketh | http://paste.openstack.org/show/474770/ | 01:17 |
jhesketh | looks to be at the right version | 01:17 |
clarkb | the wsgi thing is installed at the wrong version though | 01:17 |
clarkb | and that paste lgtm | 01:17 |
lifeless | clarkb: wrong version, may be right content | 01:18 |
clarkb | and templates/ is present now | 01:18 |
lifeless | clarkb: my theory is setuptools pkg_resources was the original break | 01:18 |
clarkb | lifeless: that makes sense | 01:18 |
clarkb | old setuptools is just :( | 01:18 |
lifeless | so does it work now? | 01:18 |
clarkb | I think we need to restart apache now | 01:19 |
clarkb | jhesketh: ^ | 01:19 |
jhesketh | I reloaded apache | 01:19 |
jhesketh | so indexes work, but logs don't now :p | 01:19 |
jhesketh | http://logs.openstack.org/51/226751/14/check/nodepool-coverage/ | 01:19 |
lifeless | oh, you did do a new install | 01:19 |
lifeless | ok | 01:19 |
jhesketh | http://paste.openstack.org/show/474771/ | 01:19 |
lifeless | so - we should add setuptools to the things we take away from the os's control | 01:19 |
jhesketh | so looks like now we're hitting an os-loganalyze bug | 01:20 |
lifeless | I'm surprised it wasn't already there :) | 01:20 |
clarkb | jhesketh: ok should we remove the flag to gen indexes and fix later or is that insuffiient? | 01:20 |
*** bltavares has quit IRC | 01:21 | |
jhesketh | clarkb: the indexes are working... it's a bug in a previous os-loganlayze version | 01:21 |
jhesketh | since it hasn't been updated for a while we've missed it | 01:21 |
clarkb | oh fun | 01:21 |
*** armax has quit IRC | 01:21 | |
jhesketh | could be because of the new swiftclient version | 01:21 |
clarkb | ya looks like its complaining about the type returned by swiftclient | 01:22 |
clarkb | output is an _ObjectBody not a tring | 01:22 |
*** dramalho has joined #openstack-infra | 01:22 | |
jhesketh | why that's failing now though is the mystery | 01:22 |
*** sridhar_ram has quit IRC | 01:23 | |
jhesketh | clarkb: http://docs.openstack.org/developer/python-swiftclient/swiftclient.html#swiftclient.client.get_object | 01:24 |
clarkb | looks like it was added in 2.3 | 01:24 |
clarkb | which means swiftlcient likely needed to 3.0 instead :/ | 01:24 |
*** ociuhandu has joined #openstack-infra | 01:24 | |
clarkb | notmyname: ^ btw | 01:24 |
jhesketh | it should be returning the object contents | 01:24 |
jhesketh | clarkb: which was added in 2.3? | 01:24 |
clarkb | jhesketh: _ObjectBody | 01:25 |
clarkb | er it was added after 2.3 acoording to git describe --always so whatever release happeend after | 01:25 |
jhesketh | we've got 2.6 installed and this is the 2.6 documentation | 01:25 |
clarkb | man even updates the test to pass :/ | 01:25 |
*** dramalho has quit IRC | 01:26 | |
clarkb | it does still return a tuple but the last entry in the tuple is _ObjectBody() not a string type | 01:27 |
clarkb | so maybe quick solution is to downgrade swiftclient? | 01:27 |
jhesketh | possibly... do we know what was working before? | 01:27 |
jhesketh | I also suspect if we downgrade swiftclient we may need to downgrade keystone client | 01:27 |
clarkb | so ship has sailed but at the very least changes like this really really need to come with semver versions | 01:28 |
clarkb | (we can't hardly gripe about it in other projects if we don't do it ourselves) | 01:28 |
clarkb | jhesketh: 2.3.1 is the newest version without that change to the api | 01:29 |
lifeless | clarkb: if only we had backwards compat testing ... | 01:29 |
clarkb | lifeless: oh believe me I am a proponent for that tesitng | 01:29 |
clarkb | lifeless: I had a major sad when we reworked everything to rip it out | 01:29 |
clarkb | lifeless: but I am saying that me saying we shouldn't do this couldn't stand up to devs saying NO | 01:30 |
openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci: Fix tripleo-incubator build workaround https://review.openstack.org/229210 | 01:30 |
*** morgan is now known as notreallymorgan | 01:30 | |
*** shashank_hegde has quit IRC | 01:30 | |
*** notreallymorgan is now known as morgan | 01:30 | |
jhesketh | clarkb: any way of telling if we need to downgrade keystoneclient? | 01:31 |
clarkb | lifeless: there are changes we can revert to get the testing back, we *did* have it | 01:31 |
clarkb | jhesketh: uh let me look at the versions in requirements for swiftcleitn | 01:31 |
lifeless | clarkb: it'll need to be a little different due to constraints | 01:31 |
clarkb | jhesketh: it says >=0.7.0 | 01:31 |
lifeless | clarkb: also I'm proposing two variants - upgrade-just-x, and all-upgraded | 01:31 |
clarkb | jhesketh: so maybe :) | 01:31 |
lifeless | clarkb: but yes, its about the discussion and consensus. Helping me help other folk understand the pain would be super useful :) | 01:32 |
clarkb | jhesketh: but that version hasn't changed in latest swiftclient | 01:32 |
lifeless | clarkb: e.g. comemnts, suggested prose on the 'what is the problem' section of my spec | 01:32 |
jhesketh | clarkb: okay, so shall we downgrade swiftclient and play with that - see how we go? | 01:32 |
*** tiswanso has joined #openstack-infra | 01:32 | |
clarkb | jhesketh: yup I think thats a good next step | 01:32 |
clarkb | lifeless: ya would need edits likely | 01:32 |
*** fawadkhaliq has joined #openstack-infra | 01:33 | |
clarkb | lifeless: I did read the two variants in the spec and I am not sure we need both yet but need to devote some brainpower to it first | 01:33 |
jhesketh | clarkb: okay, would you like me to do sudo -H pip install "python-swiftclient==2.3.1" now? | 01:33 |
clarkb | jhesketh: yup | 01:34 |
lifeless | clarkb: thats detail we can address once we have buy-in to doing it;) | 01:34 |
jhesketh | clarkb: that seemed to get it http://logs.openstack.org/51/226751/14/check/nodepool-coverage/d06565b/console.html | 01:34 |
openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci: Fix tripleo-incubator build workaround https://review.openstack.org/229210 | 01:34 |
clarkb | lifeless: the general issue we ran into is that the devs don't wnat to bother with backward compat when under the gun for $newfeature | 01:34 |
clarkb | lifeless: so you end up with almost no one that actually cares about it and without people caring it can't happen | 01:34 |
jhesketh | clarkb: cool, indexes and pages now working | 01:34 |
*** unicell has quit IRC | 01:35 | |
jhesketh | clarkb: so we should pin down swiftclient in osloganalyze? | 01:35 |
clarkb | jhesketh: woot | 01:35 |
lifeless | clarkb: sure, but we have to do backwards compat to land anything anyway | 01:35 |
clarkb | jhesketh: if you add a pin to swiftclient in os loganalyze I can review that and lifeless' pbr hange | 01:35 |
clarkb | lifeless: not with the branch separation | 01:35 |
lifeless | clarkb: mine was a blanket update-requirements run | 01:35 |
clarkb | so maybe edit lifeless' change to fix swiftclient | 01:35 |
lifeless | yeah, doing that | 01:36 |
clarkb | thanks | 01:36 |
openstackgerrit | lifeless proposed openstack-infra/os-loganalyze: Update requirements https://review.openstack.org/229198 | 01:36 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/os-loganalyze: Pin python-swifclient https://review.openstack.org/229213 | 01:36 |
jhesketh | clarkb, lifeless: ^ | 01:36 |
clarkb | haha :) proably my fault, sorry | 01:36 |
jhesketh | ah, lifeless beat me | 01:36 |
jhesketh | I'll abandon | 01:36 |
*** yamamoto has joined #openstack-infra | 01:37 | |
*** Kennnan2 is now known as Kennan | 01:37 | |
clarkb | approved, Idid check the versions to make sure current set matches reality and seems good | 01:38 |
*** mudassirlatif has quit IRC | 01:38 | |
*** tiswanso_ has joined #openstack-infra | 01:39 | |
clarkb | lifeless: https://review.openstack.org/#/c/174371/ is the history behind removing stable compat jobs | 01:42 |
*** tiswanso has quit IRC | 01:42 | |
openstackgerrit | Merged openstack-infra/os-loganalyze: Update requirements https://review.openstack.org/229198 | 01:42 |
clarkb | I am all for adding that back in again | 01:42 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/puppet-openstackci: Handle all requests through os-loganalyze https://review.openstack.org/229216 | 01:43 |
jhesketh | clarkb: ^ so there's the next big test | 01:43 |
clarkb | the way those jobs were setup was install current change for $lib and install everything else on latest version of $brnach | 01:43 |
clarkb | that is a little more complicated now that we have constraints | 01:43 |
clarkb | simplest way to handle that may be to shove the client under test in a venv and everything else global? | 01:43 |
*** ociuhandu has quit IRC | 01:44 | |
lifeless | clarkb: thank you | 01:44 |
lifeless | so the key thing is the interaction with de-aggregated servers | 01:45 |
clarkb | lifeless: we can also obviously be smarter about what jobs give coverage for specific libs too, the old setup appeared to just use neutron devstack | 01:45 |
lifeless | to me | 01:45 |
clarkb | lifeless: ah so in venv doesn't help much | 01:45 |
clarkb | good point | 01:45 |
lifeless | clarkb: so the etherpad has a larger matrix | 01:46 |
lifeless | clarkb: where we looked at 'clients in servers' and 'clients as CLI' as separate points | 01:46 |
*** ociuhandu has joined #openstack-infra | 01:46 | |
lifeless | clarkb: in the spec I've collapsed that down to be pragmatic (and make it feasible to do :)) | 01:47 |
*** amotoki has joined #openstack-infra | 01:48 | |
*** yamamoto has quit IRC | 01:49 | |
*** mrmartin has joined #openstack-infra | 01:49 | |
openstackgerrit | fumihiko kakuma proposed openstack-infra/project-config: Add job for the ovs mechanism driver with native mode https://review.openstack.org/221143 | 01:50 |
clarkb | lifeless: ^ clearly we need to test the entire matrix >_> | 01:51 |
*** angdraug has quit IRC | 01:51 | |
clarkb | thats ovs drivers x dvr/nodvr | 01:52 |
clarkb | seems odd that we need >1 ovs driver though | 01:52 |
lifeless | clarkb: I think it would be ok to start with one test, and then on anything threading through add another job for coverage | 01:52 |
clarkb | though I guess there is opendaylight and ovn and akanda et al | 01:52 |
lifeless | clarkb: diminishing returns etc tc | 01:52 |
clarkb | lifeless: ya | 01:53 |
clarkb | lifeless: I am just contrasting your appraoch with neutrons | 01:53 |
clarkb | neutrons is clearly test all the things! | 01:53 |
lifeless | oh ah | 01:53 |
*** amotoki has quit IRC | 01:53 | |
lifeless | sure | 01:53 |
lifeless | is there anyway to unsubscribe from a single change? | 01:53 |
lifeless | https://review.openstack.org/88736 is making me sad | 01:54 |
clarkb | lifeless: you may have an X next to your name in the vote graph | 01:54 |
clarkb | if you click that should remove you (and if thats admin only I guess I can do it) | 01:54 |
clarkb | though if you already have a project wide subscribe that wont help | 01:54 |
lifeless | I can see it | 01:54 |
lifeless | thanks | 01:54 |
clarkb | a better soltion there may be to remoe swift from syncing | 01:54 |
lifeless | well | 01:55 |
clarkb | we are going to test them with the sync anyways | 01:55 |
clarkb | if they don't want to merge then me | 01:55 |
clarkb | *meh | 01:55 |
*** kengo_sakai has joined #openstack-infra | 01:55 | |
*** sdake has joined #openstack-infra | 01:58 | |
*** mrmartin has quit IRC | 02:01 | |
*** stevemar has quit IRC | 02:02 | |
*** shashank_hegde has joined #openstack-infra | 02:05 | |
*** rfolco has quit IRC | 02:05 | |
*** rfolco has joined #openstack-infra | 02:06 | |
*** zhenguo has joined #openstack-infra | 02:06 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/os-testr: Better blacklist - tested with Nova https://review.openstack.org/229223 | 02:07 |
*** pahuang has quit IRC | 02:07 | |
*** pharic has joined #openstack-infra | 02:08 | |
* pharic slaps pharic around a bit with a large fishbot | 02:11 | |
*** pharic has quit IRC | 02:11 | |
*** ddieterly has quit IRC | 02:12 | |
*** stevemar has joined #openstack-infra | 02:12 | |
*** odgrim1 has quit IRC | 02:13 | |
*** larainema has quit IRC | 02:15 | |
*** stevemar has quit IRC | 02:17 | |
*** changbl has joined #openstack-infra | 02:18 | |
*** rguillebert has quit IRC | 02:18 | |
*** pahuang has joined #openstack-infra | 02:19 | |
*** changbl has quit IRC | 02:23 | |
*** changbl has joined #openstack-infra | 02:25 | |
*** sridhar_ram has joined #openstack-infra | 02:25 | |
*** asettle has joined #openstack-infra | 02:27 | |
*** kengo_sa_ has joined #openstack-infra | 02:27 | |
*** larainema has joined #openstack-infra | 02:28 | |
*** kengo_sakai has quit IRC | 02:29 | |
*** yamamoto has joined #openstack-infra | 02:35 | |
*** yamamoto has quit IRC | 02:35 | |
*** yamamoto has joined #openstack-infra | 02:36 | |
*** stevemar has joined #openstack-infra | 02:36 | |
*** changbl has quit IRC | 02:42 | |
*** achanda has joined #openstack-infra | 02:43 | |
*** armax has joined #openstack-infra | 02:44 | |
*** spzala has quit IRC | 02:47 | |
*** lawrancejing has quit IRC | 02:47 | |
*** lawrancejing has joined #openstack-infra | 02:48 | |
*** pcrews_ has quit IRC | 02:48 | |
*** stevemar has quit IRC | 02:49 | |
*** stevemar has joined #openstack-infra | 02:50 | |
*** sabeen has joined #openstack-infra | 02:51 | |
*** gyee has quit IRC | 02:52 | |
*** sridhar_ram has quit IRC | 02:52 | |
*** pcrews_ has joined #openstack-infra | 02:53 | |
*** stevemar has quit IRC | 02:54 | |
*** unicell has joined #openstack-infra | 02:56 | |
*** dims_ has quit IRC | 02:57 | |
*** salv-orlando has joined #openstack-infra | 03:00 | |
*** amotoki has joined #openstack-infra | 03:02 | |
*** tlian has quit IRC | 03:03 | |
*** amotoki has quit IRC | 03:07 | |
*** mrmartin has joined #openstack-infra | 03:12 | |
*** otter768 has quit IRC | 03:12 | |
*** ddieterly has joined #openstack-infra | 03:13 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack-infra/project-config: midonet: Make unit tests non-voting https://review.openstack.org/220431 | 03:13 |
*** rfolco has quit IRC | 03:14 | |
*** tiswanso_ has quit IRC | 03:15 | |
*** dahoo has quit IRC | 03:16 | |
*** puranamr has joined #openstack-infra | 03:17 | |
*** rfolco has joined #openstack-infra | 03:17 | |
*** ddieterly has quit IRC | 03:18 | |
*** claudiub has quit IRC | 03:18 | |
*** puranamr has quit IRC | 03:19 | |
*** puranamr has joined #openstack-infra | 03:20 | |
*** vkmc has quit IRC | 03:20 | |
*** rfolco_ has joined #openstack-infra | 03:21 | |
*** rfolco has quit IRC | 03:22 | |
*** stevemar has joined #openstack-infra | 03:22 | |
*** puranamr has quit IRC | 03:23 | |
*** sam_wan has joined #openstack-infra | 03:24 | |
*** puranamr has joined #openstack-infra | 03:24 | |
*** vkmc has joined #openstack-infra | 03:24 | |
*** stevemar_ has joined #openstack-infra | 03:24 | |
*** salv-orlando has quit IRC | 03:25 | |
*** stevemar has quit IRC | 03:27 | |
*** puranamr has quit IRC | 03:28 | |
*** stevemar_ has quit IRC | 03:28 | |
*** changbl has joined #openstack-infra | 03:31 | |
*** stevemar has joined #openstack-infra | 03:31 | |
*** pvaneck has joined #openstack-infra | 03:32 | |
*** pvaneck has quit IRC | 03:33 | |
*** dingyichen has quit IRC | 03:33 | |
*** pahuang has quit IRC | 03:35 | |
*** Sukhdev has joined #openstack-infra | 03:35 | |
*** puranamr has joined #openstack-infra | 03:35 | |
*** hdd has joined #openstack-infra | 03:38 | |
*** sdake has quit IRC | 03:38 | |
*** changbl has quit IRC | 03:39 | |
*** sdake has joined #openstack-infra | 03:41 | |
*** amotoki has joined #openstack-infra | 03:42 | |
ruagair | Is there any doco floating about on how to link to Storyboard in git-review / gerrit? | 03:43 |
*** Daisy has quit IRC | 03:43 | |
*** Daisy has joined #openstack-infra | 03:44 | |
*** sdake has quit IRC | 03:46 | |
*** mrmartin has quit IRC | 03:47 | |
*** Daisy has quit IRC | 03:48 | |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul: Update to statsd 3.0 https://review.openstack.org/229232 | 03:49 |
*** sdake has joined #openstack-infra | 03:49 | |
ianw | ruagair: you mean other than using "Story:" in the cl ? or are you talking about something else? | 03:51 |
*** pahuang has joined #openstack-infra | 03:52 | |
*** puranamr has quit IRC | 03:53 | |
*** links has joined #openstack-infra | 03:53 | |
*** baoli has quit IRC | 03:54 | |
*** sridhar_ram has joined #openstack-infra | 03:57 | |
*** dims has joined #openstack-infra | 03:57 | |
openstackgerrit | Tony Breeds proposed openstack-infra/irc-meetings: Update meeting info for 3rd party ci working group https://review.openstack.org/229178 | 03:58 |
*** yamahata has joined #openstack-infra | 03:59 | |
*** amotoki has quit IRC | 04:01 | |
*** dims has quit IRC | 04:03 | |
openstackgerrit | Sridhar Ramaswamy proposed openstack-infra/project-config: Add py27 and non-experimental tacker gate job https://review.openstack.org/228649 | 04:03 |
*** sridhar_ram1 has joined #openstack-infra | 04:04 | |
*** sridhar_ram has quit IRC | 04:04 | |
*** jamespage has quit IRC | 04:05 | |
*** gnuoy has quit IRC | 04:05 | |
*** gnuoy has joined #openstack-infra | 04:06 | |
*** jamespage has joined #openstack-infra | 04:06 | |
openstackgerrit | Merged openstack-infra/irc-meetings: Update meeting info for 3rd party ci working group https://review.openstack.org/229178 | 04:07 |
*** odgrim has joined #openstack-infra | 04:07 | |
*** gildub has quit IRC | 04:08 | |
*** amotoki has joined #openstack-infra | 04:11 | |
*** kiran-r has joined #openstack-infra | 04:11 | |
*** fawadkhaliq has quit IRC | 04:12 | |
*** sridhar_ram1 has quit IRC | 04:12 | |
*** ddieterly has joined #openstack-infra | 04:14 | |
*** yamahata has quit IRC | 04:15 | |
*** yamahata has joined #openstack-infra | 04:16 | |
*** kushal has joined #openstack-infra | 04:17 | |
*** oomichi has quit IRC | 04:19 | |
openstackgerrit | Skyler Berg proposed openstack-infra/project-config: Add ciwatch - CI monitoring tool https://review.openstack.org/216840 | 04:19 |
*** ddieterly has quit IRC | 04:19 | |
*** amotoki has quit IRC | 04:20 | |
*** ociuhandu has quit IRC | 04:22 | |
*** kiran-r has quit IRC | 04:22 | |
*** ender has joined #openstack-infra | 04:25 | |
*** gildub has joined #openstack-infra | 04:27 | |
*** odgrim has quit IRC | 04:27 | |
*** esmute has joined #openstack-infra | 04:27 | |
*** sdake has quit IRC | 04:28 | |
*** amotoki has joined #openstack-infra | 04:30 | |
*** ender has left #openstack-infra | 04:30 | |
*** fawadkhaliq has joined #openstack-infra | 04:34 | |
*** fawadkhaliq has quit IRC | 04:35 | |
*** fawadkhaliq has joined #openstack-infra | 04:35 | |
*** ayoung has quit IRC | 04:36 | |
*** links has quit IRC | 04:36 | |
*** finch has quit IRC | 04:36 | |
*** Sree has joined #openstack-infra | 04:37 | |
*** finch has joined #openstack-infra | 04:37 | |
*** Sree has quit IRC | 04:37 | |
*** grantbow has quit IRC | 04:37 | |
*** kambiz has quit IRC | 04:37 | |
*** Sree has joined #openstack-infra | 04:37 | |
*** grantbow has joined #openstack-infra | 04:39 | |
*** fawadkhaliq has quit IRC | 04:41 | |
*** kambiz has joined #openstack-infra | 04:41 | |
*** pahuang has quit IRC | 04:43 | |
*** dingyichen has joined #openstack-infra | 04:45 | |
*** mudassirlatif has joined #openstack-infra | 04:45 | |
*** amotoki has quit IRC | 04:48 | |
*** salv-orlando has joined #openstack-infra | 04:55 | |
*** pahuang has joined #openstack-infra | 04:56 | |
*** Sree_ has joined #openstack-infra | 05:00 | |
openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: Use xml instead of job description for is_managed https://review.openstack.org/225562 | 05:01 |
*** Sree_ is now known as Guest61017 | 05:01 | |
openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: Use xml instead of job description for is_managed https://review.openstack.org/225562 | 05:02 |
*** earlephilhower has quit IRC | 05:02 | |
*** sridhar_ram has joined #openstack-infra | 05:03 | |
*** Sree has quit IRC | 05:04 | |
*** BharatK has joined #openstack-infra | 05:05 | |
*** mgrosz has joined #openstack-infra | 05:06 | |
*** thiagop has quit IRC | 05:08 | |
*** kfox1111 has quit IRC | 05:09 | |
*** salv-orlando has quit IRC | 05:09 | |
*** hdd has quit IRC | 05:10 | |
*** amotoki has joined #openstack-infra | 05:11 | |
*** links has joined #openstack-infra | 05:12 | |
openstackgerrit | Jens Rosenboom proposed openstack-infra/project-config: IPv6 experimental job - set SERVICE_HOST to ::1 https://review.openstack.org/229108 | 05:13 |
*** otter768 has joined #openstack-infra | 05:13 | |
*** ddieterly has joined #openstack-infra | 05:15 | |
*** mgrosz has quit IRC | 05:16 | |
*** mgrosz has joined #openstack-infra | 05:16 | |
ruagair | Using Story: is probably allI need. Thanks ianw | 05:17 |
*** pcrews_ has quit IRC | 05:17 | |
*** mgrosz has quit IRC | 05:17 | |
*** enikanorov has joined #openstack-infra | 05:18 | |
*** otter768 has quit IRC | 05:18 | |
*** bltavares has joined #openstack-infra | 05:18 | |
*** asmigala has joined #openstack-infra | 05:19 | |
*** ddieterly has quit IRC | 05:20 | |
*** enikanorov_ has quit IRC | 05:21 | |
*** pcrews_ has joined #openstack-infra | 05:22 | |
*** bltavares has quit IRC | 05:23 | |
*** olaph has quit IRC | 05:26 | |
openstackgerrit | Sachi King proposed openstack-infra/project-config: Add pep8-constraints jobs https://review.openstack.org/228344 | 05:26 |
openstackgerrit | Sachi King proposed openstack-infra/project-config: Add cover-constraints jobs https://review.openstack.org/228345 | 05:26 |
openstackgerrit | Sachi King proposed openstack-infra/project-config: Add tarball-constraints jobs https://review.openstack.org/228346 | 05:26 |
openstackgerrit | Sachi King proposed openstack-infra/project-config: Add docs-constraints jobs https://review.openstack.org/228343 | 05:26 |
openstackgerrit | Sachi King proposed openstack-infra/project-config: Don't run non-voting jobs on gate https://review.openstack.org/229242 | 05:26 |
ruagair | Go Nakato :-) | 05:27 |
*** BharatK has quit IRC | 05:28 | |
*** Sukhdev has quit IRC | 05:29 | |
*** alexpilotti has quit IRC | 05:33 | |
*** mgrosz has joined #openstack-infra | 05:34 | |
*** dingyichen has quit IRC | 05:35 | |
*** dingyichen has joined #openstack-infra | 05:36 | |
*** mudassirlatif has quit IRC | 05:38 | |
*** Guest61017 has quit IRC | 05:39 | |
*** mgrosz has quit IRC | 05:39 | |
*** BharatK has joined #openstack-infra | 05:40 | |
*** dingyichen has quit IRC | 05:40 | |
*** hrou has quit IRC | 05:40 | |
*** dingyichen has joined #openstack-infra | 05:41 | |
*** olaph has joined #openstack-infra | 05:41 | |
openstackgerrit | Skyler Berg proposed openstack-infra/project-config: Add ciwatch - CI monitoring tool https://review.openstack.org/216840 | 05:44 |
*** mudassirlatif has joined #openstack-infra | 05:45 | |
*** dingyichen has quit IRC | 05:45 | |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul: Update to statsd 3.0 https://review.openstack.org/229232 | 05:45 |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul: Use 127.0.0.1 rather than localhost for statsd host https://review.openstack.org/229247 | 05:45 |
*** dingyichen has joined #openstack-infra | 05:45 | |
*** esmute has quit IRC | 05:46 | |
*** dra_ has joined #openstack-infra | 05:47 | |
*** Sree has joined #openstack-infra | 05:48 | |
*** denisra has quit IRC | 05:49 | |
*** dingyichen has quit IRC | 05:50 | |
*** dingyichen has joined #openstack-infra | 05:50 | |
*** deepakcs has joined #openstack-infra | 05:51 | |
*** Sree has quit IRC | 05:52 | |
*** Sree has joined #openstack-infra | 05:53 | |
*** thingee has quit IRC | 05:56 | |
*** garyk1 has joined #openstack-infra | 05:58 | |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Prevent overwriting of user modified blacklist.conf https://review.openstack.org/220883 | 05:58 |
*** garyk has quit IRC | 05:59 | |
*** dingyichen has quit IRC | 06:01 | |
*** sam_wan has quit IRC | 06:02 | |
*** stevemar has quit IRC | 06:04 | |
*** stevemar has joined #openstack-infra | 06:04 | |
*** sridhar_ram has quit IRC | 06:05 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/226816 | 06:08 |
*** stevemar has quit IRC | 06:08 | |
*** dingyichen has joined #openstack-infra | 06:11 | |
*** mudassirlatif has quit IRC | 06:12 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Translate Ironic for Liberty as well https://review.openstack.org/229252 | 06:14 |
*** sabeen has quit IRC | 06:14 | |
*** ddieterly has joined #openstack-infra | 06:16 | |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Reorder the script number of 'elements/dkms/post-install.d/99-dkms' https://review.openstack.org/220885 | 06:16 |
*** sileht has quit IRC | 06:17 | |
yolanda | morning | 06:19 |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Add a new element hpdsa https://review.openstack.org/220888 | 06:20 |
*** ddieterly has quit IRC | 06:21 | |
ruagair | o/ | 06:24 |
*** ParsectiX has joined #openstack-infra | 06:25 | |
*** sdake has joined #openstack-infra | 06:25 | |
*** nadya has joined #openstack-infra | 06:28 | |
*** scheuran has joined #openstack-infra | 06:28 | |
*** sileht has joined #openstack-infra | 06:28 | |
*** dingyichen has quit IRC | 06:28 | |
*** dingyichen has joined #openstack-infra | 06:30 | |
*** sdake has quit IRC | 06:32 | |
*** sergmelikyan has joined #openstack-infra | 06:32 | |
*** dingyichen has quit IRC | 06:33 | |
*** jhesketh has quit IRC | 06:34 | |
openstackgerrit | Maty Grosz proposed openstack-infra/project-config: Changes: Vitrage - Initialize new OpenStack project https://review.openstack.org/229259 | 06:34 |
*** odgrim has joined #openstack-infra | 06:35 | |
*** jhesketh has joined #openstack-infra | 06:35 | |
*** dingyichen has joined #openstack-infra | 06:35 | |
*** asettle has quit IRC | 06:39 | |
*** salv-orlando has joined #openstack-infra | 06:41 | |
*** matbu has quit IRC | 06:47 | |
*** matbu has joined #openstack-infra | 06:48 | |
*** salv-orlando has quit IRC | 06:48 | |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Set cinder api to 1 in rally-gate-dsvm-cli job https://review.openstack.org/229264 | 06:48 |
*** matbu_ has joined #openstack-infra | 06:51 | |
*** flepied has joined #openstack-infra | 06:53 | |
*** matbu has quit IRC | 06:53 | |
*** matrohon has joined #openstack-infra | 06:53 | |
*** matbu has joined #openstack-infra | 06:53 | |
*** ParsectiX has quit IRC | 06:54 | |
*** ParsectiX has joined #openstack-infra | 06:54 | |
*** smcginnis has quit IRC | 06:56 | |
*** matbu_ has quit IRC | 06:57 | |
*** armax has quit IRC | 06:57 | |
*** k4n0 has joined #openstack-infra | 06:58 | |
*** peristeri has joined #openstack-infra | 07:00 | |
*** Sree has quit IRC | 07:01 | |
*** afazekas_ has joined #openstack-infra | 07:01 | |
openstackgerrit | Merged openstack-infra/project-config: Also look for stackforge puppet modules when copying logs https://review.openstack.org/229136 | 07:02 |
*** markus_z has joined #openstack-infra | 07:02 | |
*** kevinbenton has quit IRC | 07:03 | |
*** jlvillal has quit IRC | 07:05 | |
*** Daisy has joined #openstack-infra | 07:08 | |
Daisy | StevenK: there ? | 07:09 |
*** odgrim has quit IRC | 07:13 | |
*** lawrancejing has quit IRC | 07:13 | |
*** otter768 has joined #openstack-infra | 07:14 | |
*** lawrancejing has joined #openstack-infra | 07:15 | |
*** dahoo has joined #openstack-infra | 07:15 | |
*** ddieterly has joined #openstack-infra | 07:16 | |
*** otter768 has quit IRC | 07:19 | |
*** sergmelikyan has quit IRC | 07:19 | |
*** arxcruz has joined #openstack-infra | 07:19 | |
*** kevinbenton has joined #openstack-infra | 07:21 | |
*** ddieterly has quit IRC | 07:22 | |
*** Kennan has quit IRC | 07:23 | |
*** hashar has joined #openstack-infra | 07:23 | |
*** alexpilotti has joined #openstack-infra | 07:24 | |
*** sdake has joined #openstack-infra | 07:25 | |
*** Kennan has joined #openstack-infra | 07:25 | |
*** scheuran has quit IRC | 07:25 | |
*** ifarkas has joined #openstack-infra | 07:26 | |
*** hashar_ has joined #openstack-infra | 07:27 | |
*** scheuran has joined #openstack-infra | 07:27 | |
*** shardy has joined #openstack-infra | 07:29 | |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Prevent overwriting of user modified blacklist.conf https://review.openstack.org/220883 | 07:30 |
*** hashar has quit IRC | 07:30 | |
*** asmigala1 has joined #openstack-infra | 07:30 | |
*** d^_^b has quit IRC | 07:31 | |
*** asmigala has quit IRC | 07:31 | |
*** shashank_hegde has quit IRC | 07:32 | |
*** sergmelikyan has joined #openstack-infra | 07:33 | |
StevenK | Daisy: Hi | 07:33 |
StevenK | Daisy: Right, horizon is special, let me generate one and reply back | 07:34 |
*** oomichi has joined #openstack-infra | 07:34 | |
Daisy | Thank you, StevenK. | 07:35 |
openstackgerrit | Merged openstack-infra/system-config: Force gzip to run on log archives https://review.openstack.org/218462 | 07:37 |
*** yamamoto has quit IRC | 07:37 | |
*** yamamoto has joined #openstack-infra | 07:38 | |
openstackgerrit | Qiming Teng proposed openstack-infra/project-config: Adjust SDK jobs to use ceilometer plugin https://review.openstack.org/229273 | 07:38 |
*** afazekas_ has quit IRC | 07:40 | |
*** Sree has joined #openstack-infra | 07:41 | |
StevenK | Daisy: I've replied. | 07:42 |
Daisy | StevenK: let me take a look. | 07:42 |
*** jlanoux has joined #openstack-infra | 07:43 | |
*** yamamoto has quit IRC | 07:43 | |
*** rossella_s has quit IRC | 07:47 | |
*** aviau has quit IRC | 07:47 | |
Daisy | StevenK: I didn't get your email. | 07:47 |
*** rossella_s has joined #openstack-infra | 07:47 | |
*** achanda has quit IRC | 07:48 | |
*** gildub has quit IRC | 07:48 | |
*** notnownikki has joined #openstack-infra | 07:50 | |
*** sergmelikyan has quit IRC | 07:51 | |
StevenK | Daisy: The IBM mailserver accepted it eight minutes ago. But, I've pasted it for you in http://paste.openstack.org/show/474786/ | 07:51 |
openstackgerrit | Sam Wan proposed openstack-infra/devstack-gate: ovs_gre_bridge check address existence before 'ip addr add' https://review.openstack.org/229277 | 07:53 |
Daisy | StevenK: It works for me. Thank you, StevenK. | 07:56 |
*** sergmelikyan has joined #openstack-infra | 07:57 | |
*** alexpilotti has quit IRC | 07:58 | |
*** ihrachys has joined #openstack-infra | 07:58 | |
*** HeOS has quit IRC | 07:58 | |
*** yamamoto has joined #openstack-infra | 08:00 | |
*** Sree_ has joined #openstack-infra | 08:00 | |
*** Sree_ is now known as Guest72568 | 08:00 | |
*** ParsectiX has quit IRC | 08:00 | |
*** aviau has joined #openstack-infra | 08:01 | |
*** fhubik has joined #openstack-infra | 08:02 | |
*** jistr has joined #openstack-infra | 08:02 | |
*** Nakato has quit IRC | 08:03 | |
*** Nakato has joined #openstack-infra | 08:03 | |
*** Sree has quit IRC | 08:03 | |
*** e0ne has joined #openstack-infra | 08:03 | |
*** Hal has joined #openstack-infra | 08:04 | |
*** ParsectiX has joined #openstack-infra | 08:05 | |
*** salv-orlando has joined #openstack-infra | 08:05 | |
*** salv-orlando has quit IRC | 08:06 | |
*** yamamoto has quit IRC | 08:07 | |
*** salv-orlando has joined #openstack-infra | 08:07 | |
*** Daisy has quit IRC | 08:09 | |
*** Daisy has joined #openstack-infra | 08:09 | |
*** ildikov has quit IRC | 08:10 | |
*** yamamoto has joined #openstack-infra | 08:10 | |
*** Daisy has quit IRC | 08:11 | |
*** Daisy has joined #openstack-infra | 08:11 | |
*** claudiub has joined #openstack-infra | 08:12 | |
*** alexpilotti has joined #openstack-infra | 08:13 | |
*** devvesa has joined #openstack-infra | 08:14 | |
*** Ala has joined #openstack-infra | 08:14 | |
*** Daisy has quit IRC | 08:15 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Nothing to see here https://review.openstack.org/111011 | 08:15 |
*** derekh has joined #openstack-infra | 08:16 | |
*** stevemar has joined #openstack-infra | 08:16 | |
*** ddieterly has joined #openstack-infra | 08:17 | |
*** Guest72568 has quit IRC | 08:18 | |
*** Daisy has joined #openstack-infra | 08:18 | |
*** Sree has joined #openstack-infra | 08:19 | |
*** markvoelker has quit IRC | 08:20 | |
*** stevemar has quit IRC | 08:21 | |
openstackgerrit | Merged openstack-infra/tripleo-ci: Set the delorean baseurl to /centos7 https://review.openstack.org/228184 | 08:21 |
*** ddieterly has quit IRC | 08:22 | |
*** dahoo has quit IRC | 08:22 | |
*** yamamoto has quit IRC | 08:23 | |
*** yamamoto has joined #openstack-infra | 08:23 | |
*** asmigala1 has quit IRC | 08:29 | |
*** Sree has quit IRC | 08:30 | |
*** rlucio_ has joined #openstack-infra | 08:30 | |
*** Sree has joined #openstack-infra | 08:31 | |
*** yamamoto has quit IRC | 08:32 | |
*** bharathm has joined #openstack-infra | 08:33 | |
*** rlucio has quit IRC | 08:34 | |
*** yamamoto has joined #openstack-infra | 08:35 | |
*** asmigala has joined #openstack-infra | 08:42 | |
*** dingyichen has quit IRC | 08:44 | |
*** mpaolino has joined #openstack-infra | 08:46 | |
*** amotoki_ has joined #openstack-infra | 08:49 | |
*** shardy_ has joined #openstack-infra | 08:49 | |
*** shardy has quit IRC | 08:51 | |
*** jordanP has joined #openstack-infra | 08:51 | |
*** tcammann_ has quit IRC | 08:54 | |
*** shardy_ has quit IRC | 08:55 | |
*** shardy has joined #openstack-infra | 08:55 | |
*** ParsectiX has quit IRC | 08:56 | |
*** cdent has joined #openstack-infra | 08:57 | |
*** e0ne has quit IRC | 09:00 | |
*** e0ne has joined #openstack-infra | 09:00 | |
*** rguillebert has joined #openstack-infra | 09:00 | |
*** nijaba has quit IRC | 09:01 | |
*** BharatK has quit IRC | 09:01 | |
*** ParsectiX has joined #openstack-infra | 09:02 | |
*** claudiub has quit IRC | 09:02 | |
*** alexpilotti has quit IRC | 09:04 | |
openstackgerrit | Maty Grosz proposed openstack-infra/project-config: Changes: Vitrage - Initialize new OpenStack project https://review.openstack.org/229259 | 09:05 |
openstackgerrit | Dominic Cleal proposed openstack-infra/jenkins-job-builder: Support schedule-failed-builds plugin https://review.openstack.org/229309 | 09:05 |
*** amotoki_ has quit IRC | 09:07 | |
*** nijaba has joined #openstack-infra | 09:08 | |
*** amotoki has quit IRC | 09:08 | |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Prevent overwriting of user modified blacklist.conf https://review.openstack.org/220883 | 09:08 |
*** hashar has joined #openstack-infra | 09:08 | |
*** hashar_ has quit IRC | 09:09 | |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Reorder the script number of 'elements/dkms/post-install.d/99-dkms' https://review.openstack.org/220885 | 09:10 |
*** dtantsur|afk is now known as dtantsur | 09:10 | |
*** claudiub has joined #openstack-infra | 09:10 | |
*** amotoki has joined #openstack-infra | 09:13 | |
*** otter768 has joined #openstack-infra | 09:15 | |
*** BharatK has joined #openstack-infra | 09:15 | |
*** pelix has joined #openstack-infra | 09:16 | |
*** rlandy has joined #openstack-infra | 09:17 | |
*** amotoki has quit IRC | 09:18 | |
*** ddieterly has joined #openstack-infra | 09:18 | |
*** otter768 has quit IRC | 09:19 | |
openstackgerrit | Dominic Cleal proposed openstack-infra/jenkins-job-builder: Support schedule-failed-builds plugin https://review.openstack.org/229309 | 09:20 |
*** markvoelker has joined #openstack-infra | 09:21 | |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add openstack-ansible-security repository https://review.openstack.org/229318 | 09:22 |
*** ddieterly has quit IRC | 09:23 | |
*** HeOS has joined #openstack-infra | 09:24 | |
*** abregman has joined #openstack-infra | 09:26 | |
*** markvoelker has quit IRC | 09:26 | |
*** abregman|pto has quit IRC | 09:27 | |
*** garyk1 has quit IRC | 09:27 | |
*** dims has joined #openstack-infra | 09:32 | |
openstackgerrit | Dominic Cleal proposed openstack-infra/jenkins-job-builder: Support schedule-failed-builds plugin https://review.openstack.org/229309 | 09:34 |
*** dims has quit IRC | 09:39 | |
openstackgerrit | Dmitry Tantsur proposed openstack/diskimage-builder: agent: ensure vmlinuz file does not exist before hard-linking into it https://review.openstack.org/229326 | 09:39 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Move nv openstack-ansible experimental job to check queue https://review.openstack.org/228408 | 09:40 |
*** abregman_ has joined #openstack-infra | 09:40 | |
*** dims has joined #openstack-infra | 09:42 | |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Add a new element hpdsa https://review.openstack.org/220888 | 09:42 |
*** abregman has quit IRC | 09:42 | |
*** amotoki has joined #openstack-infra | 09:46 | |
openstackgerrit | Maty Grosz proposed openstack-infra/project-config: Changes: Vitrage - Initialize new OpenStack project https://review.openstack.org/229259 | 09:47 |
*** enikanorov_ has joined #openstack-infra | 09:48 | |
*** matrohon has quit IRC | 09:48 | |
*** hashar is now known as hasharReviews | 09:48 | |
openstackgerrit | Merged openstack-infra/tripleo-ci: Add setup/tox/requirements files https://review.openstack.org/228474 | 09:50 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add openstack-ansible-security repository https://review.openstack.org/229318 | 09:50 |
*** sergmelikyan has quit IRC | 09:51 | |
*** marzif has joined #openstack-infra | 09:51 | |
*** enikanorov has quit IRC | 09:52 | |
*** lawrancejing has quit IRC | 09:54 | |
e0ne | hi. can anybody help me with adding new job to Cinder, please? I have no mind why tests for https://review.openstack.org/#/c/228857/ failed | 09:55 |
*** yamahata has quit IRC | 09:55 | |
openstackgerrit | Merged openstack-infra/tripleo-ci: Update CI status report to generate HTML table. https://review.openstack.org/228475 | 09:55 |
openstackgerrit | Dmitry Tantsur proposed openstack/diskimage-builder: Fix tests/test_functions.bash https://review.openstack.org/229334 | 09:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/diskimage-builder: Add a tox target to run functional tests locally https://review.openstack.org/229335 | 09:56 |
*** tlian has joined #openstack-infra | 09:57 | |
*** amotoki has quit IRC | 09:58 | |
*** fhubik is now known as fhubik_brb | 10:01 | |
*** pc_m has joined #openstack-infra | 10:01 | |
*** marzif has quit IRC | 10:03 | |
openstackgerrit | Merged openstack-infra/jenkins-job-builder: Allow JJB to set connection timeout to jenkins server https://review.openstack.org/151472 | 10:06 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add pep8 and bashate non-voting checks to openstack-ansible https://review.openstack.org/228406 | 10:06 |
*** rossella_s has quit IRC | 10:07 | |
*** ParsectiX has quit IRC | 10:10 | |
*** ZZelle has quit IRC | 10:12 | |
*** ZZelle has joined #openstack-infra | 10:13 | |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add pep8 and bashate non-voting checks to openstack-ansible https://review.openstack.org/228406 | 10:15 |
*** jasondotstar_afk is now known as jasondotstar | 10:19 | |
*** ddieterly has joined #openstack-infra | 10:19 | |
*** amotoki has joined #openstack-infra | 10:20 | |
*** ildikov has joined #openstack-infra | 10:20 | |
*** ddieterly has quit IRC | 10:23 | |
*** sergmelikyan has joined #openstack-infra | 10:23 | |
*** yamamoto has quit IRC | 10:24 | |
*** pahuang has quit IRC | 10:24 | |
*** abregman_ has quit IRC | 10:25 | |
*** pahuang has joined #openstack-infra | 10:27 | |
*** BharatK has quit IRC | 10:27 | |
*** kushal has quit IRC | 10:28 | |
*** ildikov has quit IRC | 10:33 | |
*** e0ne has quit IRC | 10:36 | |
*** mkj_wk has quit IRC | 10:37 | |
*** mkj_wk has joined #openstack-infra | 10:38 | |
*** shardy_ has joined #openstack-infra | 10:39 | |
*** Sree has quit IRC | 10:40 | |
*** BharatK has joined #openstack-infra | 10:40 | |
*** Sree has joined #openstack-infra | 10:40 | |
*** shardy has quit IRC | 10:41 | |
*** mgrosz has joined #openstack-infra | 10:42 | |
mgrosz | Hello, can someone review change https://review.openstack.org/#/c/229259/3 | 10:43 |
mgrosz | ? | 10:43 |
wznoinsk | hi infra | 10:43 |
*** trown is now known as trown|relocating | 10:44 | |
wznoinsk | whenever someone (fungi? pleia2? others?) has a moment, could you share what you use (software and methods) to create mirror(s) of openstack git repos? | 10:44 |
*** shardy_ has quit IRC | 10:45 | |
*** Sree has quit IRC | 10:45 | |
*** shardy has joined #openstack-infra | 10:45 | |
*** Hal has quit IRC | 10:46 | |
*** mgrosz has quit IRC | 10:48 | |
*** ildikov has joined #openstack-infra | 10:49 | |
openstackgerrit | Vitaly Gridnev proposed openstack-infra/project-config: Add non-voting job to test Cinder with Block Device Driver backend https://review.openstack.org/228857 | 10:49 |
openstackgerrit | Dmitry Tantsur proposed openstack/diskimage-builder: Add a tox target to run functional tests locally https://review.openstack.org/229335 | 10:52 |
*** Daisy has quit IRC | 10:56 | |
*** Daisy has joined #openstack-infra | 10:57 | |
*** jordanP has quit IRC | 10:59 | |
*** ParsectiX has joined #openstack-infra | 10:59 | |
*** Daisy has quit IRC | 11:01 | |
*** lawrancejing has joined #openstack-infra | 11:02 | |
*** yamamoto has joined #openstack-infra | 11:08 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: update grenade branch logic for mitaka https://review.openstack.org/229363 | 11:08 |
sdague | so, I had the order wrong on these patches | 11:08 |
sdague | which means we're going to be in a bit of pain for a moment | 11:09 |
*** baoli has joined #openstack-infra | 11:10 | |
*** baoli has quit IRC | 11:10 | |
sdague | jhesketh / fungi / SergeyLukjanov - anyone up for looking at that d-g change? | 11:12 |
*** Daisy has joined #openstack-infra | 11:13 | |
*** yamamoto has quit IRC | 11:13 | |
*** otter768 has joined #openstack-infra | 11:16 | |
*** e0ne has joined #openstack-infra | 11:18 | |
fungi | sdague: 229363? looking now | 11:18 |
sdague | yeh | 11:18 |
*** baoli has joined #openstack-infra | 11:18 | |
*** baoli has quit IRC | 11:18 | |
sdague | turns out I forgot the branch computation wasn't right | 11:19 |
*** sergmelikyan has quit IRC | 11:19 | |
*** ddieterly has joined #openstack-infra | 11:20 | |
*** bltavares has joined #openstack-infra | 11:20 | |
*** otter768 has quit IRC | 11:21 | |
StevenK | fungi: Hai, could you be convinced to approve a trivial infra-specs change, https://review.openstack.org/228732 , or shall I wait for jeblair? | 11:21 |
fungi | we can also rip out scenarios which involve stable/icehouse, though i suppose that can wait | 11:21 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Use tripleo.sh to drive ci https://review.openstack.org/225776 | 11:22 |
*** markvoelker has joined #openstack-infra | 11:22 | |
*** ddieterly has quit IRC | 11:24 | |
sdague | yeh, I'd prefer to just sort those out later | 11:24 |
*** bltavares has quit IRC | 11:24 | |
*** maishsk has joined #openstack-infra | 11:24 | |
*** BharatK has quit IRC | 11:25 | |
openstackgerrit | Merged openstack-infra/infra-specs: Remove topic:zanata from review link https://review.openstack.org/228732 | 11:26 |
*** kengo_sa_ has quit IRC | 11:26 | |
*** markvoelker has quit IRC | 11:27 | |
*** baoli has joined #openstack-infra | 11:27 | |
*** baoli_ has joined #openstack-infra | 11:28 | |
*** mpaolino has quit IRC | 11:29 | |
*** ldnunes has joined #openstack-infra | 11:29 | |
krotscheck | What version of puppet are we currently using? | 11:29 |
fungi | wznoinsk: do you mean what we use to maintain the git.openstack.org server farm, or what we run on our personal workstations to keep local clones up to date? | 11:30 |
StevenK | fungi: Thank you for the +A | 11:30 |
fungi | krotscheck: 3.7.0 | 11:31 |
krotscheck | fungi: danke. | 11:31 |
*** sergmelikyan has joined #openstack-infra | 11:31 | |
*** baoli has quit IRC | 11:32 | |
*** nadya has quit IRC | 11:34 | |
*** julim has quit IRC | 11:34 | |
*** aarefiev has quit IRC | 11:35 | |
*** zz_dimtruck is now known as dimtruck | 11:38 | |
krotscheck | nibalizer: That issue from 2 days ago is apparently a bug in puppet that was fixed with 3.7. My testenv is 3.4.3, so that's probably why I was hitting it. | 11:38 |
*** aarefiev has joined #openstack-infra | 11:38 | |
*** julim has joined #openstack-infra | 11:39 | |
*** abregman has joined #openstack-infra | 11:40 | |
*** abregman is now known as abregman|pto | 11:42 | |
*** gordc has joined #openstack-infra | 11:42 | |
*** julim has quit IRC | 11:44 | |
openstackgerrit | Jiri Stransky proposed openstack-infra/tripleo-ci: Remove a duplicate variable export https://review.openstack.org/229376 | 11:44 |
fungi | sdague: 229363 is failing grenade | 11:50 |
fungi | sdague: http://logs.openstack.org/63/229363/1/check/gate-grenade-dsvm/078cd89/ | 11:51 |
openstackgerrit | Michael Krotscheck proposed openstack-infra/system-config: Remove distro-package npm from release slave. https://review.openstack.org/229380 | 11:51 |
openstackgerrit | Michael Krotscheck proposed openstack-infra/system-config: Added recent version of npm to release slave https://review.openstack.org/229381 | 11:51 |
krotscheck | fungi: ^^ Try #32235543 | 11:51 |
*** mgrosz has joined #openstack-infra | 11:51 | |
*** mgrosz has quit IRC | 11:52 | |
*** mgrosz has joined #openstack-infra | 11:52 | |
*** thorst has joined #openstack-infra | 11:53 | |
*** bdemers_ has quit IRC | 11:54 | |
* fungi has no idea what #32235543 is but will look at the two changes you just uploaded shortly | 11:54 | |
*** maishsk_ has joined #openstack-infra | 11:56 | |
*** maishsk has quit IRC | 11:57 | |
*** maishsk_ is now known as maishsk | 11:57 | |
*** dizquierdo has joined #openstack-infra | 11:58 | |
*** fhubik_brb is now known as fhubik | 11:59 | |
*** asmigala1 has joined #openstack-infra | 12:01 | |
krotscheck | fungi: It's my exasperation with how many times I've had to revise this | 12:01 |
krotscheck | Aaanyway, back to bed. Silly infant keeping me up. | 12:01 |
sdague | fungi: ok, let me take a look | 12:02 |
fungi | krotscheck: ahh. well, i _did_ say i was okay with just doing the /usr/etc/npmrc thing. but thanks for trying to find a cleaner solution | 12:02 |
fungi | sdague: i started digging in... something looks unhappy with cinder there | 12:02 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add openstack-ansible-security repository https://review.openstack.org/229318 | 12:03 |
sdague | yeh, so.... I think we're maybe wedged | 12:03 |
sdague | this is the cinder v1 disable bit | 12:03 |
*** asmigala has quit IRC | 12:03 | |
*** markvoelker has joined #openstack-infra | 12:03 | |
*** julim has joined #openstack-infra | 12:03 | |
sdague | oh, it looks like in master I can land the revert | 12:03 |
sdague | cinder v1 service removal trips up another thing | 12:04 |
sdague | https://review.openstack.org/#/c/229348/ | 12:04 |
*** jtomasek has joined #openstack-infra | 12:04 | |
sdague | so... opinion time, I could land all that, because a lot of things remain broken by it | 12:04 |
*** mgrosz has quit IRC | 12:04 | |
sdague | fungi: what do you think? mtreinish not awake yet | 12:04 |
fungi | looking | 12:05 |
sdague | basically, the grenade code requires cinder v1 because cinder v2 didn't work at all as recently as six months ago | 12:05 |
sdague | with openstack client | 12:05 |
*** sridhar_ram has joined #openstack-infra | 12:06 | |
*** amotoki has quit IRC | 12:06 | |
fungi | is 229348 going to fail without 229363 (do we have a mutual wedge)? | 12:06 |
*** ildikov has quit IRC | 12:06 | |
*** salv-orlando has quit IRC | 12:06 | |
sdague | no, 229348 already passed | 12:06 |
*** salv-orlando has joined #openstack-infra | 12:07 | |
fungi | okay, so adding a depends-on in 229363 likely queues these in an order which will pass | 12:07 |
fungi | i agree the revert in devstack is likely the next step, though i'm not a devstack core reviewer | 12:08 |
*** dprince has joined #openstack-infra | 12:08 | |
*** pahuang has quit IRC | 12:08 | |
*** aysyd has joined #openstack-infra | 12:09 | |
fungi | if one of the other devstack cores here wants to approve that (or if you want to self-approve it) i'll help guide it in | 12:09 |
*** pahuang has joined #openstack-infra | 12:09 | |
*** sridhar_ram has quit IRC | 12:10 | |
*** deepakcs has quit IRC | 12:10 | |
sdague | yeh, I'm comfortable self approving, I just want another opinion | 12:11 |
sdague | to ensure that I'm not being crazy about it | 12:11 |
fungi | you've posted on the ml thread about it, seems perfectly reasonable | 12:11 |
*** hdd has joined #openstack-infra | 12:12 | |
sdague | ok, I approved the master and liberty reverts | 12:12 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: update grenade branch logic for mitaka https://review.openstack.org/229363 | 12:13 |
sdague | that depends on should address it | 12:13 |
*** eantyshev has left #openstack-infra | 12:14 | |
*** yamamoto has joined #openstack-infra | 12:14 | |
*** weshay_xchat has joined #openstack-infra | 12:16 | |
*** dimtruck is now known as zz_dimtruck | 12:17 | |
*** jlanoux has quit IRC | 12:17 | |
*** jlanoux has joined #openstack-infra | 12:17 | |
*** zz_dimtruck is now known as dimtruck | 12:18 | |
*** yamamoto has quit IRC | 12:19 | |
*** dingyichen has joined #openstack-infra | 12:20 | |
*** ddieterly has joined #openstack-infra | 12:20 | |
fungi | i went ahead and approved fungi@zuul:~$ sudo zuul enqueue --trigger gerrit --project openstack-infra/devstack-gate --pipeline gate --change 229363,2 | 12:21 |
fungi | grr, wrong paste | 12:21 |
sdague | ok, cool | 12:21 |
fungi | but yeah, it's in the gate now | 12:21 |
sdague | yep, looks good | 12:22 |
fungi | since its dependencies were already there | 12:22 |
sdague | hopefully enough people are still kind of asleep that this will all be resolved | 12:22 |
sdague | yeh | 12:22 |
*** smcginnis has joined #openstack-infra | 12:22 | |
*** sdake has quit IRC | 12:22 | |
sdague | all seems sane, ok, I'm going to go make a real breakfast | 12:23 |
sdague | and check back in in a bit once those results have sorted | 12:23 |
fungi | i need to do the same | 12:23 |
cdent | sdague: I had no idea you were a geograph-ist | 12:23 |
* cdent shuns sdague | 12:23 | |
cdent | :) | 12:23 |
sdague | :) | 12:23 |
*** sdake has joined #openstack-infra | 12:23 | |
sdague | hey, you could be taking a nap :) | 12:23 |
cdent | I wish I was, I have a cold. | 12:23 |
sdague | the gate was small enough that it didn't seem like a firedrill | 12:24 |
*** smcginnis has quit IRC | 12:25 | |
*** ddieterly has quit IRC | 12:26 | |
rcarrillocruz | zaro: heya, too early for you to be up , right? | 12:26 |
*** luqas has joined #openstack-infra | 12:26 | |
*** smcginnis has joined #openstack-infra | 12:27 | |
*** Daisy has quit IRC | 12:28 | |
*** Daisy has joined #openstack-infra | 12:28 | |
*** trown|relocating is now known as trown | 12:29 | |
*** ildikov has joined #openstack-infra | 12:30 | |
*** marcusvrn_ has joined #openstack-infra | 12:30 | |
*** julim has quit IRC | 12:30 | |
*** Hal has joined #openstack-infra | 12:32 | |
*** maishsk has quit IRC | 12:32 | |
*** Daisy has quit IRC | 12:32 | |
clarkb | didnt the v1 removal go in days ago? why are we just having issues now? | 12:33 |
*** Daisy has joined #openstack-infra | 12:33 | |
*** julim has joined #openstack-infra | 12:34 | |
*** maishsk has joined #openstack-infra | 12:35 | |
*** bswartz has quit IRC | 12:36 | |
*** Daisy has quit IRC | 12:38 | |
*** Kennan has quit IRC | 12:38 | |
*** Kennan has joined #openstack-infra | 12:39 | |
*** ddieterly has joined #openstack-infra | 12:40 | |
*** jtomasek has quit IRC | 12:41 | |
*** jtomasek has joined #openstack-infra | 12:41 | |
*** marzif has joined #openstack-infra | 12:43 | |
*** yamamoto has joined #openstack-infra | 12:43 | |
*** Daisy has joined #openstack-infra | 12:43 | |
mordred | sdague, clarkb: why in the WORLD would we DISABLE cinder v1? | 12:44 |
*** achanda has joined #openstack-infra | 12:44 | |
mordred | would we not want to add v2, switch all the services to use it by default | 12:44 |
clarkb | I do not know | 12:44 |
mordred | and then run both v1 and v2 in parallel for at least 2 or 3 years? | 12:44 |
smcginnis | Cinder v1 has been deprecated for some time. V2 has also been available. | 12:44 |
smcginnis | I believe it has been a couple years, but TBH I wasn't around when that work happened. | 12:45 |
mordred | smcginnis: as I understand it, literally nobody uses v2 yet because it did not work until just recently | 12:45 |
mordred | smcginnis: and there are zero vendors shipping it | 12:45 |
smcginnis | mordred: It does work. Maybe that's been the perception though. | 12:45 |
mordred | so we're basically at the same spot as keystone v3 | 12:45 |
mordred | whichis that we SHOULD move to v2 | 12:45 |
mordred | but v1 is still a reality in the world | 12:45 |
mordred | and we should account for that | 12:45 |
smcginnis | I think we definitely need to work on getting the word out. | 12:45 |
clarkb | but still why did it take days to find grenade fails? | 12:45 |
mordred | smcginnis: ++ | 12:45 |
*** hdd has quit IRC | 12:45 | |
smcginnis | And work with the client consumers to make sure they get updated first. | 12:45 |
mordred | smcginnis: I'm mainly arguing that we should not disable v1 anywhere until it's actually more dead than it is now | 12:46 |
smcginnis | mordred: To be fair, if you're talking about the change I think you are. | 12:46 |
smcginnis | It was to make 2 the default, not remove 1. | 12:46 |
mordred | smcginnis: but we should definitely push to get clients and whatnot to consume v2 | 12:46 |
mordred | smcginnis: k. awesome | 12:46 |
clarkb | iirc it removed v1 | 12:46 |
smcginnis | mordred: Definitely | 12:46 |
clarkb | broke rally | 12:47 |
mordred | Revert "Disable Cinder v1 API support by default" | 12:47 |
mordred | that's the title of the patch | 12:47 |
clarkb | because no v2 in rally | 12:47 |
mordred | https://review.openstack.org/#/c/194726/ | 12:47 |
mordred | "Disable Cinder v1 API support by default | 12:47 |
mordred | " | 12:47 |
mordred | original patch | 12:47 |
smcginnis | mordred: Thanks, I was just going to ask for the link. ;) | 12:47 |
mordred | smcginnis: :) | 12:47 |
*** mgrosz has joined #openstack-infra | 12:48 | |
smcginnis | OK, so it disabled it be default, but it can be enabled with config. | 12:48 |
smcginnis | It's not removed. | 12:48 |
mordred | yes | 12:48 |
mordred | but that should not be done | 12:48 |
smcginnis | But that doesn't really help in gate. | 12:48 |
mordred | for at least another few cycles | 12:48 |
smcginnis | mordred: I do agree. | 12:48 |
mordred | smcginnis: woot! | 12:48 |
* mordred loves morning argreement | 12:48 | |
smcginnis | ;) | 12:48 |
mordred | now - to find the conference room I'm supposed to be in ... | 12:48 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/os-loganalyze: Skip whitespace for html view https://review.openstack.org/226858 | 12:48 |
*** mgrosz has quit IRC | 12:49 | |
clarkb | its removed from devstack | 12:49 |
clarkb | in this case disabled == removed | 12:49 |
*** ala_ has joined #openstack-infra | 12:49 | |
clarkb | its not useable | 12:49 |
clarkb | there is no functional difference | 12:50 |
pabelanger | anteaya: yes, on my list of things to work on. | 12:50 |
clarkb | whether or not the code is on disk is a completely different issue | 12:50 |
*** Daisy has quit IRC | 12:51 | |
clarkb | but still eould be good to figure out why it took so long to see problems with the change | 12:51 |
*** Ala has quit IRC | 12:51 | |
fungi | also devstack-gate patch 229363,2 to add stable/liberty is still failing even with the cinder v2 revert | 12:52 |
fungi | sdague: ^ when you get back from eating | 12:53 |
*** bltavares has joined #openstack-infra | 12:53 | |
*** edmondsw has joined #openstack-infra | 12:54 | |
clarkb | fungi is that what precipitated the revert? so wasnt existing fails but new ones when flipping branches. that would explain the delay | 12:54 |
fungi | clarkb: well, following the ml thread, there were other issues noticed as well | 12:55 |
clarkb | there were but none deemed revert worthy | 12:55 |
clarkb | rally was told to explocitly enable v1, the tempest volume migration test got a patch to handle v2 and so on | 12:55 |
clarkb | so I am curious why all of a sudden the revert had to happen | 12:56 |
clarkb | (not that the revert is bad just curious about what changed) | 12:56 |
fungi | but yeah, cinder_grenade_vol never becomes usable according to http://logs.openstack.org/63/229363/2/gate/gate-grenade-dsvm/8709a78//logs/grenade.sh.txt.gz#_2015-09-30_12_50_33_971 | 12:57 |
fungi | in the liberty build | 12:57 |
openstackgerrit | Sergey Skripnick proposed openstack-infra/project-config: Remove usage of jenkins/slave_scripts in rally job https://review.openstack.org/229408 | 12:58 |
openstackgerrit | Pranav Salunke proposed openstack-infra/project-config: Create new project training-labs https://review.openstack.org/218345 | 12:58 |
fungi | poking around in the screen logs to see if i can spot why | 12:58 |
sdague | clarkb: so yesterday it looked like rally was the only thing affected | 12:58 |
sdague | ok, that's a one off, but today there seem to be a bunch of affected things | 12:58 |
sdague | http://logs.openstack.org/63/229363/2/gate/gate-grenade-dsvm/8709a78//logs/grenade.sh.txt.gz#_2015-09-30_12_49_28_921 | 12:59 |
sdague | some times the piling on of data to show how deep the issue is becomes an issue | 12:59 |
*** rfolco_ has quit IRC | 13:00 | |
*** ddieterly has quit IRC | 13:00 | |
jhesketh | yolanda: could you take a look at https://review.openstack.org/#/c/118993/ when you get a chance please | 13:00 |
*** achanda has quit IRC | 13:00 | |
yolanda | sure | 13:00 |
*** jaypipes has quit IRC | 13:00 | |
openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Move Item formatting into Reporters https://review.openstack.org/223298 | 13:00 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add support for sqlalchemy reporter https://review.openstack.org/223333 | 13:00 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Tidy up loggers https://review.openstack.org/224336 | 13:00 |
*** kushal has joined #openstack-infra | 13:00 | |
sdague | smcginnis: openstack client did not work v2 until a couple of months ago | 13:01 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add openstack-ansible-security repository https://review.openstack.org/229318 | 13:01 |
*** Hal1 has joined #openstack-infra | 13:02 | |
*** spzala has joined #openstack-infra | 13:03 | |
*** Hal has quit IRC | 13:03 | |
*** nadya has joined #openstack-infra | 13:04 | |
*** spzala has quit IRC | 13:05 | |
sdague | fungi: so it looks like possibly in liberty the cinder v1 code is not working? | 13:05 |
sdague | http://logs.openstack.org/63/229363/2/gate/gate-grenade-dsvm/8709a78//logs/old/screen-c-api.txt.gz#_2015-09-30_12_49_31_083 | 13:05 |
*** asmigala1 has quit IRC | 13:06 | |
fungi | but is working in master? (or devstack/tempest jobs in master aren't testing that?) | 13:06 |
sdague | they may not be touching this path | 13:07 |
*** rellerreller has joined #openstack-infra | 13:07 | |
*** dramalho has joined #openstack-infra | 13:07 | |
fungi | certainly seems possible | 13:07 |
*** redixin has joined #openstack-infra | 13:08 | |
sdague | the issue is "openstack volume show cinder_grenade_vol" | 13:08 |
sdague | doesn't return anything | 13:08 |
mordred | sdague: for 1 second I thought that someone had implemented a command called cinder_grenade_vol | 13:09 |
*** ccarrara has joined #openstack-infra | 13:09 | |
mordred | sdague: but then I drank more coffee | 13:09 |
sdague | heh | 13:09 |
fungi | there were some tenant-related config issues warned in one of the other logs which might indicate it's searching te wrong tenant | 13:09 |
redixin | dear all. could you please merge this small simple patch. we have -1 from jenkins for all our CRs https://review.openstack.org/#/c/229408/ | 13:10 |
fungi | Unable to get internal tenant context: Missing required config parameters. | 13:10 |
cdent | Just a heads up:I'm getting 504 Gateway Time-out when trying to subscribe to the openstack-infra list. I can submit the subscription form but then it times out while processing. | 13:10 |
*** rfolco has joined #openstack-infra | 13:10 | |
openstackgerrit | Antoine Musso proposed openstack-infra/zuul: Enhance CRD doc with diagrams https://review.openstack.org/229416 | 13:10 |
fungi | sdague: http://logs.openstack.org/63/229363/2/gate/gate-grenade-dsvm/8709a78//logs/old/screen-c-vol.txt.gz#_2015-09-30_12_45_58_066 | 13:11 |
*** hasharReviews is now known as hashar | 13:11 | |
fungi | not sure if that's benign since i don't stare at these often enough | 13:11 |
sdague | fungi: logstash votes it's probably noise | 13:12 |
sdague | 91% success rate | 13:12 |
fungi | yep, just checked the same | 13:12 |
*** hrou has joined #openstack-infra | 13:12 | |
*** dizquierdo has quit IRC | 13:12 | |
sdague | smcginnis: could use some assistance here, cinder is definitely locking us up | 13:13 |
*** julim has quit IRC | 13:13 | |
*** geoffarnold has joined #openstack-infra | 13:14 | |
*** zeih has joined #openstack-infra | 13:14 | |
*** Daisy has joined #openstack-infra | 13:14 | |
*** ajmiller has quit IRC | 13:14 | |
*** julim has joined #openstack-infra | 13:15 | |
*** geoffarn_ has joined #openstack-infra | 13:15 | |
*** julim has quit IRC | 13:16 | |
*** xyang1 has joined #openstack-infra | 13:16 | |
*** otter768 has joined #openstack-infra | 13:17 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-infra/project-config: dsvm full job for devstack-zmq https://review.openstack.org/229419 | 13:18 |
*** asmigala has joined #openstack-infra | 13:18 | |
*** geoffarnold has quit IRC | 13:18 | |
*** tiswanso has joined #openstack-infra | 13:18 | |
*** mriedem has joined #openstack-infra | 13:19 | |
*** eharney has joined #openstack-infra | 13:21 | |
sdague | fungi: this is weird, I can't see the POST call in the logs here to build that volume | 13:21 |
fungi | oh, i didn't think to start tracing it that far back | 13:21 |
*** otter768 has quit IRC | 13:21 | |
sdague | right, so that's indicative of this being the root cause - http://logs.openstack.org/63/229363/2/gate/gate-grenade-dsvm/8709a78//logs/grenade.sh.txt.gz#_2015-09-30_12_49_28_921 | 13:22 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/os-loganalyze: Skip whitespace for html view https://review.openstack.org/226858 | 13:22 |
sdague | need stevemar or dtroyer | 13:22 |
sdague | yeh, I was comparing a working log and a non working log | 13:22 |
*** Daisy has quit IRC | 13:22 | |
*** Daisy has joined #openstack-infra | 13:23 | |
*** mriedem has quit IRC | 13:23 | |
*** alivigni has joined #openstack-infra | 13:24 | |
*** ildikov has quit IRC | 13:24 | |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Move nv openstack-ansible experimental job to check queue https://review.openstack.org/228408 | 13:24 |
*** dkehn has quit IRC | 13:24 | |
*** rvasilets_ has joined #openstack-infra | 13:25 | |
fungi | it's using the latest python-openstackclient (1.7.0 from a week ago) in that call, looks like | 13:26 |
*** mriedem has joined #openstack-infra | 13:26 | |
rvasilets_ | Hi. Could you vote for https://review.openstack.org/#/c/229264/ ? Thx | 13:26 |
sdague | fungi: yeh, which apparently no longer works with cinder v1.... | 13:26 |
sdague | :( | 13:26 |
*** dizquierdo has joined #openstack-infra | 13:27 | |
*** julim has joined #openstack-infra | 13:27 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: update grenade branch logic for mitaka https://review.openstack.org/229363 | 13:27 |
*** doug-fish has quit IRC | 13:27 | |
sdague | ok, so that's stacked on top of a grenade work around | 13:28 |
fungi | seems like we have some significant missing test coverage for cinder volume create in tempest and/or openstackclient functional jobs | 13:28 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/os-loganalyze: Skip whitespace for html view https://review.openstack.org/226858 | 13:28 |
*** ayoung has joined #openstack-infra | 13:28 | |
sdague | I think it's mostly openstack client | 13:29 |
yolanda | hi, i was wondering about https://review.openstack.org/#/c/189762/ - nodepool secure.conf | 13:29 |
sdague | the moment it went over to v2, it stopped working on v1 | 13:29 |
sdague | which provides no real migration for folks | 13:29 |
yolanda | it needs some more eyes, and land it soon, as i sent the email about these changes some time ago | 13:29 |
fungi | sdague: yes, seems like a liberty release blocker | 13:30 |
*** Daisy has quit IRC | 13:30 | |
*** ccarrara has quit IRC | 13:30 | |
sdague | all this feels like this whole thing is too rushed, and we need to take a step back | 13:30 |
asselin_ | yolanda, I was looking at it yesterday. there are a lot of common-ci spec changes depending on that. If we can merge it soon, it will unblock all those other patches. | 13:32 |
*** ddieterly has joined #openstack-infra | 13:32 | |
*** hdd has joined #openstack-infra | 13:32 | |
pabelanger | yolanda: Ya, I think people agreed to merged it. Somebody just needs to pull trigger and baby sit. | 13:32 |
yolanda | it needs an extra +2 | 13:33 |
*** ddieterly has quit IRC | 13:33 | |
*** ttx has quit IRC | 13:33 | |
*** jsavak has joined #openstack-infra | 13:33 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Use tripleo.sh to build packages https://review.openstack.org/229425 | 13:33 |
asselin_ | I have the dependent patches listed here: https://etherpad.openstack.org/p/common-ci-sprint | 13:33 |
*** ttx has joined #openstack-infra | 13:33 | |
*** ddieterly has joined #openstack-infra | 13:33 | |
yolanda | fungi, clarkb, can you look at this patch when you have an slot? | 13:34 |
*** mpaolino has joined #openstack-infra | 13:34 | |
*** regXboi has joined #openstack-infra | 13:34 | |
mtreinish | fungi: we have coverage for cinder v2 in tempest | 13:34 |
mtreinish | although I spent yesterday fixing a bunch of places where we didn't use v2 if v1 wasn't enabled | 13:34 |
*** geoffarn_ has quit IRC | 13:35 | |
fungi | mtreinish: right, but it's cinder v1 volume create which either isn't working correctly any longer or somehow got support silently dropped in latest openstackclient | 13:35 |
mtreinish | ah, ok missed the context skimming the back log | 13:35 |
*** geoffarnold has joined #openstack-infra | 13:35 | |
fungi | so wondering if tempest master stopped testing some v1 api calls it used to test | 13:36 |
mtreinish | yeah osc probably needs real testing | 13:36 |
fungi | er, well tempest when testing master | 13:36 |
mtreinish | fungi: nope, tempest still will test both | 13:36 |
fungi | okay, cool, so this _could_ be a regression in openstackclient i guess | 13:36 |
fungi | rather than missing test coverage for cinder itself | 13:37 |
clarkb | well tempest stopped testing both recently | 13:37 |
clarkb | because v1 was not available | 13:37 |
mtreinish | clarkb: yeah, when devstack turned off v1 tempest couldn't test that path anymore | 13:38 |
*** dimtruck is now known as zz_dimtruck | 13:38 | |
*** bdemers has joined #openstack-infra | 13:38 | |
*** dkehn has joined #openstack-infra | 13:38 | |
asselin_ | mtreinish, fungi clarkb fyi there's a os client patch here that tries to fix it: https://review.openstack.org/229039 | 13:38 |
clarkb | so it is possibly a regression in cinder itself | 13:38 |
fungi | so also possible changes merged to cinder in at least stable/liberty which broke v1 image create | 13:38 |
fungi | yeah | 13:38 |
* dtroyer catches up… | 13:39 | |
fungi | after the devstack change went in to disable v1 | 13:39 |
sdague | clarkb: the request isn't getting to cinder | 13:39 |
*** mpaolino has quit IRC | 13:39 | |
sdague | it's not in the cinder api log | 13:39 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-infra/project-config: dsvm full job for devstack-zmq https://review.openstack.org/229419 | 13:39 |
*** zeih_ has joined #openstack-infra | 13:39 | |
dtroyer | that error sdague posted about 20 min ago is an OSC volume v2 only error due to how glanceclient v2 is implemented (thanks warlock) | 13:39 |
clarkb | asselin_ thats just client co fig | 13:39 |
dtroyer | so if you think it's on v1, it isn't at that point | 13:39 |
fungi | oh | 13:39 |
sdague | dtroyer: hmmm... it's specifically calling v1 then | 13:40 |
openstackgerrit | Merged openstack/os-testr: Better blacklist - tested with Nova https://review.openstack.org/229223 | 13:40 |
dtroyer | ok, hang on a sec…early for me... | 13:40 |
dtroyer | I'm conflating image and volume there | 13:40 |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: Add puppet-openstack_spec_helper project https://review.openstack.org/229126 | 13:40 |
sdague | dtroyer: yeh, this is voluem | 13:40 |
dtroyer | dangit | 13:41 |
*** vmb_out is now known as vmbrasseur | 13:42 | |
sdague | so, I'm potentially just working around it in grenade, however, that seems like a kind of big deal because there hasn't been any overlap of an osc that works on v1 and v2 that we've had in infrastructure | 13:42 |
*** marzif has quit IRC | 13:43 | |
*** zeih has quit IRC | 13:43 | |
dtroyer | really? volume v2 happened while I was unconsious, looks like I need to get up to speed on it | 13:43 |
*** doug-fish has joined #openstack-infra | 13:43 | |
dtroyer | and if that broke v1, it's a huge fubar on our part, that's one reason we c-n-p so friggin much code around, to avoid breakage | 13:44 |
pabelanger | any infra-core mind helping https://review.openstack.org/#/c/226047/ and https://review.openstack.org/#/c/226002/ land for grafyaml? We have the votes, just not 2 of them :D | 13:44 |
dtroyer | is it just a default issue? | 13:44 |
*** ttx has quit IRC | 13:44 | |
sdague | dtroyer: well, grenade explicitly set osc cinder version to 1 because 2 wasn't implemented and exploded | 13:45 |
sdague | and that works with the kilo version | 13:45 |
*** julim has quit IRC | 13:45 | |
sdague | but as soon as we cut the branch so we're doing liberty -> master | 13:45 |
sdague | it doesn't work | 13:45 |
sdague | with that error I linked to | 13:45 |
*** ttx has joined #openstack-infra | 13:45 | |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: puppet: enable integrations jobs for puppet-swift https://review.openstack.org/229432 | 13:46 |
*** k4n0 has quit IRC | 13:47 | |
openstackgerrit | Eric Harney proposed openstack-infra/project-config: Add dsvm-full-lio to os-brick jobs https://review.openstack.org/223258 | 13:47 |
*** doug-fish has quit IRC | 13:47 | |
*** doug-fish has joined #openstack-infra | 13:48 | |
*** dingyichen has quit IRC | 13:49 | |
*** dims has quit IRC | 13:49 | |
*** dims has joined #openstack-infra | 13:49 | |
*** ociuhandu has joined #openstack-infra | 13:50 | |
*** mpaolino has joined #openstack-infra | 13:51 | |
*** julim has joined #openstack-infra | 13:51 | |
*** kgiusti has joined #openstack-infra | 13:51 | |
clarkb | sdague and v2 -> v2 doesnt work for other reasons right? | 13:51 |
dtroyer | sdague: OSC changed the default volume API to v2 on 8/24, so is in the 1.7.0 release. it's possible that it isn't downshifting properly for a v1 setting from the env/cmd line | 13:51 |
clarkb | in theory you could v1 and v2 on liberty then v2 only on M but that doesnt work because v1 and v2 both unhappy? | 13:51 |
dtroyer | i'm heading off to look at that next | 13:52 |
*** doug-fish has quit IRC | 13:52 | |
openstackgerrit | Jakub Ruzicka proposed openstack-infra/project-config: Add new project pymod2pkg to Packaging-RPM team https://review.openstack.org/227891 | 13:54 |
openstackgerrit | Jakub Ruzicka proposed openstack-infra/project-config: Add new project renderspec to Packaging-RPM team https://review.openstack.org/227892 | 13:54 |
*** sergmelikyan has quit IRC | 13:54 | |
*** doug-fish has joined #openstack-infra | 13:54 | |
*** pradk has joined #openstack-infra | 13:54 | |
*** stevemar has joined #openstack-infra | 13:56 | |
*** geoffarnold has quit IRC | 13:56 | |
smcginnis | sdague: Sorry, was away but I saw your ping. | 13:56 |
sdague | clarkb: I'm deleting the forced downshift bit in grenade, we'll see | 13:56 |
smcginnis | sdague: Still issues? | 13:56 |
sdague | smcginnis: yes | 13:56 |
*** jaypipes has joined #openstack-infra | 13:56 | |
smcginnis | :/ | 13:56 |
*** geoffarnold has joined #openstack-infra | 13:57 | |
*** mriedem is now known as mriedem_away | 13:57 | |
smcginnis | sdague: Are there additional issues on the cinder side we need to address? | 13:57 |
sdague | v1 does not work in liberty with osc | 13:57 |
fungi | so this still means that osc 1.7.0 won't work with openstack deployments that lack cinder v2 | 13:58 |
smcginnis | sdague: Were changes merged in osc for the v1/v2 change? I thought those didn't go through yet. | 13:58 |
sdague | fungi: right | 13:58 |
fungi | smcginnis: back in august according to dtroyer | 13:58 |
*** scheuran has quit IRC | 13:58 | |
smcginnis | e0ne: You around? | 13:58 |
e0ne | smcginnis: yep | 13:59 |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:59 | |
*** placko_ has joined #openstack-infra | 13:59 | |
smcginnis | e0ne: Are you aware of what's going on? I didn't think we changed version support in August. | 13:59 |
smcginnis | e0ne: But I wasn't paying as close attention then. | 13:59 |
sdague | also, because this is based on explicitly changing the service catalog type I expect that in the field it will take a really long time for people to flip over | 13:59 |
e0ne | smcginnis: do you mean osc? | 13:59 |
*** julim has quit IRC | 14:00 | |
smcginnis | dtroyer: Is that what you were referring to? ^^ | 14:00 |
* smcginnis hasn't read all the backlog | 14:00 | |
fungi | smcginnis: yes | 14:00 |
smcginnis | fungi: Thanks. | 14:00 |
fungi | cinder v2 support in python-openstackclient in august appears in the 1.7.0 release from last week | 14:00 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Remove a duplicate variable export https://review.openstack.org/229376 | 14:01 |
fungi | and seems to have inadvertently ceased working with at least some cinder v1 calls when that happened | 14:01 |
smcginnis | fungi: v2 support, but I didn't think we default to v2 in any way yet. | 14:01 |
e0ne | openstackclient works with cinder api v2 only if it passes explicit via --os-volume-api-version or env variable due to https://review.openstack.org/229039 but | 14:01 |
e0ne | fungi: ^^ | 14:01 |
*** zeih_ has quit IRC | 14:02 | |
e0ne | dtroyer: ^^ | 14:02 |
stevemar | e0ne: you can set OS_VOLUME_API_VERSION=2 | 14:02 |
e0ne | stevemar: sure | 14:02 |
e0ne | stevemar: asselin did it on their 3rd party CI yesterday and it works | 14:02 |
dtroyer | so I'm getting image create with osc 1.7.0 working manually on both v1 and v2 with a devstack about a week old | 14:03 |
asselin_ | ack | 14:03 |
sdague | dtroyer: volume create | 14:03 |
sdague | volume create is what's not getting sent to the server on v1 | 14:04 |
*** sergmelikyan has joined #openstack-infra | 14:04 | |
sdague | in 1.7.0 | 14:04 |
openstackgerrit | Rob Cresswell proposed openstack/requirements: Bump minimum Django version https://review.openstack.org/229446 | 14:04 |
dtroyer | grrrrrrrr I promise I tyoed volume create in my test/// ;) | 14:04 |
e0ne | dtroyer: it works only with OS_VOLUME_API_VERSION=2 | 14:05 |
dtroyer | it feels like something in the environment… | 14:05 |
*** ccarrara has joined #openstack-infra | 14:05 | |
dtroyer | works for me with v1 | 14:05 |
sdague | https://github.com/openstack-dev/grenade/blob/9b3c56e60b6d26e160c98c559717f87f71b7e93b/projects/70_cinder/resources.sh#L81 | 14:05 |
sdague | that's what fails | 14:05 |
e0ne | dtroyer: i've mentioned fix for it in os-client-onfig earlier | 14:05 |
sdague | fails to send to the server | 14:05 |
fungi | volume create here isn't getting picked up in the c-api screen log http://logs.openstack.org/63/229363/2/gate/gate-grenade-dsvm/8709a78//logs/grenade.sh.txt.gz#_2015-09-30_12_49_28_921 | 14:05 |
e0ne | openstack volume create 1 ----- doesn't work!!! | 14:06 |
e0ne | openstack volume --os-volume-api-version 2 create 1 -- it works!! | 14:06 |
dtroyer | openstack volume create —size 1 vol-name | 14:06 |
*** bswartz has joined #openstack-infra | 14:06 | |
*** jsavak has quit IRC | 14:06 | |
*** amotoki has joined #openstack-infra | 14:06 | |
e0ne | we've got a bug in osc https://bugs.launchpad.net/python-openstackclient/+bug/1500938 | 14:06 |
openstack | Launchpad bug 1500938 in os-client-config "openstack client doesn't work correctly with cinder v2 api" [Undecided,New] | 14:06 |
fungi | e0ne: point is, this is in a job with cinder v2 api disabled | 14:07 |
asselin_ | e0ne, stevemar this is what we do: line 1: http://15.126.198.151/08/226308/11/check/3par-iscsi-driver-master-client-pip-eos10-dsvm/644838b/logs/localrc.txt.gz | 14:07 |
openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Subscriptions https://review.openstack.org/223562 | 14:07 |
*** tdurakov has joined #openstack-infra | 14:07 | |
*** jsavak has joined #openstack-infra | 14:07 | |
*** julim has joined #openstack-infra | 14:07 | |
sdague | fungi: well, it was. The reverts landed now | 14:08 |
e0ne | fungi: where do we have disabled v2? | 14:08 |
*** Sree has joined #openstack-infra | 14:08 | |
sdague | fungi: oh, sorry, no v2 was always up | 14:08 |
sdague | volumes v2 has been default on for 3 cycles | 14:08 |
sdague | or something | 14:08 |
tdurakov | hi, where can I find config for gate-tempest-dsvm-multinode-full job? | 14:08 |
*** dkranz has joined #openstack-infra | 14:09 | |
fungi | oh, hrm. so the volume calls should have been using v2 instead of v1 as far back as juno? | 14:09 |
sdague | fungi: volume calls *could* have been using it | 14:09 |
sdague | yes | 14:09 |
e0ne | :) | 14:10 |
sdague | but no one did | 14:10 |
sdague | and there was no collective effort in enabling that | 14:10 |
e0ne | and no one will do it while v1 is working | 14:10 |
clarkb | tempest does/did | 14:10 |
sdague | right, tempest did | 14:10 |
clarkb | but nothing else aiui | 14:10 |
*** amotoki has quit IRC | 14:10 | |
fungi | oh, i see. so osc <1.7.0 made volume calls via the v1 api, 1.7.0 makes them via the v2 api and can no longer perform them via the v1 api? | 14:10 |
sdague | e0ne: making people move by breaking everyone in the world is not an ok option | 14:11 |
e0ne | fungi: osc 1.7.0 could use both v1 and v2 | 14:11 |
*** Hal1 has quit IRC | 14:11 | |
dtroyer | fungi: that should not be the case, I haven't been able to duplicate that scenario yet | 14:11 |
sdague | if you want people on a new API you need to provide compelling reasons | 14:11 |
e0ne | sdague: agree | 14:11 |
e0ne | sdague: we don't extend old api at least 3 releases | 14:11 |
sdague | dtroyer: did you try to create the volume from an image? | 14:12 |
smcginnis | e0ne: So were you saying volume create with v1 is broken now? Not sure I followed completely. | 14:12 |
*** j_king has joined #openstack-infra | 14:12 | |
dtroyer | ah, missed that detail…and there's where image v2 comes into the picture!!!!!!!!!! | 14:12 |
sdague | dtroyer: could be | 14:12 |
dtroyer | I was right about that particular error, it's a friggin glacneclient2 problem!!!!!!!! | 14:12 |
sdague | dtroyer: https://github.com/openstack-dev/grenade/blob/9b3c56e60b6d26e160c98c559717f87f71b7e93b/projects/70_cinder/resources.sh#L81 | 14:12 |
e0ne | mcginnis: no | 14:12 |
sdague | oh, really? | 14:12 |
smcginnis | e0ne: OK, good. | 14:13 |
dtroyer | and that's why I am trying to get us off glanceclient first into the sdk... | 14:13 |
sdague | o_O ... flaper87 | 14:13 |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: copy_puppet_logs: use sudo to rename horizon logs https://review.openstack.org/229452 | 14:13 |
*** ParsectiX has quit IRC | 14:13 | |
e0ne | sdague: that will use api v1, not v2 | 14:13 |
*** shardy_ has joined #openstack-infra | 14:13 | |
smcginnis | dtroyer: The issue is actually with glance? That's why we're not seeing c-api logs? | 14:13 |
dtroyer | so lemme go absorb v2 image crewate... | 14:13 |
e0ne | sdague: you need to explicit set v2 api for openstack client | 14:13 |
dtroyer | dammit…volume create | 14:13 |
sdague | e0ne: if you have to explicitly set v2 api in osc, then turning off v1 in devstack is a non starter | 14:14 |
sdague | because that's just hitting users in the face with a pointy stick | 14:14 |
*** shardy has quit IRC | 14:14 | |
dtroyer | and here it is: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/volume/v2/volume.py#L111 | 14:15 |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: copy_puppet_logs: use sudo to rename horizon config https://review.openstack.org/229452 | 14:15 |
dtroyer | we need OS_API_IMAGE_VERSION=1 for this to work, first thing it does is look up the image name | 14:15 |
sdague | dtroyer: for serious? | 14:15 |
dtroyer | so we can use names instead of IDs | 14:15 |
sdague | flaper87 ... | 14:16 |
dtroyer | and I think we've addressed that problem, but it isn't released yet…I smell a 1.7.1 coming | 14:16 |
dtroyer | its in OSC due to the f-ing warlock-based resources | 14:16 |
dtroyer | so I oknow what I'm doing the rest of the week... | 14:16 |
sdague | ok, so this is blocking completion of the stable/liberty branch | 14:17 |
*** j_king has left #openstack-infra | 14:17 | |
sdague | because we can't test liberty -> master until this is fix | 14:17 |
*** dtantsur is now known as dtantsur|afk | 14:17 | |
*** amotoki has joined #openstack-infra | 14:17 | |
e0ne | dtroyer: what issue in osc do you mean? | 14:17 |
*** geoffarnold has quit IRC | 14:17 | |
fungi | was this bug 1498092? | 14:18 |
openstack | bug 1498092 in python-openstackclient "Glance - Set Image - Resolve fracturing" [Undecided,Fix committed] https://launchpad.net/bugs/1498092 - Assigned to Niall Bunting (niall-bunting) | 14:18 |
*** hdd has quit IRC | 14:18 | |
*** shardy_ has quit IRC | 14:18 | |
*** geoffarnold has joined #openstack-infra | 14:18 | |
*** shardy has joined #openstack-infra | 14:19 | |
*** jsavak has quit IRC | 14:20 | |
*** jsavak has joined #openstack-infra | 14:21 | |
*** scheuran has joined #openstack-infra | 14:21 | |
*** ccarrara has quit IRC | 14:21 | |
*** scheuran has quit IRC | 14:22 | |
*** zeih has joined #openstack-infra | 14:22 | |
*** tjones has joined #openstack-infra | 14:22 | |
*** asselin__ has joined #openstack-infra | 14:22 | |
*** yamamoto_ has joined #openstack-infra | 14:23 | |
*** hdd has joined #openstack-infra | 14:23 | |
dtroyer | e0ne: image v2 needs special handling and can't use our otherwise common find_resource() code. it's special. I'm not sure if that fix is committed yet, I know it isn't released yet | 14:24 |
*** amotoki has quit IRC | 14:24 | |
*** amuller has joined #openstack-infra | 14:25 | |
sdague | dtroyer: so, is there actually a work around? | 14:25 |
sdague | that we could limp with until there is a release | 14:25 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Move nv openstack-ansible experimental job to check queue https://review.openstack.org/228408 | 14:26 |
*** asselin_ has quit IRC | 14:26 | |
*** yamamoto has quit IRC | 14:26 | |
anteaya | jaypipes: so how violent are mirantis project names going to get before mirantis says "hey that is crossing a line?" | 14:26 |
dtroyer | sdague: yeah, OS_IMAGE_API_VERSION=1 | 14:26 |
sdague | ok, let me try that | 14:26 |
dtroyer | v2 is a <censored /> | 14:27 |
mtreinish | dtroyer: heh, I like that workaround us v1 | 14:27 |
mtreinish | s/us/use | 14:27 |
sdague | dtroyer: is there a bug for that? | 14:27 |
dtroyer | and a proposed fix that is buried in a chain of image v2 fixes | 14:28 |
dtroyer | image v2 is a mess in OSC | 14:28 |
dtroyer | I'm trying to fix it... | 14:28 |
jaypipes | anteaya: ? | 14:28 |
*** tjones has left #openstack-infra | 14:28 | |
sdague | dtroyer: is there a bug I can reference here? | 14:28 |
dtroyer | and will start the merges in a minute | 14:28 |
dtroyer | not sure, lemme look | 14:28 |
sdague | because then I can get grenade and friends working again | 14:28 |
*** amotoki has joined #openstack-infra | 14:28 | |
anteaya | jaypipes: http://lists.openstack.org/pipermail/openstack-dev/2015-September/075843.html | 14:28 |
*** Slashme has left #openstack-infra | 14:29 | |
dtroyer | I'll create one…most of that review chain was still considered 'feature' so no bug | 14:29 |
sdague | dtroyer: ok, yeh, just give me an artifact I can reference so we'll know when to remove the work around | 14:29 |
*** zz_dimtruck is now known as dimtruck | 14:30 | |
*** kgiusti has quit IRC | 14:30 | |
pelix | zaro waynr zxiiro: https://review.openstack.org/226331 fixes a random test bug and https://review.openstack.org/226107 associates logging events with tests (and cleans up the console output for test runs), both would be nice to land. | 14:31 |
*** zeih has quit IRC | 14:31 | |
*** zeih has joined #openstack-infra | 14:32 | |
jaypipes | anteaya: it's a component inside the Fuel web framework, nothing more. And, for the record, gun and war-related component names are fairly common in software projects... | 14:32 |
anteaya | oh so I should just shutup and deal? | 14:32 |
jaypipes | anteaya: I didn't say that. | 14:32 |
anteaya | because I am really not a fan of this kind of language or approach to naming | 14:32 |
*** mpaolino has quit IRC | 14:32 | |
anteaya | I have rather taken pride in the fact that openstack heretofore has not stooped to that level | 14:33 |
jaypipes | anteaya: uhm, "grenade"? | 14:33 |
*** amotoki has quit IRC | 14:33 | |
anteaya | it brings in an energy dynamic that is harmful to cooperative working | 14:33 |
anteaya | jaypipes: how are you not saying shutup and deal | 14:33 |
dtroyer | sdague: https://bugs.launchpad.net/python-openstackclient/+bug/1501362 | 14:33 |
openstack | Launchpad bug 1501362 in python-openstackclient "Image v2 lookup issues" [Critical,Triaged] - Assigned to Dean Troyer (dtroyer) | 14:33 |
anteaya | I ask again, where is the line? | 14:33 |
*** mriedem_away is now known as mriedem | 14:34 | |
jaypipes | anteaya: I don't know where the line is, Anita. | 14:34 |
anteaya | is there no respect for common shareed space anymore | 14:34 |
anteaya | jaypipes: I can accept that | 14:34 |
*** mudassirlatif has joined #openstack-infra | 14:34 | |
anteaya | I feel it has been crossed | 14:34 |
jaypipes | anteaya: I think you are ascribing intent or malice here where there is none. | 14:34 |
anteaya | I also feel noone seems to care | 14:34 |
anteaya | the word cares it | 14:35 |
*** doug-fish has quit IRC | 14:35 | |
jaypipes | anteaya: the component is a shotgun-effect broadcaster. | 14:35 |
anteaya | the most I can ascribe to intent it the lack of awareness | 14:35 |
jaypipes | anteaya: if you want to propose renaming it, I have no problem with that. | 14:35 |
*** geoffarnold is now known as geoffarnoldX | 14:35 | |
anteaya | I think it is more awareness | 14:35 |
*** ajmiller has joined #openstack-infra | 14:35 | |
anteaya | and understanding of the nature of names | 14:35 |
jaypipes | anteaya: but I will point out that this is not something specific to Mirantis. see my comment above about "grenade", which sdague and dtroyer should be familiar with. | 14:36 |
anteaya | jaypipes: I am not saying miranits is alone in this | 14:36 |
anteaya | just they happen to have sent the email to which I am referencing | 14:36 |
dtroyer | I know the origin of that name and will leave it at that…I was overruled | 14:36 |
anteaya | and you happen to work this them | 14:36 |
zxiiro | pelix: merged | 14:36 |
anteaya | I think this behooves a larger conversation | 14:37 |
anteaya | around trust | 14:37 |
anteaya | and working towards it | 14:37 |
*** amotoki has joined #openstack-infra | 14:37 | |
anteaya | and selecting thought forms supportive of it | 14:37 |
pelix | zxiiro: tks :) | 14:37 |
*** scheuran has joined #openstack-infra | 14:37 | |
zxiiro | pelix: reviewing the 2nd one now | 14:37 |
anteaya | trust is getting some airtime now, which is nice to see | 14:37 |
jaypipes | anteaya: I understand and respect your point of view, but I do not share the same degree of concern. | 14:37 |
anteaya | I think as a group we have agreement that is what we want | 14:38 |
*** geoffarnoldX has quit IRC | 14:38 | |
anteaya | jaypipes: that is fine but do you feel it is a word that supports and fosters an enviroment of trust | 14:38 |
openstackgerrit | Merged openstack-infra/jenkins-job-builder: Allow save to be called more than once https://review.openstack.org/226331 | 14:38 |
pelix | zxiiro: take a look at the console output from other test runs, you'll see the main benefit there, to test you'll probably want to download and deliberately break some tests that log output | 14:38 |
anteaya | because I think we as a group need to start becoming conscious of needing to do so | 14:38 |
anteaya | and not take it for granted | 14:38 |
anteaya | as it is eroding | 14:38 |
anteaya | that much I believe as a group we can also see | 14:39 |
sdague | ok, new grenade patch set up to hopefully work around the glance issue | 14:39 |
jaypipes | anteaya: It is not something I have spent much time considering. | 14:39 |
dtroyer | I am concerned about a growing speech censorship and 'correctness' that has entered our community. part of the corporatization that Termie talked about in YVR. while possibly unavoidable, it is also sad to see intolerance and imposition of specific views onto the wider community | 14:39 |
*** geoffarnold has joined #openstack-infra | 14:39 | |
anteaya | jaypipes: could you do me the honour of perhaps setting aside a few minutes this week to perhaps consider it? | 14:39 |
sdague | fungi: the devstack-gate is rechecked on top of this | 14:39 |
anteaya | jaypipes: I'd be grateful if you could | 14:40 |
sdague | so... maybe this time ? | 14:40 |
anteaya | dtroyer: good point | 14:40 |
anteaya | dtroyer: how do we grow a culture of trust? | 14:40 |
anteaya | dtroyer: I think that is the larger issue | 14:40 |
waynr | pelix: i reviewed the test logging fixture change last week and forgot to +2 it, definitely looking forward to that one! | 14:41 |
anteaya | you can point to me as someone you feel is censoring speach if you wish and you would have a point | 14:41 |
anteaya | but have you considered why? | 14:41 |
*** doug-fish has joined #openstack-infra | 14:41 | |
anteaya | have you thought about what is being lost | 14:41 |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for libvirt firewall race bug 1501366 https://review.openstack.org/229460 | 14:41 |
openstack | bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed] https://launchpad.net/bugs/1501366 | 14:41 |
anteaya | by condoning growing use of words of violence and agression? | 14:42 |
dtroyer | I understand why, and you totally have that responsibility if you feel stringly to make it known. but it is a two-way street and when someone you disagree with takes your process and uses it to their advantage, … | 14:42 |
*** shardy_ has joined #openstack-infra | 14:42 | |
anteaya | takes my process and uses it to their advantage, I havne't seen that, have you any examples? | 14:42 |
openstackgerrit | Rikard Anglerud proposed openstack-infra/jenkins-job-builder: Add credentialsId support to the StashNotifier https://review.openstack.org/227257 | 14:43 |
anteaya | and I agree actions are a two-way street | 14:43 |
anteaya | balance must be maintained in all things | 14:43 |
*** Hal has joined #openstack-infra | 14:44 | |
jaypipes | anteaya: my best advice would be to start the conversation on the ML and/or submit a resolution to the governance repository that the TC can vote on that talks about avoiding violent or aggressive naming of projects or components. | 14:44 |
*** shardy has quit IRC | 14:44 | |
anteaya | I think that is avoiding the personal responsibility we all have | 14:44 |
anteaya | my trying to make this "my" problem | 14:45 |
jaypipes | anteaya: I don't have a responsibility to agree with you. I have a responsibility to consider your viewpoints. | 14:45 |
*** tsekiyama has joined #openstack-infra | 14:45 | |
*** dramalho has quit IRC | 14:45 | |
anteaya | jaypipes: you don't even have a responsiblity to consider my viewpoint | 14:45 |
jaypipes | anteaya: the onus is on you to submit a resolution and build support for your ideas, no? | 14:45 |
anteaya | you can choose to if you wish | 14:45 |
stevemar | dtroyer: so we're going to make a whole bunch of osc changes now :D | 14:45 |
stevemar | dtroyer: moar image commands! | 14:45 |
anteaya | jaypipes: well you said you hadn't considered the notion of making choices to support and environment of trust | 14:46 |
*** doug-fish has quit IRC | 14:46 | |
anteaya | jaypipes: you don't have to consider it if you choose not to | 14:46 |
anteaya | I had asked you to, but you are welcome to refuse | 14:46 |
jaypipes | anteaya: no, that's not what I mean. I mean that I do not agree with you that "shotgun" is offensive. I *do* agree with you that we should be building a culture of trust and respect. | 14:47 |
*** doug-fish has joined #openstack-infra | 14:47 | |
anteaya | jaypipes: because if you as a person aren't willing to take the time to think about it why should I waste my energy proposing a resolution | 14:47 |
anteaya | which won't do anythhing anyway | 14:47 |
anteaya | jaypipes: great thank you | 14:47 |
jaypipes | anteaya: I just don't happen to believe that the term "shotgun" (or "grenade" or "zuul" for that matter) in any way reduces the level of trust that our community displays to and with each othger. | 14:48 |
*** shardy_ has quit IRC | 14:48 | |
anteaya | I think if we are successful at building a culture of trust and respect that such proposals against certain words are a waste of time and focus | 14:48 |
zxiiro | pelix: I miss the diff view that it shows if your test fails | 14:48 |
zxiiro | pelix: anyway to keep it? | 14:48 |
anteaya | jaypipes: wonderful, then what supports can you point to that do foster trust? | 14:48 |
anteaya | and how can we help group energy support them? | 14:49 |
*** shardy has joined #openstack-infra | 14:49 | |
*** mtanino has joined #openstack-infra | 14:49 | |
zxiiro | pelix: instead of a diff, i now get "TypeError: 'NoneType' object is not iterable" for all failures which seems to be too generic. Unless my local setup is missing some new dependencies? | 14:50 |
*** dramalho has joined #openstack-infra | 14:50 | |
*** mpaolino has joined #openstack-infra | 14:51 | |
waynr | zxiiro pelix actually i just rand it locally and i am seeing some strangeness also, my +2 was a little premature... | 14:51 |
*** mpaolino has quit IRC | 14:51 | |
*** doug-fish has quit IRC | 14:51 | |
jaypipes | anteaya: I wish I had answers for you. Unfortunately, I don't right now. | 14:51 |
waynr | I am seeing significantly fewer tests actually run when all i do is change the expect xml output of a single test fixture | 14:52 |
anteaya | jaypipes: neither do I | 14:52 |
*** mpaolino has joined #openstack-infra | 14:52 | |
waynr | will comment with what i am seeing in-line | 14:52 |
anteaya | jaypipes: and that make me sad | 14:52 |
*** ajmiller_ has joined #openstack-infra | 14:52 | |
*** ccarrara has joined #openstack-infra | 14:52 | |
anteaya | jaypipes: but thank you for talking with me about it | 14:52 |
anteaya | jaypipes: I appreciate your time | 14:52 |
jaypipes | of course, and I appreciate yours as well | 14:52 |
*** tjones has joined #openstack-infra | 14:53 | |
*** bswartz has quit IRC | 14:53 | |
*** tjones has left #openstack-infra | 14:54 | |
*** maishsk has quit IRC | 14:54 | |
*** bltavares has quit IRC | 14:54 | |
*** kiran-r has joined #openstack-infra | 14:55 | |
*** ajmiller has quit IRC | 14:55 | |
*** sabeen has joined #openstack-infra | 14:56 | |
sdague | dtroyer: so... this definitely exposes osc testing deficiencies | 14:57 |
*** Sree has quit IRC | 14:57 | |
pelix | waynr: I'll have a look at that, this should only be capturing the logs and outputting them along with the failure | 14:57 |
*** Sree has joined #openstack-infra | 14:57 | |
*** dangers_away is now known as dangers | 14:57 | |
*** Sree_ has joined #openstack-infra | 14:59 | |
dtroyer | sdague: those are legion unfortunately | 14:59 |
*** Sree_ is now known as Guest42276 | 14:59 | |
waynr | it looks like maybe the LoggingFixture class isn't quite set up correctly | 14:59 |
*** mgrosz has joined #openstack-infra | 14:59 | |
*** sabeen3 has joined #openstack-infra | 14:59 | |
*** kushal has quit IRC | 14:59 | |
waynr | okay time to go to work | 15:00 |
*** doug-fish has joined #openstack-infra | 15:00 | |
*** sabeen has quit IRC | 15:00 | |
*** geoffarnold has quit IRC | 15:00 | |
*** geoffarn_ has joined #openstack-infra | 15:01 | |
*** mgrosz has quit IRC | 15:01 | |
*** r-daneel has joined #openstack-infra | 15:01 | |
pelix | waynr zxiiro: guessing the problem is with tests failing that didn't cause any logging output | 15:01 |
*** hashar has quit IRC | 15:01 | |
*** Sree has quit IRC | 15:02 | |
*** fhubik is now known as fhubik_brb | 15:02 | |
*** rfolco has quit IRC | 15:02 | |
*** dkehn has quit IRC | 15:03 | |
*** Guest42276 has quit IRC | 15:03 | |
*** rfolco has joined #openstack-infra | 15:04 | |
openstackgerrit | Elena Ezhova proposed openstack/requirements: Update oslo.service version to 0.9.0 https://review.openstack.org/229469 | 15:04 |
*** mpaolino has quit IRC | 15:04 | |
*** signed8bit has joined #openstack-infra | 15:04 | |
*** jsavak has quit IRC | 15:05 | |
*** jsavak has joined #openstack-infra | 15:05 | |
*** dkehn has joined #openstack-infra | 15:05 | |
*** jistr has quit IRC | 15:07 | |
*** pblaho has quit IRC | 15:08 | |
*** sdake has quit IRC | 15:08 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for libvirt firewall race bug 1501366 https://review.openstack.org/229460 | 15:09 |
openstack | bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed] https://launchpad.net/bugs/1501366 | 15:09 |
*** asmigala has quit IRC | 15:09 | |
*** sdake has joined #openstack-infra | 15:10 | |
*** glauco_ has joined #openstack-infra | 15:10 | |
*** glauco_ has quit IRC | 15:10 | |
*** glauco_ has joined #openstack-infra | 15:11 | |
*** bltavares has joined #openstack-infra | 15:12 | |
*** maishsk has joined #openstack-infra | 15:12 | |
*** hemnafk is now known as hemna | 15:14 | |
*** glauco___ has quit IRC | 15:14 | |
openstackgerrit | Jim Rollenhagen proposed openstack-infra/project-config: Add coverage post job to ironic-lib https://review.openstack.org/229473 | 15:16 |
*** dims_ has joined #openstack-infra | 15:16 | |
*** kiran-r has quit IRC | 15:17 | |
*** mpaolino has joined #openstack-infra | 15:18 | |
*** otter768 has joined #openstack-infra | 15:18 | |
*** amotoki has quit IRC | 15:18 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Nothing to see here https://review.openstack.org/111011 | 15:18 |
*** bharathm has quit IRC | 15:19 | |
pabelanger | yolanda: timrc: mind reviewing https://review.openstack.org/#/c/226986/ ? Adds code coverage for requests | 15:19 |
pleia2 | good morning | 15:19 |
*** fhubik_brb is now known as fhubik | 15:19 | |
pabelanger | pleia2: o/ | 15:19 |
*** mgrosz has joined #openstack-infra | 15:19 | |
timrc | pabelanger, Will look in just a minute. | 15:20 |
*** kiran-r has joined #openstack-infra | 15:20 | |
*** dims has quit IRC | 15:20 | |
yolanda | sure | 15:20 |
pabelanger | timrc: yolanda: mostly adds missing requests dependency and starts the mock process | 15:20 |
*** geoffarn_ has quit IRC | 15:21 | |
*** links has quit IRC | 15:21 | |
pabelanger | pleia2: if you have some time, looking for help getting https://review.openstack.org/#/c/226047/ and https://review.openstack.org/#/c/226002/ approved. Lacking core reviewers and reluctant to self approve them | 15:21 |
pleia2 | pabelanger: sure, I'll have a look | 15:22 |
*** geoffarnold has joined #openstack-infra | 15:22 | |
*** otter768 has quit IRC | 15:23 | |
*** dramalho has quit IRC | 15:24 | |
openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Capture logs with FakeLogging Fixture https://review.openstack.org/226107 | 15:27 |
*** flepied has quit IRC | 15:27 | |
pelix | waynr zxiiro: think I've got that one fixed now, was trying too hard previously | 15:27 |
*** jtomasek has quit IRC | 15:27 | |
*** signed8bit is now known as signed8bit_ZZZzz | 15:28 | |
*** julim has quit IRC | 15:28 | |
*** mgrosz has quit IRC | 15:30 | |
*** jsavak has quit IRC | 15:30 | |
*** jsavak has joined #openstack-infra | 15:31 | |
krotscheck | fungi: Well, you can always approve https://review.openstack.org/#/c/227019/ rather than https://review.openstack.org/#/c/229380/ or https://review.openstack.org/#/c/229381/ | 15:31 |
*** odgrim has joined #openstack-infra | 15:31 | |
*** julim has joined #openstack-infra | 15:31 | |
sdague | fungi: fyi, I'm going to take all the d-g / grenade branch flips issues over to #openstack-qa | 15:32 |
*** Sukhdev_ has joined #openstack-infra | 15:32 | |
openstackgerrit | Merged openstack-infra/grafyaml: Add a slug to better track dashboards https://review.openstack.org/226047 | 15:33 |
fungi | sdague: thanks, ping me there if there are any remaining patches which need to get prioritized | 15:33 |
*** tpeoples has joined #openstack-infra | 15:33 | |
sdague | fungi: will do | 15:34 |
*** amotoki has joined #openstack-infra | 15:35 | |
*** julim has quit IRC | 15:36 | |
*** zeih has quit IRC | 15:37 | |
*** julim has joined #openstack-infra | 15:37 | |
*** Swami has joined #openstack-infra | 15:38 | |
*** signed8bit_ZZZzz is now known as signed8bit | 15:38 | |
*** jogo has quit IRC | 15:39 | |
*** pcrews_ has quit IRC | 15:40 | |
openstackgerrit | Paul Michali proposed openstack-infra/project-config: Make new functional jobs voting https://review.openstack.org/229488 | 15:40 |
openstackgerrit | Jim Rollenhagen proposed openstack-infra/project-config: Add coverage post job to ironic-lib https://review.openstack.org/229473 | 15:40 |
*** jogo has joined #openstack-infra | 15:40 | |
*** pcrews has joined #openstack-infra | 15:41 | |
*** marzif has joined #openstack-infra | 15:41 | |
*** tpeoples has left #openstack-infra | 15:41 | |
*** Daisy has joined #openstack-infra | 15:42 | |
anteaya | mestery: working on your question on the ml | 15:43 |
*** markvan_ has joined #openstack-infra | 15:43 | |
*** salv-orl_ has joined #openstack-infra | 15:43 | |
*** geoffarn_ has joined #openstack-infra | 15:43 | |
*** doug-fis_ has joined #openstack-infra | 15:43 | |
*** geoffarnold has quit IRC | 15:43 | |
*** mwagner_ has quit IRC | 15:44 | |
*** mriedem1 has joined #openstack-infra | 15:44 | |
*** markvan__ has joined #openstack-infra | 15:44 | |
*** mriedem has quit IRC | 15:44 | |
anteaya | mestery: so far all but one of the gerrit acl files give tag permissions to neutron-release | 15:44 |
anteaya | mestery: this is the only outlier: http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerrit/acls/openstack/networking-powervm.config | 15:44 |
*** doug-fi__ has joined #openstack-infra | 15:44 | |
*** adreznec_ has joined #openstack-infra | 15:44 | |
*** markvan has quit IRC | 15:44 | |
*** salv-orlando has quit IRC | 15:46 | |
clarkb | anteaya check if they have publish jobs configured | 15:46 |
*** doug-fish has quit IRC | 15:46 | |
anteaya | mestery: I think one issue might be pypi ownership | 15:46 |
anteaya | clarkb: thanks will do | 15:46 |
*** mpaolino has quit IRC | 15:46 | |
anteaya | mestery: if you see the job that worked: https://pypi.python.org/pypi/networking-onos/1.0.0 | 15:46 |
*** mrmartin has joined #openstack-infra | 15:46 | |
anteaya | mestery: Package Index Owner: openstackci, vikram.choudhary | 15:46 |
*** Daisy has quit IRC | 15:47 | |
*** adreznec has quit IRC | 15:47 | |
anteaya | mestery: if openstackci isn't named as an owner of the pypi project it won't accept the package our ci tries to offer it | 15:47 |
*** ccarrara has quit IRC | 15:47 | |
*** glauco_ has quit IRC | 15:47 | |
*** doug-fis_ has quit IRC | 15:48 | |
anteaya | mestery: okay for octavia it has openstackci as a pypi owner: Package Index Owner: openstackci, johnsom | 15:48 |
*** markvan_ has quit IRC | 15:48 | |
anteaya | and it has permissions for neutron-release to push tags: http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerrit/acls/openstack/octavia.config | 15:49 |
anteaya | mestery: but octavia doesn't have pypi jobs: http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n2923 | 15:49 |
*** dkehn has quit IRC | 15:50 | |
anteaya | mestery: whereas onos does, see publish-to-pypi: http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n2691 | 15:50 |
openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: Use xml instead of job description for is_managed https://review.openstack.org/225562 | 15:50 |
anteaya | mestery: I'll reply to the email | 15:50 |
*** maishsk has quit IRC | 15:51 | |
*** dkehn has joined #openstack-infra | 15:51 | |
*** arxcruz has quit IRC | 15:51 | |
nibalizer | krotscheck: awesome | 15:53 |
*** tiswanso has quit IRC | 15:53 | |
openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: Add delete-unmanaged feature https://review.openstack.org/221145 | 15:53 |
*** dosaboy_ is now known as dosaboy | 15:54 | |
mrmartin | nibalizer: I made a new patch for git_resource's commit issue: https://github.com/puppet-community/puppet-git_resource/pull/17 seems much simpler than the previous one. can you make a new release including this one? | 15:54 |
*** bpokorny has joined #openstack-infra | 15:54 | |
*** tiswanso has joined #openstack-infra | 15:55 | |
*** fhubik has quit IRC | 15:55 | |
*** bharathm has joined #openstack-infra | 15:55 | |
*** bdemers has quit IRC | 15:56 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: update grenade branch logic for mitaka https://review.openstack.org/229363 | 15:56 |
*** bdemers has joined #openstack-infra | 15:58 | |
*** ddieterl_ has joined #openstack-infra | 15:58 | |
*** ddieterl_ has quit IRC | 15:58 | |
*** ddieterly has quit IRC | 15:59 | |
*** hashar has joined #openstack-infra | 15:59 | |
*** hashar is now known as hasharMeeting | 15:59 | |
*** ddieterly has joined #openstack-infra | 15:59 | |
pleia2 | pabelanger: couldn't give me *easy* patches ;) I have to run off to a dr appt, I'll get to the 2nd upon my return if no one else does | 16:00 |
*** yamahata has joined #openstack-infra | 16:01 | |
*** sridhar_ram has joined #openstack-infra | 16:01 | |
*** mpaolino has joined #openstack-infra | 16:02 | |
anteaya | pleia2: hope all goes well | 16:02 |
pleia2 | anteaya: thanks! | 16:03 |
odgrim | pelix waynr: looks like you guys were busy, I patched some merge conflicts for that is_unmanaged fix and let jenkins take a whack at it. Seems ok so far | 16:03 |
*** geoffarn_ has quit IRC | 16:04 | |
*** marzif has quit IRC | 16:04 | |
*** geoffarnold has joined #openstack-infra | 16:04 | |
krotscheck | nibalizer: I know, right? | 16:05 |
*** erlon has joined #openstack-infra | 16:05 | |
*** jlanoux has quit IRC | 16:05 | |
*** tonytan4ever has joined #openstack-infra | 16:06 | |
*** bswartz has joined #openstack-infra | 16:06 | |
*** dims_ has quit IRC | 16:06 | |
*** armax has joined #openstack-infra | 16:06 | |
*** ifarkas has quit IRC | 16:07 | |
krotscheck | nibalizer: If your eyeballs have had enough coffee, I'd greatly appreciate a look at https://review.openstack.org/#/c/229380/ | 16:08 |
*** peristeri has quit IRC | 16:08 | |
openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: Add delete-unmanaged feature to supplement delete-old. https://review.openstack.org/221145 | 16:10 |
openstackgerrit | Merged openstack-infra/zuul: Document CRD https://review.openstack.org/228631 | 16:10 |
*** jlvillal has joined #openstack-infra | 16:10 | |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Move nv openstack-ansible experimental job to check queue https://review.openstack.org/228408 | 16:11 |
*** flepied has joined #openstack-infra | 16:13 | |
*** ddieterly has quit IRC | 16:13 | |
*** zhenguo has quit IRC | 16:14 | |
*** ddieterly has joined #openstack-infra | 16:14 | |
openstackgerrit | Merged stackforge/gertty: Make permalink clickable https://review.openstack.org/224851 | 16:16 |
*** ajmiller has joined #openstack-infra | 16:16 | |
*** amuller is now known as amuller_afk | 16:16 | |
*** olaph has quit IRC | 16:18 | |
*** olaph has joined #openstack-infra | 16:18 | |
*** trown is now known as trown|lunch | 16:18 | |
*** devvesa has quit IRC | 16:19 | |
*** alexsyip has joined #openstack-infra | 16:19 | |
zaro | morning | 16:20 |
anteaya | morning zaro | 16:21 |
pabelanger | pleia2: :) I did provide unit tests for all the code. So, I have that going for me | 16:22 |
*** gyee has joined #openstack-infra | 16:22 | |
tdurakov | bauzas, hi again | 16:22 |
*** mwagner_ has joined #openstack-infra | 16:22 | |
zaro | signed8bit: to answer your question. yes, gerrit-review.googlesource.com is the right place for gerrit plugins. send a msg to ML to request one of the cores to create a repo for you. | 16:22 |
zaro | rcarrillocruz: hi | 16:22 |
tdurakov | bauzas, oh, wrong channel... | 16:23 |
signed8bit | Thanks zaro! I will. | 16:23 |
*** dramalho has joined #openstack-infra | 16:25 | |
*** geoffarnold has quit IRC | 16:25 | |
*** geoffarnold has joined #openstack-infra | 16:25 | |
*** mrmartin has quit IRC | 16:26 | |
*** ala_ has quit IRC | 16:26 | |
fungi | zaro: if you haven't looked at e-mail yet, there's been several encouraging responses with potential willing help | 16:27 |
*** hasharMeeting is now known as hashar | 16:28 | |
*** hashar has quit IRC | 16:29 | |
*** ajmiller_ has quit IRC | 16:29 | |
*** glauco has joined #openstack-infra | 16:30 | |
waynr | pelix: awesome, will hopefully get another chance to check it out today | 16:30 |
*** dramalho has quit IRC | 16:30 | |
*** doug-fi__ is now known as doug-fish | 16:30 | |
zaro | fungi: ooo. checking now. | 16:33 |
*** hdd has quit IRC | 16:33 | |
waynr | odgrim: yeah i would have got to review your patch last night but noticed that my phone has not been receiving calls at all for the past month and had to remediate with my carrier | 16:33 |
*** bltavares has quit IRC | 16:33 | |
*** dims_ has joined #openstack-infra | 16:33 | |
*** bltavares has joined #openstack-infra | 16:34 | |
fungi | waynr: wow, you apparently use phones about as often as i do | 16:34 |
odgrim | waynr: At least the month was peaceful XD | 16:34 |
*** stevemar has quit IRC | 16:35 | |
*** bltavares has quit IRC | 16:35 | |
*** markus_z has quit IRC | 16:36 | |
waynr | ha yeah | 16:36 |
*** dzamboni has joined #openstack-infra | 16:36 | |
waynr | i only noticed because i was expecting a call from a friend | 16:36 |
waynr | and instead only received a new voicemail notification | 16:36 |
waynr | if every new voicemail notification i recieved for the past month was people calling me then i missed quite a few calls... | 16:36 |
*** armax has quit IRC | 16:37 | |
*** hdd has joined #openstack-infra | 16:37 | |
*** zeih has joined #openstack-infra | 16:38 | |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Use tripleo.sh to build packages https://review.openstack.org/229425 | 16:38 |
*** unicell has quit IRC | 16:39 | |
*** dramalho has joined #openstack-infra | 16:40 | |
zaro | anteaya: thanks for answering ML quesitons. | 16:40 |
anteaya | zaro: welcome, please add your responses as well | 16:41 |
anteaya | zaro: as soon as they show up in channel I'm just going to point them at you anyway | 16:42 |
openstackgerrit | Michael Johnson proposed openstack-infra/project-config: Add release jobs to the octavia project https://review.openstack.org/229518 | 16:43 |
anteaya | zaro: was just trying to open the door as wide as possible to get them to join the channel | 16:43 |
*** pc_m is now known as pc_m_AWAY | 16:43 | |
*** claudiub has quit IRC | 16:44 | |
anteaya | I have to run an errand for a bit this afternoon, back later | 16:44 |
openstackgerrit | Merged openstack-infra/project-config: Set cinder api to 1 in rally-gate-dsvm-cli job https://review.openstack.org/229264 | 16:44 |
*** bdemers has quit IRC | 16:45 | |
*** bdemers has joined #openstack-infra | 16:45 | |
openstackgerrit | Derek Higgins proposed openstack-infra/system-config: Remove unused node templates from tripleo cloud https://review.openstack.org/187676 | 16:45 |
*** notnownikki has quit IRC | 16:45 | |
*** odgrim has quit IRC | 16:47 | |
*** bdemers has quit IRC | 16:49 | |
*** bdemers has joined #openstack-infra | 16:49 | |
*** devlaps has joined #openstack-infra | 16:50 | |
*** placko_ has quit IRC | 16:51 | |
mmedvede | fungi: how many rollcall-votes would a spec need? I probably need more for https://review.openstack.org/#/c/194437/ | 16:51 |
*** cdent has quit IRC | 16:52 | |
*** mpaolino has quit IRC | 16:52 | |
*** harlowja has quit IRC | 16:52 | |
*** tdurakov has quit IRC | 16:53 | |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Add a new element hpdsa https://review.openstack.org/220888 | 16:55 |
*** flaper87 has quit IRC | 16:55 | |
*** flaper87 has joined #openstack-infra | 16:55 | |
openstackgerrit | Ari proposed openstack-infra/jenkins-job-builder: Implements: additions for OS3 plugin entry points in builders and scm https://review.openstack.org/229100 | 16:55 |
*** bdemers has quit IRC | 16:56 | |
*** dkehn has quit IRC | 16:56 | |
*** derekh has quit IRC | 16:57 | |
*** amuller_afk is now known as amuller | 16:57 | |
*** tiswanso has quit IRC | 16:57 | |
*** bltavares has joined #openstack-infra | 16:57 | |
*** achanda has joined #openstack-infra | 16:58 | |
*** bltavares has quit IRC | 16:59 | |
*** tiswanso has joined #openstack-infra | 16:59 | |
*** bltavares has joined #openstack-infra | 16:59 | |
*** scheuran has quit IRC | 17:01 | |
*** achanda has quit IRC | 17:01 | |
*** mudassirlatif_ has joined #openstack-infra | 17:01 | |
*** vkmc is now known as vkmc-afk | 17:03 | |
*** spzala has joined #openstack-infra | 17:03 | |
*** hemna is now known as hemnafk | 17:03 | |
*** dizquierdo has quit IRC | 17:03 | |
*** dkranz has quit IRC | 17:03 | |
*** dkehn has joined #openstack-infra | 17:04 | |
*** mudassirlatif has quit IRC | 17:04 | |
*** mudassirlatif_ is now known as mudassirlatif | 17:04 | |
*** bdemers has joined #openstack-infra | 17:04 | |
openstackgerrit | Matthew Treinish proposed openstack/os-testr: Force utf8 encoding on subunit attachments output https://review.openstack.org/229527 | 17:05 |
*** annegentle has joined #openstack-infra | 17:06 | |
*** geoffarnold has quit IRC | 17:06 | |
*** abitha has joined #openstack-infra | 17:07 | |
*** geoffarnold has joined #openstack-infra | 17:07 | |
*** trown|lunch is now known as trown | 17:07 | |
*** stevemar has joined #openstack-infra | 17:07 | |
fungi | mmedvede: there's no hard and fast rule. our team documentation says "a majority vote" which, given the size of the council, i think we've been interpreting as "a majority of those who vote" | 17:07 |
*** nadya has quit IRC | 17:08 | |
*** earlephilhower has joined #openstack-infra | 17:08 | |
fungi | so it's sort of at the ptl's discretion whether the spec needs to gather more consensus | 17:08 |
*** bdemers has quit IRC | 17:08 | |
fungi | that said, 3 council votes is on the low-ish side | 17:09 |
mmedvede | fungi: ok. I am trying to understand what my actions should be. E.g. just wait, or solicit more reviews | 17:09 |
openstackgerrit | Paul Belanger proposed openstack-infra/nodepool: Add dib success or failues to statsd https://review.openstack.org/226481 | 17:09 |
fungi | mmedvede: you are free to do either | 17:09 |
fungi | if we have a few (2? 3?) more rollcall +1 votes and no strong dissent tomorrow (or sometime thereafter when i get around to checking again) i'll approve it | 17:10 |
pabelanger | mmedvede: fungi: To me, once we get to the roll call vote, I don't see how it could fail. Since so much work has gone into the spec to start | 17:11 |
fungi | the deadline of 1900 utc tomorrow is for people to know that i won't approve it _before_ then so that they have time to register their opposition | 17:12 |
*** ihrachys has quit IRC | 17:12 | |
*** unicell has joined #openstack-infra | 17:12 | |
*** signed8bit is now known as signed8bit_ZZZzz | 17:12 | |
clarkb | my only weak opposition was the quota thing and I am told that we have quota now just need to confirm it | 17:13 |
*** sridhar_ram has quit IRC | 17:13 | |
mmedvede | fungi: oh, that makes more sense. | 17:13 |
clarkb | why dont I go do that now while I am thinking about it | 17:13 |
*** ashleighfarnham has joined #openstack-infra | 17:13 | |
*** ashleighfarnham has quit IRC | 17:13 | |
*** yamahata has quit IRC | 17:14 | |
*** ashleighfarnham has joined #openstack-infra | 17:14 | |
mmedvede | somehow I thought it is 19:00 utc today | 17:14 |
asselin | decided to try out gertty. First stacktrace: http://paste.openstack.org/show/474898/ | 17:14 |
*** e0ne has quit IRC | 17:14 | |
clarkb | I am still seeing the same quota limits on the openstackci account in rax | 17:15 |
clarkb | mordred: ^ | 17:15 |
*** signed8bit_ZZZzz is now known as signed8bit | 17:15 | |
clarkb | sorry rax dfw | 17:15 |
clarkb | so my preference would be to not commit to any work right now that requires new VMs | 17:15 |
mmedvede | clarkb: thanks for checking | 17:15 |
clarkb | fungi: ^ not sure if you want to approve and wait or what though | 17:16 |
*** KanagarajM has joined #openstack-infra | 17:16 | |
clarkb | or as an alternative we can start using the other regions for things more than test slaves and backups | 17:16 |
mmedvede | clarkb: I am ok with spec going in, as long as there is going to be resource later at some point | 17:16 |
openstackgerrit | Ari proposed openstack-infra/jenkins-job-builder: Implements: additions for OS3 plugin entry points in builders and scm https://review.openstack.org/229100 | 17:16 |
clarkb | mmedvede: well thats the question :) | 17:16 |
KanagarajM | Hi, I am looking for help to fix an issue facing in heat-template project http://logs.openstack.org/44/226244/3/check/gate-heat-templates-dsvm/8df60b1/logs/devstack-gate-setup-workspace-new.txt.gz#_2015-09-23_06_33_50_036 could any one please help here ? | 17:17 |
openstackgerrit | Michael Johnson proposed openstack-infra/project-config: Add release jobs to the octavia project https://review.openstack.org/229518 | 17:17 |
annegentle | anteaya: or anyone who knows jenkins templating, I have a Q about what I can fill into a template and from where. The use case is to insert "compute" instead of "nova" in an FTP publisher instruction. | 17:17 |
rcarrillocruz | zaro: heya, was chatting with darragh right now. There's no easy way to get the build history of a node in python-jenkins , is it? | 17:17 |
asselin | any reason it couldn't run on the same vm as other services? | 17:17 |
rcarrillocruz | i mean, what jobs have run on a node | 17:17 |
rcarrillocruz | success fail | 17:17 |
rcarrillocruz | etc | 17:17 |
asselin | mmedvede, clarkb ^ | 17:17 |
clarkb | asselin: we don't typically like tying sercies together like that | 17:17 |
zaro | rcarrillocruz: is there an REST api for that? | 17:18 |
clarkb | asselin: it makes upgrades and reboots trickier | 17:18 |
clarkb | asselin: and make puppet more complicated | 17:18 |
mmedvede | asselin: I am also concerned deploying something new with already existing service, you never know how stable it would be | 17:18 |
rcarrillocruz | dunno, haven't checked, just if you know off the top of your head | 17:18 |
rcarrillocruz | cos in the UI of Jenkins you can get that | 17:18 |
rcarrillocruz | but not sure if that's pulled from a REST call or via other mechanism | 17:18 |
fungi | mmedvede: when in doubt, http://eavesdrop.openstack.org/meetings/infra/2015/infra.2015-09-29-19.01.html | 17:19 |
clarkb | btw I am using nova quota-show which I think is going to give me correct info | 17:19 |
*** otter768 has joined #openstack-infra | 17:19 | |
annegentle | I see name, envlist, and node but nothing like "service" which is probably what I want. | 17:19 |
annegentle | hm | 17:19 |
*** abitha has quit IRC | 17:19 | |
fungi | clarkb: one thing worth revisiting now is whether, since the centos 7 upgrade, we might be able to scale back our git farm by ~50% | 17:20 |
*** abitha has joined #openstack-infra | 17:20 | |
clarkb | fungi: right but we can't replace git.o.o right now | 17:20 |
fungi | yeah, not without clearing more resources | 17:20 |
clarkb | fungi: so we need more quota to be able to perform operational tasks as is | 17:20 |
clarkb | more free quota either by bumping or reducing | 17:21 |
*** mriedem1 has quit IRC | 17:21 | |
zaro | rcarrillocruz: not off the top of my head but lemme look | 17:21 |
fungi | in the short term, if we do scale down the git farm, we could free up 32gb ram from four of those instances | 17:21 |
Piet | Sorry for the persistence and cross postings. We still need a handful folks currently usng Nova Networks to complete our survey. https://www.surveymonkey.com/r/osnetworking | 17:21 |
fungi | clarkb: which i think is the headroom we need for a replacement haproxy lb? | 17:22 |
clarkb | fungi: ya ~30GB for proxy | 17:22 |
krotscheck | The npm-run macro is still broken, could I get some core's to look at https://review.openstack.org/#/c/226596/ ? | 17:22 |
*** ociuhandu has quit IRC | 17:22 | |
clarkb | I am looking at cacti graphs now | 17:22 |
*** mriedem has joined #openstack-infra | 17:22 | |
*** lawrancejing has quit IRC | 17:23 | |
fungi | load averages drop by a factor of 2-3x at the point where we upgraded | 17:23 |
clarkb | ya you can see the cpu utilization graph falls off pretty decently on the yearly graph | 17:23 |
clarkb | looking at memory too | 17:23 |
*** jsavak has quit IRC | 17:23 | |
fungi | memory looks constant | 17:23 |
*** otter768 has quit IRC | 17:23 | |
*** jsavak has joined #openstack-infra | 17:24 | |
clarkb | that is what I see too | 17:24 |
clarkb | http://cacti.openstack.org/cacti/graph_image.php?action=view&local_graph_id=925&rra_id=4 | 17:24 |
fungi | and we're also not even using all available ram for buffers/cache on them currently | 17:24 |
*** abitha has quit IRC | 17:24 | |
*** stevemar has quit IRC | 17:24 | |
*** sergmelikyan has quit IRC | 17:24 | |
*** Sukhdev_ has quit IRC | 17:24 | |
*** bltavares has quit IRC | 17:24 | |
clarkb | so next step is likely just disabling 4 hosts in haproxy but not deleting them entirely | 17:25 |
clarkb | perhaps we should do that after relase though? | 17:25 |
*** abitha has joined #openstack-infra | 17:25 | |
fungi | yeah, may not want to anger the release managers unnecessarily ;) | 17:25 |
openstackgerrit | Ramy Asselin proposed openstack-infra/project-config: Add an example project-config repo https://review.openstack.org/228540 | 17:25 |
*** AJaeger has joined #openstack-infra | 17:26 | |
*** kgiusti has joined #openstack-infra | 17:26 | |
*** stevemar has joined #openstack-infra | 17:26 | |
*** abitha has quit IRC | 17:27 | |
AJaeger | anteaya, pleia2, clarkb: I have one more translation request - enable Ironic for Liberty. Could you review before the next import run (tomorrow 6am), please? https://review.openstack.org/229252 | 17:27 |
fungi | clarkb: interestingly, eth0 traffic also drops considerable starting at the upgrade... and i'm not sure that's entirely release related since we don't have a similar lull in april | 17:28 |
*** jsavak has quit IRC | 17:28 | |
*** abitha has joined #openstack-infra | 17:28 | |
*** geoffarn_ has joined #openstack-infra | 17:28 | |
fungi | so maybe newer git is not only more cpu-efficient but also more network-efficient? | 17:28 |
clarkb | AJaeger: approved (I single +2'd but those changes are becoming pretty standard at the end of the release now) | 17:28 |
*** geoffarnold has quit IRC | 17:28 | |
AJaeger | thanks, clarkb ! | 17:28 |
KanagarajM | clarkb: I am facing an jenkins failure and looking for help, could you please help to fix http://logs.openstack.org/44/226244/3/check/gate-heat-templates-dsvm/8df60b1/logs/devstack-gate-setup-workspace-new.txt.gz#_2015-09-23_06_33_50_036 ? | 17:29 |
*** jsavak has joined #openstack-infra | 17:29 | |
clarkb | fungi: that could be. Negotiates better compression or uses better defaults for compression? | 17:29 |
openstackgerrit | Michael Johnson proposed openstack-infra/project-config: Add release jobs to the octavia project https://review.openstack.org/229518 | 17:29 |
zaro | rcarrillocruz: you can do this https://stackoverflow.com/questions/30882670/list-all-jobs-which-were-running-on-a-specific-slave-node | 17:29 |
fungi | clarkb: no, wait, it doesn't drop on the lb. i think i'm looking at where we scaled out the farm | 17:29 |
AJaeger | anteaya, pleia2: clarkb already approved... | 17:29 |
zaro | rcarrillocruz: python-jenkins has a run_script method that allows you to run groovy scripts | 17:29 |
rcarrillocruz | oh | 17:29 |
rcarrillocruz | the same way with Jenkins CLI you can run groovy | 17:30 |
rcarrillocruz | excellent! | 17:30 |
rcarrillocruz | thx sir :-) | 17:30 |
fungi | clarkb: so we may be mixing multiple performance-altering changes there around the same timeframe | 17:30 |
rcarrillocruz | i'll go down that route | 17:30 |
krotscheck | (ping #2) The npm-run macro is still broken, could I get some eyeballs on https://review.openstack.org/#/c/226596/ ? | 17:30 |
clarkb | KanagarajM: looks like you have heat-templates listed in PROJECTS twice, just remove the second one | 17:30 |
mestery | anteaya fungi: Thanks for the email reply on the gerrit tag issue! I had to step out to run some errands after I sent that, I'm digesting now and will get this fixed up. Much appreciated! | 17:30 |
*** odgrim has joined #openstack-infra | 17:31 | |
fungi | clarkb: ahh, actually i do see the drop on the lb network graph too... http://cacti.openstack.org/cacti/graph.php?action=zoom&local_graph_id=862&rra_id=4&view_type=&graph_start=1435125210&graph_end=1443284661&graph_height=120&graph_width=500&title_font_size=10 | 17:31 |
clarkb | rcarrillocruz: ou can just make the http get for it | 17:31 |
*** tlian has quit IRC | 17:31 | |
clarkb | rcarrillocruz: iirc thats how some of sdague's graphing tools worked | 17:31 |
clarkb | rcarrillocruz: it comes back as json iirc | 17:32 |
*** mudassirlatif has quit IRC | 17:32 | |
*** yamahata has joined #openstack-infra | 17:32 | |
sdague | yeh, you can get json ifyou like | 17:32 |
clarkb | fungi: looks like 40% ish too thats neat | 17:32 |
openstackgerrit | Paul Belanger proposed openstack-infra/nodepool: Add dib success or failues to statsd https://review.openstack.org/226481 | 17:32 |
fungi | clarkb: yep, very encouraging! | 17:32 |
sdague | rcarrillocruz: https://github.com/sdague/tempest-timings | 17:33 |
sdague | examples of using the json api in jenkins | 17:33 |
pabelanger | fungi: clarkb: do you know when we can land the coverage parsing fix for HTML? | 17:34 |
pabelanger | not sure the review number | 17:34 |
clarkb | fungi: so ya I think a good goal/plan for immediately after realse would be to scale back the cluster but leave the hosts in place for a bit | 17:34 |
sdague | clarkb: gdi, now I have python 2.7.9 and it cares about the certs :) | 17:34 |
KanagarajM | clarkb: sorry, i am not sure whether i got your point. is there a bug in the jenkins job for heat-template? | 17:34 |
*** mgrosz has joined #openstack-infra | 17:35 | |
sdague | is there a way to make python not care? | 17:35 |
clarkb | pabelanger: I aksed jhesketh to update the tests to cover it if thats done then I ma happy to +2 it | 17:35 |
clarkb | sdague: hahaha, I am not sure | 17:35 |
clarkb | sdague: but at least you dont get insecure platform warnings now | 17:35 |
clarkb | KanagarajM: let me look more closely | 17:35 |
zaro | waynr: clojure is probably unlikely for core gerrit, but might be possible for it's plugins. | 17:35 |
signed8bit | zaro: I'll be in Tokyo for any gerrit chat | 17:36 |
mgrosz | Hello, I will be happy to receive another +2 to me change https://review.openstack.org/#/c/229259/ and continue with the initialization of the new Vitrage project. Thanks | 17:36 |
openstackgerrit | Merged openstack-infra/project-config: Translate Ironic for Liberty as well https://review.openstack.org/229252 | 17:36 |
clarkb | KanagarajM: https://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/heat.yaml#n21 and https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/devstack-vm-gate-wrap.sh#n57 conflict with each other. One will have to be removed | 17:36 |
zaro | signed8bit: ahh cool, lets plan to meetup. | 17:36 |
clarkb | pabelanger: looks like https://review.openstack.org/#/c/226858/ was updated, I am reviewing it now | 17:37 |
KanagarajM | clarkb: sure. looking them | 17:37 |
signed8bit | zaro: for sure! | 17:37 |
*** achanda has joined #openstack-infra | 17:37 | |
*** tlian has joined #openstack-infra | 17:37 | |
*** pfallenop has quit IRC | 17:38 | |
mgrosz | Someone? | 17:38 |
*** pfallenop has joined #openstack-infra | 17:38 | |
*** vkmc-afk is now known as vkmc | 17:38 | |
*** vkmc has quit IRC | 17:38 | |
*** vkmc has joined #openstack-infra | 17:38 | |
*** dahoo has joined #openstack-infra | 17:38 | |
*** shashank_hegde has joined #openstack-infra | 17:40 | |
*** dramalho has quit IRC | 17:40 | |
yamamoto_ | jeblair: hi | 17:40 |
pabelanger | clarkb: great, checking it out too | 17:40 |
clarkb | KanagarajM: I think removing it from the job (the first link) is going to be safest | 17:41 |
clarkb | KanagarajM: other jobs may use heat-templates via the line in devstack-gate | 17:41 |
*** ZZelle_ has joined #openstack-infra | 17:41 | |
clarkb | KanagarajM: but I want to reduce the list in devstack-gate over time tomake it more minimal. Haven't started that work as I expect it to be disruptive at times and the release is more important right now | 17:42 |
nibalizer | krotscheck: looking | 17:42 |
rcarrillocruz | nice, looks like have a couple vectors to tackle it, thx sdague clarkb | 17:42 |
*** bltavares has joined #openstack-infra | 17:44 | |
KanagarajM | clarkb: sure. i will remove the entry from heat.yaml and sure. thanks. | 17:44 |
*** mrmartin has joined #openstack-infra | 17:44 | |
krotscheck | nibalizer: thanks | 17:44 |
*** HeOS has quit IRC | 17:44 | |
mgrosz | Hello, I will be happy to receive another +2 to me change https://review.openstack.org/#/c/229259/ and continue with the initialization of the new Vitrage project. Thanks | 17:45 |
clarkb | mgrosz: adding new projects is sort of a low priority right now during release. We are focused on getting upgrade testing working, translations imported, debugging the last few major bugs we have found etc | 17:45 |
*** armax has joined #openstack-infra | 17:45 | |
clarkb | mgrosz: reviewers should get around to that in time | 17:45 |
mgrosz | clarkb: Thank you. | 17:46 |
*** kushal has joined #openstack-infra | 17:46 | |
*** dramalho has joined #openstack-infra | 17:48 | |
*** geoffarn_ has quit IRC | 17:49 | |
*** geoffarnold has joined #openstack-infra | 17:49 | |
*** rossella_s has joined #openstack-infra | 17:50 | |
*** mrmartin has quit IRC | 17:50 | |
*** dprince has quit IRC | 17:50 | |
*** pelix has quit IRC | 17:51 | |
alivigni | zaro# When you get a chance can you look at this review - https://review.openstack.org/#/c/229100/ | 17:51 |
zaro | rcarrillocruz: not sure if this is waht you want, but can also get node build history status with this api <host>/computer/linux-slave2/api/json?depth=3 | 17:51 |
fungi | wow, namecheap has "improved" their webui in such a way now that you can't see a list of active certs | 17:52 |
zaro | rcarrillocruz: pytho-jenkins get_node_info() does that. | 17:52 |
alivigni | zxiiro# Can you have a look at this review when you get a chance - https://review.openstack.org/#/c/229100/ | 17:52 |
*** michaelxin has joined #openstack-infra | 17:52 | |
clarkb | fungi: its like dns webuis that don't show you the full ipv6 addr on AAAA records :) | 17:52 |
fungi | indeed | 17:52 |
fungi | in this case it shows me that i have 10 active ssl certs issued for openstack.org subdomains, just not what those subdomains are | 17:53 |
fungi | i have to go 2 clicks deep into another page for each of them to find out the cn | 17:53 |
sdague | man, it is amazing how much slower HP nodes are than RAX ones on the neutron jobs | 17:55 |
sdague | http://dl.dropbox.com/u/6514884/gate-tempest-dsvm-neutron-full.png | 17:55 |
*** mrmartin has joined #openstack-infra | 17:55 | |
*** nadya has joined #openstack-infra | 17:56 | |
asselin | fungi, are you ok with this patch? 228540 | 17:56 |
*** stevemar has quit IRC | 17:57 | |
*** bcourt has quit IRC | 17:57 | |
*** stevemar has joined #openstack-infra | 17:57 | |
*** mrmartin has quit IRC | 17:58 | |
*** doug-fis_ has joined #openstack-infra | 18:00 | |
*** michaelxin has quit IRC | 18:00 | |
*** stevemar has quit IRC | 18:01 | |
fungi | asselin: sure, i was hoping my +1 on the related governance change would be sufficient to indicate i was okay with the addition | 18:03 |
asselin | nibalizer, clarkb this change should be safe to merge (what we discussed yesterday) https://review.openstack.org/#/c/229162/4 | 18:03 |
*** doug-fish has quit IRC | 18:04 | |
clarkb | sdague: I wonder if they would be more receptive to our feedback now? | 18:04 |
*** doug-fis_ has quit IRC | 18:04 | |
*** hemnafk is now known as hemna | 18:04 | |
asselin | fungi, thanks | 18:04 |
*** lg782s has joined #openstack-infra | 18:05 | |
*** rossella_s has quit IRC | 18:06 | |
*** rossella_s has joined #openstack-infra | 18:06 | |
openstackgerrit | Danilo Ramalho proposed openstack-infra/puppet-gerrit: Setup preconditions for acceptance tests. https://review.openstack.org/219400 | 18:07 |
*** KanagarajM has quit IRC | 18:07 | |
*** dprince has joined #openstack-infra | 18:07 | |
openstackgerrit | Merged openstack-infra/project-config: Create the new project 'git-repo-manifests' https://review.openstack.org/228004 | 18:07 |
*** KanagarajM has joined #openstack-infra | 18:08 | |
openstackgerrit | Kanagaraj Manickam proposed openstack-infra/project-config: heat-template dsvm failure https://review.openstack.org/229555 | 18:08 |
nibalizer | asselin: fungi lgtm but I'm not in a position to babysit | 18:08 |
*** Rockyg has joined #openstack-infra | 18:08 | |
*** amotoki has quit IRC | 18:09 | |
*** pc_m_AWAY is now known as pc_m | 18:09 | |
*** austin81 has joined #openstack-infra | 18:10 | |
openstackgerrit | Merged openstack-infra/project-config: Add an example project-config repo https://review.openstack.org/228540 | 18:10 |
openstackgerrit | Merged openstack-infra/project-config: IPv6 only services neutron experimental job https://review.openstack.org/226013 | 18:10 |
*** geoffarnold has quit IRC | 18:10 | |
docaedo | fungi: just reminding you about SSL cert for app catalog, related to https://review.openstack.org/226984 :) | 18:10 |
*** geoffarnold has joined #openstack-infra | 18:11 | |
sdague | clarkb: good question | 18:11 |
*** dmsimard is now known as dmsimard|afk | 18:11 | |
fungi | docaedo: yep, i'm working on that now. have a couple i need to order and domaincheap has done things to make this much less easy | 18:11 |
openstackgerrit | Kanagaraj Manickam proposed openstack-infra/project-config: heat-template dsvm failure https://review.openstack.org/229557 | 18:11 |
openstackgerrit | Ari proposed openstack-infra/jenkins-job-builder: Implements: additions for OS3 plugin entry points in builders and scm https://review.openstack.org/229100 | 18:11 |
docaedo | fungi: oh great that you're working on it! sorry the process is less easy | 18:11 |
*** harlowja has joined #openstack-infra | 18:13 | |
*** e0ne has joined #openstack-infra | 18:13 | |
*** doug-fish has joined #openstack-infra | 18:13 | |
*** mgrosz has quit IRC | 18:15 | |
openstackgerrit | Merged openstack-infra/project-config: Added npm publish job. https://review.openstack.org/199715 | 18:16 |
*** openstackgerrit has quit IRC | 18:16 | |
*** openstackgerrit has joined #openstack-infra | 18:16 | |
*** austin81 has left #openstack-infra | 18:17 | |
*** mwagone has joined #openstack-infra | 18:19 | |
*** austin81 has joined #openstack-infra | 18:20 | |
*** bcourt has joined #openstack-infra | 18:22 | |
*** ybathia has joined #openstack-infra | 18:22 | |
*** e0ne has quit IRC | 18:23 | |
*** mudassirlatif has joined #openstack-infra | 18:23 | |
*** rossella_s has quit IRC | 18:23 | |
*** maishsk has joined #openstack-infra | 18:24 | |
asselin | nibalizer, upstream is not yet using openstackci::nodepool, so no baby-sitting is needed | 18:24 |
openstackgerrit | Merged openstack-infra/project-config: Added new Openstack repository for Fuel Cinder Block Device plugin integration https://review.openstack.org/227230 | 18:24 |
openstackgerrit | Merged openstack-infra/project-config: Changes: Vitrage - Initialize new OpenStack project https://review.openstack.org/229259 | 18:24 |
nibalizer | asselin: oh okay | 18:25 |
nibalizer | +a'd | 18:25 |
clarkb | hrm I guess existing projct-config isnt a good example? | 18:25 |
asselin | clarkb, it's too much for newbies | 18:26 |
*** signed8bit is now known as signed8bit_ZZZzz | 18:26 | |
openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Add pypi publishing jobs for neutron sub-projects https://review.openstack.org/229564 | 18:27 |
*** mgrosz has joined #openstack-infra | 18:27 | |
*** sridhar_ram has joined #openstack-infra | 18:28 | |
clarkb | asselin maybe we could provide docs in that repo or an example subdir? | 18:29 |
clarkb | I really dislike "new repo for everything" I personally find that confusing to deal with though tools like hound simplify | 18:29 |
*** e0ne has joined #openstack-infra | 18:29 | |
mgrosz | Hey, I am following the documentation http://docs.openstack.org/infra/manual/creators.html to initialize a new project - Vitrage. I am currently in the "Wait Here" stage, waiting for the infra team to finilize the import and configure the repository. | 18:29 |
mgrosz | To what exactly do I wait? an email? else? | 18:30 |
*** abregman|pto has quit IRC | 18:30 | |
mgrosz | My new project configuration was merged successfuly to the project-config | 18:30 |
asselin | clarkb, the idea was to have a working simple project-config people can use directly as their initial project-config git url | 18:30 |
clarkb | so its not an example its a real project-config? | 18:31 |
asselin | clarkb, it's real in the sence that it's a working version. It's an example in the sence that it needs to be tailored to your environment. | 18:31 |
mgrosz | "Update the Gerrit Group Members" - who can add me to the created groups in gerrit vitrage-core and vitrage-release? | 18:32 |
*** geoffarnold has quit IRC | 18:32 | |
*** geoffarn_ has joined #openstack-infra | 18:32 | |
* asselin doesn't know how to spell sense | 18:32 | |
clarkb | mgrosz we need puppet to run and add the poject to gerrit, you can log into gerrit and check the project list for your project. once it shows up let us know and we can seed the gerrit group | 18:32 |
mgrosz | someone? | 18:32 |
clarkb | asselin ok | 18:33 |
mgrosz | clarkb: where can I see gerrit project list? | 18:33 |
waynr | zaro: any particular reason it's unlikely for core? | 18:33 |
mgrosz | (found...) | 18:33 |
openstackgerrit | Merged openstack-infra/puppet-openstackci: Add support for os-client https://review.openstack.org/229162 | 18:33 |
clarkb | mgrosz login to https://review.openstack.org the go to admin-> projects | 18:33 |
waynr | I guess I should just famliarize myself with the project's architecture ;) | 18:33 |
*** spzala has quit IRC | 18:34 | |
*** tsekiyama has quit IRC | 18:34 | |
waynr | most of the product developers I support work on clojure projects so i am bias in favor of clojure over java | 18:34 |
mgrosz | clarkb: found. Thanks | 18:35 |
AJaeger | mestery: you need to setup pypi for all projects as well. Have you done this? | 18:36 |
mgrosz | clarkb: how long does it take to puppet to add to project list? | 18:36 |
AJaeger | mestery: I checked calico and don't see it... | 18:36 |
mestery | AJaeger: Aye carumba, I need to do that! | 18:36 |
asselin | clarkb, I hope it's clear in https://git.openstack.org/cgit/openstack-infra/project-config-example/tree/README.md | 18:36 |
mestery | AJaeger: Let me mark that WIP and get that sorted | 18:36 |
clarkb | mgrosz usually no longer than an hour | 18:36 |
mestery | AJaeger: Thanks for the quick reivew as always, much appreciated! :) | 18:36 |
*** notnownikki has joined #openstack-infra | 18:37 | |
*** lg782s_ has joined #openstack-infra | 18:37 | |
*** lg782s has quit IRC | 18:37 | |
*** julim has quit IRC | 18:41 | |
*** bltavares has quit IRC | 18:41 | |
*** lg782s_ has quit IRC | 18:42 | |
openstackgerrit | Ramy Asselin proposed openstack-infra/project-config-example: test initial project ci jobs https://review.openstack.org/229566 | 18:42 |
clarkb | mordred: http://logs.openstack.org/51/226751/16/check/gate-nodepool-python27/4bfb9c7/console.html#_2015-09-29_18_39_35_600 maybe my suggestion around having the get_openstack_vars() function do the work needs more thought | 18:42 |
*** julim has joined #openstack-infra | 18:43 | |
clarkb | mordred: http://logs.openstack.org/51/226751/16/check/gate-nodepool-python27/4bfb9c7/console.html#_2015-09-29_18_40_40_105 that one is really neat | 18:43 |
*** bltavares has joined #openstack-infra | 18:44 | |
*** dims_ has quit IRC | 18:46 | |
*** openstackgerrit has quit IRC | 18:46 | |
krotscheck | AJaeger: Thanks for the +2 | 18:46 |
*** dims_ has joined #openstack-infra | 18:46 | |
*** openstackgerrit has joined #openstack-infra | 18:46 | |
mordred | clarkb: wtf? | 18:47 |
krotscheck | fungi: Would you prefer I WIP the weird /usr/etc/npmrc patch in favor of the cleaner solution? | 18:47 |
krotscheck | I don't want it to accidentlaly get merged. | 18:47 |
krotscheck | n/m, I just did it | 18:48 |
fungi | krotscheck: yep, sounds good | 18:48 |
*** yamamoto_ has quit IRC | 18:48 | |
*** rguillebert has quit IRC | 18:48 | |
krotscheck | fungi: lmk when the release crazyiness is done, so we can revisit this. | 18:48 |
clarkb | mordred right | 18:48 |
*** achanda has quit IRC | 18:50 | |
*** bcourt has quit IRC | 18:50 | |
*** ayoung is now known as ayoung-meeting | 18:51 | |
*** amotoki has joined #openstack-infra | 18:51 | |
*** doug-fish has quit IRC | 18:52 | |
*** doug-fish has joined #openstack-infra | 18:52 | |
*** achanda has joined #openstack-infra | 18:53 | |
nibalizer | waynr: another thing to look at is the 'vinz' project which as of now has zero commits | 18:53 |
*** bcourt has joined #openstack-infra | 18:53 | |
*** Sukhdev has joined #openstack-infra | 18:53 | |
nibalizer | but the idea was to build a web application on top of gerrit that used the rest api | 18:53 |
nibalizer | and didn't suck | 18:53 |
*** tonytan4ever has quit IRC | 18:54 | |
*** notnownikki has quit IRC | 18:54 | |
*** Sukhdev has quit IRC | 18:54 | |
*** tonytan4ever has joined #openstack-infra | 18:55 | |
fungi | though our current gerrit upgrade blocker is a bug in the backend, not the frontend | 18:55 |
*** david-lyle has quit IRC | 18:55 | |
jeblair | clarkb, fungi: we should have the quota for the replacement lb | 18:55 |
*** david-lyle has joined #openstack-infra | 18:55 | |
nibalizer | fungi: yes | 18:56 |
nibalizer | but waynr said on the ML that text-input was a specific pain point for him | 18:56 |
nibalizer | (i shouldn't be crossing the streams though) | 18:57 |
*** amotoki has quit IRC | 18:57 | |
nibalizer | also I checked my ssh access yesterday and it works | 18:57 |
fungi | ahh | 18:57 |
jeblair | clarkb: i thought similar for the example project-config, but then i thought, the actual api in puppet-project_config is to point it at a git repo, so i thought it's appropriate. | 18:57 |
nibalizer | is there a preference for sudo 'command' vs sudo -i; command ? | 18:57 |
*** annegentle has quit IRC | 18:57 | |
jeblair | clarkb, fungi: that is to say that our quota increase request was approved and should be in place | 18:57 |
clarkb | jeblair not accoeding to nova quota-show | 18:58 |
clarkb | mordred roped me into the thread trying to sort it out now | 18:58 |
fungi | nibalizer: sudo command. doing sudo -i to get a root shell loses per-command logging | 18:58 |
clarkb | its still 100 instances and 1TB ram | 18:59 |
clarkb | (we were bumping up against both) | 18:59 |
fungi | nibalizer: there are a few exceptions, particularly like with puppet agent which tends not to work well under sudo, but in general sudo when you can | 18:59 |
waynr | nibalizer: so obviously there are a large number of details missing from my very vaguely proposed gerrit-in-clojure question, but the idea isn't to wrap gerrit with another app, but to integrate gerrit into an app that re-uses gerrit components at the source code api level rather than a rest api level | 18:59 |
jeblair | clarkb: i agree, it does not appear to be updated. | 19:00 |
*** Swami has quit IRC | 19:00 | |
*** rguillebert has joined #openstack-infra | 19:00 | |
clarkb | jeblair hopefully we will have an answer soon | 19:00 |
*** megm has quit IRC | 19:00 | |
waynr | anyway...time will tell if i actually have time to wrap my head around gerrit enough to make a more concrete proposal. i will say that clojure is at least an order of magnitude more enjoyable to develop in than java in my opinion ;) | 19:00 |
*** annegentle has joined #openstack-infra | 19:00 | |
clarkb | waynr this is the completely wrong venue for that | 19:00 |
mgrosz | clarkb: Vitrage new project is in gerrit. How do we continue with the groups? | 19:01 |
clarkb | #gerrit is where you want to talk | 19:01 |
clarkb | mgrosz who should be the first member in the groups? they can add everyone else | 19:01 |
*** Sukhdev has joined #openstack-infra | 19:02 | |
*** megm has joined #openstack-infra | 19:02 | |
*** achanda has quit IRC | 19:02 | |
clarkb | waynr keep in mind that gerrit intentionally uses google web toolkit because they like it | 19:02 |
mgrosz | clarkb: It will be me | 19:02 |
waynr | clarkb: cool, thanks...at the moment i am just responding to responses to my mailing list question coming to me in this channel, again I'm not serious enough to commit (or ask anyone else to commit) | 19:02 |
clarkb | waynr its very much a java project | 19:02 |
*** matrohon has joined #openstack-infra | 19:02 | |
mgrosz | clarkb: what details you need from me? | 19:02 |
clarkb | mgrosz just your gerrit account but I can probably get that from the original change | 19:03 |
*** dmsimard|afk is now known as dmsimard | 19:03 | |
mgrosz | clarkb: OK. There are two groups -core and release | 19:03 |
mgrosz | clarkb: vitrage-core and vitrage-release | 19:04 |
ZZelle_ | Hi, everyone | 19:04 |
*** nadya has quit IRC | 19:04 | |
ZZelle_ | I am looking for a script which allows to unset assignee and target for LP bugs without activity in the last x months | 19:05 |
clarkb | msgrosz gerrit groups are set now | 19:05 |
*** armax has quit IRC | 19:06 | |
*** armax has joined #openstack-infra | 19:06 | |
*** mudassirlatif has quit IRC | 19:07 | |
*** maishsk has quit IRC | 19:07 | |
ZZelle_ | Does such script exist? If not should i share my implementation (in which repo?) | 19:07 |
*** signed8bit_ZZZzz is now known as signed8bit | 19:08 | |
*** signed8bit is now known as signed8bit_ZZZzz | 19:08 | |
*** signed8bit_ZZZzz is now known as signed8bit | 19:08 | |
*** mudassirlatif has joined #openstack-infra | 19:09 | |
*** maishsk has joined #openstack-infra | 19:09 | |
Rockyg | folks, I need some quick gerrit help. git review -s returns "we don't know where your gerrit is. Please manually create a remote named "gerrit" and try again." | 19:09 |
*** odgrim has quit IRC | 19:10 | |
fungi | Rockyg: does `git review --version` report 1.25.0 or something earlier? | 19:10 |
mgrosz | clarkb: THANK YOU! | 19:10 |
*** mudassirlatif has quit IRC | 19:11 | |
Rockyg | 1.25.0 | 19:11 |
*** spzala has joined #openstack-infra | 19:11 | |
Rockyg | ^fungi | 19:11 |
*** bcourt has quit IRC | 19:11 | |
clarkb | ZZelle_ ttx is probably your best bet though I think his secret is sending email to lp | 19:11 |
fungi | Rockyg: try `git review -s -v` and paste the entire output into http://paste.openstack.org/ and then let us know the url to it | 19:12 |
*** matbu has quit IRC | 19:12 | |
fungi | Rockyg: usually that error is a result of not being able to confirm the account is working (either because credentials are incorrect or the account isn't completely set up yet) | 19:13 |
clarkb | waynr mostly I dont want to derail the serious discussion of lets fix some problems in gerrit by talking about a clojure rewrite | 19:13 |
fungi | Rockyg: though sometimes it can also be due to connectivity issues | 19:13 |
Rockyg | lloks like it might be my ssh key(s) | 19:13 |
*** matbu has joined #openstack-infra | 19:14 | |
*** geoffarn_ has quit IRC | 19:14 | |
*** hashar has joined #openstack-infra | 19:14 | |
*** geoffarnold has joined #openstack-infra | 19:14 | |
*** kiran-r has quit IRC | 19:15 | |
mgrosz | clarkb: I am in the stage "Choosing the Right cookiecutter Template" - executing cookiecutter https://git.openstack.org/openstack-dev/cookiecutter. | 19:15 |
waynr | clarkb: understood and i really want to avoid being argumentative so i won't try to explain my intent any further. | 19:16 |
mgrosz | I am getting this error: ValueError: zero length field name in format | 19:16 |
clarkb | waynr: there have been previous discssions of a rewrite fwiw and unfortunately haven't helped us get over this hump with gerrit | 19:16 |
Rockyg | fungi, I think I should recreate my .ssh folder | 19:16 |
openstackgerrit | Mikhail S Medvedev proposed openstack-infra/zuul: Call parameter_function prior to creating swift params https://review.openstack.org/229582 | 19:16 |
mgrosz | clark: What is wrong? I am following the instractions... | 19:16 |
clarkb | mgrosz: that sounds like you need to set a name? I have never used cookiecutter before | 19:17 |
Clint | waynr: i don't think rewriting gerrit in clojure will fix any jgit bugs | 19:18 |
*** tiswanso has quit IRC | 19:18 | |
jlvillal | Hi, can I get added to the groups: git-repo-manifests-release and git-repo-manifests-core https://review.openstack.org/#/c/228004/5 I = patch author | 19:19 |
clarkb | jlvillal: ya one moment | 19:19 |
waynr | Clint: point taken, again i'm not here to argue and I will do my best to keep the discussion constructive in the future | 19:19 |
jlvillal | clarkb, Thanks | 19:19 |
*** otter768 has joined #openstack-infra | 19:20 | |
openstackgerrit | Paul Belanger proposed openstack-infra/nodepool: Add dib success or failues to statsd https://review.openstack.org/226481 | 19:20 |
clarkb | jlvillal: done | 19:20 |
jlvillal | clarkb, Woot! :) | 19:20 |
Clint | waynr: now, rewriting it in haskell... ;) | 19:21 |
clarkb | Clint: be careful, google might actually go for that (they rewrote ganeti in it apparently) | 19:21 |
Clint | clarkb: only part | 19:21 |
clarkb | mgrosz: was that enough to get cookiecutter working? | 19:22 |
openstackgerrit | Merged openstack-infra/tripleo-ci: Fix tripleo-incubator build workaround https://review.openstack.org/229210 | 19:22 |
waynr | Clint++ | 19:22 |
*** tiswanso has joined #openstack-infra | 19:22 | |
*** david-lyle has quit IRC | 19:22 | |
clarkb | mgrosz: basically cookiecutter is a repo templater so you have to feed in the data it needs | 19:22 |
*** bdemers has joined #openstack-infra | 19:22 | |
*** david-lyle has joined #openstack-infra | 19:22 | |
*** tonytan4ever has quit IRC | 19:23 | |
*** dkranz has joined #openstack-infra | 19:24 | |
*** otter768 has quit IRC | 19:25 | |
*** stevemar has joined #openstack-infra | 19:26 | |
openstackgerrit | Sridhar Ramaswamy proposed openstack-infra/project-config: Add py27 and fixup functional job for tacker https://review.openstack.org/228649 | 19:26 |
mgrosz | clarkb: I understand this is what I need. I passed the above error. Now I am asked to enter some template parameters. What should be the modeule_name? Vitrage? | 19:27 |
*** tonytan4ever has joined #openstack-infra | 19:28 | |
mgrosz | clarkb: What is the Python Module? Vitrage? | 19:28 |
*** abregman has joined #openstack-infra | 19:28 | |
*** abregman is now known as abregman|pto | 19:29 | |
*** bdemers has quit IRC | 19:29 | |
clarkb | probably vitrage | 19:29 |
clarkb | no caps | 19:29 |
*** dprince has quit IRC | 19:29 | |
openstackgerrit | Merged openstack-infra/nodepool: Add a test for the list command https://review.openstack.org/226591 | 19:32 |
mgrosz | clarkb: getting error... I am follwing exactly the instractions: First executing git clone "git://git.openstack.org/openstack/vitrage". This creates alocal directory named vitrage. Then I run the command "cookiecutter https://git.openstack.org/openstack-dev/cookiecutter". I am giving all the values it asks me to. Then I am getting the error: "Error | 19:33 |
mgrosz | : "vitrage" directory already exists"... But, it is there because a previous step... | 19:33 |
mgrosz | ??? | 19:33 |
*** jsavak has quit IRC | 19:33 | |
*** pvaneck has joined #openstack-infra | 19:33 | |
*** david-lyle has quit IRC | 19:35 | |
*** AJaeger has quit IRC | 19:35 | |
*** jsavak has joined #openstack-infra | 19:35 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Add Liberty compat jobs to tempest jobs https://review.openstack.org/229586 | 19:36 |
openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Add devstack-gate nv stable compat jobs https://review.openstack.org/229587 | 19:36 |
mtreinish | clarkb: ^^^ I think that should do the things | 19:36 |
clarkb | mgrosz: sorry I don't know | 19:37 |
clarkb | mgrosz: I have never used cookiecutter before | 19:37 |
mtreinish | mgrosz: cookiecutter will make the dir and do the repo init for you | 19:38 |
yolanda | clarkb, jeblair, can you look at nodepool secure.conf file? https://review.openstack.org/#/c/189762/ | 19:38 |
mtreinish | mgrosz: it asks you a bunch of questions before it does anything and it figures out the git infor from that | 19:38 |
yolanda | it needs an extra +2 and final approval, are we ok to land? | 19:38 |
mgrosz | mtreinish: thanks! | 19:39 |
clarkb | mtreinish: lgtm thank you | 19:39 |
clarkb | mtreinish: last change would be to add the periodic jobs | 19:39 |
*** marzif has joined #openstack-infra | 19:40 | |
mtreinish | clarkb: ah ok, I'll add that right now | 19:40 |
*** dangers is now known as dangers_away | 19:40 | |
dims_ | anteaya: can you please bless this? https://review.openstack.org/#/c/229419/ | 19:42 |
openstackgerrit | Merged openstack/diskimage-builder: Fix variable misspelling error https://review.openstack.org/224928 | 19:42 |
clarkb | dims_: instead of setting the branch to only master what do you think about excluding juno, liberty, kilo? | 19:43 |
clarkb | dims_: that way mitaka will be properly tested and N O P etc | 19:43 |
*** signed8bit is now known as signed8bit_ZZZzz | 19:43 | |
pleia2 | back from this morning's errands, lmk if there's anything I should look at | 19:44 |
dims_ | clarkb: ack, happy to do that | 19:44 |
openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Add periodic stable liberty jobs https://review.openstack.org/229589 | 19:44 |
mtreinish | clarkb: ^^^ | 19:44 |
openstackgerrit | James Polley proposed openstack-dev/pbr: Add test for pbr_info_writer check https://review.openstack.org/226646 | 19:44 |
*** mgrosz has quit IRC | 19:45 | |
*** thingee has joined #openstack-infra | 19:46 | |
*** signed8bit_ZZZzz is now known as signed8bit | 19:46 | |
*** amotoki has joined #openstack-infra | 19:46 | |
*** rellerreller has quit IRC | 19:47 | |
*** mgrosz has joined #openstack-infra | 19:47 | |
*** hemna is now known as hemnafk | 19:47 | |
clarkb | mtreinish: that may also need a jenkins/jobs/projects.yaml update to add the periodic jobs | 19:47 |
clarkb | mtreinish: if you grep for periodic in that file it should be clear how we define those | 19:47 |
mtreinish | clarkb: ok, I'll fix that | 19:47 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-infra/project-config: dsvm full job for devstack-zmq https://review.openstack.org/229419 | 19:47 |
clarkb | pleia2: just release things lots of reviews | 19:47 |
dims_ | clarkb: done ^^ | 19:48 |
*** annegentle has quit IRC | 19:48 | |
pleia2 | clarkb: gotcha, on it | 19:48 |
*** yamamoto has joined #openstack-infra | 19:48 | |
*** amotoki has quit IRC | 19:50 | |
clarkb | dims_: thank you | 19:51 |
mtreinish | clarkb: I think we look ok on the bitrot job patch: http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/python-bitrot-jobs.yaml | 19:51 |
dims_ | thank you clarkb! | 19:51 |
mtreinish | I don't think that needs any other definitions | 19:51 |
pleia2 | dims_: lgtm | 19:52 |
clarkb | mtreinish: we need to add a liberty job-group and apply that to the projects I think | 19:52 |
*** david-lyle has joined #openstack-infra | 19:52 | |
clarkb | mtreinish: I just see juno and kilo | 19:52 |
mtreinish | clarkb: oh, yep missed that sry | 19:53 |
dims_ | pleia2: thanks! | 19:53 |
*** yamamoto has quit IRC | 19:53 | |
openstackgerrit | Merged openstack-infra/project-config: Add Liberty compat jobs to tempest jobs https://review.openstack.org/229586 | 19:53 |
*** esker has joined #openstack-infra | 19:53 | |
pleia2 | fungi: since you engaged them on the mailing list, could you take a look at this mailing list for openstack-ux review and chime in? https://review.openstack.org/#/c/218409/ | 19:54 |
openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Add periodic stable liberty jobs https://review.openstack.org/229589 | 19:54 |
mtreinish | clarkb: ^^^ | 19:54 |
*** matrohon has quit IRC | 19:55 | |
clarkb | mtreinish: oh you also need it for the periodic dsvm jobs | 19:55 |
clarkb | mtreinish: let me get a link | 19:55 |
*** mgrosz has quit IRC | 19:55 | |
clarkb | I think we crossed the streams a bit (need the new stuff too) | 19:55 |
*** geoffarnold has quit IRC | 19:56 | |
*** mgrosz has joined #openstack-infra | 19:56 | |
clarkb | mtreinish: https://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/projects.yaml#n3560 that bit needs a new liberty section | 19:56 |
*** geoffarnold has joined #openstack-infra | 19:56 | |
mtreinish | clarkb: ok, I'll respin that in a sec | 19:56 |
clarkb | mtreinish: and for the bitrot jobs we need to update projects.yaml to apply that template to all the projects with bitrot jobs now, then update layout.yaml to run the jobs | 19:57 |
clarkb | mtreinish: that might be better in a separate change but up to you | 19:57 |
openstackgerrit | Merged openstack-infra/project-config: Add devstack-gate nv stable compat jobs https://review.openstack.org/229587 | 19:57 |
Rockyg | fungi, been reading and trying stuff. Still no go. Here's a paste with a couple of tries, plus various config info: http://paste.openstack.org/show/474928/ | 19:57 |
*** mgrosz has quit IRC | 19:57 | |
*** mudassirlatif has joined #openstack-infra | 19:58 | |
*** mgrosz has joined #openstack-infra | 19:59 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Setup periodic stable liberty jobs https://review.openstack.org/229589 | 19:59 |
*** bltavares has quit IRC | 20:00 | |
clarkb | Rockyg: can you check if `ssh -p29418 rockyg@review.openstack.org gerrit ls-projects` works? | 20:00 |
mtreinish | clarkb: ^^^ ok, I'll look at the additional layout.yaml changes in a follow on patch | 20:00 |
*** bltavares has joined #openstack-infra | 20:00 | |
anteaya | dims_: looks like you got what you needed | 20:01 |
dims_ | anteaya: yes, thanks! | 20:01 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Move nv openstack-ansible experimental job to check queue https://review.openstack.org/228408 | 20:01 |
*** mgrosz has quit IRC | 20:02 | |
*** mgrosz has joined #openstack-infra | 20:02 | |
openstackgerrit | Merged openstack-infra/project-config: dsvm full job for devstack-zmq https://review.openstack.org/229419 | 20:03 |
*** odgrim has joined #openstack-infra | 20:03 | |
clarkb | Rockyg: that hits gerrit directly without git review in the way and can help us rule out some issues | 20:03 |
openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Add the periodic liberty template to projects https://review.openstack.org/229596 | 20:03 |
anteaya | mmedvede fungi yes I have been operating with the belief that a rollcall vote is a majority of those who vote | 20:03 |
mtreinish | clarkb: ^^^ | 20:03 |
*** mgrosz has quit IRC | 20:04 | |
anteaya | annegentle sorry I wasn't here for your question, I hope you got what you need, if not do ping again | 20:05 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add pep8 and bashate non-voting checks to openstack-ansible https://review.openstack.org/228406 | 20:05 |
clarkb | mtreinish: lgtm. At some point we may want to "audit" that the projects with bitrot jobs are complete and we don't need to induct some new ones | 20:05 |
*** mgrosz has joined #openstack-infra | 20:05 | |
clarkb | mtreinish: but one thing at a time | 20:06 |
mtreinish | clarkb: meh, no one really looks at the results anyway | 20:06 |
*** Sukhdev has quit IRC | 20:06 | |
clarkb | true, but maybe with the new dashboard you are building people will start? | 20:06 |
mtreinish | that's my hope | 20:06 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Move nv openstack-ansible experimental job to check queue https://review.openstack.org/228408 | 20:07 |
*** david-ly_ has joined #openstack-infra | 20:07 | |
mtreinish | although we do need to figure out the data collection for subunit2sql outside of the gate queue to enable that | 20:07 |
mtreinish | right now we're still in the get things working stage | 20:07 |
pabelanger | pleia2: welcome back. Let me know if you have any issues with the review | 20:07 |
*** david-lyle has quit IRC | 20:07 | |
mtreinish | but once that hurdle is done, we'll need to start collecting data on other queues too | 20:07 |
pleia2 | pabelanger: so far so good, thanks | 20:07 |
mtreinish | but I'm thinking we might want a 2nd db for that | 20:07 |
mtreinish | it'll be easy enough to have the rest api query different dbs based on the queue | 20:08 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Move nv openstack-ansible experimental job to check queue https://review.openstack.org/228408 | 20:08 |
Rockyg | so, I can get there, then I can't get it to take my pw for the keys. ^fungi, clarkb | 20:08 |
*** david-ly_ is now known as david-lyle | 20:08 | |
mtreinish | and that'll maintain the data purity from having gate jobs only | 20:08 |
clarkb | Rockyg: was it asking you for the key passphrase when git review failed too? | 20:09 |
clarkb | Rockyg: if so that is likely the issue | 20:09 |
Rockyg | clarkb, yes, but with review, I could get it to take the passphrase on the DSA | 20:09 |
Rockyg | I *thougt* | 20:09 |
clarkb | mtreinish: periodic is low load can probably add it in safely? and in theroy its all stuff that passed the gate | 20:09 |
clarkb | Rockyg: oh you should really use rsa | 20:10 |
*** bcourt has joined #openstack-infra | 20:10 | |
*** jsavak has quit IRC | 20:10 | |
clarkb | dsa should work but I believe its gernally considered much less secure | 20:10 |
mtreinish | clarkb: well I was thinking one for periodic + experimental | 20:10 |
mtreinish | and there is also value in having check because of non voting jobs too | 20:10 |
Rockyg | well, it first prompts for RSA, fails, moves to DSA, and half the time I get the passphrase right, clarkb | 20:10 |
mtreinish | but that's even worse | 20:10 |
clarkb | mtreinish: thats true, need check anyways | 20:10 |
*** claudiub has joined #openstack-infra | 20:10 | |
mtreinish | clarkb: also we'll have to expand it beyond tempest jobs at some point too... | 20:11 |
clarkb | Rockyg: unfortunately I can't help you with ssh key passphrases :) if you just want to convince yourself it works you could temporarily use a passphraseless key | 20:11 |
* mtreinish wonders how long 300 GB would last on check | 20:11 | |
Rockyg | how? | 20:11 |
clarkb | mtreinish: hahahah I mean no long :) | 20:11 |
*** mgrosz has quit IRC | 20:11 | |
Rockyg | ^ clarkb | 20:12 |
clarkb | Rockyg: ssh-keygen prompts for a passphrase and you just hit enter a couple times | 20:12 |
clarkb | then put that public key in gerrit | 20:12 |
Rockyg | ah. gotcha. | 20:12 |
Rockyg | thanks | 20:12 |
mtreinish | clarkb: I don't ssh prompts you for anything if there isn't a passphrase on a key | 20:12 |
clarkb | EmilienM: crinkle fyi https://github.com/openstack/puppet-cinder/pull/23 | 20:12 |
clarkb | mtreinish: it doesn't when you ssh but does when generating the key | 20:12 |
openstackgerrit | Merged openstack-infra/project-config: Setup periodic stable liberty jobs https://review.openstack.org/229589 | 20:13 |
mtreinish | ah, yeah | 20:13 |
Clint | or you can just do -N '' | 20:13 |
*** abregman|pto has quit IRC | 20:13 | |
EmilienM | clarkb: thx | 20:13 |
clarkb | mtreinish: I am suggesting making a new throw away key rather than converting an existing key (since we don't appear to know the current phrase and that is required) | 20:13 |
*** jsavak has joined #openstack-infra | 20:13 | |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add pep8 and bashate non-voting checks to openstack-ansible https://review.openstack.org/228406 | 20:13 |
mtreinish | ok, sure that makes sense | 20:13 |
openstackgerrit | Jim Rollenhagen proposed openstack-infra/project-config: Add coverage post job to ironic-lib https://review.openstack.org/229473 | 20:14 |
*** shardy has quit IRC | 20:14 | |
anteaya | Clint: do you want to work on testing your script? | 20:14 |
openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add pep8 and bashate non-voting checks to openstack-ansible https://review.openstack.org/228406 | 20:15 |
Clint | anteaya: sure | 20:15 |
clarkb | pleia2: jeblair mordred fungi nibalizer yolanda https://review.openstack.org/#/c/226858/ that should correct a current issue with coverage report html logs | 20:15 |
openstackgerrit | Merged openstack-infra/grafyaml: Add caching support https://review.openstack.org/226002 | 20:15 |
*** dkranz has quit IRC | 20:16 | |
*** gyee has quit IRC | 20:16 | |
Clint | anteaya: what's the plan? | 20:16 |
anteaya | Clint: great so if you head to the wikipage for the renames grap the list, sort it alphabetically and run it with the first 10 names | 20:16 |
anteaya | then you can offer a patch to gerrit for me to review | 20:16 |
pleia2 | clarkb: ah, cool | 20:16 |
anteaya | I suggest a title like "testing the rename scripts" | 20:16 |
anteaya | so folks don't review it unless they want to help us | 20:16 |
*** dkranz has joined #openstack-infra | 20:16 | |
anteaya | then we can see if there are names that your script chokes on | 20:17 |
*** geoffarn_ has joined #openstack-infra | 20:17 | |
*** geoffarnold has quit IRC | 20:18 | |
openstackgerrit | Merged openstack-infra/os-loganalyze: Skip whitespace for html view https://review.openstack.org/226858 | 20:19 |
Clint | yay, found a bug | 20:19 |
anteaya | Clint: woooo | 20:19 |
anteaya | Clint: good work | 20:19 |
*** odgrim has quit IRC | 20:20 | |
austin81 | mtreinish: What needs to happen to get a db setup for periodic jobs? | 20:20 |
*** bltavares has quit IRC | 20:21 | |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 20:21 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: WIP - testing the rename scripts https://review.openstack.org/229601 | 20:21 |
Clint | argh | 20:21 |
mtreinish | austin81: well if that's the direction we decide to go we need someone to create the trove instance (or a regular server to deploy mysql on) | 20:21 |
anteaya | Clint: is that your yay I submitted a patch noise? | 20:22 |
pleia2 | lol | 20:22 |
mtreinish | austin81: and then we'll need to modify the subunit gearman worker and log gearman client to know to talk to a different db on periodic and experimental results | 20:22 |
mtreinish | err well we might not need to change the gearman client | 20:22 |
Clint | anteaya: no, that's my git-review-did-a-thing-i-didn't-intend-again noise | 20:23 |
anteaya | Clint: ah | 20:23 |
mtreinish | austin81: yeah I don't think we would, it would just be the subunit gearman worker that needs to be updated | 20:23 |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for heat functional cinder v1 bug 1500714 https://review.openstack.org/229602 | 20:23 |
openstack | bug 1500714 in heat "publicURL endpoint for volume service in RegionOne region not found" [High,Fix committed] https://launchpad.net/bugs/1500714 - Assigned to Rabi Mishra (rabi) | 20:23 |
mtreinish | but either way it's not too hard | 20:23 |
anteaya | Clint: what did you intend and what did it do? | 20:23 |
Clint | anteaya: i didn't intend for it to rebase the script change, but that's okay | 20:24 |
anteaya | oh | 20:24 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 20:24 |
Clint | and now it doesn't matter | 20:24 |
*** baoli_ has quit IRC | 20:26 | |
fungi | okay, i've bought ssl certs for apps, openstackid-resources, and a multi-domain cert for static/governance/logs/security/docs-draft/specs/tarballs/trystack | 20:26 |
fungi | none are in hiera yet | 20:26 |
fungi | i've confirmed them and am waiting for the actual issuing to happen which i've seen take upwards of a few hours in the past | 20:26 |
clarkb | neat | 20:26 |
fungi | but should have them in hiera later this evening | 20:26 |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Remove query for largeops bug 1491949 https://review.openstack.org/229603 | 20:27 |
openstack | bug 1491949 in devstack "gate-tempest-dsvm-large-ops fails to deallocate instance network due to rpc timeout" [Undecided,Fix released] https://launchpad.net/bugs/1491949 - Assigned to Matt Riedemann (mriedem) | 20:27 |
anteaya | Clint: okay what is the bug you found? | 20:27 |
anteaya | Clint: the patch looks fine to me | 20:27 |
Clint | no, i broke it again | 20:27 |
Clint | anteaya: it was outputting the /home/gerrit2 paths for git mv instead of the in-repository paths | 20:27 |
Clint | but now it's doing the wrong thing in the file | 20:27 |
anteaya | oh | 20:28 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 20:28 |
docaedo | fungi: thanks for the cert update - ping me if you need anything else from me | 20:28 |
*** bswartz has quit IRC | 20:29 | |
Clint | anteaya: okay, i think it's all good again | 20:29 |
anteaya | fungi: yay | 20:29 |
anteaya | Clint: okay lets see a paste of your mv commands | 20:29 |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for heat functional cinder v1 bug 1500714 https://review.openstack.org/229602 | 20:30 |
openstack | bug 1500714 in heat "publicURL endpoint for volume service in RegionOne region not found" [High,Fix committed] https://launchpad.net/bugs/1500714 - Assigned to Rabi Mishra (rabi) | 20:30 |
*** signed8bit is now known as signed8bit_ZZZzz | 20:30 | |
anteaya | fungi: so from yesterday just before the meeting jd__ was asking about having a publish job publish docs to an .xyz domain name | 20:30 |
anteaya | fungi: I don't think we have done that before, have you an opinion? | 20:30 |
anteaya | clarkb pleia2 ^^ | 20:31 |
anteaya | jeblair mordred ^^ | 20:31 |
clarkb | anteaya: no I think we want control of what we publish to | 20:31 |
fungi | anteaya: we've been trying to consolidate all the random vanity domains to things under openstack.org for simplicity | 20:31 |
*** salv-orl_ has quit IRC | 20:31 | |
clarkb | anteaya: for example the devstack docs | 20:31 |
openstackgerrit | Merged openstack-infra/project-config: Move nv openstack-ansible experimental job to check queue https://review.openstack.org/228408 | 20:31 |
anteaya | okay great thank you | 20:31 |
clarkb | anteaya: did jd__ have specific details? | 20:31 |
anteaya | clarkb: I'll get a link | 20:32 |
fungi | yeah, see recent moves of devstack.org, trystack.org, refstack.org... | 20:32 |
Clint | anteaya: well that did not come out well | 20:32 |
anteaya | Clint: paste | 20:32 |
Clint | anteaya: http://paste.openstack.org/show/474942/ | 20:32 |
pleia2 | clarkb: it was for publishing a doc job on their gnocchi.xyz | 20:32 |
anteaya | clarkb: it starts here: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2015-09-29.log.html#t2015-09-29T18:57:16 | 20:32 |
anteaya | pleia2: yes that was it | 20:33 |
fungi | pleia2: clarkb: anteaya: in that case my response on the tripleo.org docs publishing thread is likely 100% applicable here too | 20:33 |
anteaya | fungi: I will look up that thread | 20:33 |
pleia2 | anteaya: it was just a couple days ago | 20:33 |
fungi | (e.g. pushing them to instead publish their additional content to docs.openstack.org/developer/tripleo or if necessary a tripleo.openstack.org site though that seems like overkill) | 20:34 |
openstackgerrit | Merged openstack-infra/elastic-recheck: Remove query for largeops bug 1491949 https://review.openstack.org/229603 | 20:34 |
openstack | bug 1491949 in devstack "gate-tempest-dsvm-large-ops fails to deallocate instance network due to rpc timeout" [Undecided,Fix released] https://launchpad.net/bugs/1491949 - Assigned to Matt Riedemann (mriedem) | 20:34 |
anteaya | Clint: have you the link to the most recent rename etherpad that shows the expected mv commands? | 20:34 |
pleia2 | anteaya: http://lists.openstack.org/pipermail/openstack-dev/2015-September/075634.html | 20:34 |
anteaya | Clint: I can't recall what I should expect here | 20:34 |
anteaya | pleia2: thank you | 20:34 |
Clint | anteaya: is it https://etherpad.openstack.org/p/project-renames-Septemeber-2015 | 20:34 |
*** ihrachys has joined #openstack-infra | 20:34 | |
anteaya | Clint: thank you | 20:35 |
jd__ | fungi: it's just to have duplicate contents less OpenStack branded | 20:35 |
*** signed8bit_ZZZzz is now known as signed8bit | 20:36 | |
fungi | jd__: you have complete control over the theming of your content on docs.openstack.org too. you don't technically have to have openstack logos/branding on that either | 20:36 |
anteaya | Clint: yeah so if we are going for generating commands to satisfy step 6, I agree we are not there yet | 20:36 |
*** esker has quit IRC | 20:37 | |
jd__ | fungi: yeah though I think it's cool to have it over docs.openstack.org, I'm just looking for a way of having a separate branding somehow | 20:37 |
jd__ | it = OpenStack branded | 20:37 |
Clint | anteaya: oh, i thought that part was getting done by the ansible whatevers | 20:38 |
*** geoffarn_ has quit IRC | 20:38 | |
*** geoffarnold has joined #openstack-infra | 20:38 | |
*** sergmelikyan has joined #openstack-infra | 20:39 | |
anteaya | Clint: oh okay, what was your goal with the mv commands your script was outputting? | 20:39 |
anteaya | the acl changes in the patch you submitted? | 20:39 |
Clint | anteaya: right, i thought that's what fungi was talking about with the mvs | 20:39 |
*** stevemar has quit IRC | 20:39 | |
anteaya | Clint: well one set of them anyway | 20:39 |
anteaya | okay so now I will take a closer look at the patch and see if it missed anything | 20:40 |
*** stevemar has joined #openstack-infra | 20:40 | |
*** amotoki has joined #openstack-infra | 20:40 | |
anteaya | this will take me more time, I'd say give me 20 or 30 minutes | 20:40 |
* Clint nods. | 20:41 | |
anteaya | Clint: do you want to take a look at the layout error the patch is throwing? | 20:41 |
*** armax has quit IRC | 20:41 | |
Clint | anteaya: yes | 20:41 |
*** flepied has quit IRC | 20:42 | |
*** hemnafk is now known as hemna | 20:42 | |
Clint | right, so when it was said that we could use sed because the ordering didn't matter, that would appear to have been incorrect | 20:43 |
fungi | i'd be okay with turning off the jenkins/zuul configuration ordering checks for a while | 20:44 |
fungi | i don't think they make a lot of sense after the namespace collapse | 20:44 |
fungi | or at least need revisiting regardless | 20:45 |
*** stevemar has quit IRC | 20:45 | |
Clint | fungi: ok | 20:45 |
*** amotoki has quit IRC | 20:45 | |
anteaya | that makes it easier | 20:46 |
anteaya | fungi: do you want to turn them off just prior to the renaming? | 20:47 |
fungi | anytime from here on is likely fine since we're really not creating new stackforge/.* repos for the most part | 20:47 |
fungi | i need to pop out and do some grocery shopping before the store turns into more of a zoo though. everyone's bracing for the worst | 20:48 |
anteaya | Clint: do you want to look into a patch that disables those tests? | 20:48 |
fungi | back in a bit | 20:48 |
anteaya | fungi: happy jossling for food | 20:48 |
anteaya | and batteries | 20:48 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: WIP - Disable alphabetical ordering check for zuul/layout.yaml https://review.openstack.org/229617 | 20:48 |
Clint | anteaya: think it should just be that one-liner ^ | 20:49 |
openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Add pypi publishing jobs for neutron sub-projects https://review.openstack.org/229564 | 20:49 |
anteaya | Clint: let's remove wip from the commit title and temporarliy from the comment | 20:50 |
anteaya | and I agree that seems to be the function | 20:50 |
*** odgrim has joined #openstack-infra | 20:50 | |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Disable alphabetical ordering check for zuul/layout.yaml https://review.openstack.org/229617 | 20:51 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: WIP - testing the rename scripts https://review.openstack.org/229601 | 20:52 |
anteaya | Clint: can we lose (temporarily?) as well please? | 20:52 |
Clint | anteaya: oh, in the code | 20:52 |
anteaya | yes | 20:52 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Disable alphabetical ordering check for zuul/layout.yaml https://review.openstack.org/229617 | 20:53 |
*** stevemar has joined #openstack-infra | 20:53 | |
*** hrou has quit IRC | 20:53 | |
openstackgerrit | Sean M. Collins proposed openstack-infra/devstack-gate: Stop hard coding SERVICE_HOST https://review.openstack.org/229111 | 20:53 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: WIP - testing the rename scripts https://review.openstack.org/229601 | 20:54 |
pleia2 | Clint: I haven't been following this, is it a temporary disable? Can you make a note accordingly in the commit message? | 20:54 |
Clint | pleia2: i don't know | 20:54 |
pleia2 | ok :) | 20:54 |
openstackgerrit | Matthew Treinish proposed openstack/os-testr: Force utf8 encoding on subunit attachments output https://review.openstack.org/229527 | 20:54 |
pleia2 | I'd like to see it get back to capitalized after whatever issues there are with the rename | 20:55 |
Clint | capitalized? | 20:56 |
*** bltavares has joined #openstack-infra | 20:56 | |
*** bcourt has quit IRC | 20:57 | |
pleia2 | er, alphabetized | 20:57 |
Clint | ah yes | 20:57 |
Clint | i am not opposed to this | 20:57 |
pleia2 | sorry, post-lunch brain :) | 20:57 |
*** austin81 has left #openstack-infra | 20:58 | |
*** reed has quit IRC | 20:58 | |
*** dkehn has quit IRC | 20:59 | |
*** geoffarnold has quit IRC | 20:59 | |
anteaya | pleia2: fungi said we could disable it now until after the renames have happened | 21:00 |
openstackgerrit | Merged openstack-infra/project-config: Add dsvm-full-lio to os-brick jobs https://review.openstack.org/223258 | 21:00 |
*** geoffarnold has joined #openstack-infra | 21:00 | |
anteaya | now since the stackforge namespace is less important that part isn't as useful post-rename | 21:00 |
anteaya | but I do agree the alphabetizing is useful | 21:00 |
pleia2 | anteaya: right, context in the commit message would be lovely :) | 21:00 |
anteaya | agreed | 21:00 |
anteaya | Clint: could you be so kind as to add those details to the commit message? | 21:01 |
*** dkehn has joined #openstack-infra | 21:01 | |
Clint | anteaya: which details exactly? | 21:01 |
anteaya | Clint: and I have some feedback in the form of a link to a paste as a comment on the wip patch | 21:01 |
pleia2 | mostly that it's temporary and we can revert it post-stackforge things | 21:01 |
anteaya | that we are disabling the test now | 21:01 |
anteaya | well taht alphabetizing is important that will comeback after rename | 21:02 |
Clint | ok | 21:02 |
zigo | Could someone help me make sense of the error message I get in Tempest? | 21:02 |
zigo | Details: {u'message': u'Image b3df2c6c-c2f6-400e-addd-3c22b90b1368 is not active.', u'code': 400} | 21:02 |
zigo | What could be the reason for such a message? | 21:02 |
anteaya | zigo: well tempest is more the forte of the -qa team | 21:02 |
*** Rockyg has quit IRC | 21:02 | |
zigo | Yup. | 21:02 |
zigo | Switching channel. | 21:02 |
zigo | Thanks. | 21:02 |
anteaya | thank you | 21:02 |
openstackgerrit | Merged openstack/diskimage-builder: Install 'gdisk' when building ramdisk with ironic-agent https://review.openstack.org/215095 | 21:03 |
*** Rockyg has joined #openstack-infra | 21:03 | |
*** david-lyle has quit IRC | 21:03 | |
*** david-ly_ has joined #openstack-infra | 21:03 | |
clarkb | http://logs.openstack.org/91/226591/2/check/nodepool-coverage/92c0ac6/cover/ is back to working again | 21:03 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Disable alphabetical ordering check for zuul/layout.yaml https://review.openstack.org/229617 | 21:03 |
*** david-ly_ is now known as david-lyle | 21:05 | |
*** jsavak has quit IRC | 21:05 | |
*** bpokorny_ has joined #openstack-infra | 21:05 | |
*** signed8bit is now known as signed8bit_ZZZzz | 21:06 | |
anteaya | Clint: so two concerns on the findings with the acl file moves | 21:06 |
*** amuller has quit IRC | 21:06 | |
anteaya | one, we are missing some files taht need to be moved | 21:06 |
*** gildub has joined #openstack-infra | 21:06 | |
*** bpokorny has quit IRC | 21:07 | |
anteaya | two, we are moving files because one project that is being moved is using it, but we are not checking if that is the only project using it before it is being moved | 21:07 |
anteaya | or if all the projects using that acl file are being moved | 21:07 |
*** signed8bit_ZZZzz is now known as signed8bit | 21:08 | |
*** Sukhdev has joined #openstack-infra | 21:08 | |
Clint | anteaya: right, two was expected but not a problem in practice with the entire set right now | 21:08 |
Clint | not sure what's going on with one | 21:08 |
anteaya | let's focus on one first | 21:09 |
anteaya | then we can discuss two | 21:09 |
* Clint nods. | 21:09 | |
anteaya | thanks | 21:09 |
mmmpork | anteaya: are these points you're bringing up being captured anywhere? | 21:09 |
mmmpork | i'm working on tests for this move script right now and collecting the things you've tested and your results in hopes of writing tests to cover it | 21:10 |
anteaya | mmmpork: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/ | 21:10 |
Clint | anteaya: okay, so those four don't have any acl files specified in gerrit/projects.yaml .. am i misunderstanding how that works? | 21:10 |
anteaya | mmmpork: that is the place we capture conversations | 21:10 |
anteaya | and reference them if we need to find them again | 21:10 |
anteaya | Clint: ah okay yes | 21:11 |
mmmpork | anteaya: that's not very helpful | 21:11 |
Clint | clearly i am | 21:11 |
*** KanagarajM has quit IRC | 21:11 | |
Clint | anteaya: so it defaults to the name? | 21:11 |
anteaya | Clint: so in gerrit/projects.yaml if a project is named the same as the config file it is found | 21:11 |
anteaya | mmmpork: well that is our workflow | 21:11 |
Clint | great | 21:11 |
mmmpork | maybe we should add a space on a wiki or other document so we can capture what you've done to test? | 21:11 |
anteaya | mmmpork: as you like | 21:11 |
anteaya | mmmpork: this is a one off thing | 21:12 |
anteaya | and we hope never to repeat it | 21:12 |
mmmpork | anteaya: yes but we should still have test coverage so we can release this with confidence | 21:12 |
anteaya | so documenting what I am doing for the next time is kind of time wasted | 21:12 |
*** trown is now known as trown|outttypeww | 21:12 | |
anteaya | we aren't releasing it anywhere | 21:12 |
mmmpork | i'm documenting what you are doing so i can write tests around the script | 21:12 |
*** thorst has quit IRC | 21:12 | |
anteaya | mmmpork: as you see fit | 21:12 |
mmmpork | how were you planning to test this before the real run? | 21:13 |
anteaya | that is what Clint and I are doing now | 21:13 |
mmmpork | yeah but it's not really documented what you've tested | 21:13 |
anteaya | do as you see fit | 21:13 |
*** Rockyg has quit IRC | 21:14 | |
mmmpork | look, i've been volunteered to help out with this project and i see this as a way to help out | 21:14 |
*** ociuhandu has joined #openstack-infra | 21:14 | |
anteaya | I think you are wasted your time | 21:14 |
anteaya | I don't know who said you needed to do this | 21:14 |
anteaya | but it is yours to do what you wish with | 21:14 |
mmmpork | ok so i guess you don't want my help? | 21:14 |
mmmpork | is what i'm hearing? | 21:15 |
mmmpork | also re: respectful community and all that jazz, i'm not really getting that from you right now :) | 21:15 |
*** shashank_hegde has quit IRC | 21:15 | |
anteaya | I don't think what you are offering to do is helpful to me at this time | 21:15 |
*** alivigni has quit IRC | 21:15 | |
anteaya | because what you are wanting to do serves no purpose | 21:15 |
*** hashar has quit IRC | 21:15 | |
anteaya | I don't need it | 21:15 |
mmmpork | i think because you are thinking in terms of "me" instead of looking at things from a big picture | 21:15 |
anteaya | and don't know who would consume it | 21:15 |
anteaya | do as you like | 21:15 |
mmmpork | it would give us some confidence that the thing we are doing is going to do the thing that we want | 21:15 |
*** ldnunes has quit IRC | 21:15 | |
anteaya | Clint: let me know when you have something more to test | 21:16 |
mmmpork | and allow others to review what we've tested | 21:16 |
*** Rockyg has joined #openstack-infra | 21:16 | |
mmmpork | it provides documentation of what we've done so if there are errors after running this thing we have a clear way to debug and rule out possible issues | 21:16 |
mmmpork | this seems like a fairly invasive change that could have repercusions, i don't think i'm out of line to try to have some test coverage around this so we have some confidence that this will do what we expect it to | 21:17 |
*** dimtruck is now known as zz_dimtruck | 21:17 | |
mordred | I'm only partially here, but I believe that test coverage and documentation are generally valuable, as is double-checking work | 21:18 |
jeblair | mmmpork, anteaya: maybe we could start an etherpad with the things that have been caught so far and things we think we should double check so that they are documented and can be verified with any followup patches? | 21:19 |
*** shashank_hegde has joined #openstack-infra | 21:19 | |
*** shashank_hegde has quit IRC | 21:19 | |
anteaya | jeblair: well they are all in the comments on Clint's patches | 21:20 |
jeblair | mmmpork, anteaya: (whether that followup testing ends up being formalized testing, or just manually running through a checklist again, it would be good to have them documented) | 21:20 |
anteaya | if someone wants to copy the comments on Clint's patches to an etherpad I am not stopping them | 21:20 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 21:20 |
*** otter768 has joined #openstack-infra | 21:21 | |
mmmpork | jeblair, mordred: i can take care of that | 21:21 |
*** armax has joined #openstack-infra | 21:21 | |
pleia2 | jeblair: I think that's a good idea, and in line with what we tend to do with these kind of one-off maintanence tasks | 21:21 |
*** abregman has joined #openstack-infra | 21:22 | |
*** Rockyg has quit IRC | 21:23 | |
mmmpork | i'll start an etherpad based on what i see in the etherpad | 21:23 |
mmmpork | i mean gerrit, sorry ;D | 21:23 |
mmmpork | thinking about other things and typing haha | 21:23 |
pleia2 | no inception etherpads | 21:24 |
mmmpork | i already started a test that checks the actual rename in the code itself so that's some level of verification | 21:24 |
jeblair | mmmpork: closing the feedback loop? :) | 21:24 |
*** shashank_hegde has joined #openstack-infra | 21:24 | |
mmmpork | yo dawg, i heard you liek etherpadz | 21:24 |
openstackgerrit | Nisha Agarwal proposed openstack/diskimage-builder: Add a new element hpdsa https://review.openstack.org/220888 | 21:24 |
clarkb | greghaynes: do you have tests for dib image list/delete and image upload in your stack? | 21:24 |
clarkb | greghaynes: I am putting together another set of test changes for nodepool to cover the commands and through that get more coverage | 21:25 |
*** otter768 has quit IRC | 21:25 | |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 21:26 |
greghaynes | clarkb: thedre is https://review.openstack.org/#/c/215275/ | 21:26 |
*** woodster_ has joined #openstack-infra | 21:26 | |
greghaynes | clarkb: which does delete, but thats it | 21:26 |
clarkb | greghaynes: thanks and its not a command test either so we wont conflict | 21:26 |
jeblair | clarkb: i don't understand your comment about the task manager on 226751. we want a taskmanager in nodepool to do rate limiting -- are you proposing we get rid of that? | 21:26 |
*** gabriel-bezerra has quit IRC | 21:26 | |
*** david-lyle has quit IRC | 21:26 | |
clarkb | jeblair: no, I am proposing we get rid of ProviderManager as a subclass of the taskmanager | 21:26 |
clarkb | jeblair: basically shade is hte ProviderManager now and takes a TaskManager as the manager | 21:27 |
*** sergmelikyan has quit IRC | 21:27 | |
jeblair | clarkb: ooh, i think i parse correctly now; let me rethink with that new understanding. | 21:27 |
*** dkranz has quit IRC | 21:28 | |
jeblair | clarkb: yes, i think that's the next step in principle. | 21:28 |
*** david-lyle has joined #openstack-infra | 21:29 | |
clarkb | jeblair: because right now we basically just have a reunddant pass through to shade | 21:29 |
clarkb | or will once that change works :) | 21:29 |
jeblair | yup | 21:30 |
*** sergmelikyan has joined #openstack-infra | 21:30 | |
*** tiswanso has quit IRC | 21:32 | |
anteaya | Clint: I found the error in this patch: https://review.openstack.org/#/c/229617/ you commented out the initialization of a variable, now it has no value elsewhere in the file | 21:33 |
Clint | anteaya: ah, foolish | 21:33 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Add script to facilitate project renaming in {projects,channels,layout}.yaml https://review.openstack.org/228002 | 21:33 |
anteaya | Clint: I missed it too | 21:34 |
mordred | it turns out variable initialization is important | 21:34 |
anteaya | it turns out | 21:34 |
*** angdraug has joined #openstack-infra | 21:34 | |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: Disable alphabetical ordering check for zuul/layout.yaml https://review.openstack.org/229617 | 21:34 |
*** amotoki has joined #openstack-infra | 21:34 | |
*** regXboi has quit IRC | 21:35 | |
*** doug-fish has quit IRC | 21:36 | |
*** doug-fish has joined #openstack-infra | 21:36 | |
Clint | anteaya: okay, that solves acl file problem #1, i believe | 21:36 |
anteaya | wooo, let's see | 21:36 |
*** Sukhdev has quit IRC | 21:36 | |
anteaya | may I have a new wip patch? | 21:36 |
asselin | jeblair, when you get a chance could you re-review this patch? it's a rebase since your last +2: https://review.openstack.org/#/c/189762/ | 21:38 |
openstackgerrit | Clint Adams proposed openstack-infra/project-config: WIP - testing the rename scripts https://review.openstack.org/229601 | 21:38 |
Clint | anteaya: ^ | 21:38 |
asselin | jeblair, also would be nice to find a time to merge it | 21:38 |
anteaya | Clint: thank you | 21:38 |
*** HeOS has joined #openstack-infra | 21:39 | |
*** doug-fis_ has joined #openstack-infra | 21:39 | |
openstackgerrit | Danilo Ramalho proposed openstack-infra/puppet-gerrit: Setup preconditions for acceptance tests. https://review.openstack.org/219400 | 21:39 |
anteaya | Clint: agreed that catches the ones that were missed as a part of issue 1 | 21:39 |
anteaya | Clint: well done | 21:39 |
anteaya | Clint: do you have a sense of what the second issue is? | 21:39 |
*** amotoki has quit IRC | 21:39 | |
Clint | anteaya: yes, so we could solve that by a) copying the file to both places or b) having the un-renamed project use the file in openstack/ | 21:40 |
*** doug-fish has quit IRC | 21:41 | |
mmmpork | i've got an etherpad and am working on filling things out for the project-rename stuff - https://etherpad.openstack.org/p/project-config-rename-issues | 21:41 |
anteaya | Clint: let's back up a bit | 21:41 |
anteaya | Clint: let's make sure you understand what is happening | 21:41 |
anteaya | so if one project is a lone project it has it's own acl file | 21:42 |
anteaya | if there is a group of projects they may share a single acl file | 21:42 |
*** aeng has quit IRC | 21:42 | |
*** geoffarnold has quit IRC | 21:42 | |
*** doug-fish has joined #openstack-infra | 21:42 | |
Clint | right, like the myriad xstatic ones | 21:42 |
*** geoffarn_ has joined #openstack-infra | 21:42 | |
anteaya | yes | 21:42 |
anteaya | exactly right | 21:42 |
anteaya | so here is my problem | 21:42 |
*** Rockyg has joined #openstack-infra | 21:43 | |
anteaya | we have zero guarentees that all of the projects that share an acl file will end up in the same namespace | 21:43 |
anteaya | and I don't know what to do about that | 21:43 |
*** doug-fis_ has quit IRC | 21:43 | |
anteaya | you mentioned copying the acl file | 21:43 |
anteaya | perhaps that is what we have to do | 21:43 |
anteaya | then removing dups after the patch is generated | 21:44 |
anteaya | that would ensure we don't have acl-less projects after the rename | 21:44 |
anteaya | hmmmmm | 21:44 |
anteaya | anyone have any other thoughts? | 21:44 |
*** aeng has joined #openstack-infra | 21:44 | |
anteaya | Clint: what do you think? | 21:46 |
*** thingee has quit IRC | 21:46 | |
openstackgerrit | Merged openstack-infra/jenkins-job-builder: Add support for Artifactory plugin https://review.openstack.org/173932 | 21:46 |
*** doug-fish has quit IRC | 21:46 | |
anteaya | then we have one acl file in openstack and one in stackforge and remove dups afterwards | 21:46 |
anteaya | I think having some spare acl files that aren't being used left around is better than too few | 21:47 |
mmmpork | what's going to happen to projects that are not renamed? | 21:47 |
zaro | waynr: i guess just like any established project, switching languages is not easy to do. | 21:47 |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: Create puppet-barbican repository https://review.openstack.org/229643 | 21:47 |
anteaya | mmmpork: https://wiki.openstack.org/wiki/Stackforge_Namespace_Retirement | 21:48 |
*** david-lyle has quit IRC | 21:48 | |
anteaya | mmmpork: http://lists.openstack.org/pipermail/openstack-dev/2015-August/071816.html | 21:48 |
anteaya | mmmpork: http://lists.openstack.org/pipermail/openstack-dev/2015-August/072140.html | 21:49 |
mmmpork | right, i understand that | 21:49 |
mmmpork | i'm just asking, what does "retired" mean | 21:49 |
mmmpork | are they going to be run? | 21:49 |
Rockyg | folks, back with more error messages from gerrit: http://paste.openstack.org/show/474955/ clarkb fungi | 21:49 |
mmmpork | why do they need acl files if they aren't going to be run? | 21:49 |
*** sergmelikyan has quit IRC | 21:49 | |
Clint | anteaya: i'm not sure, i think the problem might be avoided by the full list | 21:50 |
anteaya | Clint: possibly | 21:50 |
anteaya | Clint: but the full list will make my eyes sore | 21:50 |
*** pc_m has quit IRC | 21:50 | |
anteaya | Clint: I'm trying to do this in steps | 21:50 |
* Clint nods. | 21:50 | |
anteaya | and let's look at the list that is not moving | 21:50 |
mmmpork | why are we not moving the full list? i'm just curious, what's the reason behind it? | 21:50 |
nibalizer | i believe we need the acl file in placee to set that the repo should no longer accept packages | 21:50 |
mmmpork | nibalizer: ah ok, that makes sense, thanks | 21:51 |
nibalizer | eg https://review.openstack.org/#/c/204685/ | 21:51 |
anteaya | Clint: there are some puppet ones moving and some not | 21:51 |
anteaya | Clint: and I'm going with many of them share acl files | 21:52 |
*** bltavares has quit IRC | 21:52 | |
*** dzamboni has quit IRC | 21:52 | |
anteaya | Clint: so if we just look at puppet as an example | 21:53 |
anteaya | hang on | 21:53 |
anteaya | Clint: the ones not being moved all will have new acl files generated for them anyway | 21:53 |
anteaya | Clint: that move the permissions to read only | 21:53 |
anteaya | Clint: there we are, issue solved, I think | 21:54 |
*** doug-fish has joined #openstack-infra | 21:54 | |
Clint | ok | 21:54 |
*** david-lyle has joined #openstack-infra | 21:54 | |
*** sdake has quit IRC | 21:55 | |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Test config-validate command https://review.openstack.org/229650 | 21:55 |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Simplify command test asserts https://review.openstack.org/229651 | 21:55 |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add dib image command tests https://review.openstack.org/229652 | 21:55 |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add node command tests https://review.openstack.org/229653 | 21:55 |
clarkb | I hvaen't acutally run those locally because I don't have my mysql container handy | 21:56 |
clarkb | but yay more tests | 21:56 |
zaro | signed8bit: do you review upstream Gerrit much? | 21:56 |
clarkb | Rockyg: that is saying that public key auth was denied | 21:56 |
mmmpork | i'm going to add that we expect the new acl files to be generated to the list of things to test | 21:56 |
openstackgerrit | Skyler Berg proposed openstack-infra/project-config: Add ciwatch - CI monitoring tool https://review.openstack.org/216840 | 21:57 |
clarkb | Rockyg: so make sure that your pubkey is in gerrit and matches what you have locally | 21:57 |
mmmpork | i'm not sure how we'd test that but we can at least verify it somewhow | 21:57 |
signed8bit | zaro: nope, new to that... working on getting all setup there in terms of upstream contrib with my company etc. | 21:57 |
*** sdake has joined #openstack-infra | 21:57 | |
Rockyg | clarkb, k, but didn't I use that public ke to do the ssh? | 21:58 |
*** ihrachys has quit IRC | 21:58 | |
clarkb | Rockyg: you did, gerrit doesn't support any other type of ssh auth method | 21:58 |
*** gordc has quit IRC | 21:58 | |
*** doug-fish has quit IRC | 21:58 | |
anteaya | Clint: all projects that don't move have their acl file changed to: https://review.openstack.org/#/c/227016/1/gerrit/acls/openstack/magnetodb.config | 21:59 |
Rockyg | clarkb, so the site accepts it but gerrit doesn't. Hmmm. | 22:00 |
Clint | anteaya: good | 22:00 |
clarkb | Rockyg: when you say site what do you mean? | 22:00 |
Rockyg | review.openstack.org | 22:00 |
clarkb | Rockyg: the direct ssh command? | 22:00 |
Rockyg | clarkb, yes | 22:00 |
clarkb | Rockyg: ya I think what happened there is scp is using your default key | 22:00 |
anteaya | Clint: so I guess that will be a new script for dealing with the projects that are going to be read only | 22:00 |
clarkb | Rockyg: but your ssh command specified a key | 22:00 |
Clint | jasondotstar: is that something you're doing? | 22:01 |
anteaya | Clint: I'm guess jeblair already has a script that makes that change | 22:01 |
clarkb | hrm no thats not the case I just fai lat reading logs | 22:01 |
clarkb | Rockyg: oh! its scping from git.o.o that wont work | 22:01 |
*** glauco___ has joined #openstack-infra | 22:01 | |
clarkb | Rockyg: what does `git remote -v` look like? | 22:01 |
*** pradk has quit IRC | 22:02 | |
*** armax has left #openstack-infra | 22:02 | |
Rockyg | Ah. That's it. It wasn't set. And the only docs I could find to set it set it wrong. | 22:03 |
*** eharney has quit IRC | 22:03 | |
*** geoffarnold has joined #openstack-infra | 22:04 | |
*** glauco has quit IRC | 22:04 | |
Rockyg | both fetch and push are set to git.openstack.org clarkb | 22:04 |
fungi | Rockyg: probably the reason why you're not finding docs on how to set it is that git review is supposed to be setting it for you (after testing the credentials/connection to make sure it succeeds) | 22:04 |
*** sballe has joined #openstack-infra | 22:04 | |
*** geoffarn_ has quit IRC | 22:04 | |
*** svasheka has quit IRC | 22:05 | |
anteaya | fungi: when you have a minute, have you any thoughts on the best way to deal with the multiple projects moving off stackforge that share an acl file? | 22:05 |
fungi | your earlier errors in http://paste.openstack.org/show/474928/ were causing it to fail to test the supplied credentials, so the gerrit remote wasn't being added for you by git review as a result | 22:05 |
*** thingee has joined #openstack-infra | 22:06 | |
*** armax has joined #openstack-infra | 22:06 | |
Rockyg | how to fix? fungi? | 22:06 |
anteaya | fungi: or do you think that just moving the acl file if any project uses it and turning any remaining acl files to read only is the best approach | 22:06 |
*** dramalho has quit IRC | 22:07 | |
*** andreykurilin has quit IRC | 22:08 | |
*** maishsk_ has joined #openstack-infra | 22:08 | |
fungi | anteaya: Clint: mmmpork: probably the easiest thing is to evaluate the projects.yaml the same way manage-projects does... specifically assume that if no acl-file parameter is specified then the acl path should change when the repo namespace changes. if there is an acl-file parameter on a project pointing at teh name of an acl which is a default acl for a project whose namespace is changing, then also | 22:09 |
fungi | edit the path in the acl-file parameter (and likely we could just move any from stackforge to openstack if they're mentioned in an ac-file parameter for a repo which is also on the rename list) | 22:09 |
*** tiswanso has joined #openstack-infra | 22:09 | |
*** aarefiev has quit IRC | 22:09 | |
Clint | fungi: that's what's happening now, unless i misunderstood that last line | 22:09 |
*** maishsk has quit IRC | 22:10 | |
*** maishsk_ is now known as maishsk | 22:10 | |
anteaya | Clint: no I think that is what we are doing | 22:10 |
fungi | worth noting we'll also likely have some orpaned acls which can be deleted too, if they're not the default acl file name for any project and are mentioned in the acl-file parameter of only projects which are being retred | 22:10 |
anteaya | Clint: then after the rename round up any acl files still in teh stackforge namespace and make them read only | 22:10 |
anteaya | fungi: right | 22:10 |
fungi | Rockyg: git remote rm gerrit ; git review -s | 22:10 |
anteaya | Clint: so there will be some post rename clean up | 22:11 |
anteaya | Clint: but that workflow means any renamed project should have an acl file | 22:11 |
* Clint nods. | 22:11 | |
Rockyg | fungi, "Cound not remove dofing section 'remote.gerrit' | 22:11 |
*** tsekiyama has joined #openstack-infra | 22:11 | |
Rockyg | s/dofing/config | 22:11 |
*** vponomaryov has quit IRC | 22:12 | |
Rockyg | git review -s | 22:12 |
Rockyg | sorry... | 22:12 |
*** Sukhdev has joined #openstack-infra | 22:12 | |
fungi | Rockyg: does `git remote -v` mention any lines starting with "gerrit"? | 22:13 |
Rockyg | fungi: no good. won't remove and won't rewrite. Ain't life fun with me around? | 22:13 |
*** tonytan4ever has quit IRC | 22:13 | |
* jasondotstar reads back | 22:13 | |
Rockyg | fungi, nope | 22:13 |
jasondotstar | Clint: ^ | 22:13 |
*** rvasilets_ has quit IRC | 22:13 | |
Clint | jasondotstar: sorry, i was just trying to figure out what the ansible stuff will/won't be doing | 22:14 |
*** tiswanso has quit IRC | 22:14 | |
fungi | Rockyg: okay, then that's why it's failing to remove them at least (or the error message was misleading) | 22:14 |
Clint | jasondotstar: for the stackforge renames | 22:14 |
*** david-lyle has quit IRC | 22:15 | |
jasondotstar | Clint: ah... right. yeah the ansible playbook(s) are suppose to do some of the stop/starting of services.... pre/post flight tasks | 22:15 |
Clint | jasondotstar: okay, so no actually renaming of files/directories or the mysql stuff? | 22:16 |
*** glauco___ has quit IRC | 22:16 | |
*** armax has quit IRC | 22:16 | |
fungi | Rockyg: can you do `git review -s -v` and put the result on paste.openstack.org? the -v will add some debugging output so we can see what it does before breaking | 22:16 |
*** glauco_ has joined #openstack-infra | 22:16 | |
Rockyg | will do | 22:17 |
openstackgerrit | Monty Taylor proposed openstack-infra/shade: Provide short-circuit for finding server networks https://review.openstack.org/229140 | 22:17 |
openstackgerrit | Monty Taylor proposed openstack-infra/shade: Dont throw exception on missing service https://review.openstack.org/229664 | 22:17 |
jasondotstar | Clint: well, we were starting down that road | 22:17 |
Clint | jasondotstar: oh good | 22:17 |
*** mwagner_ has quit IRC | 22:18 | |
jasondotstar | trying to figure out how to hand the ansible playbook a list of projects that require renaming | 22:18 |
Rockyg | fungi, http://paste.openstack.org/show/474966/ | 22:18 |
jasondotstar | do the things that only need to be done once, then iterate of the list to perform the renaming and deletion of hosts (if required) | 22:19 |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add dib image command tests https://review.openstack.org/229652 | 22:19 |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add node command tests https://review.openstack.org/229653 | 22:19 |
clarkb | I think that will fix the race that I hit | 22:19 |
Clint | jasondotstar: ok, is there a WIP in gerrit somewhere? | 22:19 |
jasondotstar | Clint: I'm working through the best way to go about it, but that's the goal AFAICT | 22:19 |
jasondotstar | Clint: we want to improve what's here: https://review.openstack.org/#/c/105057/ | 22:20 |
Clint | jasondotstar: ok, thanks | 22:20 |
jasondotstar | Clint: np. | 22:20 |
*** redixin has quit IRC | 22:21 | |
fungi | Rockyg: okay, so i think i see part of the problem... mordred added a feature to support assuming that the push url could be the gerrit remote. i need to look back at the code but i think this should only come into play if there is no .gitreview hostname/port | 22:21 |
openstackgerrit | fumihiko kakuma proposed openstack-infra/project-config: Add job for the ovs mechanism driver with native mode https://review.openstack.org/221143 | 22:22 |
*** ociuhandu has quit IRC | 22:22 | |
fungi | Rockyg: can you put the results of `cat .gitreview` on paste.openstack.org? | 22:22 |
*** mriedem has quit IRC | 22:22 | |
Rockyg | fungi: http://paste.openstack.org/show/474968/ | 22:24 |
*** ociuhandu has joined #openstack-infra | 22:24 | |
*** marcusvrn_ has quit IRC | 22:25 | |
fungi | Rockyg: weird. i wonder if something is causing git-review not to read that. looking in the source code for it now for potential causes | 22:25 |
*** ddieterly has quit IRC | 22:28 | |
anteaya | Rockyg: I can't seem to find tristanC or tonyb online at the moment, but the point of using the git repo was not to exclude candidates | 22:29 |
*** sabeen3 has quit IRC | 22:29 | |
anteaya | Rockyg: be sure to email both of them with your candidate statement well before the deadline with an explaination of the issue and a link to this channel log | 22:29 |
Rockyg | Hey, story of my life. If it's a computer and has software, it will break for me ;-) | 22:30 |
*** devlaps has quit IRC | 22:30 | |
anteaya | i hear ya | 22:30 |
*** bpokorny_ has quit IRC | 22:30 | |
anteaya | but the point is to be clear about intentions | 22:30 |
Rockyg | And thank you anteaya. I'll send off some emails. | 22:30 |
anteaya | so be clear about your intentions to the electoral officials before the deadline | 22:30 |
*** bpokorny has joined #openstack-infra | 22:30 | |
openstackgerrit | Merged openstack-infra/project-config: Disable alphabetical ordering check for zuul/layout.yaml https://review.openstack.org/229617 | 22:30 |
*** dims__ has joined #openstack-infra | 22:30 | |
anteaya | then come back and we will continue to help but I don't want you to feel pressured about the dealine | 22:31 |
*** earlephilhower has quit IRC | 22:32 | |
*** e0ne has quit IRC | 22:33 | |
*** dims_ has quit IRC | 22:33 | |
*** glauco_ has quit IRC | 22:34 | |
*** david-lyle has joined #openstack-infra | 22:36 | |
Rockyg | anteaya, I've shot them an email with the file. I'm going to post to dev and keep an eye out here. | 22:36 |
anteaya | Rockyg: cover all your bases | 22:37 |
anteaya | tristanC is probably done for the day and hopefully tonyb should be online soon | 22:37 |
tonyb | Reading scrollback .... | 22:39 |
tristanC | Rockyg: Greeting sir | 22:40 |
*** tiswanso has joined #openstack-infra | 22:40 | |
* fungi feels like he's stepped into a peanuts comic strip | 22:40 | |
anteaya | tristanC: tonyb oh great | 22:40 |
tonyb | fungi: :) | 22:40 |
tonyb | fungi: good greif ;p | 22:40 |
anteaya | tristanC: Rockyg is a woman | 22:40 |
anteaya | and is having an issue with ssh keys and the election repo, which we are trying to help with | 22:41 |
anteaya | but the point is her candidacy | 22:41 |
anteaya | I didn't want her to miss the deadline on a technical issue | 22:41 |
tonyb | fungi: is there *any* change that that error is related to not havign signed the ICA (or whatever it's called) | 22:41 |
anteaya | tonyb: ICLA | 22:42 |
anteaya | not that we have seen | 22:42 |
fungi | tonyb: nope, i've checked that | 22:42 |
anteaya | seems to be ssh key related, permissions issue | 22:42 |
fungi | or a git/git-review configuration problem, or a bug in git-review (possibly only exposed under whatever platform she's running) | 22:43 |
anteaya | tristanC: I'm looking on the original etherpad for the git repo workflow: https://etherpad.openstack.org/p/scaling-election-tooling | 22:43 |
Rockyg | Yeah. Hate my platform. Win7 with lots of stuff added by the internal IT group | 22:44 |
anteaya | I had thought I put a provision in there about someone having the option to email their candidacy to an election official who could post to the repo for them | 22:44 |
anteaya | but I can't seem to find that written down | 22:44 |
anteaya | Rockyg: woooooo | 22:44 |
*** xyang1 has quit IRC | 22:45 | |
jeblair | mordred, jhesketh: in https://review.openstack.org/156178 why is the 'ipv6-preferred' option necessary? | 22:45 |
jeblair | Rockyg: do you know if jerry zhao is on irc? | 22:45 |
Rockyg | I need to exit this for a short bit to get the emails out, (can't email from outside corporate; can't IRC inside corporate) but I'll be back! | 22:45 |
tristanC | Rockyg: apologize, I meant Madam :) Did you send election officials your candidacy ? | 22:45 |
tonyb | anteaya: I havn't talked that over with tristanC but I'd be happy to s=do that with a review comment and Rockyg's +1 | 22:45 |
*** thingee has quit IRC | 22:45 | |
pleia2 | so, general mailing list creation question, how much of gatekeepers should we be WRT creation of new lists? seems like this should just be a topic on -dev like everything else, but I'm not sure about our role here https://review.openstack.org/#/c/223556/ | 22:45 |
*** ybathia has quit IRC | 22:46 | |
anteaya | tonyb tristanC talk amongst yourselves, just sharing thoughts, the decision is yours to make | 22:46 |
jeblair | pleia2: we should push back a bit | 22:46 |
*** salv-orlando has joined #openstack-infra | 22:46 | |
pleia2 | jeblair: ok | 22:46 |
tristanC | tonyb: agreed | 22:46 |
jeblair | fungi, clarkb: you also reviewed https://review.openstack.org/156178 ... do you know why the ipv6-preferred option is there? | 22:47 |
anteaya | sounds like she may have had to leave irc to send the emails | 22:47 |
fungi | jeblair: he's jerryz when he does pop in here | 22:48 |
*** abregman has quit IRC | 22:48 | |
jeblair | fungi: thx | 22:48 |
anteaya | thanks for understanding tristanC and tonyb :) | 22:48 |
*** Rockyg has quit IRC | 22:48 | |
* anteaya goes to cut kindling, time for a fire | 22:48 | |
clarkb | jeblair jerryz had an ipv6 only cloud | 22:48 |
clarkb | that was an easy way to support the use case | 22:49 |
*** claudiub has quit IRC | 22:49 | |
jeblair | clarkb: yeah, so i mean, why is ipv6 not always preferred for everyone? | 22:49 |
openstackgerrit | Merged openstack/diskimage-builder: Output failing lines when dib-lint finds wrong indents https://review.openstack.org/227223 | 22:49 |
fungi | jeblair: clarkb: well, a cloud where the only _global_ addressing was ipv6 but there were still non-globally-routable ipv4 addresses on the instances | 22:49 |
*** Rockyg has joined #openstack-infra | 22:49 | |
clarkb | jeblair the way the db schema is set up would be hard to always support | 22:49 |
jeblair | clarkb: er, change the schema? | 22:50 |
Rockyg | ok, I'm back. I think I got the messages posted to tonyb, tristanC and the dev mailing list. | 22:50 |
fungi | and i'd be in favor of an "if there's an ipv6 address use it" default behavior change in nodepool | 22:50 |
jeblair | fungi: yeah, i mean, that's kind of what every ipv6 program should do, right? | 22:50 |
clarkb | jeblair sure but thats hard because no one will review mordreds change to support migrations | 22:50 |
fungi | that's what _i_ do, and i guess i'm sort of an ipv6 program | 22:50 |
tonyb | Rockyg: We'll post a revoew for you to +1 pretty soon, are you going to be around for a bit? | 22:50 |
tristanC | Rockyg: can you +1 your candidacy here: https://review.openstack.org/229668 | 22:50 |
fungi | that was fast! | 22:50 |
*** tjones has joined #openstack-infra | 22:50 | |
Rockyg | Yup. | 22:51 |
anteaya | yay! | 22:51 |
* anteaya goes outside now | 22:51 | |
clarkb | jeblair its not perfect but the changes yo make it better languish | 22:51 |
clarkb | I think the migration change even came with tests not sure what statr it is in now | 22:51 |
jeblair | clarkb: okay, that doesn't mean we have to limp along... if we can't merge that change, then just change the schema. to date nodepool schema changes are "drop the relevant table"; that's not so hard. | 22:51 |
fungi | i'd still like to dig into the git-review error a bit, and at least get a bug report opened with the salient details, though i have a sinking suspicion i'll be unable to reproduce that behavior without windows 7 | 22:52 |
Rockyg | Done. I tried to add my IRC handle to the commit message, but you guys will have to. | 22:52 |
jeblair | i mean, i'm looking at how to fix mordred's shade change, and i'm running into code from that that clearly wasn't designed for the long haul | 22:52 |
Rockyg | Thanks anteaya! | 22:52 |
*** reed has joined #openstack-infra | 22:53 | |
jeblair | clarkb, fungi: so anyway, what i haven't heard is "this option is necessary because..." | 22:53 |
* Rockyg waves to a retreating anteaya enrobed in the light of the setting sun | 22:53 | |
*** Daisy has joined #openstack-infra | 22:53 | |
fungi | jeblair: i think it was "this option is necessary to avoid a default behavior change" but i agree it seems like a good default behavior, so i'm in favor of just having it change | 22:53 |
jeblair | clarkb, fungi: so would assuming ipv6 is preferred always work? | 22:53 |
jeblair | fungi: ya, sounds like you think so | 22:53 |
Rockyg | tristanC, tonyb Done! | 22:54 |
fungi | i think it solves the use case described to us before, and in much simpler to implement ways | 22:54 |
fungi | (no additional knobs required) | 22:54 |
fungi | though i could be convinced of a prefer-ipv4 option for people with broken environments (if someone steps forward with a compelling reason/patch) | 22:55 |
clarkb | yes would be fine | 22:55 |
tonyb | Rockyg: great. I'll get it merged ASAP. | 22:56 |
clarkb | but it would be double plus good to get migrations in before a new schema since we have the change written | 22:56 |
*** spzala has quit IRC | 22:56 | |
fungi | tristanC: tonyb: i'm putting together the requested magnum election rolls now. should have it ready shortly | 22:57 |
jeblair | clarkb: yeah. it's been in merge conflict since march | 22:57 |
*** spzala has joined #openstack-infra | 22:57 | |
tonyb | fungi: Thanks. | 22:57 |
reed | if I want to see which security group is applied to an instance, using shade, is there a special API call? | 22:57 |
tristanC | fungi: awesome! | 22:57 |
*** bdemers has joined #openstack-infra | 22:58 | |
jeblair | clarkb, fungi: the ipv6 change is broken as-is -- it does not handle periodic connectivity checks :( | 22:58 |
mtreinish | reed: the security groups for a server should be in the response from nova, it should still be there when using shade | 22:59 |
jeblair | clarkb, fungi: oh, wait, it will because of the db field override | 22:59 |
jeblair | overload rather | 22:59 |
asselin | jeblair, when you get a chance could you re-review this patch? it's a rebase since your last +2: https://review.openstack.org/#/c/189762/ | 22:59 |
Rockyg | jeblair, I don't see him, but I can ping his phone if you need him.. | 22:59 |
jeblair | Rockyg: i think i have what i need, thanks | 23:00 |
*** mriedem has joined #openstack-infra | 23:00 | |
*** tiswanso has quit IRC | 23:01 | |
*** Krinkle has left #openstack-infra | 23:04 | |
*** Daisy_ has joined #openstack-infra | 23:05 | |
*** ociuhandu has quit IRC | 23:05 | |
*** odgrim has quit IRC | 23:05 | |
openstackgerrit | Merged openstack-infra/puppet-elasticsearch: Add acceptance tests for puppet-elasticsearch. https://review.openstack.org/224301 | 23:07 |
*** geoffarnold has quit IRC | 23:07 | |
*** signed8bit is now known as signed8bit_ZZZzz | 23:07 | |
*** geoffarnold has joined #openstack-infra | 23:07 | |
*** jaypipes has quit IRC | 23:07 | |
*** otter768 has joined #openstack-infra | 23:08 | |
*** Daisy_ has quit IRC | 23:08 | |
*** Daisy_ has joined #openstack-infra | 23:08 | |
*** erlon has quit IRC | 23:08 | |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Use shade for all OpenStack interactions https://review.openstack.org/226751 | 23:09 |
*** Daisy has quit IRC | 23:09 | |
reed | mtreinish, yeah, indeed... I was doublechecking if there is a specific call like there is in libcloud | 23:09 |
jeblair | clarkb: ^ | 23:10 |
*** marzif has quit IRC | 23:10 | |
*** tsekiyama has quit IRC | 23:10 | |
jeblair | clarkb: i did not implement the ipv6 changes, i just made it work with what we have now for "simplicity" :) | 23:10 |
*** signed8bit_ZZZzz is now known as signed8bit | 23:10 | |
*** tsekiyama has joined #openstack-infra | 23:10 | |
mnaser | https://git.openstack.org/openstack-infra/nodepool | 23:12 |
mnaser | blank page? | 23:12 |
* anteaya pulls her robe of the setting sun a little closer | 23:12 | |
anteaya | thanks Rockyg | 23:12 |
jeblair | mnaser: https://git.openstack.org/cgit/openstack-infra/nodepool | 23:12 |
mnaser | in that case, broken URL on http://docs.openstack.org/infra/system-config/nodepool.html :) | 23:13 |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add dib image command tests https://review.openstack.org/229652 | 23:13 |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add node command tests https://review.openstack.org/229653 | 23:13 |
jeblair | mnaser: yeah, cgit would probably be better for that | 23:14 |
pleia2 | mnaser: do you want to submit the patch to openstack-infra/system-config to fix that up? | 23:14 |
jeblair | clarkb: ps18 fixes a lot of the tests; i'm working on the leak test now | 23:14 |
mnaser | sure ill do it in a sec, pleia2 | 23:14 |
clarkb | jeblair: ok | 23:14 |
clarkb | I have a stack of independent test changes that should give us even more coverage | 23:15 |
clarkb | granted its for things less likely to be affected by shade | 23:15 |
*** hdd has quit IRC | 23:16 | |
openstackgerrit | Mohammed Naser proposed openstack-infra/system-config: Fix project URL for Nodepool https://review.openstack.org/229671 | 23:17 |
*** sc` has quit IRC | 23:18 | |
jeblair | mnaser: thanks! :) | 23:20 |
mnaser | np.. trying to figure out a way to automate our image builds, it's become quite hectic | 23:20 |
*** armax has joined #openstack-infra | 23:21 | |
jeblair | mnaser: oh interesting -- thinking of using nodepool for the side effect of scheduled image builds, but not actually for the node provisioning? | 23:21 |
jhesketh | Morning | 23:21 |
*** vmbrasseur is now known as vmb_out | 23:21 | |
timrc | I really wish Gerrit allowed you to configure how to review a patch or at least made it easier to navigate files. A Gertty diff makes it a little better, but not much. | 23:21 |
anteaya | morning jhesketh | 23:22 |
mnaser | yes jeblair, we already use diskimage-builder to build the images, but it's not ideal.. seems like nodepool has the whole build-it-and-test-it process nailed down | 23:22 |
anteaya | timrc: have you tried gertty? | 23:22 |
mnaser | don't need to provision nodes but the building part is what's ideal | 23:22 |
timrc | anteaya, Indeed. | 23:22 |
jeblair | mnaser: it doesn't do acceptance tests on the images yet, but i think yolanda has been working on a spec for that | 23:22 |
anteaya | timrc: does it make it easier for you to navigate files? | 23:22 |
*** rellerreller has joined #openstack-infra | 23:22 | |
*** amotoki has joined #openstack-infra | 23:22 | |
openstackgerrit | Michael Johnson proposed openstack-infra/project-config: Add release jobs to the octavia project https://review.openstack.org/229518 | 23:23 |
*** thorst has joined #openstack-infra | 23:23 | |
jeblair | timrc: i would happily accept a gertty patch which was "skip to the next/prev file in diff" :) | 23:23 |
*** stevemar has quit IRC | 23:23 | |
jeblair | timrc: if that's the sort of thing you're talking about | 23:23 |
mnaser | jeblair: i see, just still scouting out different solutions... just checking if it boots and becomes pingable should be enough | 23:23 |
*** stevemar has joined #openstack-infra | 23:24 | |
zaro | timrc: have you used a review system that does it in a better way? | 23:24 |
timrc | jeblair, Well that's cool and all, but I actually thing alphabetical ordering is arbitrary and mostly not an intuitive ordering. | 23:24 |
fungi | tristanC: tonyb: oops, i forgot to attach to my first e-mail. another is on the way | 23:25 |
timrc | zaro, No. But it's a persistent pain point. | 23:25 |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Use shade for all OpenStack interactions https://review.openstack.org/226751 | 23:25 |
zaro | jeblair: wasn't there a shortcut for that? | 23:25 |
timrc | I'd almost like it if the reviewer could offer a "recommended" way of reviewing the change that I could opt for. | 23:25 |
openstackgerrit | Merged openstack-infra/irc-meetings: Change Solum meeting time https://review.openstack.org/228441 | 23:25 |
mmedvede | jhesketh: good morning. Can you please look at https://review.openstack.org/#/c/229582/1, we are using zuul swift upload in our CI, and that would help. | 23:25 |
*** sigmavirus24 is now known as sigmavirus24_awa | 23:26 | |
jhesketh | mmedvede: will do | 23:26 |
mmedvede | jhesketh: thank you. Let me know if I am going the wrong way about it | 23:27 |
*** amotoki has quit IRC | 23:27 | |
timrc | zaro, I'm not sure if enforcing ordering is a good thing or not or what opinions are better than others. For example, maybe things that appear tests should be ordered before any implementation code? For me, at least, that's usually a good starting point for me. | 23:27 |
*** tjones has quit IRC | 23:28 | |
timrc | appear to be tests* | 23:28 |
*** geoffarnold has quit IRC | 23:28 | |
*** stevemar has quit IRC | 23:28 | |
*** geoffarnold has joined #openstack-infra | 23:29 | |
*** sc` has joined #openstack-infra | 23:31 | |
*** tjones has joined #openstack-infra | 23:31 | |
*** aysyd has quit IRC | 23:32 | |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add dib image command tests https://review.openstack.org/229652 | 23:32 |
openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add node command tests https://review.openstack.org/229653 | 23:32 |
clarkb | now with fewer bugs | 23:32 |
*** mudassirlatif_ has joined #openstack-infra | 23:32 | |
anteaya | yay fewer bugs | 23:32 |
mmmpork | i have a general infra question.. do we use a particular unit test library in python in our infra code? | 23:33 |
mmmpork | i've been doing some digging but haven't found anything specific | 23:33 |
*** thorst has quit IRC | 23:33 | |
*** darvon has quit IRC | 23:33 | |
mmmpork | if one was going to write unit tests, so far i've seen the practice is to manually error and then report the errors | 23:33 |
mmmpork | but i haven't looked too deeply | 23:33 |
clarkb | mmmpork: we use testtools | 23:33 |
mmmpork | clarkb: awesome, thanks | 23:34 |
clarkb | as it comes along with subunit which is required for testr | 23:34 |
mmmpork | ok, pleia2 had pointed me at testr but we weren't sure about infra specific stuff | 23:34 |
*** darvon has joined #openstack-infra | 23:34 | |
clarkb | mmmpork: well its not infra specific | 23:34 |
clarkb | but we use it | 23:34 |
*** mudassirlatif has quit IRC | 23:34 | |
*** mudassirlatif_ is now known as mudassirlatif | 23:34 | |
clarkb | mmmpork: zuul and nodepool have python unittest suites | 23:34 |
clarkb | may serve as examples | 23:35 |
mmmpork | thanks! | 23:35 |
mmmpork | i'm working on some tests for the project-config script we're writing | 23:35 |
*** mudassirlatif has quit IRC | 23:35 | |
timrc | mmmpork, We also use mock in places inside of our unit tests. | 23:36 |
mmmpork | timrc: that's right i'm remember that from the nodepool tests now i think | 23:36 |
*** Swami has joined #openstack-infra | 23:36 | |
mmmpork | my brain feels so tiny hahaha | 23:36 |
mmmpork | and infra is sooo big :D | 23:36 |
jeblair | timrc: git has the idea of an orderFile, but it's a local repository option. so you could check an orderfile in, but you'd need to set diff.orderFile in your local repo in order to use it. | 23:36 |
jeblair | timrc: gerrit would not know to use it | 23:36 |
jeblair | timrc: however, it would be possible to add that as a project config option in gerrit | 23:37 |
timrc | jeblair, Hm. Interesting | 23:37 |
jeblair | timrc: and it would be possible to have gertty obey that too (though it might actually obey diff.orderFile today, i haven't checked) | 23:37 |
anteaya | and everyone could make their own orderfile? | 23:37 |
anteaya | because I can't see folks agreeing on order of files | 23:38 |
jeblair | anteaya: either way -- you can make your own right now, if you want to check one in and suggest other people use it, you can do that too. | 23:38 |
*** ybathia has joined #openstack-infra | 23:38 | |
openstackgerrit | Armando Migliaccio proposed openstack-infra/project-config: Skip lbaasv2-dsvm-minimal on doc changes https://review.openstack.org/229680 | 23:38 |
anteaya | jeblair: ah great | 23:38 |
timrc | jeblair, I might just play around with that an see if gertty does the Right Thing with its diff file. | 23:39 |
jeblair | anteaya: there is no current mechanism to "force" or even suggest that gerrit or gertty use it; that's what i've suggested timrc could implement, though i would hope such implementations would be user customizable (they certainly would in gertty, hopefully in gerrit) | 23:39 |
timrc | and* | 23:39 |
*** otter768 has quit IRC | 23:39 | |
jeblair | timrc: it does | 23:39 |
jeblair | timrc: i just checked | 23:39 |
anteaya | ah it would be interesting | 23:40 |
waynr | can anyone point me to documentation or tell me of a sweet command to easily create my own local JJB package for use as described here: http://stackoverflow.com/questions/14737500/get-pip-to-work-with-git-and-github-repository | 23:40 |
clarkb | we have more quota now \o/ | 23:40 |
clarkb | mordred: jeblair fungi pleia2 jhesketh et al | 23:40 |
fungi | clarkb: w00t! | 23:40 |
timrc | I'm curious to know how other people feel about this. I feel certain level of frustration conforming to the default. I always think to myself "Why am I reviewing this patch like this" and then I end up jumping around. | 23:40 |
timrc | And losing context. | 23:40 |
* fungi changes into his w00t suit | 23:40 | |
waynr | basically I am working on tooling that needs to consume JJB as a dependency | 23:40 |
jeblair | timrc: i think the fastest way to happiness for you is to create ~timrc/.git-order-file, and then set "git config --global diff.orderFile ~timrc/.git-order-file" | 23:41 |
jeblair | timrc: then all of your git repos would obey rules like "tests first" or whatever using standard diff tools (including gertty) | 23:41 |
anteaya | yayyaya | 23:41 |
anteaya | more quota | 23:41 |
anteaya | clarkb: do we know how much? | 23:41 |
clarkb | anteaya: yes 30 more instances and 128GB more ram | 23:42 |
*** mudassirlatif has joined #openstack-infra | 23:42 | |
*** dingyichen has joined #openstack-infra | 23:42 | |
anteaya | woooooo | 23:42 |
*** mudassirlatif has quit IRC | 23:43 | |
pleia2 | great | 23:43 |
clarkb | waynr: you can pip install the git path | 23:44 |
openstackgerrit | Merged openstack-infra/system-config: Make apply test more configurable https://review.openstack.org/226605 | 23:44 |
fungi | timrc: i also jump around a lot in the diff when reviewing and often resort to checking out (c in gertty) the change so i can inspect things differently on larger changes. a couple features i've been thinking of submitting that might help are expand/collapse files with macros for expand/collapse all (and maybe a default preference option), and also keyword searching in the diff view | 23:44 |
clarkb | waynr: pip has docs but its something like pip install -e git+https://git.openstack.org/openstack-infra/jenkins-job-builder | 23:44 |
waynr | sounds promising, I will try that out--thanks clarkb! | 23:45 |
fungi | timrc: though i haven't spent much time in the diff presentation layer to see how painful it's going to be to implement those | 23:46 |
fungi | er, much time looking at the code for the diff presentation | 23:47 |
*** ddieterly has joined #openstack-infra | 23:49 | |
*** geoffarnold has quit IRC | 23:50 | |
*** geoffarn_ has joined #openstack-infra | 23:50 | |
timrc | fungi, keyword searching would be huge... | 23:50 |
timrc | I do feel like if we default to a linear progression that ordering changes by subject would be better. So: commit message, docs, testing, implementation would be more intuitive to me because presumably this progression attempts to maximize understanding. | 23:54 |
timrc | I think this could largely be achieved with regex. | 23:55 |
timrc | But dunno that sounds awfully opinionated :) | 23:55 |
*** ZZelle_ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!