Wednesday, 2014-06-04

sdaguemordred: that I don't know, deflate is pretty late in the request processing00:01
sdagueso it might actually be gzipping after the cache00:01
openstackgerritKhai Do proposed a change to openstack-infra/config: upgrade jenkins build-timeout plugin due to a performance bug  https://review.openstack.org/9765100:01
openstackgerritClark Boylan proposed a change to openstack-infra/zuul: Cache zuul status.json for one second.  https://review.openstack.org/9765200:02
sdagueIn random web world where I've lived you'd often put varnish on top of all of that to cache the output00:02
clarkbjhesketh: mordred ^ that should do caching. I looked to see if paste.httpserver has it built in but didn't see anything so meh00:02
*** trinaths has joined #openstack-infra00:02
mordredsdague: that seems excessive for one file...00:03
*** yamahata has joined #openstack-infra00:03
sdaguemordred: agreed00:04
sdagueapache should be pretty fast here regardless, faster than doing this in python00:05
mordredclarkb: I agree with you00:05
*** shivharis has joined #openstack-infra00:07
jheskethmordred: why did you get rid of the check for the deflate apache module?00:07
mordredjhesketh: sdague said it wasn't needed00:07
jheskethright, I just figured it can't hurt00:08
sdaguejhesketh: we don't need it on the other box where we use this00:08
clarkbjhesketh: mordred I am actually wondering if I should keep the Vary: Accept-Encoding in the zuul change00:08
clarkbor should we set that in the apache change. apache is the one doing the deflate so should deal with that instead00:09
sdagueclarkb: I think apache will just do it with magic00:09
harlowjamagci!00:09
clarkboh good00:09
sdagueit's pretty easily testable once it's in place00:09
clarkbbecause zuul doesn't need to set it as it only ever has one encoding with my change00:09
clarkband zuul shouldn't assume it is proxied so apache should magic00:10
sdagueyeh00:10
mordredyah00:10
*** praneshp has joined #openstack-infra00:11
clarkbok stuff is approved00:14
*** ArxCruz has joined #openstack-infra00:14
*** ArxCruz has quit IRC00:15
clarkbmordred: oh crap there is even a test in zuul to make sure that the data is gzipped...00:15
* clarkb rips this out00:16
*** sarob has quit IRC00:16
*** sarob_ has joined #openstack-infra00:16
*** homeless has quit IRC00:17
mordredclarkb: haha00:18
*** mriedem has joined #openstack-infra00:19
*** mmaglana has joined #openstack-infra00:21
*** radez` is now known as radez_g0n300:21
*** lcheng_ has quit IRC00:22
openstackgerritClark Boylan proposed a change to openstack-infra/zuul: Stop gzipping zuul status json in zuul  https://review.openstack.org/9764900:22
openstackgerritClark Boylan proposed a change to openstack-infra/zuul: Cache zuul status.json for one second.  https://review.openstack.org/9765200:22
clarkbmordred: jhesketh ^ that should do it00:22
*** matsuhashi has joined #openstack-infra00:23
clarkband sorry for not actually checking. of course there would be a test to test this problem :)00:23
morganfainbergclarkb, it's the test you expect to not exist that does (and break things) ... always...00:23
clarkbmorganfainberg: indeed00:24
harlowjaqq, are reviews that should be running through the gate actively working/processing?00:24
clarkbharlowja: yes00:24
harlowjahmmm00:24
harlowjak00:24
clarkbharlowja: slowly but yes00:24
harlowjagotcha00:24
clarkbharlowja: do you have an example of something that isn't right?00:24
mordredclarkb: you haz reviews from jhesketh00:25
*** swes has quit IRC00:25
*** _nadya_ has quit IRC00:25
*** unicell has joined #openstack-infra00:26
*** olaph has quit IRC00:26
harlowjahmmm, clarkb probably a mess up in my czuul tool, shows up on the zuul site, odd00:26
*** arnaud has joined #openstack-infra00:26
harlowjaclarkb ya, its something broke in my tool, hmmm00:27
harlowjanm then :-P00:28
harlowjanot showing ones that are full queued for some reason00:28
*** zhiyan_ is now known as zhiyan00:28
sdagueoh mordred while you seem to be active, this is what you were talking about at summit right - https://review.openstack.org/#/c/97442/00:32
*** nati_uen_ has quit IRC00:32
*** nati_ueno has joined #openstack-infra00:33
*** sarob_ has quit IRC00:33
openstackgerritK Jonathan Harker proposed a change to openstack-infra/config: Move variables in manifests/site.pp to top of file  https://review.openstack.org/9765900:33
mordredsdague: yup00:33
sdagueI think the neutron fail is a normal race, and not an actual issue00:34
sdaguebut I'll wait for things to calm down a little before rechecking that00:34
*** bcrochet is now known as bcrochet|g0ne00:34
*** lcostantino has quit IRC00:35
*** lcheng_ has joined #openstack-infra00:35
sdaguehey, ianw so the fact that Red Hat CI is running check experimental automatically is putting some additional load back on all the things. Can we have it stop that?00:35
sdaguethat kicks off another 6 dsvm jobs for every devstack change00:36
ianwsdague: yes00:36
sdagueianw: cool00:37
*** otter768 has joined #openstack-infra00:37
sdaguejust thinking about ways to save load until we recover00:37
ianwsdague: no worries; just restarted it and it won't post any more experimental requests for now00:38
*** nati_ueno has quit IRC00:38
sdagueawesome, thanks00:39
*** nati_ueno has joined #openstack-infra00:39
*** ramashri has quit IRC00:40
sdagueianw: this was mostly because of the f20 job?00:40
sdaguewe could probably move that to devstack non-voting00:40
sdagueit looks close00:40
ianwsdague: yes, i was only interested in the stability of that00:40
sdagueyou want to propose moving that out of experimental to main non-voting?00:40
sdagueI'd be ok with that00:41
ianwsdague: per infra meeting this morning, i'm just validating the f20 partner image on hpcloud to give it some redundancy00:41
openstackgerritClark Boylan proposed a change to openstack-infra/zuul: Cache zuul status.json for one second.  https://review.openstack.org/9765200:41
clarkbjhesketh: mordred ^00:41
ianwsdague: so i'd say let's get that up, and then we can move to non-voting00:41
sdaguecool00:41
ianws/morning/time of day for you/ :)00:42
sdagueianw: yeh, what TZ are you in? I realize I have no grounding on that00:42
clarkbmordred: jhesketh I think I am going to relocate to home now00:43
ianwsdague: AEST (UTC+10)00:43
clarkbmordred: jhesketh but afterwards maybe we do the scary thing and restart zuul00:43
clarkband apply those apache changes00:43
jheskethsure00:43
sdagueoh, cool. I thought UTC+1 for some reason.00:44
*** lcheng_ has quit IRC00:45
trinathsanteaya: What is the criteria for CI to be good for voting.00:45
*** kevinbenton has joined #openstack-infra00:45
mordredclarkb: ++00:46
*** nati_ueno has quit IRC00:47
sdaguewhen did zuul get a favicon?00:47
sdagueor is that a side effect of the apache front end00:47
* krotscheck looks askance at the current puppet module versions in config....00:47
*** nati_ueno has joined #openstack-infra00:48
sdagueor a side effect of me not noticing things00:48
clarkbok suspending now back in a bit ish00:48
clarkbactually wait. going to make a change ot install yappi on zuul nodes00:49
*** mmaglana has quit IRC00:51
openstackgerritClark Boylan proposed a change to openstack-infra/config: Install yappi on zuul nodes.  https://review.openstack.org/9766400:51
clarkbjhesketh: mordred ^00:51
*** mmaglana has joined #openstack-infra00:51
*** SumitNaiksatam has quit IRC00:53
*** yamahata has quit IRC00:53
*** yamahata has joined #openstack-infra00:54
*** mmaglana has quit IRC00:55
*** trinath_s has joined #openstack-infra00:57
fungiclarkb: 97652 lgtm. any reason not to approve yet?00:59
clarkbfungi: probably not01:00
* fungi is caught back up on all the exciting01:00
clarkbit passes tests locally01:00
clarkbfungi: I still need to move myself to home01:00
*** maurosr has quit IRC01:00
clarkbbut next steps are to merge things and restart zuul and apply new apache config01:00
fungiright. i'm here for the restart. have a good stroll01:01
clarkbcool01:01
clarkbok really suspending now01:01
clarkbback in ~20 minutes01:01
*** maurosr has joined #openstack-infra01:02
*** marcoemorais has quit IRC01:04
*** _nadya_ has joined #openstack-infra01:04
*** mmaglana has joined #openstack-infra01:08
*** arnaud has quit IRC01:08
openstackgerritIan Wienand proposed a change to openstack-infra/config: Make install_puppet.sh more robust against failure  https://review.openstack.org/9766501:10
*** trinaths has quit IRC01:11
*** camunoz has quit IRC01:16
openstackgerritA change was merged to openstack-infra/nodepool: Check the returned image status  https://review.openstack.org/9756401:17
openstackgerritA change was merged to openstack-infra/nodepool: Prevent listserver tasks from piling up  https://review.openstack.org/9757001:18
openstackgerritA change was merged to openstack-infra/nodepool: Log task manager queue length  https://review.openstack.org/9757401:18
*** nati_uen_ has joined #openstack-infra01:18
openstackgerritA change was merged to openstack-infra/nodepool: Log task durations  https://review.openstack.org/9757501:18
*** shivharis has quit IRC01:21
openstackgerritIan Wienand proposed a change to openstack-infra/config: Make install_puppet.sh more robust against failure  https://review.openstack.org/9766501:21
*** nati_ueno has quit IRC01:22
*** otherwiseguy has joined #openstack-infra01:24
mordredfungi: in the 30G nodes - are we making use of all 8 cpus?01:25
fungimordred: good question. tempest bases its parallelism on cpu count i think01:27
mordredalso -01:27
funginot sure whether straight testr runs do01:27
clarkbwe are01:28
lifelesstestr --parallel works off of cpu count by default01:28
fungithat answers that, then01:28
clarkbttempest uses 401:29
clarkbservices use the others01:29
*** camunoz has joined #openstack-infra01:29
*** gokrokve has joined #openstack-infra01:29
clarkband for unittests we use all 801:29
bodepdmy latest build blew up: http://logs.openstack.org/57/97357/3/check/gate-config-layout/26774e2/console.html01:29
bodepdI assume I just wait patiently ;)01:29
*** jhesketh has quit IRC01:30
*** praneshp has quit IRC01:30
bodepdthis has weird puppet failures: http://logs.openstack.org/57/97357/3/check/gate-config-puppet-apply-precise/87c0a49/console.html01:31
jesusaurusbodepd: yeah, puppet-apply-precise is a pet project of mine. hopefully soon it will start passing as i work out the issues01:32
jesusaurusthere's also puppet-apply-centos6 as an experimental job01:33
ianwbodepd: i'm seeing that on hpcloud too01:33
ianwi'm manually running some of the setup scripts, and getting various "error: [Errno 104] Connection reset by peer"01:33
ianwfrom curl get's of scripts, pip installs01:33
jesusaurusis pypi.python having a sad?01:33
ianwhttp://logs.openstack.org/57/97357/3/check/gate-config-layout/26774e2/console.html <- you're seeing it on pip01:34
ianwjesusaurus: i had issues wget'ing from openstack git, so not pypi specific01:34
jesusaurusive noticed it a few times, always with docutils...01:34
ianwi wonder if it's more flaky networking on the hpcloud side01:34
harlowjaclarkb alright fixed up gerrit-view, was wondering why i wasn't seeing any queued up jobs :-P01:36
fungihttps://community.hpcloud.com/status/incident/261901:36
fungibodepd: jesusaurus: ianw: ^01:36
*** bknudson has quit IRC01:37
jesusaurusfungi: i don't think that's causing the error in http://logs.openstack.org/57/97357/3/check/gate-config-layout/26774e2/console.html01:37
ianwfungi: yeah, yum updates did seem *slow*01:37
bodepdjesusaurus: wrong like, I meant to post this one: http://logs.openstack.org/57/97357/1/check/check-projects-yaml-alphabetized/fa31884/console.html01:38
*** nosnos has joined #openstack-infra01:38
*** jp_at_hp has quit IRC01:39
*** nati_uen_ has quit IRC01:39
jesusaurusoh, that's strange01:39
bodepdI am totally confused about how the spell check treats - vs. _ vs. [a-z]01:39
fungijesusaurus: looks like a struggling network device somewhere. pip got a stray tcp/rst during download01:39
*** nati_ueno has joined #openstack-infra01:39
*** amotoki has quit IRC01:40
fungijesusaurus: it's sorting with LC_ALL=C (remember that sort order is locale-dependent)01:40
fungier, bodepd: ^01:41
jesusaurusfungi: could be, but that brief incident description makes it sound like the issue causes nova boot to hang, not random network outages01:41
fungijesusaurus: ahh, yes agreed01:42
*** jhesketh_ has joined #openstack-infra01:42
*** jhesketh_ is now known as jhesketh01:43
ianwfungi, jesusaurs: http://paste.openstack.org/show/82705/01:44
fungiianw: nice random fail there01:45
ianwfungi: yeah, it will keep failing just enough that you think it's working, then it isn't :) (that's how i noticed https://review.openstack.org/97665)01:47
*** amcrn has quit IRC01:48
*** _nadya__ has joined #openstack-infra01:49
ianwthat's on a vm in az2, which i guess is not us-east from https://community.hpcloud.com/status/incident/261901:49
*** wenlock has joined #openstack-infra01:49
*** _nadya_ has quit IRC01:50
*** otherwiseguy has quit IRC01:50
fungiwell, us east has an az2 as well01:52
*** otherwiseguy has joined #openstack-infra01:52
*** Ryan_Lane has quit IRC01:52
fungibut it's an actual os-ext-az availability zone, not a region called an az01:52
*** _nadya__ has quit IRC01:53
ianwoh, i see, yeah it's in us east01:54
ianwsilly me thought az might mean the state, and that seems like it would be "west" :)01:55
fungibwahahaha01:55
KiallLOL01:55
fungithat would be awesome though01:55
KiallAlso - https://community.hpcloud.com/status/incident/2619 should have zero effect on networking for running instances01:57
openstackgerritA change was merged to openstack-infra/config: Use Apache to compress status.json  https://review.openstack.org/9764801:57
openstackgerritA change was merged to openstack-infra/config: Cache status.json for 5 seconds in Apache  https://review.openstack.org/9765001:59
*** gokrokve has quit IRC01:59
*** hs634 has left #openstack-infra02:00
*** mmaglana has quit IRC02:05
*** mmaglana has joined #openstack-infra02:06
*** mmaglana has quit IRC02:10
clarkbfungi: mordred should we do the zuul stuff soonish?02:15
clarkblooks like the zuul and config changes are about to finish check testing02:16
fungithey'll end up flying through the gate once they do02:16
clarkbcool02:16
clarkbmay as well let them go through that then02:16
*** _nadya_ has joined #openstack-infra02:17
clarkbmy dinner is just about ready though02:17
mordredclarkb: yes02:18
*** crc32 has quit IRC02:20
openstackgerritA change was merged to openstack-infra/zuul: Stop gzipping zuul status json in zuul  https://review.openstack.org/9764902:22
openstackgerritA change was merged to openstack-infra/config: Install yappi on zuul nodes.  https://review.openstack.org/9766402:23
*** zz_gondoi is now known as gondoi02:23
openstackgerritA change was merged to openstack-infra/zuul: Cache zuul status.json for one second.  https://review.openstack.org/9765202:23
*** gyee has quit IRC02:27
*** mriedem has quit IRC02:27
*** mbacchi has quit IRC02:27
clarkbfungi: mordred it looks like config changes have merged02:30
clarkbnow we are waiting on zuul?02:30
clarkbfungi: https://review.openstack.org/#/c/97641/ though that didn't end up in the queues like the others so maybe we restart for that another day?02:31
clarkbthe info is potentially valuable for debugging though02:31
clarkbapache change is in place though02:32
mordredwoot02:33
clarkbmost important is the zuul gzipping02:33
clarkbI am ok with doing that without the profiler change02:33
clarkblet me know what you guys think02:33
mordredclarkb: I think we can do it without the profiler change02:35
*** camunoz has quit IRC02:36
*** gokrokve has joined #openstack-infra02:37
*** gondoi is now known as zz_gondoi02:39
fungiclarkb: for 97641, we would kill -USR2 twice (once to start the profiler and again to stop)?02:39
*** otter768 has quit IRC02:41
clarkbfungi: yup02:42
clarkbfungi: mostly want a way to stop it so that an performance impact can be bypassed once we have data02:42
*** harlowja is now known as harlowja_away02:42
clarkbit is upposed to be light weight though02:43
*** melwitt has quit IRC02:44
clarkbI'm eating quickly now but can be ready to restart zuul whenever02:44
clarkbfungi: mordred just let me know when you are ready and we can coordinate this02:44
*** dims has quit IRC02:45
clarkbthe code in /opt/zuul is up to date on zuul.o.o02:45
fungiwell, the profiler change has been approved and ought to merge momentarily since it already had a +1 check02:45
mordredneat02:45
clarkbfungi: perfect I will eat more then02:46
fungibecause gate priority02:46
fungisupposed eta 2 minutes02:46
*** mmaglana has joined #openstack-infra02:46
*** asettle is now known as asettle-luncheon02:47
openstackgerritA change was merged to openstack-infra/zuul: Add toggleable yappi profiling to zuul  https://review.openstack.org/9764102:48
fungimergola02:48
clarkbnow to get that in /opt and installed02:48
*** camunoz has joined #openstack-infra02:49
*** trinath_s has quit IRC02:50
clarkbfungi: mordred can one of you kick off a puppet run on zuul.o.o? that should get us to the latest version of all e things02:51
clarkbI am refreshing on zuul killing process02:53
clarkblooks like grab zuul changes with script in root's home dir then init script stop start02:54
clarkbthen run script to reenqueue change02:54
clarkband looks like zuul yappi is on zuul host02:55
clarkbeveryone ready?02:55
ianwjesusaurus / fungi: i raised an issue with hpcloud about the network issues; the support guy could replicate it so it's been escalated, apparently02:55
clarkbI can start pipeline capture as soon as I get a node from someone not me :)02:56
clarkbs/node/nod/02:58
clarkbmordred: fungi ^03:00
funginod03:02
fungiyes03:02
clarkbok capturing pipelines now03:02
fungiyou lost me at node ;)03:02
clarkbwill ping here before I actually stop zuul03:02
clarkbok check gate and check-tripleo captured03:04
clarkbready to stop zuul?03:04
* clarkb does it03:04
fungiyeah03:04
clarkbgot to requeue gate now03:05
fungilooks like it restarted03:05
fungii guess we lost anything in post03:06
clarkbfungi: yeah I checked it and no docs so meh03:06
clarkband nothing in release03:06
fungiprobably not critical at this point... i manually triggered the remaining branch tarballs for apevec's call for 2014.1.1 testing03:06
fungiand those all appeared a while before the restart03:06
clarkbcheck is queueing now03:07
fungiand stable/icehouse is basically in freeze now, so shouldn't have been any stragglers03:07
clarkband now tripleo03:08
*** changbl has joined #openstack-infra03:08
clarkbfungi: can you do some curling against the status.json endpoint and see if it is really caching and gzipping03:09
*** gokrokve has quit IRC03:09
fungiyep, on it03:10
clarkbdanke03:10
*** rwsu has quit IRC03:10
fungiLength: 1493566 (1.4M) [application/json]03:11
clarkbfungi: from my testing it grabs the gzip if you do curl -H 'Accept-Encoding: gzip'03:12
clarkbbut I get differnt zip files each time03:13
clarkbgoing to inflate and check contents now03:13
openstackgerritMarc Abramowitz proposed a change to stackforge/python-jenkins: Fix "H404  multi line docstring should start with a summary" hacking warnings  https://review.openstack.org/9767603:14
clarkbmordred: ^ I am not sure your five second thing is working03:15
openstackgerritMarc Abramowitz proposed a change to stackforge/python-jenkins: Fix hacking docstring warnings  https://review.openstack.org/9767603:15
mordredclarkb: no?. damm03:15
fungiunless it's caching the uncompressed content and then compressing on each request03:16
fungioh, or you get different uncompressed content03:17
clarkbfungi: well I uncompressed too and get differnet file sizes there03:17
fungifun03:17
clarkbnothign in the error log indicating apache is unhappy03:18
clarkblet me format this json and look at it in a readable way03:18
clarkbfungi: the diffs appear to be in elapsed and remaining time03:19
clarkbso definitely different03:19
clarkbbut zuul should cache it for at least a second now. going to see if I can do two subsecond requests and get the same one that way03:19
*** unicell has quit IRC03:19
mordredand apache is gzipping it, yeah?03:21
mordredthat should also be nice03:21
clarkbmordred: yes apache is definitely gzipping if you set Accept-Encoding03:21
clarkbso zuul should be much happier even without the 5 second cache03:21
*** asettle-luncheon is now known as asettle03:22
clarkbyup really quickly curling I get the same value back each time03:22
clarkbso the 1 second cache is working I think03:23
fungijust not at the apache layer03:23
clarkbya03:23
fungiand that we can tweak without a zuul restart03:23
clarkbyup03:23
* clarkb goes to debug apache now03:23
fungii'm not finding a smoking gun in the apache logs yet03:23
*** W00dy has joined #openstack-infra03:25
*** W00dy is now known as Guest8735003:25
clarkbI think  Ifound it03:25
clarkbMCacheMaxObjectSize 204803:26
clarkbis way to conservative (that value is bytes according to apache docs)03:26
clarkbwe need to tweak those numbers03:26
lifelessclarkb: you could put a real cache (squid/varnish) in there03:26
clarkblifeless: as mordred said earlier that is a bit heavyweight for one file03:27
openstackgerritLance Bragstad proposed a change to openstack/requirements: Add bash8 to global-requirements  https://review.openstack.org/9762503:27
lifelessclarkb: mmm I guess03:27
clarkbcurrently status.json is 1.7MB. I think jeblair said it was in the 3MB range before. What if we set the MCacheSize to ~12MB and max object size to ~6MB03:27
mordredclarkb: ++03:28
clarkbor we can be even less conservative. That box isn't memory constrained03:28
clarkbalso the streaming buffer may be important as we are proxying this file03:29
*** nosnos has quit IRC03:29
mordredclarkb: we could make zuul return protobuf, and then have an apache module that transforms that into json ...03:30
* mordred ducks03:30
*** matsuhashi has quit IRC03:31
*** unicell has joined #openstack-infra03:31
clarkbarg apache uses kilobytes in some places and bytes in others03:32
clarkbthis isn't confusing03:32
*** Guest87350 has quit IRC03:33
*** Longgeek has joined #openstack-infra03:33
mordredclarkb: I blame perl03:34
clarkbmordred: fungi: you ok with me ninja applying apache config really quicly to see if it works?03:36
fungiyeah, should be safe03:36
*** talluri has joined #openstack-infra03:36
fungiand 2k is definitely not enough to cover this file ;)03:37
clarkbgah puppet overwrote my changes very quicky;03:38
* clarkb figures out how to make it not do that03:38
clarkbpuppet disabled on zuul.o.o03:39
clarkbnope that still seems to not work03:41
clarkbI am doing curl ; sleep 2 ; curl ; sleep 2; curl03:41
clarkbat least 2 of the 3 curls should be identical03:41
*** otherwiseguy has quit IRC03:42
mattoliverauclarkb: what return value are you getting from curl?03:42
clarkboh wait03:42
clarkbmod mem cache is per process03:42
clarkbderp03:42
clarkbso this may be working with my changes just harder to confirm03:42
*** JoshNang has quit IRC03:43
* clarkb sets some LogLevel debug03:46
mattoliverauclarkb: you could turn the cacheheader directiive on and look at the returned x-headers in curl.03:47
mordredclarkb: should we switch to cache_disk?03:47
clarkbmordred: oh good point03:47
clarkbmordred: maybe?03:47
*** sballe has joined #openstack-infra03:47
mordredI read in another place that it does weird things with user agent combos03:48
mordredmaybe we _Should_ just stick a varnish there03:48
clarkbI am not against doing that :)03:48
clarkbmod_cache.c(635): cache: /status.json not cached. Reason: No Last-Modified, Etag, Expires, Cache-Control:max-age or Cache-Control:s-maxage headers03:50
clarkbwell that solves it03:50
clarkbI will propose my apache changes as they will be needed anyways and reenable puppet03:50
clarkbthen tomorrow we can fix the zuul wsgi app03:50
clarkband scheduel another restart03:50
mordredclarkb: ++03:52
openstackgerritClark Boylan proposed a change to openstack-infra/config: Better apache mem cache tuneable values  https://review.openstack.org/9767803:52
*** otherwiseguy has joined #openstack-infra03:52
clarkbthat ^ is what I had running while testing03:52
*** trinath_s has joined #openstack-infra03:53
*** trinaths has joined #openstack-infra03:55
clarkbok puppet ran and put us back to normal03:57
clarkbI am going to shift back to more normal evening things03:57
*** trinath_s has quit IRC03:57
*** amotoki has joined #openstack-infra03:58
mordredclarkb: like beer?03:59
clarkband television04:00
clarkbalso paste.httpserver doesn't support http/1.1 out of the box04:00
clarkbmight be worth investing in a different server so that we can get things like Last-Modified for free04:00
fungior we could just format the additional headers we want in the response04:02
clarkbyeah I am working on that real qucik right now04:03
clarkbprobably the quickest path forward04:03
fungioh, though if it reports http/1.0 then apache might ignore them. eh, worth testing04:03
*** lcheng_ has joined #openstack-infra04:08
clarkb< HTTP/1.1 200 OK from apache though that may just be apache being nice04:09
clarkbbut webob response objects have a last_modified attribute we can set04:12
clarkbso thats simple04:12
*** mburned is now known as mburned_out04:12
*** zhiyan is now known as zhiyan_04:15
*** zhiyan_ is now known as zhiyan04:17
*** mmaglana has quit IRC04:18
mordredclarkb: does pecan/wsme?04:19
ryanpetrellopecan.response is just a webob.Response instance04:20
mordredah04:20
ryanpetrello(so yes)04:20
mordredbut if we're already using webob, in this case, it's probably not going to get us much?04:21
mordredryanpetrello: http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/webapp.py04:21
ryanpetrelloyea, if you’re already just using webob, pecan’s not going to do anything special for you w/ Last-Modified04:22
openstackgerritClark Boylan proposed a change to openstack-infra/zuul: Set Last-Modified header when serving status.json  https://review.openstack.org/9768204:23
clarkbmordred: fungi ryanpetrello ^ pretty simple04:24
ryanpetrelloyep04:24
ryanpetrellothat’d do it04:24
clarkbor should I do it in the object creation?04:24
clarkbmeh04:24
mordredclarkb: lgtm04:25
*** matsuhashi has joined #openstack-infra04:25
*** nosnos has joined #openstack-infra04:26
*** oomichi has quit IRC04:26
*** lcheng_ has quit IRC04:27
*** rcarrill` has joined #openstack-infra04:29
*** rcarrillocruz has quit IRC04:32
*** lcheng_ has joined #openstack-infra04:36
*** amotoki has quit IRC04:39
*** pcrews has quit IRC04:44
openstackgerritA change was merged to openstack-infra/config: Better apache mem cache tuneable values  https://review.openstack.org/9767804:46
*** lcheng_ has quit IRC04:52
boris-42mordred hi there04:53
boris-42mordred can we make a bit simpler life for gates04:53
boris-42mordred and remove py33 job in rally04:53
boris-42mordred https://review.openstack.org/#/c/97448/04:53
clarkbboris-42: done04:55
*** nati_ueno has quit IRC04:55
boris-42clarkb thank you04:55
boris-42clarkb it's sad but we are not able to add support for py33 until all python clients won't have py33 voting job ..04:56
boris-42clarkb =(04:56
*** sarob has joined #openstack-infra04:56
*** Longgeek has quit IRC04:56
*** nati_uen_ has joined #openstack-infra04:57
*** nati_uen_ has quit IRC04:57
boris-42clarkb btw could you point me where can I found code that organizes  chain in gates?04:57
boris-42find*04:57
clarkbboris-42: the zuul dependent pipeline manager does that05:00
openstackgerritA change was merged to openstack-infra/config: Remove rally-python33 job  https://review.openstack.org/9744805:01
mordredboris-42: http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/scheduler.py#n162805:01
boris-42mordred thanks05:01
*** Longgeek has joined #openstack-infra05:01
*** yaguang has joined #openstack-infra05:02
*** otherwiseguy has quit IRC05:02
boris-42mordred so what I am thinking about is that limiting size of changes that are in gates queueu05:03
boris-42mordred may improve performance of gates a lot05:03
*** camunoz has quit IRC05:03
boris-42mordred I made some math to calculate overhead depending on possibility of fail + queue length + time that is required for testing patch05:04
*** ildikov has quit IRC05:04
*** yaguang has quit IRC05:04
boris-42mordred need to make one more how depends duration of merging queue on size of queue05:04
StevenKboris-42: We already limit the number of jobs that the gate runs05:04
StevenKBut not the length of the queue itself05:05
*** lcheng_ has joined #openstack-infra05:05
boris-42StevenK you mean jobs in gate chain?05:05
clarkbboris-42: we already do that05:05
boris-42StevenK or overall?05:05
clarkbif you look at the zuul status page now you will see it happening05:05
clarkbthe windowing method could probably be improved though05:06
clarkbright now it is similar to tcp slow start05:06
boris-42clarkb nice05:07
clarkbbut the gate pipelines are windowed. the window grows as changes merge and shrinks as they fail to merge05:07
clarkbanything with a gray blob next to it in the gate pipeline is currently outside of the window05:07
boris-42clarkb ah nice05:08
boris-42clarkb so how did you chose the number of jobs?05:08
boris-42clarkb experimental?05:08
clarkbboris-42: its actually number of changes instead of jobs. and yes we started at 10 iirc but found that was too small05:08
clarkbso we have 20 as the base window size and it grows by 2 ever merge iirc05:09
boris-42clarkb actually if we collect info about success possibility for tasks and avg duration of all jobs we can make it dynamic05:10
boris-42clarkb but not sure that it will help a lot05:10
clarkbyeah we can definitely make it more dynamic05:10
clarkbthe trouble is over doing it05:10
boris-42clarkb so should i continue work on formulas?05:10
clarkbthe nice thing this dumb implementation has going for it is that it has a good worst case05:10
clarkbboris-42: its probably worth looking into05:11
clarkbboris-42: a pid loop may be appropriate05:11
boris-42clarkb I mean sometimes it's better to limit by 10 (when we have a lot of races in gates, which sometimes happens)05:11
clarkbyeah and I think a pid loop can learn that05:12
boris-42clarkb sometimes we can make 20 or even more if there is no super big races05:12
*** yaguang has joined #openstack-infra05:12
boris-42clarkb I think it's nice as a first task=)05:13
StevenKboris-42: The 20 that clarkb mentioned isn't static -- it changes depending on if things merge or fail05:13
clarkbStevenK: well 20 is a static floor05:13
clarkbbut it is dynamic above that05:13
StevenKAh, right05:13
*** praneshp has joined #openstack-infra05:13
clarkbsdague found that if we let it be very dynamic we drive to 005:13
StevenKHehe05:13
boris-42clarkb heh=)05:13
StevenKSounds like an excellent plan to have stuff merge05:13
*** camunoz has joined #openstack-infra05:14
*** Kiall has quit IRC05:15
boris-42clarkb btw one more thing that I was thinking, for example I am doing change in heat05:15
boris-42clarkb why we are testing that glance, nova, keystone, cinder and other projects still work05:15
boris-42clarkb I think it's enough to run just heat benchmarks05:16
clarkbbecause it is an integrated cloud?05:16
boris-42clarkb no I mean we can do this a bit accurate05:16
clarkbboris-42: so this comes up constantly and the issue is no one actually understands how all of the compoenents talk to each other05:16
boris-42clarkb e.g. changing nova requires testing heat05:16
clarkband whenever you remove something you wedge the gate05:16
boris-42clarkb but not vise versa05:16
clarkbboris-42: but it does. what if heat does something to break their compatibility with nova?05:16
boris-42clarkb for example?)05:17
clarkbif they typo a rest call05:17
boris-42clarkb they will break heat05:17
boris-42clarkb not nova05:17
boris-42clarkb and it should be catches by heat tempest tests05:17
clarkbboris-42: yes but you won't know that its broken unless you test it against nova05:17
clarkbboris-42: I guess I don't follow then05:17
boris-42clarkb yep a bit=)05:17
boris-42clarkb lemme explain more carefully05:18
clarkboh why do the nova tempest tests run?05:18
clarkbso that goes back to my former statement05:18
boris-42clarkb yep05:18
clarkbthere are tons of implicit dependencies and people always get them wrong05:18
clarkband the gate breaks05:18
boris-42clarkb so you already try to make such experiment?05:18
clarkbboris-42: no, but we have inadverdently done it and discovered it when things break05:19
clarkbthe other hard problem is that there isn't one openstack05:19
*** Kiall has joined #openstack-infra05:19
boris-42clarkb you mean nova + neutron or nova + nova network?05:19
clarkbif you start counting all the combos you start crying05:20
boris-42clarkb yep=)05:20
clarkbboris-42: yes nova network or neutron (neutron with openvswitch, other drivers and so on). kvm vs lxc vs docker vs xen and so on. rabbit, 0mq, qpid. mysql, postgres, mongodb. and on and on05:20
boris-42clarkb hm but did you think about next thing (works for everything except ceilometer)05:21
openstackgerritA change was merged to openstack-infra/zuul: Set Last-Modified header when serving status.json  https://review.openstack.org/9768205:21
clarkbjust from my example that is 2 * 4 * 3 * 3 openstacks05:21
boris-42clarkb if project has in requirements python clients of other project05:21
clarkband we didn't even get into architecture, just components05:21
boris-42clarkb yep sure but any way, lemme try to explain non-human approach to collect dependencies05:22
clarkbboris-42: that is inaccurate because for example swiftclient doesnt depend on keystone05:22
boris-42clarkb hm05:22
clarkbyou can automate this. you just can't do it today05:22
boris-42clarkb but if we automate this stuff05:22
boris-42clarkb we can safe a lot of time05:23
clarkbyes but someone has to do the work to untangle the mess first05:23
clarkbbtw 72 is the number from above05:23
clarkband I am sure I can make that number much bigger if you actually take the time to count05:23
boris-42clarkb yep it's huge05:23
*** mmaglana has joined #openstack-infra05:23
boris-42clarkb but this number makes work on carefully choosing what to test even more important05:25
*** yaguang has quit IRC05:25
*** yaguang has joined #openstack-infra05:25
clarkbI agree05:25
clarkbbut we can't jump ahead or everything will fall apar05:25
boris-42clarkb I will try to get some resources from Mirnatis to work on this task05:25
boris-42clarkb as well I think if we change order of running tempest scenarios we can increate concurrency05:26
clarkbfor tempest specific stuff you will need to talk to the qa team05:27
clarkbI also volunteered to start documenting what we test today so that we can see that from a high level05:27
clarkbbecause really very few people know what is tested05:27
boris-42clarkb +105:27
*** praneshp_ has joined #openstack-infra05:27
clarkbwhich is another part of untangling05:27
clarkbthen we map that onto the inter project dependencies and figure it out05:28
boris-42clarkb so I will try to make some document05:29
*** praneshp has quit IRC05:29
*** praneshp_ is now known as praneshp05:29
boris-421) testing different projects simultaneously05:30
boris-422) dependencies between projects05:30
clarkbboris-42: and if we look at it from another perspective one nice thing of overtesting is we get really good statistically sound data that we can look at05:30
boris-42clarkb actually tempest testing is quite syntetic05:31
clarkbit shakes out a lot of bugs05:31
boris-42clarkb e.g. if you try to run simultaneously bunch of scenarios you may be schoked=)05:31
boris-42shocked*05:31
clarkbboris-42: as someone that runs thousnads of nodes every hour probably not :)05:31
boris-42clarkb hehe=)05:31
*** Ivo has quit IRC05:32
boris-42clarkb I was shocked when I saw this http://pavlovic.me/rally/glance_list.html05:32
boris-42clarkb 1k images list will take 20 seconds=)05:32
boris-42clarkb and it's exponential growth gg05:33
boris-42clarkb in nova same situation =)05:33
boris-42clarkb with vms=)05:33
clarkbis it exponential? that looks linear05:34
clarkbstill not good05:34
boris-42clarkb on the second graph05:34
boris-42clarkb you can click on create_image05:34
boris-42clarkb and turn it off05:34
clarkb200 iamges is 5 seconds. 1000 images is ~23 seconds05:34
*** oomichi has joined #openstack-infra05:34
boris-42clarkb it's with create05:34
boris-42clarkb on the graph on bottom (stacked are) click on glance.create_image to hide it05:35
boris-42clarkb so 200 = 2.5 sec 1000 ~20sec05:35
*** ildikov has joined #openstack-infra05:35
boris-4240 takes 0.505:36
clarkbright so 40 to 200 is linear05:36
openstackgerritIsaku Yamahata proposed a change to openstack-infra/config: Add tacker project on StackForge  https://review.openstack.org/9743505:37
boris-42but from 200 to 1000 it raises faster05:37
clarkb40 * 5 = 200. .5 * 5 = 2.505:37
clarkbyes the factor seems to change05:37
clarkbanyways we should fix that :)05:38
boris-42clarkb yep I need to get merged profiler05:38
boris-42clarkb going to move it on stackforge05:38
boris-42clarkb otherwise it's unclear what the hell is happaning=)05:38
Alex_Gaynorregardless of the scaling characferistic, 2.5 seconds to return 200 items is bonkers05:38
clarkbAlex_Gaynor: sure. just poitning out it doesn't look exponential :)05:38
clarkbstill terrible05:39
boris-42clarkb btw neutorn list ports05:39
*** alexpilotti has joined #openstack-infra05:39
boris-42clarkb will win golden poo awarad05:39
boris-42clarkb need to get data for it=)05:39
clarkbboris-42: this is a special profiler?05:39
boris-42clarkb nope this is just plain rally05:40
boris-42clarkb profiler will show what the hell is happening inside05:40
boris-42clarkb http://pavlovic.me/rally/profiler/05:40
boris-42clarkb boxes are clickable05:40
boris-42clarkb and there is a typo osapvi_volume.cinder.db not nova.db05:40
clarkboh I see it renders the call tree05:41
boris-42clarkb yep via all services of all projects05:41
boris-42clarkb and collect out of box db calls with sql statements, wsgi calls, and rpc05:41
boris-42clarkb but we can add more points05:41
boris-42clarkb so this one iteration of rally with profiling enabled in cinder/nova05:41
boris-42clarkb it creates volume from image and boot vm from this volume05:42
boris-42clarkb so for example we can see ultimate number of db request05:43
boris-42clarkb uncountable* for booting single vm05:43
*** pcrews has joined #openstack-infra05:44
*** basha has joined #openstack-infra05:44
boris-42clarkb this will organize huge load on DB if we try for example to run 500 vm simultaneosuly05:45
boris-42clarkb as well regarding to nova conductor there will be super huge load on rabbit as well05:45
*** sarob has quit IRC05:46
*** sarob has joined #openstack-infra05:46
*** yaguang has quit IRC05:47
boris-42clarkb so when I get profiler in upstream I will start working on fixing this stuff=)05:47
*** yaguang has joined #openstack-infra05:47
clarkbcool. does the profiler use a standard(ish) output format like callgrind or pstat?05:48
*** jamespage_ has joined #openstack-infra05:48
clarkbmight be interesting to feed data generated by other tools into ti too05:48
boris-42clarkb actually we are using ceilometer to collect all data05:48
clarkboh well that will put a huge laod on your cloud :)05:49
boris-42clarkb yep I know=)05:49
clarkbI was playing with yappi today which is why I am curious05:49
boris-42clarkb but swift guys said that probably we should print in logs05:49
clarkbit has learned to spit out callgrind and pstat files05:49
boris-42clarkb all messages and then collected05:49
boris-42collect05:49
clarkbwhich you can use other tools to view05:49
*** _nadya_ has quit IRC05:49
boris-42clarkb we can transform json into that format05:50
boris-42clarkb I don't see any issues here=)05:50
boris-42clarkb I have limitation in rally that I am using only API of OpenStack05:50
*** sarob has quit IRC05:50
boris-42clarkb so as we don't have in moment logging-as-a-service with python client05:50
boris-42clarkb i have to use ceilometer for collecting all resources05:51
clarkbcould you use a python profiler directly? (I am not sure how they play with eventlet)05:51
*** jlibosva has joined #openstack-infra05:52
boris-42clarkb the main goal of my profiler was to collect all traces from different services05:52
boris-42clarkb in different projects05:52
boris-42clarkb not to get full traces of all calls of everything in code05:52
boris-42in single process05:53
boris-42cause if something is broken in keystone and you are seeing issues in nova05:53
boris-42nova traces won't help you to detect what the hell is happaning05:53
boris-42=)05:53
*** jamespage_ has quit IRC05:54
clarkbwell they will help you figure out what you need to do to make nova list less bad05:54
clarkbits one thing to know nova list is bad05:54
*** alexpilotti has quit IRC05:54
morganfainbergclarkb, oh interesting yappi spitsout  callgrind? /me is going to have to check this out05:54
clarkbmorganfainberg: yup and pstat. and it works fine with threads. I am really happy with it05:54
boris-42clarkb but with profiler that I am doing you'll get trace of everything interesting05:54
morganfainbergclarkb, thats awesome!05:54
clarkbmorganfainberg: I shoved it into zuul as an optional thing today05:54
morganfainbergclarkb, ++ nice!05:55
boris-42clarkb that nova list works bad I already now even without profiler05:55
boris-42clarkb it is collected by rally only using api calls05:55
kashyapThere's no such thing as NEEDINFO flag  on Launchpad I guess?05:55
clarkbkashyap: incomplete05:55
kashyapclarkb, Okay I thought so.  but a fix is posted, and it's languishing since 8 months in review05:56
*** pcrews has quit IRC05:56
kashyapsoren, I can still mark it incomplete I guess?05:56
kashyap(Err, didn't mean to prompt him :-( sorry)05:56
clarkbkashyap: if there is a fix do you need more info?05:56
kashyapclarkb, The fix has two NACKs05:57
kashyapI should have been more clear05:57
clarkbjust because the code has been -1'd doesnt mean the bug is incomplete (or needs info)05:57
kashyapI see your point. It was just meant as a gentle ping to raise the attention05:58
*** srenatus has quit IRC05:58
clarkbI would just leave a comment in that case05:58
*** srenatus has joined #openstack-infra05:58
clarkbeither on the review or bug depending on where you need people to look05:59
*** alexpilotti has joined #openstack-infra06:01
*** andreaf has joined #openstack-infra06:03
*** sarob has joined #openstack-infra06:08
*** nati_ueno has joined #openstack-infra06:10
*** sarob has quit IRC06:12
*** mmaglana has quit IRC06:14
*** andreykurilin has joined #openstack-infra06:16
*** _nadya_ has joined #openstack-infra06:16
*** sarob has joined #openstack-infra06:17
*** sarob_ has joined #openstack-infra06:19
*** Ivo has joined #openstack-infra06:19
*** mkoderer has joined #openstack-infra06:20
*** pcrews has joined #openstack-infra06:21
kashyapYep, done that already.06:21
*** sarob has quit IRC06:21
*** sarob_ has quit IRC06:24
*** rfolco has quit IRC06:24
*** nati_ueno has quit IRC06:28
*** afazekas is now known as _afazekas06:29
*** rgerganov has joined #openstack-infra06:34
*** ccit has joined #openstack-infra06:39
*** ihrachyshka has joined #openstack-infra06:41
*** ominakov has joined #openstack-infra06:42
*** bauzas has joined #openstack-infra06:43
*** wenlock has quit IRC06:43
*** Guest22618 has joined #openstack-infra06:45
*** andreykurilin has quit IRC06:46
*** rgerganov has quit IRC06:54
*** fbo_away is now known as fbo06:56
*** alexpilotti has quit IRC06:59
*** ihrachyshka has quit IRC06:59
*** tkelsey has joined #openstack-infra07:01
*** _nadya_ has quit IRC07:04
*** freyes has joined #openstack-infra07:04
*** rgerganov has joined #openstack-infra07:06
*** ihrachyshka has joined #openstack-infra07:07
*** denis_makogon has joined #openstack-infra07:09
*** fbo is now known as fbo_away07:13
*** ihrachyshka has quit IRC07:14
*** Ivo has quit IRC07:15
*** skolekonov has joined #openstack-infra07:16
*** matsuhashi has quit IRC07:17
openstackgerritJames Polley proposed a change to openstack-dev/hacking: Clarify where the "following template" is  https://review.openstack.org/9769807:17
*** matsuhashi has joined #openstack-infra07:18
*** sarob has joined #openstack-infra07:19
sorenSpamapS: Done.07:22
*** sarob has quit IRC07:23
SpamapSsoren: cool thanks!07:24
*** ccit has quit IRC07:26
*** fifieldt has joined #openstack-infra07:29
*** _nadya_ has joined #openstack-infra07:30
*** hashar has joined #openstack-infra07:32
*** Alexei_987 has quit IRC07:33
*** zhiyan is now known as zhiyan_07:34
openstackgerritPrashanth Raghu proposed a change to openstack/requirements: Adding hiredis as a dependancy  https://review.openstack.org/9770107:36
*** srenatus has quit IRC07:37
*** srenatus has joined #openstack-infra07:38
*** oomichi has quit IRC07:38
*** lcheng_ has quit IRC07:45
*** pblaho has joined #openstack-infra07:45
*** matrohon has joined #openstack-infra07:46
*** renlt has joined #openstack-infra07:46
*** mrda is now known as mrda-away07:51
*** lcheng_ has joined #openstack-infra07:53
openstackgerritAntoine Musso proposed a change to openstack-infra/zuul: Document usage of yappi profiler  https://review.openstack.org/9770807:55
*** jpich has joined #openstack-infra07:55
*** andreaf has quit IRC07:55
*** yaguang has quit IRC07:59
*** derekh_ has joined #openstack-infra07:59
*** yaguang has joined #openstack-infra07:59
*** doude has joined #openstack-infra08:00
*** afazekas has joined #openstack-infra08:03
*** lcheng_ has quit IRC08:05
*** praneshp has quit IRC08:06
*** habib has joined #openstack-infra08:10
*** jistr has joined #openstack-infra08:10
*** viktors has quit IRC08:14
*** sarob has joined #openstack-infra08:17
*** srenatus has quit IRC08:18
*** srenatus has joined #openstack-infra08:18
*** oomichi has joined #openstack-infra08:19
*** sarob has quit IRC08:21
*** dizquierdo has joined #openstack-infra08:22
*** ihrachyshka has joined #openstack-infra08:25
*** Longgeek has quit IRC08:27
*** pelix has joined #openstack-infra08:27
*** Longgeek has joined #openstack-infra08:28
*** Longgeek has quit IRC08:29
*** markmc has joined #openstack-infra08:30
*** Longgeek has joined #openstack-infra08:30
*** flaper87|afk is now known as flaper8708:35
*** IvanBerezovskiy1 has joined #openstack-infra08:35
*** _nadya_ has quit IRC08:35
*** _nadya_ has joined #openstack-infra08:35
*** fbo_away is now known as fbo08:35
*** e0ne has joined #openstack-infra08:38
*** markmcclain has joined #openstack-infra08:40
*** jooools has joined #openstack-infra08:42
*** matsuhashi has quit IRC08:42
*** dizquierdo has quit IRC08:44
*** pblaho has quit IRC08:48
*** rgerganov has quit IRC08:48
*** mkoderer has quit IRC08:48
*** jlibosva has quit IRC08:48
*** ildikov has quit IRC08:48
*** sballe has quit IRC08:48
*** changbl has quit IRC08:48
*** openstackgerrit has quit IRC08:48
*** IvanBerezovskiy1 has quit IRC08:48
*** rickerc has quit IRC08:48
*** ityaptin has quit IRC08:48
*** pblaho has joined #openstack-infra08:48
*** rgerganov has joined #openstack-infra08:48
*** ityaptin has joined #openstack-infra08:48
*** mkoderer has joined #openstack-infra08:48
*** IvanBerezovskiy has joined #openstack-infra08:48
*** changbl has joined #openstack-infra08:49
*** ildikov has joined #openstack-infra08:49
*** IvanBerezovskiy has left #openstack-infra08:49
*** jlibosva has joined #openstack-infra08:49
*** rickerc has joined #openstack-infra08:49
*** sballe has joined #openstack-infra08:49
*** IvanBerezovskiy has joined #openstack-infra08:49
*** matsuhashi has joined #openstack-infra08:50
*** openstackgerrit has joined #openstack-infra08:50
*** e0ne has quit IRC08:53
*** liam_ has joined #openstack-infra08:54
isviridovhello infra. Could anybody give me a hint where can I check status or release job?08:54
*** liam_ has quit IRC08:54
*** jgallard has joined #openstack-infra08:54
isviridovI've pushed a tag and tar.gz is available at http://tarballs.openstack.org/magnetodb/, but not at pypi  https://pypi.python.org/pypi/magnetodb08:55
*** e0ne has joined #openstack-infra08:57
*** jp_at_hp has joined #openstack-infra08:58
*** fifieldt has quit IRC09:00
*** timrc is now known as timrc-afk09:01
openstackgerritMonty Taylor proposed a change to openstack-infra/nodepool: Build images using diskimage-builder  https://review.openstack.org/4648209:01
*** freyes has quit IRC09:02
*** andreaf has joined #openstack-infra09:05
*** freyes has joined #openstack-infra09:05
openstackgerritLe Tian Ren proposed a change to openstack-infra/config: Add acl to create branches on powervc-driver  https://review.openstack.org/9772609:06
*** SpamapS has quit IRC09:08
*** pblaho has quit IRC09:09
*** rcarrillocruz has joined #openstack-infra09:11
*** rcarrill` has quit IRC09:11
*** SpamapS has joined #openstack-infra09:11
*** ociuhandu has quit IRC09:11
*** ihrachyshka has quit IRC09:12
openstackgerritAntoine Musso proposed a change to openstack-infra/config: Bring docs jobs for openstack-infra/gerritlib  https://review.openstack.org/9500609:16
openstackgerritAntoine Musso proposed a change to openstack-infra/config: Bring docs jobs for openstack-infra/gerritlib  https://review.openstack.org/9500609:20
*** ihrachyshka has joined #openstack-infra09:40
*** e0ne has quit IRC09:43
*** e0ne has joined #openstack-infra09:43
*** markmcclain has quit IRC09:44
*** freyes has quit IRC09:44
*** e0ne_ has joined #openstack-infra09:44
*** markmcclain has joined #openstack-infra09:45
*** e0ne has quit IRC09:48
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: update URL references to project.  https://review.openstack.org/9443009:50
*** Sam-I-Am has quit IRC09:52
*** Sam-I-Am has joined #openstack-infra09:52
*** StevenK has quit IRC09:53
*** monester has quit IRC09:53
*** StevenK has joined #openstack-infra09:53
*** dizquierdo has joined #openstack-infra09:54
*** monester has joined #openstack-infra09:58
*** renlt has quit IRC09:59
*** apevec has joined #openstack-infra10:01
*** jcoufal has joined #openstack-infra10:01
*** jamespage_ has joined #openstack-infra10:05
*** denis_makogon has quit IRC10:06
*** nosnos has quit IRC10:08
*** hashar has quit IRC10:10
*** markmcclain has quit IRC10:11
*** jgallard has quit IRC10:13
*** sarob has joined #openstack-infra10:17
*** salv-orlando has joined #openstack-infra10:17
*** salv-orlando has quit IRC10:18
*** salv-orlando has joined #openstack-infra10:18
sdagueboris-42: 1k glance images is also something that I'd be surprised if you find anywhere. Rax image list is 38 entries10:19
*** e0ne_ has quit IRC10:22
*** sarob has quit IRC10:22
*** jamespage_ has quit IRC10:23
*** matsuhashi has quit IRC10:24
*** ociuhandu has joined #openstack-infra10:26
boris-42sdague in any case=)10:26
boris-42sdague it doesn't work => it should be fixed10:26
sdagueboris-42: there are a ton of things that don't work, that's a really low priority one10:27
boris-42sdague what about snapshots?10:27
boris-42sdague actually most of all functionality of openstack don't work properly10:27
boris-42sdague and they all should be just fixed10:27
boris-42sdague profiler + benchmarks will allow to find these issues fast10:28
boris-42sdague and just fix them10:28
boris-42sdague without too much discussion=)10:28
sdagueboris-42: sure, but you randomly picking some benchmark out of a hat which is bad, that almost no one will actually hit, seems like a pretty terrible way to prioritize10:28
boris-42sdague actually we hit the same perofrmance10:28
boris-42sdague in nova vm list , and cinder volume list10:28
boris-42sdague and in nova there could be much more then 1k vms10:29
boris-42sdague this bug just came from real production cloud10:29
boris-42sdague in nova10:29
boris-42sdague so we are adding benchmarks for all list commands10:29
sdagueso is it if 1k vms exist? or are returned10:29
boris-42sdague returned10:29
sdagueright, returning 1k vms is non sensible10:30
boris-42sdague it depends10:30
sdaguethere was a whole conversation about that in the paging sessions10:30
sdagueI get that we'd like these calls to be faster, however they are many orders of magnitude less important than actually getting a compute to reliably boot and connect to the network :)10:31
sdaguelike the top bugs here - http://status.openstack.org/elastic-recheck/10:31
boris-42sdague and for example nova conductor that is bottlneck10:31
boris-42sdague in real deployments10:32
sdagueboris-42: sure, I'd believe that10:32
boris-42sdague btw I made some proposal10:32
boris-42sdague don't want to publish it in mailing list10:32
sdaguewhy not?10:32
boris-42sdague but a better approach for real zero downtime upgraeds10:32
boris-42sdague nova team will hate me=)10:32
*** sulochan has joined #openstack-infra10:32
sdaguewe do everything on the mailing lists10:32
boris-42sdague it's not yet the proper time for that10:33
boris-42sdague but if you are interested take a look at https://docs.google.com/a/mirantis.com/document/d/15SVB2VrO8ZczVjk9C1rQ2uXgNTuNQ9DddZbFU1UR8UM/edit10:33
*** bcrochet|g0ne is now known as bcrochet10:33
sdaguehonestly, right now I'd be more interested in help fixing the fact that we can't reliably boot a compute and connect it to the network10:33
boris-42sdague no need in unified objects and conductor to resolve zero downtime upgrades and security issues=)10:34
sdagueI think until that's a 100% guaruntee10:34
boris-42sdague sure I understand10:34
*** Longgeek has quit IRC10:34
*** Longgeek has joined #openstack-infra10:35
*** sballe has quit IRC10:35
*** matrohon has quit IRC10:36
*** sballe has joined #openstack-infra10:40
*** lttrl has quit IRC10:42
*** sballe_ has joined #openstack-infra10:42
*** sballe has quit IRC10:45
openstackgerritValeriy Ponomaryov proposed a change to openstack-infra/config: Set env vars explicitly for manila jobs  https://review.openstack.org/9774710:46
sdagueSergeyLukjanov: you up yet?10:47
*** yamahata has quit IRC10:47
*** matsuhashi has joined #openstack-infra10:47
*** sballe_ has quit IRC10:48
*** torgomatic has quit IRC10:48
*** dims_ has joined #openstack-infra10:50
*** SpamapS has quit IRC10:50
*** _nadya_ has quit IRC10:51
*** SpamapS has joined #openstack-infra10:52
sdagueI'm staring at zuul, and the giant rush that's going through the gate now has issues. The clean check hole that SpamapS found where something can jump to the gate even if a dependant patch is -1ed10:52
*** freyes has joined #openstack-infra10:53
openstackgerritA change was merged to openstack-infra/config: Fix zuul's app.js name  https://review.openstack.org/9131410:53
*** torgomatic has joined #openstack-infra10:55
openstackgerritMark McLoughlin proposed a change to stackforge/gertty: Handle multiple child revisions of same parent  https://review.openstack.org/9774910:57
openstackgerritA change was merged to openstack-infra/config: Fix scenario name in rally-neutron job  https://review.openstack.org/9746910:57
openstackgerritA change was merged to openstack-infra/config: Make murano-dsvm jobs voting  https://review.openstack.org/9746210:57
openstackgerritDarragh Bailey proposed a change to openstack-infra/git-review: Provide nicer user message for missing remote ref  https://review.openstack.org/9775110:58
pelixwould appreciate feedback on https://review.openstack.org/#/c/48783/ as well if anyone would like to review a custom yaml loader for JJB10:59
openstackgerritA change was merged to openstack-infra/config: Fix jquery-visibility.min.js name  https://review.openstack.org/9696811:02
*** ociuhandu has quit IRC11:04
openstackgerritMark McLoughlin proposed a change to stackforge/gertty: Add ctrl-o to help dialog  https://review.openstack.org/9775311:05
*** _nadya_ has joined #openstack-infra11:05
openstackgerritMark McLoughlin proposed a change to stackforge/gertty: Set log mode to append  https://review.openstack.org/9775411:05
*** ociuhandu has joined #openstack-infra11:07
*** amotoki has joined #openstack-infra11:11
*** zzelle has joined #openstack-infra11:12
SergeyLukjanovsdague, I'm here, just wfh tody, so, not stable online11:12
sdagueok11:13
sdagueyeh, so at the current burn rate, we probably won't get through the gate queue until some time next week11:13
*** dims_ has quit IRC11:14
sdaguedefinitely not until a weekend shows up11:15
sdagueespecially given there are things like a huge heat stack of patches that are in there, but have some early parts -1ed11:16
SergeyLukjanovsdague, yeah...11:17
sdaguemy instinct is to dump the gate queue. Because the problem is that when check got to 500 a ton of stuff was +Aed, and the rush like that is something our gate can't really handle11:17
SergeyLukjanovsdague, you mean drop?11:17
*** sarob has joined #openstack-infra11:17
sdagueyeh, basically drop everything out of it11:17
SergeyLukjanovsdague, I'm ok with it, I'd like to have 5 patches in into the stable/icehouse today ;(11:18
sdagueare those the ones currently in the gate?11:20
*** dims_ has joined #openstack-infra11:20
*** sarob has quit IRC11:21
sdagueI would be surprised if they landed before tomorrow11:23
*** srenatus has quit IRC11:23
*** srenatus has joined #openstack-infra11:23
openstackgerritRadomir Dopieralski proposed a change to openstack/requirements: Require django-pyscss 1.0.1 for django-compressor 1.4 support  https://review.openstack.org/9776011:24
*** matsuhashi has quit IRC11:25
*** flaper87 is now known as flaper87|afk11:25
*** matsuhashi has joined #openstack-infra11:26
*** mburned_out is now known as mburned11:27
*** matsuhashi has quit IRC11:29
*** matsuhashi has joined #openstack-infra11:29
*** eharney has joined #openstack-infra11:34
*** yamahata has joined #openstack-infra11:41
zzellehi everyone11:47
zzelleit seems opendaylight CI is broken11:47
*** yamahata has quit IRC11:50
*** yamahata has joined #openstack-infra11:50
*** hashar has joined #openstack-infra11:52
*** alexpilotti has joined #openstack-infra11:53
*** alaski_ is now known as alaski11:53
*** e0ne has joined #openstack-infra11:56
*** adalbas has joined #openstack-infra11:57
*** weshay has joined #openstack-infra12:00
rgerganovsdague, hi, do you have a minute to ask you something about https://review.openstack.org/#/c/9574312:02
*** e0ne_ has joined #openstack-infra12:03
*** e0ne has quit IRC12:05
*** achuprin has quit IRC12:06
sdagueSergeyLukjanov: https://review.openstack.org/#/c/97768/ - any idea who is even the approvers on that?12:07
*** alexpilotti has quit IRC12:07
sdaguergerganov: sure12:07
rgerganovsdague, do I need to make any puppet changes if I want to move the CI names in external file?12:08
rgerganovsdague, and how to test this12:08
*** trinaths has quit IRC12:08
SergeyLukjanovsdague, yup, they are in gate12:09
sdaguergerganov: I don't think so12:09
*** jgallard has joined #openstack-infra12:09
rgerganovsdague, so we deploy all files in the gerrit directory and I just need to create a new one for the CI names12:10
sdaguergerganov: so I'd just say make that a separate .js file12:10
sdaguethen have the header just include it by url12:10
SergeyLukjanovsdague, I'm looking on your patch12:10
rgerganovsdague, thanks12:10
SergeyLukjanovsdague, I think sahara missed12:11
sdagueSergeyLukjanov: yeh, my concern remains that no one is awake yet, and we're basically merging about 2 changes an hour12:11
SergeyLukjanovsdague, yeah12:12
*** rfolco has joined #openstack-infra12:12
sdaguethough determining run rate is kind of hard12:14
*** esker has joined #openstack-infra12:16
*** lcostantino has joined #openstack-infra12:16
SergeyLukjanovsdague, I've added comments to https://review.openstack.org/#/c/97768/12:16
*** sarob has joined #openstack-infra12:17
*** basha has quit IRC12:17
*** akrivoka has joined #openstack-infra12:18
*** jistr is now known as jistr|eng12:18
*** ArxCruz has joined #openstack-infra12:19
*** esker has quit IRC12:21
*** sarob has quit IRC12:21
*** alexpilotti has joined #openstack-infra12:22
*** lcostantino has quit IRC12:23
*** radez_g0n3 is now known as radez12:23
*** lcostantino has joined #openstack-infra12:23
*** achuprin has joined #openstack-infra12:24
openstackgerritRadoslav Gerganov proposed a change to openstack-infra/config: Add button that shows/hides CI comments in Gerrit  https://review.openstack.org/9574312:24
*** jamespage_ has joined #openstack-infra12:28
*** ominakov_ has joined #openstack-infra12:28
*** ociuhandu has quit IRC12:28
*** dims_ has quit IRC12:29
*** aysyd has joined #openstack-infra12:29
*** jamespage_ has quit IRC12:29
*** dims_ has joined #openstack-infra12:29
sdagueSergeyLukjanov: ok, cool12:30
*** mbacchi has joined #openstack-infra12:30
sdagueI think I addressed those, though clearly I don't understand submodules, as this doesn't seem to be doing quite the right thing12:30
*** ominakov has quit IRC12:31
*** talluri has quit IRC12:31
*** talluri has joined #openstack-infra12:31
sdaguemaybe when mordred gets up he can explain what I did terribly wrong :)12:32
*** Ivo has joined #openstack-infra12:32
mordred_phonesubmodules are evil12:33
*** ominakov_ has quit IRC12:34
*** ociuhandu has joined #openstack-infra12:35
sdagueI entirely agree12:36
sdaguehowever openstack/openstack is really out of date in whats included12:36
*** talluri has quit IRC12:36
sdaguehere is my attempt to fix that - https://review.openstack.org/#/c/97768/ - but clearly I'm missing something12:37
sdaguemordred_phone or SergeyLukjanov - can someone promote this - https://review.openstack.org/#/c/96824/12:38
sdagueit will fix a bunch of failures where we are breaking because of n-cond not having clean logs12:38
SergeyLukjanovsdague, /melooking12:39
*** lttrl has joined #openstack-infra12:39
SergeyLukjanovsdague, oh, it's approved in nova, yup, I can promote it12:39
SergeyLukjanovsdague, please, ack that it should be promoted12:41
*** ominakov has joined #openstack-infra12:41
sdagueyes, it should be12:41
*** viktors has joined #openstack-infra12:43
SergeyLukjanovsdague, anything else to promote with this change?12:43
* SergeyLukjanov will run "zuul promote --pipeline gate --changes 96824,2"12:43
mordredSergeyLukjanov: ++12:43
*** viktors has quit IRC12:44
mordredsdague: ok - what thing that you did did you want me to look at?12:44
mordredsdague: got it12:44
mordredI need to work on figuring out tags and o/o too12:45
SergeyLukjanovmordred, publishing to pypi doesn't work right no, isn't it?12:46
*** dkranz has joined #openstack-infra12:46
mordredSergeyLukjanov: aroo? it should ...12:46
SergeyLukjanovmordred, magnetodb folks pushed the tag and there is no new package at pypi12:47
mordredsdague: why not read the integrated projects list from the governance repo?12:47
mordredSergeyLukjanov: how long ago?12:47
SergeyLukjanovmordred, could you please take a look too? I've not find any issues12:47
SergeyLukjanovmordred, today12:47
mordredI mean - the gate is very backed up12:47
sdaguemordred: will that include pbr, tempest, devstack?12:47
SergeyLukjanovmordred, https://github.com/stackforge/magnetodb/releases/tag/2.0.512:47
mordredhrm. maybe not12:47
mordredSergeyLukjanov: yah - but the post job may not have run yet12:47
mordredit responds to the tag landing12:48
sdagueI know that jogo and I use openstack/openstack to figure out merge velocity on the integrated gate12:48
sdaguewhich is what I was trying to heuristic out12:48
SergeyLukjanovmordred, yeah... it could be, thx12:48
SergeyLukjanovsdague, promoted12:48
sdagueSergeyLukjanov: thanks12:48
sdagueI also just kicked all the ironic changes out of the gate pipeline12:49
SergeyLukjanovsdague, I've used os/os several times for reports :)12:49
sdaguethey seem to all be failing unit tests12:49
*** dprince has joined #openstack-infra12:49
sdaguewhich means I think in the giant flush something merged there which broke everything else12:49
mordredSergeyLukjanov: looking12:49
mordredSergeyLukjanov: I see the tarball here: http://tarballs.openstack.org/magnetodb/12:51
SergeyLukjanovmordred, yup, and wheel too12:51
SergeyLukjanovmordred, but there are no runs in https://jenkins.openstack.org/job/magnetodb-pypi-both-upload/12:52
mordredSergeyLukjanov: but I don't see the upload one having run in jenkins.o.o12:52
mordredyeah12:52
mordredhrm12:52
*** yaguang has quit IRC12:52
sdaguethere is also a 17 deep patch queue in heat that has a jenkins -1 on patch #412:53
sdaguebut they all are in the gate12:53
*** mfer has joined #openstack-infra12:53
SergeyLukjanovsdague, :(12:54
sdaguemordred: so my knee jerk reaction to the gate status right now is to dump everything out of it12:54
sdaguewith the following rationale12:54
sdagueyesterday the check queue got to 500 and people just kept approving things12:55
*** ominakov has quit IRC12:55
sdagueso once we could finally process a ton of stuff all came through at once12:55
sdaguehowever 2 things happen in such a stampede12:55
sdaguethe first, is the ironic situation12:55
fungimordred: pypi.slave.openstack.org seems to have not been recreated after jeblair accidentally deleted all the long-running slaves on monday12:55
sdaguewhere code merged which breaks the unit tests for others12:56
sdaguebut our assumptions is that it will all be fine12:56
sdagueso we keep grinding12:56
sdaguethe second is the heat case12:56
*** ominakov has joined #openstack-infra12:56
sdaguewhere because of a bug in clean check it is possible to land items in the gate with dependencies that have -1 from jenkins12:57
mordredfungi: aha. that would explain it12:57
sdaguewhich heat mostly finds a way to trip because they have very long patch series12:57
fungii can start spinning up a replacement while i find some coffee12:57
sdagueand it has to do with timing of when things enter12:57
mordredhrm12:57
mordredI don't understand the ironic bug12:58
sdaguethere are 20 ironic changes in check12:58
sdaguethey all get approved simultaneously12:58
fungisdague: at the moment, because of jgit problems, gerrit tends to fail at merging everything in a series of dependent changes past about the first four12:58
sdaguechange #2 in gate changes an interface12:58
sdaguethat's not enough to create a merge conflict12:59
sdague#3 - #20 now all fail unit tests12:59
*** markmcclain has joined #openstack-infra12:59
mordredbut should not land12:59
mordredoh12:59
mordredOh. ok. I follow12:59
mordredI thought you were saying code had landed that broke unittest in trunk12:59
mordredwhich is weird12:59
mordredbut I get it now13:00
*** markmcclain1 has joined #openstack-infra13:00
mordredsdague: I'm fine with it - fungi?13:00
openstackgerritBoris Pavlovic proposed a change to openstack-infra/config: Finally add osprofiler to stackforge  https://review.openstack.org/9777813:00
fungii'd need to ponder the messaging a bit13:00
*** gokrokve has joined #openstack-infra13:00
sdaguewe actually also see it with pep813:00
sdaguewhich is the clearly explanation, the automatic merge resolution creates a pep8 violation13:01
fungiyeah, a ton of nova changes got approved and now violate pep8 tests13:01
fungiand keep causing gate resets because of it13:01
*** esker has joined #openstack-infra13:01
sdaguebasically we have a race in the fact that we have a pipeline13:01
fungisaw a bunch of those in the gate yesterday13:01
*** fanhe has joined #openstack-infra13:02
sdaguefungi: well, there is also the n-cond clean log failure13:02
sdaguewhich is what 96824,2 is about13:02
sdaguethat SergeyLukjanov just promoted13:02
mordredwell, there was another thing clark wanted to land that requires a zuul restart - we could kill 2 birds withi one stone13:02
sdaguemordred: a zuul restart will kill the queues?13:02
sdagueI thought they were often reloaded13:02
*** HenryG has quit IRC13:02
sdaguewe should land 96824,2 before we do anything, as that will reduce failure rate13:03
*** markmcclain has quit IRC13:03
mordredsdague: we usually save the queues and reload them13:05
* fungi will brb13:07
*** mriedem has joined #openstack-infra13:07
fungipypi.slave replacement is building now13:07
*** signed8bit has joined #openstack-infra13:07
*** ArxCruz has quit IRC13:07
*** markmc has quit IRC13:09
*** ArxCruz has joined #openstack-infra13:11
sdaguemordred: https://review.openstack.org/#/c/97768/13:12
sdagueupdated13:12
mordredsdague: python-ganttclient is in there13:15
*** masayukig has quit IRC13:15
sdagueit sure is13:16
Shrewssdague: such scrollback. an ironic change has broken the gate? or is that just another symptom?13:16
sdagueit's also in the integrated gate :)13:16
*** pdmars has joined #openstack-infra13:16
mordredsdague: I think that's the thing I was pointing out13:16
mordred:)13:16
sdagueyeh, but I mean if you pushed a change to it, it would run 4 tempest jobs and grenade13:16
sdaguealso, ceilometer isn't in the integrated gate definition13:17
sdagueas discovered by this13:17
mordredsdague: did we not move gantt and python-ganttclient to the attic?13:17
sdagueShrews: there were enough ironic changes in flight that something that was approved and landed merged in a way that all ironic changes now fail their unit tests13:17
*** sarob has joined #openstack-infra13:17
sdaguemordred: apparently not the client13:17
*** julim has joined #openstack-infra13:18
*** Longgeek has quit IRC13:18
Shrewslovely13:18
mordredsdague: not either13:18
*** bknudson has joined #openstack-infra13:18
mordredI'm so confused13:18
*** Longgeek has joined #openstack-infra13:18
sdaguemordred: yeh, me too, because I actually thought I'd fixed that ceilometer issue before as well13:18
*** Longgeek has quit IRC13:19
sdaguenow I understand why all of facebook is 1 git tree :)13:19
mordredsdague: :)13:19
*** Longgeek has joined #openstack-infra13:19
*** Longgeek has quit IRC13:19
*** Longgeek_ has joined #openstack-infra13:19
*** jistr|eng is now known as jistr13:20
*** Longgeek_ has quit IRC13:21
*** masayukig has joined #openstack-infra13:22
*** unicell has quit IRC13:22
*** sarob has quit IRC13:22
mordredsdague: Release Managers has +2 on o/o13:23
mordredttx: ^^13:23
*** Longgeek has joined #openstack-infra13:24
*** otherwiseguy has joined #openstack-infra13:24
openstackgerritMark McLoughlin proposed a change to stackforge/gertty: Refactor duplicated code in dependencies handling  https://review.openstack.org/9778613:26
*** radez is now known as radez_g0n313:26
*** habib has quit IRC13:27
*** palar has joined #openstack-infra13:27
*** habib has joined #openstack-infra13:28
openstackgerritLonggeek proposed a change to openstack-infra/config: Modify jenkins home directory  https://review.openstack.org/9724013:28
sdagueheh, so I guess I have +2 on it then13:29
openstackgerritBob Ball proposed a change to openstack-infra/nodepool: Support install phase with nodepool  https://review.openstack.org/9778713:29
openstackgerritPeter Belanyi proposed a change to openstack/requirements: Add nodeenv for horizon  https://review.openstack.org/9778813:30
*** alexpilotti has quit IRC13:30
Longgeeksdague: My patch?13:30
SergeyLukjanovsorry, I've been on unexpected meeting, reading scrollback13:31
*** markmcclain1 has quit IRC13:31
SergeyLukjanovsomething awful happens?13:31
*** habib has quit IRC13:33
anteayaLonggeek: read scrollback or check the channel logs13:34
*** habib has joined #openstack-infra13:34
anteayaLonggeek: sdague is in the middle of another conversation and was not referring to your patch13:34
Longgeekanteaya: ok13:34
*** jamespage_ has joined #openstack-infra13:35
SergeyLukjanovmordred, fungi, how to enqueue the release pipeline job? (there is no change id for them)13:35
SergeyLukjanovI mean to restart job13:35
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: fix bug 1324793 to determine what the issue actually is  https://review.openstack.org/9779013:36
uvirtbotLaunchpad bug 1324793 in nova "Nova conductor throws ERROR messages, which is forbidden in our contract" [Critical,In progress] https://launchpad.net/bugs/132479313:36
openstackgerritMarc Abramowitz proposed a change to stackforge/python-jenkins: Fix hacking docstring warnings  https://review.openstack.org/9767613:36
*** freyes has quit IRC13:36
*** habib has quit IRC13:37
*** habib has joined #openstack-infra13:37
*** matrohon has joined #openstack-infra13:38
*** _nadya_ has quit IRC13:38
*** ArxCruz_ has joined #openstack-infra13:39
*** ArxCruz has quit IRC13:39
SergeyLukjanovand mordred, fungi, I can help with 3rd party testing accounts creation if you point me to the process :)13:40
*** otherwiseguy has quit IRC13:41
fungiSergeyLukjanov: once i get pypi.slave.openstack.org rebuilt and readded to jenkins.openstack.org, we use the trigger script in the tools directory of zuul's git repo13:42
*** otherwiseguy has joined #openstack-infra13:43
fungiSergeyLukjanov: as for third-party service account creation, it's documented in http://ci.openstack.org/gerrit.html#third-party-testing-access13:44
*** doug-fish has joined #openstack-infra13:44
SergeyLukjanovfungi, oh, I'm always forget to try searching using third-party vs. 3rd party :(13:44
SergeyLukjanovfungi, thank you13:45
SergeyLukjanovfungi, I see the 3rd party account request, will try to create one ;)13:45
fungiSergeyLukjanov: there are actually several pending requests that i haven't gotten to yet... let me get you links13:45
srenatusok, last resort: http://memecrunch.com/meme/3P8WU/merge-97196/image.jpg (https://review.openstack.org/#/c/97196/)13:47
openstackgerritRadomir Dopieralski proposed a change to openstack/requirements: Add xstatic-packaged JavaScript libraries for Horizon  https://review.openstack.org/9779313:48
ttxmordred: missing context -- what would I use that +2 on o/o power for ?13:48
*** timrc-afk is now known as timrc13:48
sdaguettx: https://review.openstack.org/#/c/97768/13:48
fungiSergeyLukjanov: http://lists.openstack.org/pipermail/openstack-infra/2014-May/001261.html http://lists.openstack.org/pipermail/openstack-infra/2014-May/001269.html https://bugs.launchpad.net/openstack-ci/+bug/132615013:49
uvirtbotLaunchpad bug 1326150 in openstack-ci "Request 3rd Party CI Account for HP Cinder" [Undecided,New]13:49
SergeyLukjanovfungi, will do13:49
anteayaSergeyLukjanov: do you mind if we get some agreement on naming of third party accounts first please?13:49
anteayaSergeyLukjanov: I would like to go over that list on the etherpad and ensure we all agree how we want to handle multiple accounts from the same company13:49
anteayaas well as username and gerrit account name agreement13:50
anteayait will save chasing down people and renaming in future13:50
SergeyLukjanovanteaya, do we have an estimate for this agreement?13:50
anteayaas well, we had agreed at summit that they needed a wikipage before account creation and we are almost there13:50
*** zz_gondoi is now known as gondoi13:51
fungianteaya: well, some of those requests are already a couple weeks old, i've just been too busy to pick them up yet13:51
anteayawell for the wikipage creation if we get agreement at next week's meeting that we like the seed accounts, we can create wikipages13:51
SergeyLukjanovanteaya, it's not documented now, so, people don't know what should be done13:51
fungiyeah, the current requests are following the currently documented process13:51
anteayaI know, but they can't vote yet anyway and if we do this according to the new rules, it saves me a heap of time13:51
SergeyLukjanovfungi, +13:52
anteayathis just creates so much more work for me13:52
sdagueok, so we seem to have at least a triumvirant of race fails that are liking to land on each other13:52
anteayasince we had agreed to it at summit, I had thought there would be a little support for it13:52
fungii'm just considering that these vendors have been instructed (primary by the block storage program now) that they need to be doing testing, so i feel sort of bad for holding up their requests and thus their progress toward better integration with cinder13:53
anteayaI can't stop you, but this just digs the hole deeper13:53
*** _nadya_ has joined #openstack-infra13:53
*** wenlock has joined #openstack-infra13:53
ttxsdague: I suppose the new .gitmodules is the result of the script, not something you cobbled together ?13:53
anteayajgriffith: ping13:53
srenatusthanks jeremy13:53
fungithe hole is already pretty deep, so i'm not sure that a few more on top while we work out the remaining details make much difference13:53
*** mwagner_lap has joined #openstack-infra13:53
SergeyLukjanovI think that while we have now documented process we couldn't point folks to the new process13:53
sdaguettx: yep13:53
anteayafungi: the difference is for me13:54
anteayado I feel supported or alone13:54
ttxsdague: ok, +2. Will wait for another release manager to +2 or a bit more infra-core to +A before I APRV it13:54
anteayaright now I don't feel supported13:54
anteayaI feel alone13:54
*** unicell has joined #openstack-infra13:55
*** dkliban_afk is now known as dkliban13:55
ttxmordred: don't forget you're supposed to submit a scoring proposal as a governance resolution at some point13:56
fungijust to confirm, your suggestion is to announce a freeze on any new gerrit systems accounts until we determine a more consistent way to add them, and that programs should adjust their driver approval deadlines accordingly13:56
*** jgrimm has joined #openstack-infra13:56
* ttx just got home13:56
*** fanhe has quit IRC13:56
*** e0ne_ has quit IRC13:56
anteayafungi: no13:56
*** heyongli has joined #openstack-infra13:56
fungii'm not sure i understand then13:57
*** e0ne has joined #openstack-infra13:57
SergeyLukjanovme too13:57
fungiwe should or shouldn't stop adding new system accounts right now?13:57
*** fanhe has joined #openstack-infra13:58
*** dizquierdo has quit IRC13:58
anteayaI am checking timing with cinder13:58
fungiokay13:59
*** ilyashakhat has joined #openstack-infra13:59
SergeyLukjanovack, I;m on hold for creating acc13:59
fungiif there is a good synergy of schedules for them, then i can understand telling the current requestors we're changing the process, with an eta on when the new process will be unveiled for them to follow instead14:00
sdagueSergeyLukjanov / fungi: https://etherpad.openstack.org/p/gatetriage-june201414:00
fungisdague: looking those over now14:00
openstackgerritAlexandre Levine proposed a change to openstack-infra/config: Initial commit for the AWS EC2 and VPC API standalone service project  https://review.openstack.org/9779514:00
SergeyLukjanovfungi, ++ otherwise we should keep the old process unfortunately while the new one isn't ready14:00
openstackgerritBoris Pavlovic proposed a change to openstack-infra/config: Finally add osprofiler to stackforge  https://review.openstack.org/9777814:01
sdaguefungi: the tempest one isn't approved yet, just discussing with mtreinish14:01
*** habib has quit IRC14:01
sdaguethe conductor one was just at top of gate, and killed by the tempest one14:01
SergeyLukjanovsdague, candidates for promoting?14:01
sdagueSergeyLukjanov: yep14:01
fungithose all look like sound measures to me14:01
SergeyLukjanovsdague, so, I think we should wait for tempest change to be approved and the promote all three changes14:02
sdagueSergeyLukjanov: I agree14:02
fungiyeah, presumably in the order listed in that pad14:02
*** pdmars has quit IRC14:02
openstackgerritKiall Mac Innes proposed a change to openstack-infra/config: Add Designate DevStack/Requirements/Docs Jobs  https://review.openstack.org/9734814:02
fungi--changes 97755,2 96824,2 92893,514:02
*** pdmars has joined #openstack-infra14:02
KiallWould love reviews on ^ TC people holding off on incubation vote until it's running :(14:03
anteayafungi SergeyLukjanov right now cinder has no timeline that I can see: https://etherpad.openstack.org/p/juno-cinder-3rd-party-cert-and-verification14:03
anteayaI am looking to find jgriffith to get additional information14:03
fungianteaya: okay, there's no rush if he's busy (he almost certainly is). this was merely a good opportunity for SergeyLukjanov to get some additional experience with the basic account creation api calls14:04
sdaguefungi: ok, we have a +A on the tempest change14:04
*** jamespage_ has quit IRC14:05
sdagueso I think the promotes should be good to go14:05
fungianteaya: and also to help with the fact that we've not gotten back on a couple of account requests for a week or two14:05
SergeyLukjanovsdague, I'll do it14:05
fungiSergeyLukjanov: did you want to do that promote command?14:05
fungigo for it14:05
sdagueSergeyLukjanov: thanks14:05
SergeyLukjanovfungi, yup14:05
fungiKiall: adding it near the top of my pile. if my pile stops growing for the day and starts shrinking, i'll have a look14:06
KiallThanks fungi :)14:06
SergeyLukjanovzuul promote --pipeline gate --changes 97755,2 96824,2 92893,514:06
SergeyLukjanovfungi, sdague ^^14:07
anteayafungi: I understand that14:07
anteayaand thanks for letting me get some additional information on this14:07
fungiSergeyLukjanov: that looks right to me14:07
SergeyLukjanovfungi, hm, Exception: Unable to find shared change queue for 97755,214:08
*** basha has joined #openstack-infra14:08
*** markmcclain has joined #openstack-infra14:09
fungiSergeyLukjanov: that's odd...14:09
SergeyLukjanovfungi, yup...14:10
*** trinaths has joined #openstack-infra14:10
sdagueit's in check14:11
sdaguedoes that matter14:11
sdagueI guess we need to move it over to gate14:11
fungisdague: shouldn't14:11
*** [1]trinaths has joined #openstack-infra14:11
fungiSergeyLukjanov: i wonder if you tried to run it before zuul saw the approve vote. try a second time now?14:11
fungioh!14:12
fungisdague: you are correct14:12
fungiSergeyLukjanov: any of them which aren't in the gate pipeline yet need a zuul enqueue command first14:12
SergeyLukjanovfungi, okay14:12
SergeyLukjanovfungi, got it14:12
fungisudo zuul enqueue --trigger gerrit --project openstack/tempest --pipeline gate --change 97755,214:13
fungilike that14:13
* fungi clearly didn't get enough coffee14:13
fungithat will pop them onto the end of the gate, and then you can promote them all in one shot. you'll also need to delete any jenkins -1 or -2 votes14:14
fungibefore trying to enqueue14:14
*** trinaths has quit IRC14:14
*** [1]trinaths is now known as trinaths14:14
fungipossibly14:15
fungithough it looks like that's not necessary now that zuul clears the old jenkins vote when enqueuing anyway14:15
SergeyLukjanovdone14:15
SergeyLukjanovfungi, sdague ^^14:15
sdaguecool14:15
fungiyep, now they're on top and in order. awesome SergeyLukjanov14:16
sdaguethanks14:16
SergeyLukjanovfungi, it works after enqueuing them to the gate pipeline and only after that promoting14:16
fungiSergeyLukjanov: yep14:16
SergeyLukjanovfungi, adding it to my infra how to notes ;)14:16
*** pblaho has joined #openstack-infra14:16
openstackgerritBob Ball proposed a change to openstack-infra/nodepool: Support nodes with launch condition  https://review.openstack.org/9779814:17
*** sarob has joined #openstack-infra14:17
*** skolekonov has quit IRC14:18
*** wenlock has quit IRC14:20
anteayaSergeyLukjanov: who might I talk to about fuel ci and fuel watcher?14:20
SergeyLukjanovanteaya, let me check14:20
anteayathanks14:21
anteayaif there is a channel I should join, do tell me what it is14:21
anteayaBobBall: any idea who maintains CitrixJenkins?14:21
*** sarob has quit IRC14:21
anteayaBobBall: it has to be renamed and I need to talk to someone to find out what it actually does14:22
BobBallI saw that the other day on the mailing list14:22
BobBallI think it might have been from the good old days of Citrix and OpenStack14:22
BobBalli.e. probably Euan Mellor's automated account that is highly deprecated now14:23
*** Alexei_987 has joined #openstack-infra14:23
*** mrodden has quit IRC14:24
fungianteaya: yeah, worth considering some of these accounts may be defunct (and if so should just be removed from the associated groups). figuring that out might require some interesting api queries looking for the age of their most recent comments though14:24
anteayaBobBall: do you feel it is safe to mark CitrixJenkins as deprecated then?14:24
BobBallyes14:24
openstackgerritA change was merged to openstack-infra/elastic-recheck: fix bug 1324793 to determine what the issue actually is  https://review.openstack.org/9779014:24
uvirtbotLaunchpad bug 1324793 in nova "Nova conductor throws ERROR messages, which is forbidden in our contract" [Critical,In progress] https://launchpad.net/bugs/132479314:24
anteayaBobBall: thanks I will make a note of that14:25
*** dizquierdo has joined #openstack-infra14:25
anteayafungi: yes, I will compile a list of suspected deprecated accounts so they can be confirmed14:25
anteayaand then we can discuss what to do with them14:25
openstackgerritSergey Skripnick proposed a change to openstack-infra/config: Add rally-neutron-neutron job  https://review.openstack.org/9780514:27
*** Longgeek has quit IRC14:27
e0neHi Infra Team. I'm trying to setup local environment for debug using manifests from https://github.com/openstack-infra/config. Looks like it isn't flexible enough: a lot of hardcoded git repos urls inside. Does it make sense to create bug and fix it?14:29
*** heyongli has quit IRC14:29
e0nee.g.: https://github.com/openstack-infra/config/blob/master/modules/jenkins/manifests/job_builder.pp#L2714:30
e0neI propose to make git repos urls configurable14:30
tteggelheads up- hpcloud 1.1 is experiencing scheduling delay again14:35
tteggelhttps://community.hpcloud.com/status/incident/262214:35
ruheanteaya: there is #fuel-dev channel, i hope you'll find people you're looking at that channel14:36
SergeyLukjanovanteaya, Alexandra Fedorova <afedorova@mirantis.com>14:36
rcarrillocruztteggel: interesting, i created an instance in east two hours ago and it was quite slow14:36
rcarrillocruzthx for sharing14:36
anteayaruhe, SergeyLukjanov thank you14:36
*** mrodden has joined #openstack-infra14:37
openstackgerritBob Ball proposed a change to openstack-infra/nodepool: Support install phase with nodepool  https://review.openstack.org/9778714:37
openstackgerritBob Ball proposed a change to openstack-infra/nodepool: Support nodes with launch condition  https://review.openstack.org/9779814:37
fungie0ne: i believe there are already some changes proposed to do just that. asselin_ had at least one up for review in that theme14:38
*** doude has quit IRC14:39
*** rwsu has joined #openstack-infra14:40
*** andreaf has quit IRC14:41
e0nefungi, thanks, but i found only https://review.openstack.org/#/c/9653914:42
*** basha has quit IRC14:42
*** reed has joined #openstack-infra14:43
e0neit doesn't fix all issues with hardcoded git repos14:43
anteayafungi SergeyLukjanov so DuncanT in -cinder said that if we can get some agreement on naming in 24 hours, he is okay with that14:44
*** thedodd has joined #openstack-infra14:44
anteayaI am working on this etherpad: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format14:44
*** gondoi is now known as zz_gondoi14:44
anteayaand have my suggestions for new names as #comments after the current name14:44
anteayaif we can at least agree that the new accounts will follow the format of the suggestions, I am fine with creating them14:45
anteayaif we agree my suggestions for new names are good suggestions14:45
*** markmcclain has quit IRC14:45
fungii'll have a look once i'm done wrangling the bitrot in our server launch script and get release uploads back online again14:46
anteayafungi: *nod14:46
anteayayeah I know this is not top priority for you14:46
anteayajust trying to keep you up to date14:46
*** wenlock has joined #openstack-infra14:46
trinathshi. In Jenkins testing, I got this error for my patchset which doesn't relate me http://paste.openstack.org/show/82784/ . Can I go for recheck no bug14:47
*** achuprin has quit IRC14:48
* anteaya is afk for a bit14:48
*** matsuhashi has quit IRC14:49
openstackgerritA change was merged to openstack-infra/config: Fix Chef cookbook-related strainer issue  https://review.openstack.org/9719614:49
*** jpich has quit IRC14:50
*** sballe has joined #openstack-infra14:50
*** sballe_ has joined #openstack-infra14:52
*** klindgren_ has left #openstack-infra14:53
*** sballe has quit IRC14:56
openstackgerritA change was merged to stackforge/gertty: Re-add alembic to requirements  https://review.openstack.org/9680714:56
*** sballe__ has joined #openstack-infra14:57
*** jpich has joined #openstack-infra14:57
openstackgerritKiall Mac Innes proposed a change to openstack-infra/config: Add Designate DevStack/Requirements/Docs Jobs  https://review.openstack.org/9734814:58
*** basha has joined #openstack-infra14:58
openstackgerritA change was merged to stackforge/gertty: Add hyperlinks  https://review.openstack.org/9576614:59
openstackgerritA change was merged to stackforge/gertty: Correct a problem with tables at very small widths  https://review.openstack.org/9576714:59
*** sballe_ has quit IRC15:00
*** achuprin has joined #openstack-infra15:01
fungitrinaths: that looks like it might be bug 1310852 but it's very hard to tell from the limited amount of detail you provided15:01
uvirtbotLaunchpad bug 1310852 in neutron "tempest.scenario.test_security_groups_basic_ops.TestSecurityGroupsBasicOps.test_in_tenant_traffic  Timeout" [High,Confirmed] https://launchpad.net/bugs/131085215:01
*** atiwari has joined #openstack-infra15:01
*** dangers_away is now known as dangers15:02
openstackgerritA change was merged to stackforge/gertty: Add patchset selection in diff  https://review.openstack.org/9576815:03
openstackgerritA change was merged to stackforge/gertty: Handle (ignore) no-diff renames  https://review.openstack.org/9576915:03
*** basha has quit IRC15:04
*** mriedem has quit IRC15:04
fungisdague: this got hit on a cinder change and i'm having a hard time fathoming that it could be a random failure... known issue? http://logs.openstack.org/52/97452/1/check/check-tempest-dsvm-full/8e9b941/console.html15:05
russellbto request a new tag on openstack mailing lists, do i file an openstack-ci bug?  or ping someone/somehwere else?15:05
fungisdague: "Job timeout set to: -128 minutes ... timeout: invalid option -- '1'"15:05
sdaguefungi: that one I don't know15:05
openstackgerritA change was merged to stackforge/gertty: Handle missing commits  https://review.openstack.org/9680115:05
*** mriedem has joined #openstack-infra15:05
openstackgerritA change was merged to stackforge/gertty: Handle file-level comments  https://review.openstack.org/9698515:05
openstackgerritA change was merged to stackforge/gertty: Add project updated column  https://review.openstack.org/9703115:06
openstackgerritA change was merged to stackforge/gertty: Support background sync of missing refs  https://review.openstack.org/9706615:06
fungirussellb: http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev suggests reed or ttx can help15:06
openstackgerritA change was merged to stackforge/gertty: Add dependency navigation  https://review.openstack.org/9706715:06
*** sballe__ has quit IRC15:06
reedrussellb, whazzup?15:06
fungirussellb: "OpenStack-dev list run by thierry at openstack.org, stefano at openstack.org"15:06
openstackgerritA change was merged to stackforge/gertty: Remove stray debug line  https://review.openstack.org/9706815:07
russellbreed: was looking to get the "NFV" tag added to both openstack@ and openstack-dev@15:07
russellbreed: NFV team would like to have that15:07
reeddo people use tags? :)15:07
reedsure, glad to hear that people use them15:07
russellbheh, i don't know, but it came up in our meeting just now15:07
russellbpeople certainly use them when they post ... not sure about subscribers15:07
*** ociuhandu has quit IRC15:08
openstackgerritJulien Danjou proposed a change to openstack-infra/config: Gate Gnocchi against MySQL too  https://review.openstack.org/9782015:08
sdagueit seems a little odd to add a tag for not a program though, no?15:08
* reed blanking out15:08
reedwhat's NFV?15:08
fungiit stands for "very confusing acronym"15:09
openstackgerritA change was merged to stackforge/gertty: Handle multiple child revisions of same parent  https://review.openstack.org/9774915:09
ttxfungi: that would be reed's territory yes15:09
fungi(network function virtualization)15:09
rcarrillocruznetwork function virtualization iirc15:09
russellbreed: stands for Network Function Virtualization ... a big effort within the telco industry15:09
reedah15:09
reedthanks15:09
ttxrussellb: ^15:09
russellbof which OpenStack plays an important part15:09
russellbttx: thanks15:09
reedrussellb, Is there a wiki page where that is defined?15:09
russellbNFV group within OpenStack is mainly around collecting use cases, translating them to OpenStack land, and generating requirements, tracking blueprints ...15:10
russellbreed: WIP, just had first meeting, but https://wiki.openstack.org/wiki/Meetings/NFV15:10
fungiit's one of those new acronyms which means exactly the same thing as several other well-established acronyms, but each industry likes to reinvent the naming wheel15:10
reedrussellb, col15:11
reedrussellb, cool15:11
reedrussellb, topics are only active on openstack-dev15:11
russellbok15:12
russellbthat's fine15:12
fungineat, the term has its own wikipedia page: http://en.wikipedia.org/wiki/Network_Functions_Virtualization15:12
russellbfungi: that means it's legit15:12
*** zz_gondoi is now known as gondoi15:12
russellbupdated wiki page with team mission statement15:13
fungiaccording to the article, about a year and a half ago some telecommunications people had the bright idea that you could have virtual network devices, but this novel concept lacked a good name15:13
*** radez_g0n3 is now known as radez15:13
russellb:)15:13
*** spiffxp- has quit IRC15:13
russellbyeah, basically out with the old traditional hardware networking appliances, and in with sexy VMs that do the same thing.15:14
*** SlickNik has quit IRC15:14
*** SlickN1k is now known as SlickNik15:14
russellband these telcos are identifying lots of good gaps in OpenStack15:14
russellband there was lots of distributed, private coordination around these things15:14
russellbso ... just centralizing / coordinating a bit here.15:14
fungi(never mind the fact that i used virtual network devices at a service provider more than a decade ago, we just didn't have the need for a confusing term to go along with that concept)15:15
russellb:)15:15
russellbfungi: so the other thing was making #openstack-nfv official ...15:15
*** HenryG has joined #openstack-infra15:15
fungirussellb: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/accessbot/channels.yaml15:16
russellbfungi: ack thanks15:16
fungirussellb: that should get us on track with having appropriate control over the channel (though someone may need to register it with chanserv and set the openstackinfra nick as a founder first)15:17
*** sarob has joined #openstack-infra15:17
russellbfungi: k thanks15:17
sdaguefungi: all the failures are in rax iad15:17
openstackgerritA change was merged to stackforge/gertty: Expand the .gitignore file to ignore .egg files  https://review.openstack.org/9289115:17
sdaguehttp://logstash.openstack.org/#eyJzZWFyY2giOiJcInRpbWVvdXQ6IGludmFsaWQgb3B0aW9uXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjYwNDgwMCIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJ1c2VyX2ludGVydmFsIjowfSwic3RhbXAiOjE0MDE4OTQ3MzIzNzF915:18
fungirussellb: beyond that, if they want channel logging, review notifications, et cetera that's in different files15:18
russellbfungi: sounds good, appreciate the pointer, i can figure that stuff out.  i should have guessed it was in infra-config :)15:18
russellb<3 infra-config15:18
fungirussellb: http://ci.openstack.org/irc.html15:18
fungi(for the official rtfm link)15:19
fungibut i'm happy to answer any questions about that stuff if you have them15:19
sdaguefungi: any chance that the clock get moverd15:19
trinathsstackers!, I hit with this error http://paste.openstack.org/show/82786/, for a 'recheck no bug' for my patchset.15:19
*** eharney has quit IRC15:20
fungisdague: it's... possible that something changed the time on the vm before d-g tried to calculate the timeout offset15:21
sdaguefungi: it's new, and only on rax-iad15:21
sdagueI'm going to put in a failsafe to the calculation15:21
fungisdague: new as of the past hour?15:21
sdaguesee logstash link above15:21
sdaguenew as in last couple days15:21
gilliardWhat's the process for submitting a patch to something in stackforge?  Is there a gerrit?  Or a github PR?15:21
*** sarob has quit IRC15:21
fungimaybe something with the devstack-precise image in rax-iad from a little bit ago15:22
sdague73 hits15:22
sdaguestarted about 48hrs ago15:22
fungigilliard: gerrit. any stackforge project should have a valid .gitreview file which would configure the git-review utility automagically when you try to use it to submit a patch15:22
fungisdague: ahh, okay, so maybe it's a recent change to the ubuntu precise base image there15:23
*** nati_ueno has joined #openstack-infra15:23
gilliardthx fungi15:24
fungiphschwartz: do you happen to know if there was a refresh for the "Ubuntu 12.04 LTS (Precise Pangolin)" image in iad roughly 2-3 days ago?15:24
*** virmitio has joined #openstack-infra15:25
fungiphschwartz: we're seeing some oddness when booting from a snapshot there, where it looks like the system time is possibly jumping by a couple hours shortly after boot, but it only started a couple days ago15:26
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: debug time calculations  https://review.openstack.org/9782415:26
*** mmaglana has joined #openstack-infra15:26
sdaguefungi: that might at least help debug15:27
sdagueand fail more sanely15:27
openstackgerritMatthew Gilliard proposed a change to stackforge/gertty: Adds alembic to requirements.txt  https://review.openstack.org/9782515:27
fungigilliard: looks like you found it! ;)15:27
*** IvanBerezovskiy has left #openstack-infra15:27
sdaguefungi: you have a bug for this?15:28
fungisdague: bug 132623115:28
uvirtbotLaunchpad bug 1326231 in openstack-ci "Invalid timeout value cusing "gate-tempest-dsvm-full" to fail" [Undecided,New] https://launchpad.net/bugs/132623115:28
fungiskimming the new bugs, that one caught my eye as hard to be your run-of-the-mill openstack brokenness15:28
sdagueyep15:29
sdagueI'll build an er query for it15:29
*** gokrokve has quit IRC15:29
*** gokrokve has joined #openstack-infra15:29
fungithe nodepool image list suggests that we failed to get new devstack-precise and devstack-precise-check images in rax-iad today, but built fresh ones 24 and 48 hours ago, so my guess is that if it's related to the base image it may have been introduced in rax-iad between 48-72 hours ago15:30
*** olaph has joined #openstack-infra15:30
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add query for timeout calculation gone wrong  https://review.openstack.org/9782615:30
sdaguefungi: yeh, but the only thing that could be going on is if we jumped time forward by a couple hours after d-g was running15:31
fungi(though really not sure if we got an image 72 hours ago, so it could have actually been like that longer and we just wouldn't have seen it until our successful image updates on monday)15:31
fungisdague: well, ntp could be lagging, especially if it started up before its gateway was reachable15:31
*** rcarrill` has joined #openstack-infra15:31
BobBalltrying to re-run upstream nodepool for the xenserver CI I get a weird error saying a socket has no option for RCVTIMEO - http://paste.openstack.org/show/82790/ - any ideas why this might be?15:32
fungiand then adjusted the system clock just after d-g started15:32
*** wenlock has quit IRC15:32
fungiBobBall: looks familiar. consulting magic 8-ball now15:33
BobBallI did check and zmq is in the right range...15:33
BobBallroot@xs-ci-test:~/src/nodepool# pip freeze | grep zmq15:33
BobBallpyzmq==13.1.015:33
*** rcarrillocruz has quit IRC15:34
fungiBobBall: conversation starts at 19:33:19 http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2014-04-03.log15:35
BobBallwonderful - thanks fungi15:35
pelixany chance of a review on https://review.openstack.org/#/c/48783/ ?15:35
openstackgerritArun Kant proposed a change to openstack/requirements: Adding ldappool module dependency as needed by keystone bug #1320997.  https://review.openstack.org/9584215:35
uvirtbotLaunchpad bug 1320997 in keystone "Common Ldap handler connection pooling" [Medium,In progress] https://launchpad.net/bugs/132099715:35
openstackgerritDavid Shrewsbury proposed a change to openstack-infra/config: Adjust ironic tempest regex for new tests  https://review.openstack.org/9783015:35
openstackgerritMarc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: Some tweaks to get closer to Python 3 compat  https://review.openstack.org/8781015:36
BobBallfungi: that is soooo ugly! But thanks :)15:36
fungiBobBall: looks like resolution was pyzmq linking against a system installed libzmq1. try uninstalling that15:36
BobBallyup - found that + testing.15:36
*** rcarrill` is now known as rcarrillocruz15:37
fungiBobBall: if that works, maybe a couple-sentence patch to the nodepool docs is in order, to serve as a warning to other hapless adventurers15:38
*** markmc has joined #openstack-infra15:38
*** hashar has quit IRC15:38
anteayaBobBall: this is another citrix gerrit account: NetScalerAts    NetScaler TestingSystem15:39
*** gyee has joined #openstack-infra15:39
anteayaBobBall: ring any bells for you?15:39
BobBallanteaya: that one is live + used by the netscaler team (I'm on the XenServer team)15:39
BobBallor at least I believe it's live...15:39
BobBall:)15:39
anteayaBobBall: so citrix-netscaler-ci would be appropriate?15:40
*** andreykurilin_ has joined #openstack-infra15:40
BobBallyup15:40
anteayathanks15:40
BobBallthanks fungi - that does indeed fix it.15:41
*** rcarrill` has joined #openstack-infra15:41
*** markmcclain has joined #openstack-infra15:42
*** dims_ has quit IRC15:42
*** yamahata has quit IRC15:43
*** rcarrillocruz has quit IRC15:43
*** jgallard has quit IRC15:45
openstackgerritA change was merged to openstack/requirements: Add xvfbwrapper package for running Selenium tests headless  https://review.openstack.org/8039315:45
openstackgerritA change was merged to openstack/requirements: Remove amqplib from global-requirements  https://review.openstack.org/9698315:46
*** srenatus has quit IRC15:46
*** basha has joined #openstack-infra15:46
*** Sukhdev has joined #openstack-infra15:46
*** srenatus has joined #openstack-infra15:46
*** rgerganov has quit IRC15:46
openstackgerritBob Ball proposed a change to openstack-infra/nodepool: Add warnings about the installation of libzmq1  https://review.openstack.org/9783515:47
fungiBobBall: thanks!15:47
*** basha has quit IRC15:48
fungiBobBall: git review thought your example error was a commit message header though :/15:48
fungihilarious15:48
tteggelhpcloud 1.1 is back to normal scheduling latency. think we got to the root cause this time \o/15:48
fungitteggel: awesome!!! thanks for your dogged persistence on that15:49
*** ihrachyshka_ has joined #openstack-infra15:49
*** andreykurilin_ has quit IRC15:50
fungiBobBall: i'm going to test editing that in gerrit's webui15:50
openstackgerritJeremy Stanley proposed a change to openstack-infra/nodepool: Add warnings about the installation of libzmq1  https://review.openstack.org/9783515:50
*** gondoi is now known as zz_gondoi15:51
*** jistr has quit IRC15:51
*** ihrachyshka has quit IRC15:52
BobBallhah - whoops :)15:52
funginow that i've experienced editing a git commit message in a webui, i feel like i need a shower15:54
sdagueheh15:55
*** mmaglana has quit IRC15:55
*** ihrachyshka_ has quit IRC15:55
*** mmaglana has joined #openstack-infra15:56
Kiallfungi: lol15:58
*** UtahDave has joined #openstack-infra15:58
reedttx, where do we publish the schedules for future releases, beyond Juno (if we do)?15:58
openstackgerritMark McLoughlin proposed a change to stackforge/gertty: Refactor duplicated code in dependencies handling  https://review.openstack.org/9778615:58
*** rcarrill` is now known as rcarrillocruz15:59
*** nati_ueno has quit IRC15:59
*** mmaglana has quit IRC16:00
*** markmc has quit IRC16:02
*** asadoughi is now known as Swami16:02
*** Swami is now known as asdoughi16:02
*** asdoughi is now known as asadoughi16:02
*** lcheng_ has joined #openstack-infra16:03
*** ociuhandu has joined #openstack-infra16:03
*** e0ne has quit IRC16:03
*** markmcclain has quit IRC16:04
*** esker has quit IRC16:04
openstackgerritJeremy Stanley proposed a change to openstack-infra/config: Update launch script and instructions  https://review.openstack.org/9784116:06
*** pblaho has quit IRC16:06
*** afazekas has quit IRC16:07
zaromorning16:08
anteayamorning zaro16:08
*** ominakov has quit IRC16:09
anteayadprince: ola16:10
*** eharney has joined #openstack-infra16:10
anteayadprince: I'm going to have to stick CI on the end of smokestack16:10
morganfainbergmornin!16:10
anteayamorning morganfainberg16:10
fungizaro: do you happen to know the minimum gerrit acl necessary to create new branches on a project? we've had stackforge projects ask for it in the past and i've seen some with create perms on refs/* or refs/heads/* but have never taken the time to test which actually works16:10
dprinceanteaya: whats up?16:11
fungizaro: similarly, do you know whether branch deletion easier now from an acl perspective (in older gerrit you had to be set as a project owner for that)16:11
anteayadprince: we have to do a standardized naming format for automated gerrit accounts16:11
anteayadprince: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format16:11
anteayadprince: we are going to ensure ci is on the end of any automated account that votes or might vote16:12
anteayadprince: and bot on accounts that will never vote16:12
fungidprince: smokestack is now caught in the unrelenting tide of third-party ci ;)16:12
anteayadprince: I'll need to append ci to smokestack's name16:12
anteayadprince: yeah, sorry about that16:12
*** gokrokve has quit IRC16:13
anteayawish I could exempt you but I can't see that I can16:13
fungithe price of being a trail blazer16:13
anteayayeah16:13
dprinceanteaya: you have my blessing16:13
anteayathank you16:13
anteaya:D16:13
*** crc32 has joined #openstack-infra16:14
*** wenlock has joined #openstack-infra16:17
*** jcoufal has quit IRC16:17
*** otherwiseguy has quit IRC16:17
*** ewindisch has quit IRC16:18
*** jlibosva has quit IRC16:19
anteayafungi SergeyLukjanov so if you can do a round up of outstanding account requests so that I can add potential usernames and Full Names to my etherpad to facilitate the agreement process, that would make me happy16:20
*** aconrad has joined #openstack-infra16:22
*** jooools has quit IRC16:25
*** hemnafk is now known as hemna16:26
*** jpich has quit IRC16:28
*** zz_gondoi is now known as gondoi16:29
*** Guest22618 has quit IRC16:30
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: refactor cmd processor to make it slightly easier to test  https://review.openstack.org/9784816:31
*** otherwiseguy has joined #openstack-infra16:32
*** ewindisch has joined #openstack-infra16:32
*** smarcet has joined #openstack-infra16:33
*** bhuvan has joined #openstack-infra16:34
zarofungi: i think all you need is 'Create Reference' permission, https://groups.google.com/d/msg/repo-discuss/2kC1IBQ-n6k/RYjWfz5XKDMJ16:36
*** dims_ has joined #openstack-infra16:38
fungizaro: ahh, great--thanks!16:38
pleia2morning16:38
fungisdague: on bug 1326231 it looks like we're only hitting that on devstack-precise-check nodes, but not devstack-precise (which is odd... those images should in theory be identical)16:38
uvirtbotLaunchpad bug 1326231 in openstack-ci "Invalid timeout value cusing "gate-tempest-dsvm-full" to fail" [Undecided,New] https://launchpad.net/bugs/132623116:38
*** dizquierdo has quit IRC16:39
*** aconrad has quit IRC16:40
*** _nadya_ has quit IRC16:40
anteayamorning pleia216:43
*** dims_ has quit IRC16:43
clarkbmorning16:43
anteayamorning clarkb16:43
anteayajhesketh: so let us know what to do with turbo-hipster: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format16:43
anteayajhesketh: turbo-hipster-ci?16:43
openstackgerritA change was merged to stackforge/python-jenkins: Fix hacking docstring warnings  https://review.openstack.org/9767616:46
clarkbrackspace bd migration ci16:46
clarkbimo16:46
clarkber db16:46
*** Alexei_987 has quit IRC16:46
*** fbo is now known as fbo_away16:47
fungiisviridov: okay, pypi.slave is back on jenkins.o.o now, i've retriggered magnetodb's upload job and it seems to have worked https://pypi.python.org/pypi/magnetodb http://logs.openstack.org/06/06fdbf5dd339163772d6e4b31fa16b45140294ce/release/magnetodb-pypi-upload/72f6116/console.html16:47
fungiisviridov: er, premature. the job hasn't completed yet16:47
fungiisviridov: i'll let you know once it has16:47
*** ewindisch has quit IRC16:48
*** ociuhandu has quit IRC16:49
fungiaha, that 'splains it... forgot the job name is now either .*-pypi-wheel-upload or .*-pypi-both-upload16:49
*** dangers is now known as dangers_away16:50
fungiclarkb: ugh... new bug! http://logs.openstack.org/0606fdbf5dd339163772d6e4b31fa16b45140294ce/release/magnetodb-pypi-both-upload/72f6116/console.html "Uploading magnetodb-2.0.5.tar.gz ... AttributeError: 'str' object has no attribute 'read'"16:53
fungiso something is amiss with pypi-tarball-upload.sh now too16:54
anteayaSergeyLukjanov: right now your system savanna-ci is formatted in such a way that it doesn't require changing, do you want to rename it to sahara-ci?16:55
*** derekh_ has quit IRC16:56
reedmy emails to infra mlist failed to spark a conversation, I wonder why16:56
anteayareed: mostly because the gate is in such a poor state16:56
fungireed: mostly because we've been firefighting all week16:56
reedoh, that explains it16:57
reedthanks16:57
*** mmaglana has joined #openstack-infra16:57
reedkeep firefighting, my stuff can wait16:57
anteayareed: if we had had a clue during meeting time, we could have at least asked a few questions16:57
*** doude has joined #openstack-infra16:57
anteayareed: you might consider adding them to next weeks agenda so you are sure they get some attention16:57
reedanteaya, neither of the questions are time sensitive, process when time comes16:57
*** arnaud has joined #openstack-infra16:58
pleia2reed: I didn't want to hijack your activity board thread, but do you think reviewstats is a good candidate to landing on the new activity server once we have it up? http://git.openstack.org/cgit/openstack-infra/reviewstats/tree/README.rst16:58
*** doude has quit IRC16:59
*** rkukura has joined #openstack-infra16:59
pleia2trying to figure out where to land it, was going to go with status.o.o but activity seems to make 100x more sense16:59
pleia2(except for the fact that it doesn't exist in our CI yet)16:59
anteayareed: they don't have to be time sensitive to get air time during the meeting16:59
reedpleia2, totally, I actually wanted bitergia to add a panel for those stats16:59
anteayareed: but agenda items ensure we all have time to hear about the idea and ask questions16:59
reedanteaya, adding to the agenda, even if that means logging in the wiki :)17:00
pleia2reed: a bitergia-powered review script, or that specific one? (we have some reasons for having multiple different bits of code analyzing the review stats)17:00
anteayareed: yup17:00
anteayareed: unless you want me to do that for you, I will if you want17:01
reedanteaya, should I delete the June 3 agenda on https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting ?17:01
anteayano, just add june 10 to the top and add your item17:01
anteayathe chair de jour will clean it up before the meeting17:01
reedpleia2, I need bitergia to gather some stats for me from the review system, they might as well start producing the charts and plots and tables and json files we actually need17:01
anteayaadd your irc nick beside any of your items17:02
reedpleia2, s/we/you17:02
pleia2:)17:02
pleia2ok, thanks17:02
sdaguefungi: so yeh, we're jumping time a lot - http://logs.openstack.org/11/95911/2/check/check-grenade-dsvm/c91fe2f/logs/syslog.txt.gz17:03
clarkbfungi: nice!17:03
clarkbfungi: did you want to debug the upload script? I am happy to if you already have all the things on your plate this morning17:03
fungiclarkb: i can keep plugging at it. this is a continuation of discovering that now that we have working tarball jobs again, we never rebuilt the pypi slave, and once that was enrolled, remembering that we changed the job name, and now...17:04
openstackgerritJarret Raim proposed a change to openstack-infra/config: Adding barbican-specs to infra configs  https://review.openstack.org/9761617:05
reedpleia2, where can I see a sample output of reviewstats.py?17:06
pleia2reed: russellb has it on his website, I'll dig up link17:06
reedis that russellb's stuff17:06
reedok, I know that one17:06
reedhttp://russellbryant.net/openstack-stats/17:07
pleia2yeah17:07
notmynameclarkb: jeblair: mordred: is there any reason I can't start populating swift-specs?17:07
pleia2reed: since the code is now in infra, we want to host it for him :)17:07
clarkbnotmyname: I don't think so17:07
clarkbnotmyname: we have started populating ours17:07
russellbyeah i keep breaking it17:07
notmynameok, thanks17:07
*** aconrad has joined #openstack-infra17:07
openstackgerritA change was merged to openstack-infra/elastic-recheck: add query for timeout calculation gone wrong  https://review.openstack.org/9782617:08
mordred_phonenotmyname: it just needs a patch with all the specs-cookiecutter stuff17:08
notmynamemordred_phone: where is the cookiecutter repo?17:08
reedrussellb, I noticed :) most files I checked are empty17:08
russellbreed: sorry.17:09
russellbreed: you can run the scripts locally though17:09
russellbin theory17:09
russellbunleess they were busted by the gerrit upgrade17:09
mordred_phonenotmyname: openstack-dev/specs-cookiecutter17:09
russellbi'm just back this week after being out for over a month17:09
reedrussellb, no need to apologize, I just wanted to get a sample so I can ask Bitergia to make us a dashboard with those17:09
notmynamemordred_phone: thanks17:09
sdaguefungi: so on the time jump one, should we try to create a work around, or figure out why that region is weird17:10
mordred_phonenotmyname: let me know if you need help, I'm happy to make you a patch or whatnot17:10
sdaguemaybe they are booting with a different tz?17:10
notmynamemordred_phone: I'll do it today, I think. I'll ask if I have questions. and ping you with a link to the patch I submit17:10
sdaguefungi: a 4 hr jump is very suspicious to me17:10
fungisdague: probably i need to hold a node and check its syslog for signs that ntp is making a huge correction or something17:11
sdaguefungi: see url above I posted17:11
sdaguethat's exactly what's happening17:12
fungiaha!17:12
fungitoo many things at once ;)17:12
fungisdague: i suspect we've got a nova compute in rackspace iad whose time is hours off, and so instances are starting with radically incorrect time17:13
fungisdague: the start of occurrences doesn't seem to map to a nodepool image update17:13
sdaguethe fact that it's only happening in one az makes me feel like someone borked the hypervisor config there and it's set to a different timezone17:13
fungientirely possible17:14
sdaguebecause seed time should be set on boot17:14
fungisdague: and yeah, it's jumping precisely 4 hours, so almost certainly set to edt (where iad is located)17:14
fungiphschwartz: are you around?17:15
sdagueI guess we don't index syslog normally?17:18
SpamapSsdague: I found a hole where something can jump to the gate even if a dependent patch is -1'ed?17:18
sdagueSpamapS: yeh, some time ago17:18
fungiSpamapS: you're so good at exposing bugs you can't remember them all any more ;)17:18
SpamapSAh I was going to add it to my infra-breaking-trophy room ;)17:18
*** gokrokve has joined #openstack-infra17:18
sdagueat least my vague memory is that you discovered it17:18
SpamapSIt's really that I'm too lazy to test things so I just plod forward expecting them to work intuitively. ;)17:19
fungiSpamapS: change b is dependent on change a, and both have approval votes but b has failed (or not completed) check tests... when a gets enqueued into the gate b comes along for the ride automatically17:19
*** Ryan_Lane has joined #openstack-infra17:19
clarkbsdague: we index syslog17:19
fungib won't merge, but it will act as a wrecking ball all the way through the pipeline17:20
SpamapSOh I think I "found" that but I didn't actually debug it.. I just made it happen a few times. :)17:20
sdagueclarkb: .... hmmm17:20
*** dims_ has joined #openstack-infra17:20
fungiSpamapS: that's why i like to use the word "expose"17:20
clarkbsdague: we are very behind on indexing last I checked17:20
clarkbI think the tempest logs are more impactful than you expected17:21
sdagueclarkb: ok, well we can turn them off17:21
*** srenatus has quit IRC17:22
*** shivharis has joined #openstack-infra17:23
*** srenatus has joined #openstack-infra17:23
*** dims_ has quit IRC17:23
shivharisfungi: ping17:23
fungishivharis: hi there17:23
*** dims_ has joined #openstack-infra17:23
shivharisfungi: need to get onto "neutron bugs" team, how to do that...17:23
shivharisi have already sent the request,17:24
shivharisin waiting mode for about 2 weeks now17:24
sdaguethough syslog is running at 3x the # of lines of tempest due to ceilometer17:25
fungishivharis: an administrator of that group has to approve your request. try contacting one of the people listed as an administrator at https://launchpad.net/~neutron-bugs/+members (possibly via irc or e-mail as they may not be paying attention to lp notifications)17:25
clarkbsdague: oh well that may explain it17:26
shivharisthe admins like DanW and Rick C are not active, can we add Kyle Mestery?17:26
shivharisas admin17:26
clarkbsdague: is that swift leaking into syslog every time ceilometer hits it?17:26
sdagueclarkb: yep17:26
clarkbsdague: awesome :/17:26
sdaguewith good stuff like17:26
sdagueJun  4 16:38:08 devstack-precise-check-rax-iad-63313 proxy-server: Returning cached token17:26
sdagueJun  4 16:38:08 devstack-precise-check-rax-iad-63313 proxy-server: Storing token in cache17:26
clarkbdstufft: is http://logs.openstack.org/0606fdbf5dd339163772d6e4b31fa16b45140294ce/release/magnetodb-pypi-both-upload/72f6116/console.html a twine bug?17:27
clarkbfungi: ^ I wonder if server rebuild updated twine now with new bugs17:27
fungishivharis: yeah, i suspect mestery not being an administrator on that lp group was just an oversight on markmcclain's part during the ptl changing of the guard. fixed17:27
*** BobBall is now known as BobBallAway17:27
fungiclarkb: either that or we missed puppeting something like credentials17:28
shivharisfungi: thanks, i will request Kyle to admit. Much Thanks!!17:28
*** Sukhdev has quit IRC17:28
fungiclarkb: manually running it is turning up the same error consistently, as expected17:28
*** praneshp has joined #openstack-infra17:28
clarkbfungi: hrm, creds should be .pypirc iirc17:28
fungiyep, and it looks correct17:28
fungii may try downgrading twine and see where that gets me17:29
sdagueclarkb: http://paste.openstack.org/show/82805/17:29
mesteryfungi: +1! I even asked markmcclain to add me, though he's been traveling.17:29
fungimestery: problem solved now17:29
mesteryfungi: You rock man, thanks!17:29
fungimestery: any time17:29
clarkbfungi: pdb?17:30
clarkbI may give that a go17:30
dstufftclarkb: fungi it's not a bug i'm aware of, but that hardly means much17:30
fungiclarkb: i'm going to stuff older twines in a virtualenv to try to see if it's a regression, so you can pdb the system-installed one all you like17:31
clarkbfungi: why is the source tree checked out there? do we have an unneeded gerrit git prep?17:31
fungiclarkb: i plan to make that a needed ggp at some point soon anyway (so i can compare the tarball to a tag)17:32
clarkbfungi: oh ok I won't propose a change to remove it then :)17:32
*** jcoufal has joined #openstack-infra17:33
*** trinaths has left #openstack-infra17:33
dstufftthere hasn't been a new twine in awhile17:33
*** jp_at_hp has quit IRC17:33
anteayafungi: this is a paste with 6 lines that encompass the format that has come out of my suggestions on renaming gerrit automated accounts: http://paste.openstack.org/show/82807/17:34
*** nati_ueno has joined #openstack-infra17:34
fungidstufft: yeah, but we haven't rebuilt the system where we ran it in a while, and may have used an older one17:34
*** dangers_away is now known as dangers17:34
dstufftfungi: gotcha17:34
anteayafungi: when you get a chance if you can let me know if new accounts can work this this, that would make me happy17:34
*** akrivoka has quit IRC17:34
anteayas/this this/with this17:34
fungi...may have used an older one previously (thus wouldn't have noticed at release time)17:34
openstackgerritMarc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: Some tweaks to get closer to Python 3 compat  https://review.openstack.org/8781017:36
clarkbdstufft: it happens in twine-upload17:36
fungianteaya: lgtm. as we discussed, we should only enforce the new ssh username format on new accounts (though we should definitely change the display names of all old accounts). the only thing i think might be missing is an upper-bound on the lengths of the username and display name17:37
clarkband now I have to figure out how to get pdb to run in a subprocess. I guess I can just have pdd run twine-upload direclty17:37
dstuffttwine is bad software, but it's less bad tan setup.py upload17:37
dstufftI really need to get around to just making it ``pip publish``17:37
*** harlowja_away is now known as harlowja17:39
anteayafungi do you know what those upper bounds are, I do not know17:40
*** e0ne has joined #openstack-infra17:40
*** Alexei_987 has joined #openstack-infra17:41
fungianteaya: i don't know of any explicit upper bounds in gerrit (probably ~255 bytes allowed in the mysql columns), but we should pick something reasonable which fits the majority of the current systems and doesn't make the ui totally horrible looking17:41
anteayafungi: I'll see what we have right now17:41
fungidstufft: clarkb: seems to have been introduced between 1.2.2 and 1.2.317:42
anteayaafter I eat my sandwich17:42
*** e0ne has quit IRC17:42
*** e0ne has joined #openstack-infra17:43
fungiisviridov: your 2.0.5 release is up on pypi now... https://pypi.python.org/pypi/magnetodb17:43
*** masayukig has quit IRC17:43
*** e0ne has quit IRC17:43
fungidstufft: clarkb: i was able to upload with 1.2.2 but get that error with 1.2.3 and later17:43
*** sarob has joined #openstack-infra17:44
dstufftapparently i'm a bad person who didn't tag releases17:44
fungiheh17:44
openstackgerritSean Dague proposed a change to openstack-infra/os-loganalyze: construct syslog debug level  https://review.openstack.org/9786317:44
sdagueclarkb: ^^ is not tested17:45
sdaguehowever, is a proposed idea to address the logs17:45
clarkbdstufft: fields.update({k: (fn, fp.read())}) "/usr/lib/python2.7/dist-packages/requests/models.py", line 376, in send. fp has no read()17:45
clarkbsdague: thanks will look shortly17:45
sdagueoh... in the move to hpcloud 1.1 everything is 8 cpu now?17:46
dstufftclarkb: looks like an incompatible requests version?17:46
clarkbdstufft: I don't know enough of requests or twine to sort that out at least not right now17:46
clarkbdstufft: maybe?17:46
clarkbsdague: I guess?17:46
dstuffttwine has not been broken for me since 1.2.3, and i use the latest requests whenever I use it17:46
sdaguethat would explain why the sec group bug is happening more often17:47
dstufftwhat versionof requests is isntalled17:47
*** timrc is now known as timrc-afk17:47
clarkbdstufft: 0.8.217:47
dstufftoh lol17:47
dstufftthat's ancient17:47
*** lcheng_ has quit IRC17:47
clarkbdstufft: huh I guess that means something pins requests?17:48
*** timrc-afk is now known as timrc17:48
dstufftthat's coming from the OS17:48
dstufft /usr/lib/python2.7/dist-packages17:48
dstufftthat's where apt-get installs things17:48
*** fanhe has quit IRC17:48
clarkboh right17:48
fungidstufft: clarkb: i get that error with requests (2.3.0) in the virtualenv with twine (1.2.3) and later17:48
clarkbfungi: fun!17:49
*** zzelle has quit IRC17:49
fungiclarkb: try pdb'ing pypi.slave.o.o:~jenkins/twine-1.3.1/bin/twine17:49
clarkbdstufft: https://github.com/dstufft/twine/blob/1.3.1/twine/commands/upload.py#L174 is the twine side of the requests call17:49
*** masayukig has joined #openstack-infra17:50
clarkbfungi: is it a different fail?17:50
fungiclarkb: nope, same error17:50
clarkbI guess the requests code may be differen though /me does that17:50
fungi"AttributeError: 'str' object has no attribute 'read'"17:50
KiallSergeyLukjanov - if you have a chance, I've updated https://review.openstack.org/#/c/97348/ with you're suggestions (the Designate DevStack job review) :)17:51
anteayafungi: we have two usernames at 24 characters17:51
bodepdhow do I run that spell checker script? ie: where does it live?17:52
anteayathat is also the longest Full Name17:52
bodepds/spell checker/alpa-checker/17:52
clarkbdstufft: is it because files=filedata passing a string rather than files?17:52
dstufftI dunno17:52
dstufftdoesn't explain why it works for me :|17:52
*** _nadya_ has joined #openstack-infra17:52
fungiclarkb: dstufft: http://paste.openstack.org/show/8281417:53
fungidstufft: clarkb: oho! env fail17:54
*** Sukhdev has joined #openstack-infra17:54
*** jistr has joined #openstack-infra17:55
fungidstufft: clarkb: activating the venv produces different behavior than executing twine directly our of it17:55
clarkbhttp://docs.python-requests.org/en/latest/user/quickstart/#post-a-multipart-encoded-file there they pass both file objects and strings17:55
*** tkelsey has quit IRC17:55
*** ociuhandu has joined #openstack-infra17:55
dstuffterm17:55
dstufftwat17:55
dstufftI bet this is becuase of my stupid "lol let's do git style dispatch"17:55
clarkboh right17:55
fungidstufft: maybe an odd entrypoint behavior? it's definitely not something i'd expect17:56
dstufftit's not entry points17:56
clarkbfungi: twine forks twine-upload17:56
clarkbso the subprocess may derp17:56
dstufftfungi: try to repro with twine-upload instead of twine upload17:56
fungiooh, and twine-upload may be missing the parent env17:56
* fungi tries17:56
fungiyep!17:57
clarkbthat worked?17:57
*** andreykurilin_ has joined #openstack-infra17:57
clarkbif that is happy we can just update requests on that box17:57
fungiso it's the locally-installed requests getting imported by twine-upload when it gets called from twine17:57
fungicalling twine-upload directly from the venv or with the venv activated works17:57
clarkbcool lets just update requests17:58
fungidstufft: thanks!17:58
fungiclarkb: yes, let's ;)17:58
openstackgerritDan Bode proposed a change to openstack-infra/config: Add puppet-openstack project puppet-openstacklib  https://review.openstack.org/9735717:58
sdaguefungi / clarkb - how did this get into the gate - https://review.openstack.org/#/c/94813/ ?17:58
fungidstufft: want a bug for "twine subcommands don't inherit twine's environment"?17:58
fungisdague: seems to have piggy-backed on the parent 94812 change, via "the SpamapS hole"17:59
clarkbyup18:00
sdagueok, that's another one of those?18:00
*** gokrokve has quit IRC18:00
jogosdague: look at the top hit in the unclassified gate failrure: http://status.openstack.org/elastic-recheck/data/uncategorized.html hehe18:00
dstufftfungi: that or, "donald is a dolt who has bad ideas and shouldn't have made git style dispatching"18:00
sdaguejogo: yeh, so the failure you were tracking isn't the one yuo think18:00
sdagueit's not a merge conflict18:00
sdagueit's the SpamapS hole18:00
jogosdague: o_O do tell18:01
fungiyou know he's just going to configure his irc client to stop highlighting his nick in this channel any more ;)18:01
sdaguehe18:01
sdaguehehe18:01
clarkbsdague: fungi: going back to slave times. should devstack-gate do ntpdate as the first step in its run?18:01
sdagueclarkb: well I was trying to figure out why it seems to happen late here18:01
SpamapS"the SpamapS hole" was my nickname in highschool18:02
sdaguejogo: there is some process by which a child change that's approved can pull a bogus parent change into the gate with it18:02
SpamapSshortened it for IRC..18:02
clarkbalso https://jenkins03.openstack.org/job/gate-ceilometer-python27/964/console is cool18:02
sdaguehonestly, my brain power is kind of drained from diving through the other fixes this morning18:03
jogosdague: ouch18:03
sdagueso I don't fully understand the loophole18:03
sdaguejogo: you stable maint?18:04
*** e0ne has joined #openstack-infra18:04
jogosdague: nope18:04
*** sarob has quit IRC18:04
fungisdague: jogo: basically when zuul enqueues approved child changes triggered from the enqueuing of a parent change, it happens through a different routine which doesn't pass through the configured criteria18:04
sdaguefungi: ah gotcha18:04
*** sarob has joined #openstack-infra18:04
jogofungi: ahh18:05
sdagueyeh, so when we don't have a giant backlog we don't notice it18:05
sdaguebut with so much in flight18:05
fungisdague: jogo: and to fix it, that routine needs to have callouts to check that stuff, more or less18:05
jogoyeah, nova has similar issues with live migration18:05
jogoand scheduling18:05
clarkbI am debugging zuul now18:05
clarkbI don't think this is a hard bug to fix18:05
fungiwoot! yappi all the way18:06
clarkb(famous last words)18:06
clarkbwell I don't need yappi for this18:06
clarkb(famous last works part 2)18:06
clarkb*words18:06
*** amcrn has joined #openstack-infra18:06
fungiclarkb: oh, you said debugging not profiling18:06
fungiyou're tracking down the child-enqueuing hole18:06
fungigot it18:06
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: authorization property: add extended read support  https://review.openstack.org/9709718:07
*** esker has joined #openstack-infra18:08
*** praneshp_ has joined #openstack-infra18:08
*** pelix has quit IRC18:08
*** praneshp has quit IRC18:09
*** praneshp_ is now known as praneshp18:09
*** _nadya_ has quit IRC18:09
*** sarob has quit IRC18:09
*** apevec has quit IRC18:10
sdagueok, need to step away for a bit.18:10
fungiclarkb: sdague: jogo: here i think... http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/scheduler.py#n170118:10
*** e0ne has quit IRC18:11
clarkbwow transifex spam18:11
clarkbfungi: ya in enqueueChangesBehind18:11
clarkbit checks canMerge but not the pipeline constraints18:11
fungiyep, could trap it there or a few lines later in for other_change in to_enqueue: self.addChange(other_change, quiet)18:11
clarkbarg but the pipeline constraints are event based not change based. this may require more shifting of responsibility than I hoped18:15
openstackgerritAlex Gaynor proposed a change to openstack-infra/zuul: Simplified the scheduler very slightly  https://review.openstack.org/9786918:15
fungiyeah, i think the infrequency, modest impact and complexity to fix is why this has stuck around as a known issue18:16
*** yfried has joined #openstack-infra18:16
clarkbwell we have changeish_filters with approval checks, hrm why isn't this working18:21
*** zzelle has joined #openstack-infra18:22
*** nati_uen_ has joined #openstack-infra18:23
fungiclarkb: do you think we should switch pypi-tarball-upload.sh to just call twine via tox -e venv like we do with setup.py sdist? that would get latest twine and requests and whatever else into play18:23
fungiand then system python-requests deb becomes irrelevant18:23
clarkbwe would need to make every project depend on twine18:24
clarkbor I guess you could tox -evenv pip install twine && twine upload18:24
fungioh, hm... unless we told tox to do that, yeah18:24
clarkbif that works then I think that is a simple way forward18:24
clarkbactually no18:24
clarkbbecause tox will install the project18:24
clarkbwhihc is running arbitrary code18:24
clarkbwhich is why everything ever is bad18:24
fungioh, right. poop18:24
fungiso screw tox, just virtualenv directly but otherwise similar story18:25
fungior does that then introduce new random failures because we're downloading more things? grr...18:26
fungiwe need puppet virtualenv manglement18:26
*** nati_ueno has quit IRC18:26
fungiclarkb: do the changeish_filters approval checks just check for approval, or also check for other configured criteria (which are currently only associated with triggers)?18:27
fungilack of approval isn't the issue. the changes slipping in through this hole are approved with negative or missing check scores18:27
clarkbfungi: its just approval I think18:28
fungiyeah, so that part is working fine18:28
clarkbfungi: basically changeish_filters are the new thing which we are not completely migrated to18:28
clarkband if we were migrated to them this would work18:29
fungioh18:29
clarkbbut we are still using the old event filters too18:29
clarkb(this is my rudimentary understanding of the problem)18:29
clarkbyeah we depend on require-approval for jenkins votes18:29
clarkband it is deprecated and only checked on events not changes18:30
clarkbso I will stop working on this now18:30
clarkbthe bug will be fixed when we switch to approvals for everything18:30
isviridovfungi, thanks for magnetodb release18:30
*** melwitt has joined #openstack-infra18:30
fungiokay, so completing migration to changeish_filters will in theory solve this hole as a side effect18:30
fungiisviridov: you're welcome. apologies for the delay (you got caught in a chain of several different intertwined bugs there)18:31
clarkbfungi: yup because addChange() checks the changeish_filters18:31
bodepdI sorted the sort stuff. apparently _ comes after [a-z] (surprising...)18:32
fungiclarkb: right, that was begun to solve the testing as a reaction to comments on closed changes problem, and nicely addresses this one as well18:32
*** sarob has joined #openstack-infra18:32
clarkbfungi: I think there is a bug in approvals though18:32
clarkbbut we can cross that bridge when we get there18:32
fungibodepd: yeah, if you set LC_ALL=C in your environment and then run them through the sort utility you can emulate what that job does, hopefully18:32
fungiclarkb: right, that implementation got hurried through to stem the bleeding on the test loop spam problem, but still isn't well tested beyond that one use case18:34
fungibodepd: basically the sort order for a posix (non-localized) environment follows the ascii byte equivalent ordering18:35
fungibodepd: oh, except that we set it to a case-insensitive sort, so not entirely18:35
fungibecause _ normally falls between A-Z and a-z but i think case-insensitive sort uppercases it all first, which makes _ come after a-z as well18:36
openstackgerritDan Bode proposed a change to openstack-infra/config: Add puppet-openstack project puppet-openstacklib  https://review.openstack.org/9735718:36
clarkbwell maybe it wasn't a bug18:36
bodepdfungi: that didn't work (to set that to emulate)18:37
fungibodepd: huh. crazy18:37
bodepdfungi: I just guessed, submitted patches trying both orders18:37
bodepdfungi: and one worked!18:37
fungiand yeah, man sort says "-f, --ignore-case: fold lower case to upper case characters" so that explains why _ comes after a-z18:38
bodepdfungi: ah, I somehow missed the -f18:38
clarkbfungi: no I remember. Approvals are ANDed but require-approvals are OR'd18:39
clarkbfungi: so the mapping isn't quite 1:118:39
fungiclarkb: yeah, that'll make translating them somewhat of a challenge18:39
clarkbfungi: yeah especially since in the gate we say a vote of 1 or 2 is valid18:40
fungiclarkb: now you must have a vote of 1 and 2 instead. our code quality can only go up from here!18:40
clarkbfungi: right!18:40
Alex_Gaynormordred: ping18:41
*** jcoufal has quit IRC18:42
mordred_phoneAlex_Gaynor: just boarded a plane ... sup?18:43
Alex_Gaynormordred_phone: You mentioned the idea of using livegrep as a replication target, instead of needing to pull every 30 minutes. Where can I see some reference code/docs/whatever on how to set that up. Sounds way better than pulling18:44
*** denis_makogon_ has joined #openstack-infra18:44
mordred_phoneI think you want to look at our cgit mirrors18:45
*** denis_makogon_ is now known as denis_makogon18:45
mordred_phonepleia2: ^^18:45
mordred_phoneAlex_Gaynor: if you don't have it when I land, I'll send you things18:46
Alex_Gaynorthanks18:46
pleia2if I understand correctly, we're just using gerrit magic configured in puppet18:46
clarkbsdague: re the os log analyze change. My only concern is that I think those ar ethe only proxy server logs we get18:47
clarkbsdague: because ya18:47
pleia2or are we talking about something else?18:47
Alex_GaynorWould that be under the gerrit module then?18:47
pleia2I'll grab it, sec18:47
clarkbpleia2: I think that + the script to create the repos that cgit uses18:47
Alex_Gaynorthanks18:47
clarkbpleia2: Alex_Gaynor: however, could we just host livegrep on the cgit servers?18:48
clarkb(I don't know what kind of state is needed18:48
clarkband the cgit servers assume that things are mostly stateless)18:48
*** srenatus has quit IRC18:48
Alex_GaynorIt basically just needs the git repos, it can run in place with no previous prep18:48
Alex_GaynorYou can take a look at the existing puppet module to get a sense of how it works18:48
clarkboh cool then ya we might be able to just run it on top of the cgit repos18:48
*** dprince has quit IRC18:48
*** srenatus has joined #openstack-infra18:48
pleia2http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/manifests/review.pp#n13118:49
*** e0ne has joined #openstack-infra18:49
clarkbthen you don't haveto worry about how the repos show up18:49
pleia2^^ that's where we tell gerrit what to replicate to18:49
pleia2jeepyb has the script, sec18:49
pleia2https://git.openstack.org/cgit/openstack-infra/jeepyb/tree/jeepyb/cmd/create_cgitrepos.py18:49
clarkbjogo: for your silent large ops jobs how comfortable are you with just bumping the number now?18:50
clarkbjogo: you must have a reasonable sample at this point18:50
lifelessjogo: hi18:51
lifelessjogo: elastic recheck and nonvoting jobs18:51
lifelessjogo: I haven't checked the review again, thought maybe we needed some realtime bandwidth :)18:51
*** arnaud has quit IRC18:51
clarkbjogo: for the last 24 hours on the non neutron job there are 109 successes to 5 failures18:53
*** jistr has quit IRC18:55
*** rcarrillocruz has quit IRC19:00
*** marcoemorais has joined #openstack-infra19:02
clarkbfungi: for zuul approvals I think we can do something like: https://etherpad.openstack.org/p/zuul-approvals19:04
mtreinishjeblair: https://wiki.openstack.org/wiki/Qa_Infra_Meetup_2014 did you change back without telling anyone? :)19:04
fungimtreinish: he did. but now he's on a week+long hike into the wilderness, so that you won't be able to interrogate him about it ;)19:05
*** reed has quit IRC19:06
*** rcarrillocruz has joined #openstack-infra19:06
clarkbbefore I run to grab lunch19:06
*** dprince has joined #openstack-infra19:07
clarkbmtreinish: is there consensus ish on where people are hotelling?19:07
clarkbI don't want to be the ONE person in darmstadt or frankfurt19:07
fungiclarkb: mtreinish: i meant to ask the same19:07
mtreinishI still haven't even really looked for myself yet19:07
fungiknowing someone in the same hotel increases the chances we both make it to the venue successfully via whatever mode of transportation is chosen19:08
mtreinishI was thinking of staying darmstadt though just to make it easier and not having to wake up earlier19:08
clarkbmtreinish: yes I like this plan19:08
fungithat was my preference as well19:08
clarkbalso darmstadt appears to have beer++ so bases are covered19:08
*** gokrokve has joined #openstack-infra19:10
mtreinishmkoderer put 2 hotels on the wiki page, both are within a reasonable walking distance19:10
fungiideal!19:11
fungialso has anyone tried to convince Ajaeger to join us? i think he's only a couple hundred kilometers away, near nuremberg?19:12
*** andreaf has joined #openstack-infra19:12
fungigood opportunity there given he does so much infra-facing automation for the docs team19:12
*** andreykurilin_ has quit IRC19:13
mtreinishbtw, before I forget would there be oposition to swapping the qa day with the infra day on Tues.?19:13
bknudsonI wonder how this happened -- https://review.openstack.org/#/c/96265/19:14
bknudsonproposal bot got -1 that the requirements don't match19:14
fungibknudson: laaaaaag19:14
bknudson"recheck no bug" ?19:15
bknudsonor is there a laaaaaag bug?19:15
*** gokrokve has quit IRC19:15
fungibknudson: that was a guess. looking at the log i think it's something more subtle19:15
fungiit's not that it doesn't match, but rather it's rejected for not overlapping with previous requirements19:16
fungiand indeed it does not19:16
fungithe bigger question is how we tunnel ourselves out of this situation...19:16
bknudsonah19:16
bknudsonmaybe we were slow about merging a previous change19:17
fungithe problem it's attempting to prevent is that sphinx>=1.1.2,<1.2 and sphinx>=1.2.1,<1.3 do not cover any potential versions in common19:17
fungiwhich makes transitioning from one to the other problematic19:17
bknudsonpatch set 1 was 05-28 which isn't that long ago.19:18
fungithat new check was only introduced very recently i think, and is possibly being enforced in the wrong place19:18
fungii thought the intent was to enforce that check on changes to the global-requirements.txt in openstack/requirements, not on individual project reqs changes19:19
*** praneshp has quit IRC19:20
fungithe problem being that when a change to global-reqs is non-overlapping, it leave a window of broken for projects using the restricted mirror since anything behind it in the gate and anything else tested between when it merges and when the mirror updates complete will potentially fail to find viable versions19:20
devanandarequirements question and I think folks here might know the answer far better than I19:21
bknudsonfungi: I think I remember that discussion from a while back19:21
bknudsonso is this going to break the gate for keystoneclient?19:22
*** hashar has joined #openstack-infra19:22
devanandaironic may have an issue with ipmitool <= 1.8.11 and would like to add a warning if that's present, which it is in ubuntu 12.04. the issue is fixed in >= 1.8.12 (which is present in ubuntu 14.04, recent fedoras, etc)19:22
bknudsonhere's the change in requirements http://git.openstack.org/cgit/openstack/requirements/commit/global-requirements.txt?id=a78540bae70ea75b54ca0aad34fca18dc2ade56619:23
bknudsonsphinx>=1.1.2,<1.2 -> sphinx>=1.2.1,<1.319:23
bknudsonso those didn't overlap19:23
*** otherwiseguy has quit IRC19:23
fungienforcement added in https://review.openstack.org/84100 merged 2014-06-01 02:25 utc19:24
bknudsonok, that was recent!19:24
fungiwe may want to consider reverting that for now and get it readded to the correct job19:24
fungisdague: jogo: do you recall the discussion around that?19:25
fungijust want to make sure i'm not completely off base on why this was added where it was19:25
fungidevananda: so what's the question? wrt discovering the ipmitool version installed on the system?19:26
devanandafungi: a) right way to discover that, b) right thing to do if it's old19:27
fungidevananda: or just whether it's okay to depend on a version not in rhel 6 and ubuntu precise?19:27
fungidevananda: if you think you're not going to be integrated prior to rhel 7 (and since ubuntu 14.04 is out), and if nothing else depends on ipmitool besides ironic, seems like just updating ironic's reqs and the global reqs to >= 1.8.12 should be fine19:29
SergeyLukjanovoh, 503 messages in scrollback19:30
SergeyLukjanovKiall, I'll take a look19:30
fungidevananda: detecting older versions of modules from within python, if you want to still run but present a warning, can be done by importing and then looking in something like .__version__ i think19:30
devanandafungi: sorry, not module -- system package / utility19:30
SergeyLukjanovanteaya, yup, we'd like to rename it to sahara-ci, too lazy to do it ;)19:30
fungidevananda: oh, got it. probably calling ipmitool --version via a shell callout and parsing its stdout, i would guess19:31
devanandafungi: heh, ok. i had imagiend that there might be a cleaner way :)19:31
fungidevananda: since trying to discover it from distro package details is going to get real ugly real fast from a cross-platform perspective19:32
devanandaack19:32
*** Sukhdev has quit IRC19:33
*** nati_uen_ has quit IRC19:33
*** esker has quit IRC19:35
*** esker has joined #openstack-infra19:35
SergeyLukjanovanteaya, I'm here now, google-translating your message re accounts19:35
SergeyLukjanovanteaya, does you mean to provide you a list of new accounts?19:36
*** gokrokve has joined #openstack-infra19:36
bknudsonfungi sdague jogo: https://bugs.launchpad.net/openstack-ci/+bug/132650419:36
uvirtbotLaunchpad bug 1326504 in openstack-ci "Requirements change and check causes proposal failure" [Undecided,New]19:36
*** marcoemorais has quit IRC19:37
*** marcoemorais has joined #openstack-infra19:38
*** marcoemorais has quit IRC19:38
*** shivharis has quit IRC19:38
*** reed has joined #openstack-infra19:38
*** Sukhdev has joined #openstack-infra19:38
*** marcoemorais has joined #openstack-infra19:38
*** nati_ueno has joined #openstack-infra19:39
openstackgerritBrant Knudson proposed a change to openstack-infra/config: Revert "Checks if requirements overlap between head and branch reqs"  https://review.openstack.org/9789319:39
openstackgerritBrant Knudson proposed a change to openstack-infra/config: Revert "Checks if requirements overlap between head and branch reqs"  https://review.openstack.org/9789319:39
*** esker has quit IRC19:40
*** palar_ has joined #openstack-infra19:40
*** masayukig has quit IRC19:40
*** michchap has quit IRC19:40
*** jp_at_hp has joined #openstack-infra19:41
anteayaSergeyLukjanov: that could work or are you able to follow this format:19:42
anteayaCi if it votes or ever might in the future, Bot if it will never ever vote19:42
anteayaFormat:19:42
anteaya    username --> lowercase, hyphens between words, matches Full Name, -ci or -bot at the end19:42
anteaya    Full Name --> capitalize, upper(minimal) and lowercase, matches username, CI, -CI, -ci or Bot, -Bot, -bot at the end19:42
anteaya    {company name}-{thing it is testing}-{ci|bot}19:42
anteaya    remove test, testing, jenkins, openstack, tempest, user from the names19:42
anteayapaste.o.o is 500'ing19:42
KiallSergeyLukjanov: thanks :)19:42
*** palar has quit IRC19:43
*** marcoemorais has quit IRC19:44
*** marcoemorais has joined #openstack-infra19:44
anteayaSergeyLukjanov: and fungi asked for an upper bound for length, which I don't know but the longest username and Full Name we have right now is 24 characters19:44
SergeyLukjanovKiall, it looks ok, but you have a lot of non-voting jobs in gate pipeline, we prefer to avoid it due to the such stuff is totally ignored in gate19:44
*** palar_ has quit IRC19:44
KiallSergeyLukjanov: yep, I intend to change them to voting as soon as we know they work19:45
SergeyLukjanovanteaya, ack, I'll check acc requests tonight or tomorrow and make a list19:45
anteayaSergeyLukjanov: thanks I appreciate it19:45
SergeyLukjanovKiall, okay, please, response in CR to avoid new question on it19:45
*** masayukig has joined #openstack-infra19:45
KiallSergeyLukjanov: Sure19:46
anteayaSergeyLukjanov: I'll let cinder know we are processing requests since we have agreed on a naming format19:46
KiallSergeyLukjanov: done :)19:47
*** e0ne has quit IRC19:47
*** Ryan_Lane has quit IRC19:48
*** esker has joined #openstack-infra19:48
*** Ryan_Lane has joined #openstack-infra19:48
*** resker has joined #openstack-infra19:48
*** sarob has quit IRC19:49
sdaguefungi: reading scrollback19:50
SergeyLukjanovanteaya, just to ack - we agreed to wait for several days to agreed on name pattern?19:50
sdagueso the reason we were enforcing overlapping requirements is because of the gate19:50
anteayaSergeyLukjanov: well here we are19:50
anteayaI have a name pattern and am willing and waiting for agreement19:50
sdaguefungi: however, that should only be enforcing it on global requirements, not on the projects19:51
sdagueso the check seems to be in the wrong place19:51
fungisdague: okay, that confirms my recollection then19:51
anteayayou and fungi wanted to not adhere to a pattern and create a bunch more accounts19:51
*** HenryG has quit IRC19:51
anteayawhich doesn't work for me19:51
sdaguebecause otherwise you leave a wake of broken gate jobs until the mirror updates19:51
anteayacinder said can you come up with something in 24 hours so you can keep processing accounts, which I have19:51
anteayaand which was reasonable for them to give me that time19:52
anteayaso what are we waiting for?19:52
anteayaif we are waiting for something besides time to elapse, let me know whta it is so I can do it19:52
fungianteaya: i think the proposed name pattern seems fine to move forward with19:52
anteayafungi: thanks19:52
anteayaSergeyLukjanov: if you need more, please tell me so I can get it for you19:52
*** otherwiseguy has joined #openstack-infra19:53
fungisdague: yep, cool. so i guess let's rush bknudson's revert through and then figure out how to get that code transplanted to the correct place19:53
*** esker has quit IRC19:53
sdaguefungi: is there a rush on this?19:54
sdagueunless it's fixing a gate reset, I don't see the point in the rush.19:54
sdagueBecause it's going to take 2 days to land19:54
sdaguemy back of the envelope is that the gate queue is probably really 48hrs deep right now19:55
bknudsonthere's no rush for me19:55
fungisdague: well, it's currently broken proposed requirements changes for any project depending on sphinx19:56
fungiwhich would be... most projects i think19:56
fungibecause someone just got a non-overlapping sphinx version bump into global-reqs19:57
fungicausing this to -1 pretty much all the reqs proposal changes which came after19:58
bknudsonyou want a change to sphinx reqs so they overlap?19:58
sdaguebknudson: no19:58
clarkbok back from lunch19:58
*** fbo_away is now known as fbo19:58
sdagueso in the land of new fails - https://jenkins05.openstack.org/job/gate-tempest-dsvm-full/4156/console19:58
sdague2014-06-04 19:53:07.289 | sudo: /usr/sbin/apache2ctl: command not found19:59
sdaguehp region b19:59
fungisdague: infrequent, or common?19:59
clarkbsdague: devstack should install that if it needs it20:00
clarkbsdague: and iirc the job I saw that did that had a huge lag time between running devstack and that error20:00
*** fifieldt_ has joined #openstack-infra20:01
sdaguefungi: searcing20:01
*** eharney has quit IRC20:01
sdagueso it's a lot - http://logstash.openstack.org/#eyJzZWFyY2giOiJcIi91c3Ivc2Jpbi9hcGFjaGUyY3RsOiBjb21tYW5kIG5vdCBmb3VuZFwiIiwiZmllbGRzIjpbXSwib2Zmc2V0IjowLCJ0aW1lZnJhbWUiOiI2MDQ4MDAiLCJncmFwaG1vZGUiOiJjb3VudCIsInRpbWUiOnsidXNlcl9pbnRlcnZhbCI6MH0sInN0YW1wIjoxNDAxOTEyMDY4MDA0fQ==20:01
*** apevec has joined #openstack-infra20:01
*** apevec has joined #openstack-infra20:01
sdaguebut it's not always a failure20:01
fungilooking through the devstack log for that one now... i'm wondering whether it's failing to install apache2?20:02
clarkbfungi: right I think sudo: /usr/sbin/apache2ctl: command not found is just noise20:03
fungiahh20:03
*** nati_ueno has quit IRC20:04
fungiyep! http://logs.openstack.org/76/79776/15/gate/gate-tempest-dsvm-full/700ee7e/logs/devstacklog.txt.gz#_2014-06-04_19_53_49_91820:04
*** hogepodge has joined #openstack-infra20:04
fungiit died because curl -O https://pypi.python.org/packages/source/p/pip/pip-1.4.1.tar.gz threw err_trap20:04
*** sabari` has joined #openstack-infra20:04
*** fifieldt-afk has quit IRC20:04
fungijudging from the log20:04
*** sabari` is now known as sabari20:05
clarkboh fun, why is that curling pypi20:05
clarkbinstall pip should use the cached file on disk20:05
fungibecause /opt/stack/new/devstack/files/pip-1.4.1.tar.gz is missing20:05
clarkbright but we shouldn't use a pip sdist at all20:05
clarkbwe should use that crazy self extracting zip file that dstufft has hosted and we cache20:06
fungiright now it's coming from the install_pip_tarball function in devstack20:06
*** nati_uen_ has joined #openstack-infra20:07
fungioic, the machine system has pip 1.5.6 installed, so devstack decides to uninstall it because it wants 1.4.120:07
clarkboh fun20:07
fungihilariously, it finds pip-installed pip 1.5.6, freaks out, tries to apt-get remove it...20:08
fungithen goes on to look for a local tarball of 1.4.1 and then, not finding it, downloads one from pypi20:09
*** adalbas has quit IRC20:09
clarkbsdague: ^ is that something you can untangle? I am guessing 1.5.6 comes from the pip zip thing20:09
*** nati_ueno has joined #openstack-infra20:09
*** nati_uen_ has quit IRC20:10
fungimy guess is that this is legacy cruft from trying to UPGRADE pip on systems with too-old distro packages of it20:10
fungibut instead of looking to see if the version of pip is new enough, it just flattens to 1.4.1 or not 1.4.120:10
sdagueclarkb: any chance I can get hooked up with an hp region b account? I might need to poke.20:11
fungiwhich would explain the misguided assumption that the distro package tools are removing the non-conforming pip version20:11
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for neutron fwaas bug 1326510  https://review.openstack.org/9790720:11
uvirtbotLaunchpad bug 1326510 in neutron "tempest.api.network.test_fwaas_extensions.FWaaSExtensionTestJSON fails with 409" [Undecided,New] https://launchpad.net/bugs/132651020:11
clarkbsdague: I can spin up nodes if nothing else20:11
sdaguefungi: so the fact that it crokes is weird20:11
*** marcoemorais has quit IRC20:12
*** denis_makogon has quit IRC20:12
*** marcoemorais has joined #openstack-infra20:12
fungisdague: the croaking/choking looks like maybe a network failure impacting connection from the slave to pypi.python.org20:12
fungihard to tell without additional detail from curl20:12
fungibut it definitely didn't finish downloading20:13
bknudsonI'm not too familiar with the infra config and how it works, but I would expect that to check something for requirements, there should be a separate job for requirements20:13
bknudsonso the fix would be to write a new script in modules/openstack_project/files/slave_scripts/ for the code that was added to project-requirements-change.py to verify the nonoverlapping versions20:14
sdaguefungi: so nodepool isn't always installing the 1.4.1 tarball?20:14
jogoclarkb: http://jogo.github.io/gate/large-ops-testing.html I am comfortable bumping it if you are.20:15
bknudsonand then have a job in requirements to run that new script20:15
fungibknudson: yeah, or add it to its integration test or to its python unit tests or something, but probably separate job would make most sense20:15
jogoclarkb: i'll push up a patch now20:16
jogolifeless: pong20:16
clarkbsdague: it is never installing the tarball20:16
bknudsonfungi: ah, I'll check for an existing script I can add it to20:16
clarkbsdague: nodepool installs a self extracting zip of pip20:16
sdagueclarkb: I mean caching it20:16
fungibknudson: well, i think that's how we got in this mess ;)20:16
clarkbsdague: yes caching. it caches a different thing20:16
clarkbsdague: and I believe that is where 1.5.6 comes from20:16
fungibknudson: the integration test is actually running a script out of the openstack-dev/pbr repo (which is sort of strange but for historical reasons), or the unit tests are running things out of the openstack/requirements repo via tox20:17
clarkbsdague: I think there may possibly be a misunderstanding between devstack and nodepool on what should be cached20:17
sdagueok, so install_pip.sh is currently extremely specific on requested version. Think we should just set a floor?20:17
clarkbsdague: I think so20:17
sdaguehttps://github.com/openstack-dev/devstack/blob/master/tools/install_pip.sh#L3020:17
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: refactor cmd processor to make it slightly easier to test  https://review.openstack.org/9784820:18
clarkbsdague: https://github.com/openstack-dev/devstack/blob/master/tools/install_pip.sh#L47 is what we cache20:18
fungibknudson: i think maybe a new script in the openstack/requirements repo which can get called as a tox env would be a fairly simple solution, and more easily runnable for devs too20:18
clarkbsdague: so maybe use-get-pip should be default or d-g should set what ever flag is needed for that20:18
bknudsonfungi: that makes sense, and sounds pretty easy20:19
*** hartsocks has left #openstack-infra20:19
sdagueclarkb: then I'm confused20:19
sdaguebecause if get-pip.py is there it should use that20:19
sdagueor you think we're missing the flag?20:19
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: tests for JJB command processing  https://review.openstack.org/9790820:19
fungibknudson: that way if we add it to the default env list for tox.ini in the reqs project, they can just run 'tox' and see results20:19
clarkbsdague: yes I think missing the flag20:19
gilliard320:19
sdagueclarkb: I'm also happy to change the devstack default here if we think that's the better default now20:20
fungigilliard: i see your 3 and raise you 220:20
sdagueactually, is there any reason not to use get-pip?20:20
clarkbdstufft: ^20:20
clarkbsdague: the one issue may be that it is a self extracting zip file20:20
clarkbwhich may be scary to some people20:20
*** mwagner_lap has quit IRC20:20
sdaguethat is by far the least scary thing in devstack20:20
gilliardfungi: you win20:21
clarkbsdague: :)20:21
clarkbsdague: it is hosted on a reliable CDN now and so on20:21
clarkbsdague: I think its pretty safe to use20:21
sdagueok, let me propose, at least to get check results. I would like dstufft's pov on it20:21
clarkb++20:21
anteayagilliard: he's bluffing, you should have called20:22
openstackgerritJoe Gordon proposed a change to openstack-infra/config: Bump LARGE_OPS number from 50=>100  https://review.openstack.org/9791020:22
jogoclarkb sdague dansmith: ^20:22
fungianteaya sees through my poker face20:23
jogoclarkb: are we using HP cloud 1.1 now?20:23
jogoclarkb: because if not we should leave the large-ops testing around for a bit longer20:23
fungijogo: yes, for large definitions of "using"20:23
fungijogo: we haven't been running any jobs on hpcloud 1.0 all week20:23
anteayawe have been exersising hpcloud 1.120:24
anteayathink hamster wheel20:24
clarkbjogo: we are20:24
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for neutron fwaas bug 1326510  https://review.openstack.org/9790720:24
uvirtbotLaunchpad bug 1326510 in neutron "tempest.api.network.test_fwaas_extensions.FWaaSExtensionTestJSON fails with 409" [Undecided,New] https://launchpad.net/bugs/132651020:24
jogoso large-ops-testing has run on there?20:24
lifelessjogo: o/20:25
fungijogo: probably a bunch of times, yes20:25
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: tests for JJB command processing  https://review.openstack.org/9790820:25
clarkbfungi: don't forget we need to apply jeblair's nodepool changes and restart nodepool20:25
clarkbfungi: have those changes merged yet? /me looks20:25
fungiclarkb: right, and i think puppet is still disabled there too20:26
* jogo checks logstash20:26
jogowhat is the naming scheme for hpcloud 1.1 nodes?20:26
fungijogo:  devstack-precise-hpcloud-b\d-[\d]+20:26
lifelessjogo: so yeah, elastic-recheck and tripleo jobs - which are all in a nonvoting queue because $reasons20:26
clarkbhttps://review.openstack.org/#/c/97570/2 it merged20:26
lifelessjogo: but all the tripleo reviewers treat them as voting20:27
clarkbso making sure nodepool puppet is happy first then planning to restart nodepool can happen20:27
dstufftclarkb: sdague get-pip.py should be no different than doing pip install pip20:27
jogolifeless: so the patch that landed just looks for " (non-voting)" in the jenkins comment20:27
lifelessjogo: which is odd and frustrating but thats what it is20:27
jogoand I don't think the tripleo jobs have that tag20:27
dstufftso if you're OK with pip installing into wherever, it's fine to use20:27
fungijogo: the integer after the "b" is one of five separate precreated neutron nets we've divvied it up into, so as to account for the network size being a /2420:27
jogofungi: so devstack-precise-hpcloud-b2-6557220:27
sdaguedstufft: for devstack, it's probably fine20:27
fungijogo: like that, right20:28
clarkbjogo: hpcloud-b*20:28
sdaguelike I said, it's really one of the least scary bits in devstack20:28
fungis/devstack/blowawayyoursystemstack/20:29
jogoso in last 48 hours: 16 failed large-ops-tseting20:29
gilliardugh - terrible connectivity on the train.  We believe hpcloud 1.1 issues to be undercontrol now.  Thanks for the firedrill everyone.20:30
gilliardIt's been a fun week.20:30
jogoand 521 successfull :)20:30
fungigilliard: thanks for sticking with it!20:30
jogosound look good numbers to me20:30
virmitioquestion regarding Jenkins Job Builder.  I find in the docs that I can specify 'block-upstream: true' to wait on upstream jobs, but how do I specify an upstream job?20:30
jogoand failed are on both rax and HP 1.120:30
clarkbgilliard: yes thank you for hanging in there20:30
gilliardI will pass on your thanks to the team :)20:31
jogoclarkb: don't know if I have told you enough how awesome logstash.o.o is20:31
clarkbjogo: :)20:31
*** jamielennox is now known as jamielennox|away20:31
fungivirmitio: zaro or hashar might know the answer... hopefully one of them is around20:32
sdaguefungi: so the invalid timeout bug is #8 on the ER list (the top couple are on non voting jobs, so not really an issue)20:33
sdaguethoughts on how we should fix that?20:33
fungisdague: yeah, i guess clarkb's ntpdate suggestion has merit20:33
sdaguedo we trust that it's there?20:33
clarkbI am clearly biased but ++20:33
fungisdague: also, i should open a ticket with rackspace to let them know they're misconfigured one or more of their nova compute nodes20:34
clarkbsdague: I believe our base node stuff installs ntp machinery20:34
clarkbsdague: so kicking it with ntpdate should make it happy20:34
fungiclarkb: i think ntpdate comes in a separate package20:34
fungichecking20:34
clarkboh20:34
*** dangers is now known as dangers_away20:34
hasharI am there (virmitio zaro fungi)20:34
*** eharney has joined #openstack-infra20:35
sdagueoh, well one of the failed jobs would tell us20:35
sdaguefungi: it's a different package on 14.0420:35
fungihashar: virmitio has a question about defining jenkins "upstream" jobs in jjb (related to the block-upstream option)20:35
sdagueI don't have 12.04 handy20:35
fungiclarkb: sdague: looks like it's a separate package but installed20:36
hasharvirmitio: block-upstream block-downstream are generic Jenkins jobs options.    They are only used when a job is triggered by another job for example when using the publisher 'trigger' http://ci.openstack.org/jenkins-job-builder/publishers.html#publishers.trigger20:36
sdaguefungi: ok, lets risk it20:37
fungii think it might be preinstalled on rax20:37
sdaguefungi: what ntp server are we going to ask for?20:37
fungibecause it doesn't look like ntp depends on it, and i don't see it mentioned in our config repo20:38
openstackgerritEvgeny Fadeev proposed a change to openstack-infra/askbot-theme: fixed ask button on tags and other pages  https://review.openstack.org/9792020:38
clarkbfungi: we use a third party ntp module iirc20:38
clarkbone of pabelangers20:38
zarohashar: i'm thinking about making a point release of jjb to pypi to fix the missing info on pypi.20:38
hasharvirmitio: as an example you can have a job A triggering a job B and want to make sure job A does not run again until the child (B) is finished. You would set job A  with block-downstream.20:38
zarohashar: would like your help to get 2 patches thru to fix that.20:39
hasharzaro: ah sure!20:39
virmitiohashar: Thank you, much appreciated.  That wasn't clear when just hunting through the docs on my own.20:39
hasharzaro: sorry I had a strike of reviews a month ago and have been lagging since then.  I can surely get those two patches reviewed/checked.20:39
sdagueclarkb / fungi - ntp server name to use? or should I grep it out of /etc/ntp.conf?20:40
hasharvirmitio: note that you can set A with block-downstream (i.e. wait for B)   or set B with block-upstream (i.e. A will wait for B).20:40
zarovirmitio: patches to make the docs more clear would be welcome :)20:40
hasharzaro: what are the tests?  I can do that tomorrow morning (europe time)20:40
fungisdague: clarkb: puppetlabs-ntp 0.2.0 apparently20:40
fungidigging into it now to see whether it's pulling in ntpdate20:41
clarkbsdague: I think you can grep20:41
clarkbhrm though if the daemon is running ntpdate won't update the time20:41
clarkband we should have the daemon running20:42
fungisdague: looks like we just use [0123].debian.pool.ntp.org on the current nodes20:42
notmynamemordred_phone: there is a _lot_ more to a specs repo than I expected (I'll leave the "why" questions for later). if you've already got cookiecutter available, can you set it up?20:42
virmitiohashar: I'm afraid your last comment sounds like it conflicts with the docs.  Should me placing block-upstream on B cause B to wait for A to next complete?20:42
openstackgerritA change was merged to openstack-infra/askbot-theme: fixed ask button on tags and other pages  https://review.openstack.org/9792020:43
hasharvirmitio: yeah maybe I get it wrong :-/20:44
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: force ntpdate before we start the timer  https://review.openstack.org/9792420:44
virmitiohashar: I guess I'll try it and find out.20:44
clarkbI guess that change will test if it works20:44
sdagueyep20:44
zarohashar: ohh just i noticed that you've already approved the fix already.  ok new point release should be out soon.20:44
hasharzaro: wonderful!20:45
fungiclarkb: no mention of ntpdate in https://github.com/puppetlabs/puppetlabs-ntp20:45
sdagueok, I'm about to end my day as our lug meeting starts in an hour or so.20:45
*** whayutin_ has joined #openstack-infra20:45
clarkbsdague: have fun20:45
clarkbfungi: huh I wonder if maybe it is in the base image then?20:45
sdagueI bet it's in the base image20:45
fungiclarkb: sdague: that's what i'm worried about20:46
hasharzaro: you proposed the removal of a misleading example. Might as well land it in https://review.openstack.org/#/c/96309/20:46
fungiclarkb: sdague: because if it is, this may insta-break on !rax for example20:46
sdaguefungi: well we can add it to local policy20:46
zarohashar: would be good to get these 2 doc changes in, 96309 and 9627920:46
clarkbyeah easy to add to puppet20:47
sdagued-g has enough jobs that we should see hp runs on it20:47
fungisdague: yeah, i think we probably should, and to avoid waiting for image updates, you could make 97924 conditionally opportunistic20:47
sdagueclarkb / dstufft - if you want to have an opinion on this - https://review.openstack.org/#/c/97912/ would be appreciated20:47
clarkblooking20:47
dstufftoh I love opinions20:47
sdaguethat's the get-pip one20:48
*** dkliban is now known as dkliban_brb20:48
*** weshay has quit IRC20:48
*** ArxCruz_ has quit IRC20:49
*** sarob has joined #openstack-infra20:49
*** eharney has quit IRC20:50
zarohashar: you read my mind, would you mind taking a look at those 2?20:50
*** eharney has joined #openstack-infra20:50
hasharzaro: I have approved 96279 which removes the sample. That is nice.20:50
*** tcammann has joined #openstack-infra20:50
hasharzaro: the other 96309 I have to try it out.20:50
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: tests for JJB command processing  https://review.openstack.org/9790820:50
hasharzaro: was introduced with https://review.openstack.org/#/c/43013/ so I guess the example used to be correct at one point20:51
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: reference tests as examples  https://review.openstack.org/9627920:52
hasharzaro: I will do the review tomorrow and let you know.20:53
*** harlowja has quit IRC20:54
*** harlowja has joined #openstack-infra20:54
*** dizquierdo has joined #openstack-infra20:54
*** mbacchi has quit IRC20:54
zarohashar: i believe 43013 added the ability to do subsitution on job lists (i.e. - '{name}-build':) but not on the parameters of the list (i.e. pipeline-next: '{name}-upload')20:55
*** tcammann has quit IRC20:55
zarohashar: anyways, please take your time.  Not in a hurry to do a release.20:56
*** tcammann has joined #openstack-infra20:56
*** tcammann has quit IRC20:57
*** masayukig has quit IRC20:57
jogosdague: one last review before I am ready to cut hacking 0.9 https://review.openstack.org/#/c/97580/20:58
*** lcostantino has quit IRC20:58
*** pdmars has quit IRC20:58
openstackgerritAntoine Musso proposed a change to openstack-infra/zuul: Make swiftclient an optional dependency  https://review.openstack.org/9793320:59
jogodhellmann: ^21:00
jogodhellmann:  never mind you already reviewed it. fungi ^21:01
*** marcoemorais has quit IRC21:01
*** marcoemorais has joined #openstack-infra21:02
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: tests for JJB command processing  https://review.openstack.org/9790821:02
*** e0ne has joined #openstack-infra21:02
*** marcoemorais has quit IRC21:03
*** e0ne has quit IRC21:03
*** eharney has quit IRC21:03
sdaguelifeless: re jogo's er change, I think it was more limitted than you realized. It only impacts the real time reporting, not the bulk analysis21:03
*** masayukig has joined #openstack-infra21:03
sdagueand the reason for the real time reporting is to tell people the bug unrelated to their code that gave them a -121:03
sdaguewhich by definition non-voting jobs can't do21:03
*** marcoemorais has joined #openstack-infra21:03
*** marcoemorais has joined #openstack-infra21:04
hasharzaro: I will take my time. Probably add some more tests to exercise the code while I look at it hehe.21:04
jogolifeless: yeah, and we can always add whitelists of jobs if needed etc21:05
*** mfer has quit IRC21:05
sdaguejogo: well more importantly, I don't think there is ever a valid use of recheck bug for a non voting job21:05
sdaguebecause it didn't -1 you21:05
lifelesssdague: so lets split the discussion in two21:06
jogosdague: so last I looked because tripleo jobs use a different comment, they don't have the non-voting tag in them21:06
*** mfer has joined #openstack-infra21:06
lifelesssdague: if it looks for (non-voting) its fine, because we have non-voting non-voting jobs21:06
lifelesssdague: *and* non-voting voting jobs21:06
lifelesssdague: the latter, if it fails, its equivalent to a -1 in tripleo.21:07
sdaguelifeless: is it equivalent because of people, or is jenkins enforcing it?21:07
lifelesssdague: If I was allowed to, it would vote, and we wouldn't have this friction.21:07
sdaguelets step back further21:07
lifelesssdague: its equivalent by review policy, because we're not allowed to make the jobs voting21:07
*** amcrn_ has joined #openstack-infra21:07
sdaguerecheck exists for 1 reason21:07
sdaguejenkins has blocked my code, and I feel it did so incorrectly21:08
sdaguebecause you can't reason with jenkins21:08
sdaguehe's kind of a douche21:08
lifelessyup21:08
lifelessand our reviewers have a mini-jenkins in their head that says 'if jenkins says a tripleo job failed, don't merge it'21:08
sdagueso only in place where jenkins is the one blocking your code is recheck bug ### really a valid thing21:08
sdaguelifeless: ok, but you can reason with them, right?21:09
*** maxbit has joined #openstack-infra21:09
*** amcrn has quit IRC21:09
*** _nadya_ has joined #openstack-infra21:09
lifelesssdague: you can certainly argue with them21:10
sdague:)21:10
*** resker has quit IRC21:11
sdaguewell, I really don't want us solving the problem of virtual jenkins in people's heads. Because there is actually a solution there, people being reasonable21:11
*** mfer has quit IRC21:11
jogothis sounds a bit like we are going in a circle21:12
sdaguethe solution without recheck and jenkins is having to upload another patch that changes some whitespace21:12
fungithere is an entire profession focused on solving the problems of virtual things in people's heads, in fact21:12
sdague:)21:12
*** sarob_ has joined #openstack-infra21:12
fungialso a sharp blow with a blunt instrument can often sort that out21:12
anteayaor cause it21:13
* fungi follows the flintstones school of psychology21:13
*** _nadya_ has quit IRC21:14
hasharif tripleo are non voting but should be considered maybe make them voting or highlight them in <b></b> ?21:14
morganfainbergjeblair, i keep almost selecting the wrong review option in gertty because -1/-2 are at the top, not the bottom (used to the web interface). not sure if there is any real need to change that... or you know... I might need to l2readbetter :P21:14
openstackgerritGerman Eichberger proposed a change to openstack-infra/gear: adds code to not block when using eventlet (incorporated Clark's suggestions)  https://review.openstack.org/9753321:14
fungimorganfainberg: proposing a patch might get more traction. jeblair's out for a week or more21:15
*** eharney has joined #openstack-infra21:15
morganfainbergfungi, yeah.. not sure if i think it should be changed.21:15
fungiotherwise, you probably want to leave yourself a reminder to ask again21:15
morganfainbergfungi, is the point....21:15
*** sarob has quit IRC21:15
morganfainbergfungi, i might just stew on it and see how many times i make the same mistake :P21:15
fungiseems reasonable. i'm not convinced that replicating gerrit's ui design choices is a great way to make an application more usable21:16
morganfainbergfungi, I tend to agree...21:16
sdaguefungi: though being gratuitous in changing them also makes it hard for people to use things situationaly21:17
*** dkliban_brb is now known as dkliban21:18
*** ArxCruz has joined #openstack-infra21:18
morganfainbergwoohoo. just crashed gerrty21:18
* morganfainberg tries to figure out why now.21:19
*** derekh_ has joined #openstack-infra21:20
anteayaArxCruz: hey21:21
anteayaArxCruz: I need some of your time, let me know when you are available21:21
fungimorganfainberg: using the tip of master? jeblair just merged a bunch of fixes right before he left21:22
morganfainbergfungi, checking that now21:22
fungimorganfainberg: in particular there were some rendering bugs which would cause it to jettison the reactor core21:22
morganfainbergfungi, yep, still crashing. - KeyError: /COMMIT_MSG21:23
morganfainbergfungi, /me goes and pokes at it21:23
*** ArxCruz has quit IRC21:23
*** marcoemorais has quit IRC21:23
*** marcoemorais has joined #openstack-infra21:23
openstackgerritA change was merged to openstack-dev/hacking: Update localization checks to understand separate catalogs  https://review.openstack.org/9758021:24
hasharfungi: morganfainberg any clue where to report gertty bugs beside james?21:24
*** smarcet has quit IRC21:25
morganfainberghashar, propose a fix in gerrit for it?21:25
hasharNotImplementedError: op.f() feature requires SQLAlchemy 0.9.4 or greater.21:26
hashar :D21:26
fungihashar: https://storyboard.openstack.org/#!/project/69821:26
hasharsome weir dependency issue hehe21:26
morganfainbergfungi, ah storyboard!21:26
*** nati_ueno has quit IRC21:26
hasharawesome21:26
fungithough i like morganfainberg's idea. i need a bug tracker which only lets you open a bug by proposing a patch ;)21:27
morganfainbergfungi, totally want to do that now!21:27
sdaguefungi: um, isn't that just gerrit21:28
fungisdague: that was the quiet joke there, yes21:28
*** eharney has quit IRC21:29
hasharfungi: yeah that is named Gerrit isn't it ? :-D21:31
*** julim has quit IRC21:32
openstackgerritJoe Gordon proposed a change to openstack/requirements: Bump pep8 from 1.5.6 to 1.5.7  https://review.openstack.org/9794421:35
*** marcoemorais1 has joined #openstack-infra21:36
jogosdague: random question: do you think its worth holding off cutting hacking 0.9 until we switch over to pep8 1.5.7?21:37
jogopep8 changelog https://pypi.python.org/pypi/pep821:37
*** Sukhdev has quit IRC21:38
jogofungi: ^21:38
jogoI am thinking its worth it21:38
jogo(waiting that is)21:38
fungijogo: i really have just about zero opinion on pep8 1.5.7. is 1.5.6 broken, or is this just the endless march of progress?21:39
jogofungi: a few bug fixes21:40
*** andreaf has quit IRC21:40
*** marcoemorais1 has quit IRC21:40
*** marcoemorais has quit IRC21:40
jogopep8 1.5 was extremely buggy21:40
jogonot surprisingly21:40
*** andreykurilin_ has joined #openstack-infra21:41
jogofungi: the main bug that cought my eye is: Check the last line even if it does not end with a newline.21:42
*** hashar has quit IRC21:42
*** eharney has joined #openstack-infra21:43
fungiha21:43
fungiokay, that's warranted21:43
*** radez is now known as radez_g0n321:45
jogofungi: oh never mind flake raises a differnt error in that case21:46
mikalHello... Where does the openstack governance RST stuff get published to?21:46
jogoso we cannot have any of those in our code base21:46
jogomikal: this isn't the answer but https://github.com/openstack/governance/blob/master/resolutions/20131106-ceilometer-and-heat-official-names.rst ;)21:47
anteayamikal: I don't know that is is currently published anywhere21:48
fungimikal: eventually www.openstack.org21:48
fungimikal: currently git.openstack.org21:48
jogojust like specs aren't published yet either21:48
mikalHmmm, ok21:49
mikalI wanted to link to a resolution somewhere nice21:49
mikalBut git it is!21:49
melwitthi, is there a bug where gerrit isn't linking people's patches to launchpad? I've seen this happen a few times so far lately21:49
*** timrc is now known as timrc-afk21:50
anteayamelwitt: it sometimes depends on the commit message21:51
anteayamelwitt: the commit message has to mention the bug in the first patchset21:51
*** timrc-afk is now known as timrc21:51
fungimelwitt: there are several factors which can cause it. the automation account has to be a member (directly or indirectly) of the bug supervisor group for that project *if* the bug is not yet assigned to the patch committer, the gerrit change has to be targeting the same project and branch as an existing task on the bug, the bug header has to appear in the first patchset of the change which addresses21:51
fungiit21:51
fungioh, and launchpad's api has to not randomly fail21:51
anteayamelwitt: https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references21:52
melwittanteaya: okay. I have seen ones with the Closes-bug: # but it's possible it wasn't there for the initial patchset21:52
*** mburned is now known as mburned_out21:52
clarkbmelwitt: do you have an exmaple change?21:52
fungimelwitt: right, the hook which does that doesn't maintain any state, so to prevent spamming the bug over and over it only fires for the very first patchset of a changhe21:53
melwittclarkb: I just saw this one get mentioned https://review.openstack.org/#/c/96293/ https://bugs.launchpad.net/nova/+bug/132427721:53
uvirtbotLaunchpad bug 1324277 in nova "Use of finally/return considered harmful" [Medium,Confirmed]21:53
*** andreykurilin_ has quit IRC21:53
*** mriedem has quit IRC21:54
*** ArxCruz has joined #openstack-infra21:54
fungimelwitt: it looks like there's a chance the patch was uploaded for review so quickly after the bug was opened on lp that the api wasn't returning it yet21:56
fungithey were within the same minute21:56
lifelesssdague: sorry, distracted21:56
lifelesssdague: yes, you can argue with a reviewer.21:56
lifelesssdague: but these reviewers will say 'recheck the test run with the relevant bug #'21:56
melwittfungi: interesting. let me check the others I saw for similar21:56
sdaguelifeless: so that seems extremely wasteful21:57
lifelesssdague: why? we don't want regressions to land.21:57
lifelesssdague: and something that fails at step X due to a know bug may fail at step X+N due to a bug in the patch.21:57
*** markmcclain has joined #openstack-infra21:58
fungimelwitt: it would be good to correlate, for sure. i've seen some lag before between the lp webui and the api, which i think i remember hearing is intentional (lifeless?)21:58
*** jgrimm has quit IRC21:59
*** homeless has joined #openstack-infra21:59
lifelessfungi: readonly queries via the webui will be served from slave DB servers unless the session has done a write in the last N seconds (30 IIRC)21:59
lifelessfungi: but DB replication lag is typically a few seconds at most22:00
sdaguelifeless: well, I'm basically off for the night now. Maybe we can pick it up tomorrow.22:00
lifelesssdague: ok22:00
sdaguethere was a real issue with people rechecking blindly with non-voting bugs22:00
fungilifeless: okay, so if the bug is opened through the webui and then queried and attempted to be updated via the api seconds later, that should be okay?22:00
lifelessfungi: always22:01
fungilifeless: thanks!22:01
fungimelwitt: so i guess it's not backend latency causing that22:01
sdagueso we can figure out something. but it would also be good to figure out if there is a way to make the tripleo jobs just vote anyway, instead of putting the extra load on people.22:01
lifelesssdague: I would love that.22:01
clarkbthe tripleo jobs can vote using that thing jhesketh is doing22:02
fungisdague: isn't the idea that we do something for that pipeline similar to what ironic's wanting for their non-binding jobs?22:02
*** ArxCruz has quit IRC22:02
melwittfungi: another example, patch upload about one minute after lp created https://bugs.launchpad.net/python-novaclient/+bug/1324857 https://review.openstack.org/#/c/96715/22:02
uvirtbotLaunchpad bug 1324857 in python-novaclient "floating-ip-list does not list attached server anymore" [Medium,Confirmed]22:02
clarkbfungi: yup22:02
lifelessfungi: clarkb: thats still nonbinding though right? So no different.22:02
clarkblifeless: I don't follow22:02
clarkblifeless: its just not jenkins voting22:02
clarkbbut it can be a real vote22:02
lifelessclarkb: will zuul merge on such a failure if someone asks it to ?22:03
clarkblifeless: yes22:03
clarkbbut that is a different issue22:03
lifelessclarkb: if the answer is yes, the burden is still held by human reviewers, which is exactly the thing sdague said it would be good to fix.22:03
lifelessclarkb: and the thing I would love to fix.22:03
fungimelwitt: that one looks like it was uploaded a couple minutes later22:03
*** sarob_ has quit IRC22:04
*** sarob has joined #openstack-infra22:04
fungibut i would still call that a fairly tight correlation22:05
melwittfungi: yeah, 10:30:53 is the timestamp on the lp. I agree22:05
*** dizquierdo has quit IRC22:05
* jogo just pushed the hacking 0.9.0 tag22:05
jogoI think its funny how check queue is shorter then the gate queue22:06
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: tests for JJB command processing  https://review.openstack.org/9790822:06
fungimelwitt: if you find that the majority which failed to update were uploaded very, very soon after the bugs were opened, then that very well could indicate some sort of something there22:06
*** mkoderer has quit IRC22:06
fungijogo: oh, i think that may not end up on pypi without manual help, which i can apply after dinner22:07
fungijogo: i still need to decide how we're getting a newer version of the requests module onto the new pypi slave22:07
fungiso the upload job is going to die in flames22:07
clarkbwe should update the gerrit ACLs to prevent any tags from being pushed when we are in this state22:08
clarkbfungi: I don't think it is too terrible to just upgrade requests globablly22:08
fungijogo: i just uploaded it now22:08
fungiclarkb: yeah, though so far it's just been these two22:09
fungi(based on what i see workspaces created for)22:09
fungiclarkb: so right now we're installing python-requests from the distro on all slaves. is the suggestion that we globally install it from pip instead?22:10
clarkbhrm do we know why we are installing it?22:10
fungithat is a very good question. looking for clues22:10
clarkboh for pushing files to swift according to the comment22:11
clarkbin that case I would expect newer requests to work as well22:11
*** mmaglana has quit IRC22:11
*** mkoderer has joined #openstack-infra22:11
*** dkliban is now known as dkliban_afk22:11
*** sarob has quit IRC22:11
jogofungi: sanity check: hacking revs don't automatically get rolled out to projects right?22:11
*** jamespd has quit IRC22:11
*** gondoi is now known as zz_gondoi22:11
clarkbjogo: correct hacking should be pinned in every project22:12
fungijogo: right, unless they fail to pin22:12
*** jamespd has joined #openstack-infra22:12
*** doug-fish has left #openstack-infra22:12
jogoclarkb fungi: thanks, I confirmed they all pinned a while back22:12
fungiclarkb: yeah, so the trick here is to ensure absent the deb/rpm version *and* install the pip one... oh! the deb package name will not be the same as the pip package name, so all good22:12
jogofungi: you manually uploaded hacking 0.9.0 to pypi?22:12
clarkbfungi: ya should be straightfowrad. we should confirm with jhesketh that pip requests works with his script but I would expect it to22:13
jogofungi: if not, I'll hold off sending the email until its up22:13
fungijogo: sort of. i reran the failing command from the job using a virtualenv on that slave which contained a newer requests installed22:13
fungijogo: so it should be there, but it was really only sort of manual22:13
fungiclarkb: okay, change coming, then i'll disappear for dinner22:14
*** SpamapS has quit IRC22:14
*** YorikSar has quit IRC22:15
jogofungi: hmm I don't see it http://pypi.openstack.org/openstack/hacking/22:15
jogowell its on pypi.python.org so thats good22:15
*** YorikSar has joined #openstack-infra22:15
lifelessdevananda: have you considered asking for the fixs (or the nova revert) to be promoted the next time the gate queue resets?22:16
lifelessdevananda: (nova-bm was broken too..)22:16
clarkbjogo: you will have to wait for a mirror rebuild22:16
jogoclarkb: ack, thanks22:16
devanandalifeless: i thought nova-bm was fixed by that patch (not broken)22:17
*** signed8bit has quit IRC22:17
devanandalifeless: and i'm not aware that it's possible to bump a patcjh to the top of the queue22:17
devanandaclarkb: ^ ?22:17
*** signed8b_ has joined #openstack-infra22:17
clarkbdevananda: it is, but we only do it for gate breakers really22:17
devanandaclarkb: ah. ironic's gate has been broken for >12hr now22:18
*** SpamapS has joined #openstack-infra22:18
*** SpamapS has quit IRC22:18
*** SpamapS has joined #openstack-infra22:18
clarkbdevananda: what change is this?22:18
lifelessdevananda: it broke the hostmanager for both nova-bm and Ironic22:18
devanandaclarkb: https://review.openstack.org/#/c/97447/4 should unwedge our gate22:18
devanandalifeless: oh. nice22:18
*** ociuhandu has quit IRC22:19
devanandaoh great22:19
devanandahttps://review.openstack.org/#/c/97757/ is a revert on the nova side22:19
devanandaif both of those land, ironic will be broken /again/22:19
jogodevananda: it looks like the patch that we asked you guys to look at broke you :/22:19
lifelessclarkb: devananda: see  https://review.openstack.org/#/c/97447/too22:19
lifelesswhere the author says that the ironic patch has to be stopped if the nova revert goes through22:19
devanandajogo: yep. I missed the one line changing the definition of __init__22:19
clarkbdevananda: thats the first ironic change22:19
lifelesssince it will break in the opposite direction22:19
devanandajogo: and focused on the functional change to the get_host_stats22:19
clarkbso I don't think promoting it changes anything22:19
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: force ntpdate before we start the timer  https://review.openstack.org/9792422:20
lifelessclarkb: it would let checks succeeed22:20
*** masayukig has quit IRC22:20
jogodevananda: o_O I missed that too22:20
devanandaclarkb: well, it's buried under a lot of other non-related changes22:20
lifelessclarkb: in ironic and tripleo22:20
clarkblifeless: there is nothing in check22:20
clarkboh in tripleo too22:20
lifelessclarkb: but they cannot succeed while ironic-dvsm fails22:20
lifelessclarkb: and that fails because trunk is broken22:20
clarkbright, I am just trying to reason about whether or not we gain anything by promoting this change22:20
clarkband it sounds like we do22:20
lifelessclarkb: tripleo has a local revert of the nova patch to make tripleo deploy tests work, but that doesn't help the dvsm-ironic jobs22:21
lifelesswith check failing we cannot ascertain whether other patches are good or not22:21
devanandaclarkb: if we promote the ironic change(97447) and cancel the nova one (97757), then i think we're ok. lifeless - is there a corresponding fix for nova_bm up yet?22:21
clarkbso someone needs to decide on which change should die22:21
lifelessdprince: hi22:21
clarkbbecause I am not doing anything until that is sorted22:22
devanandaI agree that the nova change which caused this was the right direction. I would rather not revert it, because we'll just need to do it again eventually22:22
devanandajogo: agree yes/no ^ ?22:22
jogodevananda: agree22:22
jogodevananda: although if that is easer for whatever reason, I am fine with the revert.22:23
openstackgerritJeremy Stanley proposed a change to openstack-infra/config: Install newer requests package for twine  https://review.openstack.org/9796222:23
fungiclarkb: that ^ i guess22:23
*** bknudson has quit IRC22:23
lifelessso there are three patches in flight here22:23
devanandalifeless: what's the third?22:24
lifelesshttps://review.openstack.org/#/c/97757/ (revert nova change)22:24
lifelesshttps://review.openstack.org/#/c/97447/ (fix ironic)22:24
lifelesshttps://review.openstack.org/#/c/97806/ (fix nova)22:24
*** amcrn_ has quit IRC22:24
lifelesswith either need 97757 to land, or both 97447 and 97806 to land22:24
*** mmaglana has joined #openstack-infra22:24
clarkbfungi: I am +2. hopefully jhesketh will pop up and can confirm it is ok22:24
devanandalifeless: ++22:25
*** masayukig has joined #openstack-infra22:25
clarkb97806 has not been approved22:25
clarkbthat leaves you with 9775722:25
lifelessso, since the rollforward hasn't been approved, my suggesetion is to let the revert roll in - and promote it to unbreak everything22:26
jogoyeah still isn't passing jenkins yet22:26
lifelessand pull 97447 out of the gate22:26
jogolifeless: works for me22:26
lifelessyes we need to do the change at some point, but it can hopefully be done by adding new api, migrating all users, then removing old one.22:27
sdaguejogo: 97806 bounced off the devstack get-pip fix22:27
lifelessdevananda: ?22:27
sdaguejogo: realistically we've got about 20 coupled race bugs right now22:27
sdaguewhich is why we're grinding22:27
devanandaI may have another trivial fix, but it'd depend on someone in nova approving it22:28
devanandathe problem is the change to __init__ on L112 here https://review.openstack.org/#/c/94043/2/nova/scheduler/host_manager.py22:28
* fungi is going to go grind on some food for a while22:28
*** prad has quit IRC22:28
sdagueclarkb: is there any chance that hp 1.1 nodes have 16 cores?22:28
lifelessdevananda: and the missing stats, no ?22:28
sdaguehttp://logs.openstack.org/12/97912/1/check/check-grenade-dsvm-partial-ncpu/6f8f858/console.html22:29
fungiclarkb: we can do the nodepool restart when i get back if you're still okay with that22:29
sdagueclarkb: there definitely are 8 testr processes there22:29
devanandalifeless: ah, right - it's not addign those :(22:29
devanandayes, revert22:29
lifelessclarkb: ok we have consensus from nova core and ironic core22:29
lifelessclarkb: the revert needs to go through and we need to pull the ironic patch22:29
clarkbsdague: no 8 cores22:30
* devananda -2's the ironic patch22:30
*** otherwiseguy has quit IRC22:30
clarkbdevananda: even better is pushing a new patchset but whatever22:30
sdagueclarkb: are we sure?22:30
clarkbsdague: yes I just booted one for testing22:30
clarkbit has 822:30
*** thedodd has quit IRC22:30
sdaguein b5?22:30
clarkbya b5 is just a logical name22:30
lifelessclarkb: if you can promote https://review.openstack.org/#/c/97757/ when the gate next resets, that would be low disruption and very useful22:30
clarkblifeless: will do22:30
devanandaclarkb: k, doing that instead then22:31
lifelessdevananda: we should be able to add the stats now though as part of prep work22:31
lifelessdevananda: AIUI22:31
devanandalifeless: yep22:31
zaroanteaya: create reference permission isn't necessary?  https://review.openstack.org/#/c/90281/2/modules/openstack_project/files/gerrit/acls/openstack/dib-utils.config22:32
devanandaoh, nvm22:32
*** mburned_out is now known as mburned22:32
*** maxbit has quit IRC22:33
jheskethanteaya: At the moment turbo-hipster is only doing database migrations CI and in our deployment that's all it is likely to do22:34
jheskethwell it might do other database tests against datasets22:35
devanandalifeless: so I don't have time to figure out the right way to clean up 97447 before totally missing this appointment :(22:35
jheskethso maybe a name like 'db-datasets-ci'?22:35
devanandaclarkb: if I block 97447, it will stop it from merging, even though it's in the queue already, right?22:35
clarkbdevananda: if you -2 it will prevent merging. if you push a new patchste it will rip it out of the gate22:36
*** whayutin_ has quit IRC22:36
lifelessdevananda: want me to push up a no-op change to 97447 ?22:36
lifelessdevananda: you go, I got this22:36
jheskethclarkb: what did you need me to look at/confirm?22:36
*** fbo is now known as fbo_away22:36
*** markwash has joined #openstack-infra22:37
clarkbjhesketh: https://review.openstack.org/97962 change to how we install python requests on slaves22:37
lifelessclarkb: is a commit message only change sufficient ?22:37
devanandalifeless: thanks!22:37
clarkbjhesketh: twine (which we use to upload packages to pypi) needs newer requests22:37
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: capture cpu information when we setup_host  https://review.openstack.org/9796522:37
clarkblifeless: yes, but maybe not via gerrit UI22:37
clarkbI can't remember if that does a new patchset22:37
sdaguelifeless: yes, commit change is sufficient22:37
lifelessdone22:38
sdagueeven via the gerrit UI I believe22:38
lifelessswitched the metadata header ordering22:38
anteayazaro, correct under refs/tags/* the create line is no longer required22:38
sdaguetypically I leave a comment that just explains I've removed it from the gate, for whatever reason22:38
anteayazaro the pushSignedTag line is though22:38
sdagueso people that are surprised about it later know why22:38
*** jp_at_hp has quit IRC22:39
anteayazaro and refs/tags/* is not required for every acl file, only if that project wants to push tags22:39
jheskethclarkb: right, I have no idea of the implications of that (in terms of what is supported etc), but I don't see any obvious problems with using pip22:39
anteayajhesketh: if you like db-datasets-ci I will add that to the renaming etherpad22:40
anteayajhesketh: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format22:40
clarkbjhesketh: cool. was mostly worried about your swift uploader script22:40
zaroanteaya: i believe it used to be required, is that due to never version of gerrit?22:40
*** amcrn has joined #openstack-infra22:40
*** dprince has quit IRC22:40
jheskethanteaya: sounds good to me22:40
zaroanteaya: or does it get handled by All-Projects?22:40
jheskethmikal: Do you think 'db-datasets-ci' is appropriate for turbo-hipster ^22:41
anteayajhesketh: line 14922:41
mikalPardon?22:41
jheskethclarkb: well so long as requests still works :p22:41
jheskethclarkb, fungi: I do have a question for 97962 which I left on the review22:41
clarkbjhesketh: it should but its version 0.8.2 vs 2.3.022:41
anteayazaro correct, create used to be required with Gerrit 2.4, but is no longer requried with Gerrit 2.822:41
mtreinishsdague: I think we just forgot to do the concurrency = nprocs/2 in grenade and that's why you're seeing 8 workers there22:41
jheskethmikal: we need to make turbo-hipster's name more obvious when it leaves votes22:41
*** arnaud has joined #openstack-infra22:42
mtreinishbut I'm coming in a little late so I'm probably missing what the problem is22:42
anteayajhesketh mikal more like the standardized naming found here: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format22:42
zaroanteaya: ok.  sounds good.22:42
anteayazaro: thanks22:42
jerryzclarkb: are there any network interruption on jenksin slaves? or gem repo is down? http://logs.openstack.org/48/97948/1/check/gate-cookbook-openstack-common-chef-style/bdc6688/console.html22:42
*** marcoemorais has joined #openstack-infra22:42
anteayajhesketh mikal don't get me wrong I love turbo-hipster22:42
sdaguemtreinish: oh, really?22:43
anteayabut bringing sembelance to the masses means you have to leave out some fun stuff22:43
anteaya:(22:43
clarkbjerryz: not that I know of in IAD. and you have to ask gem people about their end22:43
jheskethI've got to run out for a little bit guys, be back soon22:43
*** gokrokve has quit IRC22:43
mtreinishsdague: yeah because we call tempest from inside grenade not devstack gate right22:44
anteayajhesketh: oh and since you are here, what should I propose a session on?22:44
sdaguemtreinish: right22:44
mtreinishbut, the concurrency stuff is set in devstack gate22:44
anteayajhesketh: when you get back22:44
jhesketh:-)22:44
mtreinishso it'll just use the default in grenade22:44
lifelessok 97447 is out of the queue22:44
*** jhesketh has quit IRC22:44
mtreinishsdague: I'll push out a fix for it22:45
*** zzelle has quit IRC22:46
sdaguemtreinish: I have it22:46
lifelessnow we just wait 8 hours or for a failure22:47
*** ArxCruz has joined #openstack-infra22:47
mikalI have no problem renaming it22:47
lifelessclarkb: actually, is it possible to promote to the next-window-in-the-predictive-queue?22:47
mikalWe were also meant to link to a wiki page in our votes, did that ever get done?22:47
lifelessclarkb: e.g to where 97126,2 is?22:47
clarkblifeless: it isn't unfortunately22:47
clarkblifeless: I was going to do it after some of these merge if no failures happen22:48
lifelessnuts22:48
lifelesscool22:48
*** maxbit has joined #openstack-infra22:48
sdaguemtreinish: that would actually explain a lot about the fact that grenade was failing more often than the other jobs22:49
sdague8 way tempest exposes way too many other issues22:49
clarkblifeless: around 92739 hitting head of queue22:49
anteayamikal: we are working on wikipage set up for automated accounts22:49
*** derekh_ has quit IRC22:50
anteayamikal: I will assign you one22:50
anteayamikal: plus I will be bugging you to fill out one for nova as well22:50
mikalanteaya: IIRC Josh needed to add a reporter tweak to get it done in zuul, but I thought he would be working on that already22:50
mikalHe appears to have dropped off IRC, so I can't ask22:50
anteayahe may be22:50
clarkbmikal: I think that merged22:51
*** dims_ has quit IRC22:51
anteayalet's pile up questions for when he returns22:51
*** sarob has joined #openstack-infra22:51
clarkbsdague: its like a salt shaker with bugs in it22:51
anteayamikal: then he will know we missed him22:51
mikalclarkb: Oh, he might just need to turn it on then?22:51
mikalI have to run to a meeting in a minute, so someone please hassle him about that22:51
mtreinishsdague: yeah, it just stresses a devstack-gate node a bit too much22:51
*** ArxCruz has quit IRC22:52
mikalHeh22:52
mattoliverauMorning all22:53
anteayamorning mattoliverau22:53
sdagueclarkb: if you are taking promotes - https://review.openstack.org/#/c/97845/22:56
sdagueis the backport of a fix that's #7 in the er bugs22:56
clarkbok22:58
*** prad has joined #openstack-infra22:59
notmynamewhat are the restrictions on force-pushing to branches in gerrit? I know -core can push merge commits. and it seems ttx can force push to milestone-proposed. what about a feature branch? specifically, I'm thinking about swift's feature/ec branch23:00
clarkbI think we do our best to avoid force pushing to anything23:01
*** olaph has quit IRC23:02
notmynameclarkb: yes I know. but that doesn't answer the question ;-)23:02
clarkbare you trying to change history (badbadbad) or just merge something on top of history23:02
clarkbfor changing history never. for merging something on top of history without going through the gate we only do that when the gate has managed to get itself into a situation where the only practical way out is to merge without testing23:02
jogosdague: ahh I see now http://jogo.github.io/gate/ we have some really high failure rates23:03
sdaguejogo: yes23:03
sdaguerealistically, I think we've been pretty bad for a while, clean check was masking it23:04
sdaguebut a 500 element check queue meant a giant flush23:04
*** sarob_ has joined #openstack-infra23:04
clarkbnotmyname: what are we trying to accomplish?23:04
notmynameclarkb: I understand and am aware of the dangers of force pushing code. but I'm curious about what roles have the ability to do it23:05
*** aconrad has quit IRC23:05
*** bhuvan_ has joined #openstack-infra23:06
clarkbnotmyname: the only role I am aware that actively has the ability to do it is the project creator account so that it can seed repos23:06
notmynameok23:06
*** sarob has quit IRC23:07
notmynameclarkb: there's a few reasons I'm asking, but the most immediate one is because the feature/ec branch is done (and has the wrong history). we are interested in reusing it for actual erasure code work23:07
*** maxbit has quit IRC23:07
clarkbnotmyname: I consider that bad23:08
*** markmcclain has quit IRC23:08
clarkbyou can do what the milestone branches used to do (which we are going to stop doing because it is bad) and delete the branch and recreate it with a different head23:08
clarkbthe thing to keep in mind is that anyone that had the branch locally will have to reset --hard to catch the update23:08
*** bhuvan has quit IRC23:08
*** dims_ has joined #openstack-infra23:08
openstackgerritK Jonathan Harker proposed a change to openstack-infra/config: Move variables in manifests/site.pp to top of file  https://review.openstack.org/9765923:09
*** maxbit has joined #openstack-infra23:09
*** atiwari has quit IRC23:09
clarkbnotmyname: this is similar to a force push but less scary with gerrit ACLs iirc23:09
clarkb(you can delete branches via the web ui without proper force push abilities)23:10
notmynameclarkb: ah yes. a delete and recreate makes sense23:10
notmynameoh really?23:10
notmynameclarkb: what web ui?23:10
clarkbthe gerrit web ui23:11
clarkblifeless: sdague changes promoted23:11
clarkba reset happened23:11
openstackgerritK Jonathan Harker proposed a change to openstack-infra/config: Move variables in manifests/site.pp to top of file  https://review.openstack.org/9765923:12
*** timrc is now known as timrc-afk23:13
notmynameclarkb: on https://review.openstack.org/#/admin/projects/openstack/swift,branches I don't see anything do delete branches23:13
clarkbnotmyname: I think you may need additional permissions to do it23:14
notmynameclarkb: ah ok23:14
clarkbthey are just more restrictive than allowing force push iirc23:14
clarkblet me figure that out23:14
notmynameclarkb: ok, thanks. no rush. I can ping you on it tomorrow23:14
*** virmitio has quit IRC23:15
*** mrodden has quit IRC23:15
clarkboh its attached to "owner"23:16
*** jhesketh_ has joined #openstack-infra23:17
anteayajhesketh_: welcome back23:17
anteayamikal wants you to do a thing23:17
jhesketh_thanks anteaya23:17
anteayaand asked me to ensure you were bugged about it23:17
jhesketh_I'm catching onto mikal's tricks of making me do work....23:18
anteayaI consider my job done23:18
anteayaha ha ha23:18
jhesketh_heh, he's also good at getting others to bug me23:18
anteayamy favourite memory of summit is him giggling for a solid 20 minutes23:18
anteayathat was the best23:18
anteayaso yeah, back to me23:19
anteayawhat am I to propose for a talk?23:19
clarkbjhesketh_: did you manage to get your pet rock through customs?23:19
anteayaha ha ha23:19
jhesketh_heh, I didn't list it, so just walked straight through23:19
anteayaha ha ha23:19
jhesketh_so I plugged it in... turns out, it works as advertised23:19
anteayaha ha ha23:19
anteayawas the blank book of instructions helpful23:20
jhesketh_exceptionally.. it's important to take good care of your pets23:20
anteayaha ha ha23:21
*** jhesketh_ is now known as jhesketh23:21
*** ArxCruz has joined #openstack-infra23:21
*** aconrad has joined #openstack-infra23:21
*** sarob_ has quit IRC23:23
*** mrda-away is now known as mrda23:24
*** ArxCruz has quit IRC23:25
dstufftwat is this redhat ci stuff i'm getting emails from23:25
*** lcostantino has joined #openstack-infra23:26
anteayadstufft: what do the email say?23:26
anteayas/email/emails23:26
dstufftanteaya: some failure stuff23:27
dstufftI'm just confused why openstack stuff is sending me redhat ci emails o.O23:27
anteayadstufft: like on any of these patches? https://review.openstack.org/#/q/reviewer:%22Red+Hat+CI+%253Credhatci%2540redhat.com%253E%22,n,z23:28
lifelessdstufft: it is from gerrit?23:28
dstuffthttps://review.openstack.org/#/c/97912/ this patch23:28
dstufftlifeless: yea23:28
dstufftis there like23:28
dstufftmultiple CI systems now23:28
dstufftoh man there's some xenserver stuff there too23:29
lifelessdstufft: so 3rd partt checks into gerrit23:29
lifelessgerrit tells you23:29
lifelesswe all cry23:29
dstufftI guess I need to get around to setting up some email filters23:30
apevecdstufft, redhat ci is 3rd party testing devstack, ianw is the contact23:32
dstufftI guess I should pay more attention or something, I had no idea ther was third party testing :]23:32
dstufftis it new?23:32
clarkbdstufft: its not new, but has picked up more steam during the icehouse cycle23:33
clarkbsmokestack was a third party test system that ran for years23:33
anteayawe liked it23:33
dstufftdo they test everything? or just certain stuff?23:33
anteayathen the crowds came23:33
anteayathey should only test certain stuff23:33
anteayabut some folks don't know how to irc or email list yet23:34
anteayaso they tend to do wild things before we can contain them23:34
dstufftprobably why I never noticed it then, I must not have been paying attention to anything that was tested by hird party things23:34
anteayawe should just clone dprince23:34
lifelessdstufft: they test all the stuff relevant to their tech23:34
lifelessdstufft: e.g. a switch vendor might test neutron jobs only23:34
dstufftmakes sense23:35
dstufftgerrit should totally have a way to record test results that doesn't add more emails23:35
dstufft(also :sadtrombone: this'll be the first thing I filter out of my inbox and don't read)23:36
*** mrodden has joined #openstack-infra23:36
openstackgerritJoe Gordon proposed a change to openstack-dev/hacking: Make import checks properly detect blank lines and comments  https://review.openstack.org/9798823:39
jogofungi: looks like I will need to cut a hacking 0.9.1  :/23:40
ianwdstufft: there's moves to standardize the names of CI accounts, which will mean possibility to filter and write some javascript to hide it23:42
*** praneshp has joined #openstack-infra23:42
*** Sukhdev has joined #openstack-infra23:45
*** maxbit has quit IRC23:45
Sukhdevanteaya: ping23:45
anteayahello Sukhdev23:46
anteayawhat is on your mind23:46
*** maxbit has joined #openstack-infra23:46
Sukhdevanteaya: I mentioned to you in the morning that I will ping you later…..23:46
Sukhdevanteaya: wanted to let you know that I did not forget - day has been crazy23:46
anteayayes, we were talking about Arista Testing becoming Arista CI23:46
anteayaSukhdev: I understand, thanks for following up23:47
Sukhdevanteaya: Yes, but, can I ask a favor - can I ping you regarding this tomorrow and go over it.23:47
anteayaSukhdev: sure23:47
anteayayou don't have to do anything23:47
Sukhdevanteaya: I want to understand it correctly so that I do not mess up - and now I am up against the clock23:47
anteayasomeone else I was talking to thought they had to do something and they created more accounts23:48
Sukhdevanteaya: wanted to just let you know that I did not forget - just things have been hectic23:48
anteayaplease don't do that23:48
anteayafine23:48
*** unicell has quit IRC23:48
*** jaypipes has quit IRC23:48
anteayajust keep in mind, you don't have to do anything23:48
Sukhdevanteaya: cool - thanks, will ping you tomorrow23:48
*** michchap has joined #openstack-infra23:48
anteayaI'll talk to you then, have a good night23:48
anteayaSukhdev: don't do anything23:49
Sukhdevanteaya: ah thats even better - I do not have to do anything :-)23:49
anteayano23:49
anteayayou don't have to do anythign23:49
Sukhdevanteaya: I will ping you tomorrow - thanks for your understanding.23:49
anteayanp23:49
Sukhdevanteaya: bye - have a wonderful night23:49
anteayathanks Sukhdev you too23:49
*** Sukhdev has quit IRC23:51
*** Sukhdev has joined #openstack-infra23:52
*** bhuvan_ has quit IRC23:53
anteayaianw yes https://etherpad.openstack.org/p/neutron-juno-mid-cycle-meeting23:53
anteayaianw: efforts ongoing in that etherpad23:53
anteayasorry wrong etherpad23:54
anteayathis one: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format23:54
*** signed8b_ is now known as signed8bit23:54
*** Sukhdev has quit IRC23:56
fungijogo: poo23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!