Monday, 2013-12-02

openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: move queries.yaml into a queries subdir  https://review.openstack.org/5930500:04
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: Fix E122,E126,E128 items in codebase  https://review.openstack.org/5927200:04
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: method extraction for readability  https://review.openstack.org/5926300:04
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: time to grow up and fit in 80 columns  https://review.openstack.org/5926200:04
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: wrapping the README.rst file to 80 cols  https://review.openstack.org/5926100:04
*** senk has joined #openstack-infra00:12
*** dstanek_afk has joined #openstack-infra00:14
*** sarob has joined #openstack-infra00:15
*** dkliban has joined #openstack-infra00:17
*** rongze has joined #openstack-infra00:18
*** sarob has quit IRC00:18
*** sarob has joined #openstack-infra00:19
*** dstanek_afk has quit IRC00:19
*** matsuhashi has joined #openstack-infra00:19
*** sarob has quit IRC00:23
*** rongze has quit IRC00:23
*** matsuhashi has quit IRC00:30
*** matsuhashi has joined #openstack-infra00:30
*** alexpilotti has quit IRC00:30
openstackgerritA change was merged to openstack-infra/jeepyb: Savanna client is now using separated LP project  https://review.openstack.org/5775200:35
*** matsuhashi has quit IRC00:35
openstackgerritA change was merged to openstack-infra/config: Remove explicit depends on transitive pip deps  https://review.openstack.org/5140200:37
*** jcooley_ has quit IRC00:42
*** alexpilotti has joined #openstack-infra00:42
lifelesswhat github repos hold official program code00:45
lifelessopenstack-infra/00:45
lifelessopenstack/00:45
lifelessdevstack/00:45
lifelessanything else ?00:45
lifelesss/github/blah/00:45
*** zul has joined #openstack-infra00:46
sdagueopenstack, openstack-dev, openstack-infra00:47
sdaguedevstack is under openstack-dev00:47
lifelessah, thanks00:47
*** CaptTofu has quit IRC00:48
cyeohsdague or any other nova core around - would you mind looking at https://review.openstack.org/#/c/59142/ - its fixes a bug which has been causing failures in the gate00:49
*** CaptTofu has joined #openstack-infra00:49
*** sarob has joined #openstack-infra00:49
sdaguecyeoh: sure, lgtm00:51
cyeohsdague: thx!00:51
*** jcooley_ has joined #openstack-infra00:52
*** jcooley_ has quit IRC00:58
*** jcooley_ has joined #openstack-infra00:59
*** jcooley_ has quit IRC01:04
*** matsuhashi has joined #openstack-infra01:05
*** dstanek_afk has joined #openstack-infra01:07
*** adalbas has quit IRC01:09
*** odyssey4me has joined #openstack-infra01:15
*** sarob has quit IRC01:21
*** alexpilotti has quit IRC01:22
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: Fix E122,E126,E128 items in codebase  https://review.openstack.org/5927201:25
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: move queries.yaml into a queries subdir  https://review.openstack.org/5925201:25
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: method extraction for readability  https://review.openstack.org/5926301:25
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: time to grow up and fit in 80 columns  https://review.openstack.org/5926201:25
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: wrapping the README.rst file to 80 cols  https://review.openstack.org/5926101:25
*** marun has joined #openstack-infra01:27
*** lianglin has joined #openstack-infra01:32
*** nosnos has joined #openstack-infra01:33
*** dims has joined #openstack-infra01:41
sdaguemordred: who can I stab for the fact that the foundation membership database is broken, and won't let me update email addresses until I add an affiliation, which it then fails to do anything when I click save?01:43
sdaguealso note: adding an affiliation is broken01:44
*** yaguang has joined #openstack-infra01:53
*** dims has quit IRC01:54
*** bingbu has joined #openstack-infra01:57
*** ljjjustin has joined #openstack-infra01:59
*** rongze has joined #openstack-infra02:01
*** rcleere has joined #openstack-infra02:01
*** mgagne has quit IRC02:12
*** dcramer_ has joined #openstack-infra02:12
*** mgagne1 has joined #openstack-infra02:12
*** mgagne1 has quit IRC02:12
*** mgagne1 has joined #openstack-infra02:12
*** sarob has joined #openstack-infra02:19
*** jishaomei has joined #openstack-infra02:21
*** bingbu has quit IRC02:23
*** sarob has quit IRC02:24
*** CaptTofu has quit IRC02:29
*** CaptTofu has joined #openstack-infra02:29
*** bingbu has joined #openstack-infra02:35
*** guohliu has joined #openstack-infra02:36
*** dkliban has quit IRC02:43
*** slong has quit IRC02:46
*** slong has joined #openstack-infra02:46
*** marun has quit IRC02:56
*** nati_ueno has joined #openstack-infra02:57
*** matsuhashi has quit IRC02:59
clarkbsdague: fungi and jeblair are your closest people to stab on irc03:01
*** sarob has joined #openstack-infra03:02
*** matsuhashi has joined #openstack-infra03:02
*** odyssey4me has quit IRC03:03
*** jcooley_ has joined #openstack-infra03:10
*** dstanek_afk has quit IRC03:25
*** mgagne1 has quit IRC03:26
*** CaptTofu has quit IRC03:26
mordredsdague: everyone03:32
mordredsdague: I've been trying for a year to get that moved under infra03:33
*** bingbu has quit IRC03:43
*** dstanek_afk has joined #openstack-infra03:51
*** nicedice has quit IRC03:53
*** bingbu has joined #openstack-infra03:55
*** dstanek_afk has quit IRC03:56
*** jcooley_ has quit IRC03:59
*** matsuhashi has quit IRC04:02
*** matsuhas_ has joined #openstack-infra04:05
*** markmcclain1 has joined #openstack-infra04:12
*** markmcclain has quit IRC04:12
*** SergeyLukjanov has joined #openstack-infra04:14
*** sarob has quit IRC04:16
*** sarob has joined #openstack-infra04:16
*** sarob has quit IRC04:21
*** chandankumar has quit IRC04:28
*** chandankumar has joined #openstack-infra04:34
*** lianglin has quit IRC04:40
*** sarob has joined #openstack-infra04:47
*** dstanek_afk has joined #openstack-infra04:54
*** DennyZhang has joined #openstack-infra04:55
*** boris-42 has joined #openstack-infra04:55
*** rongze has quit IRC04:57
*** slong has quit IRC04:58
*** dcramer_ has quit IRC04:59
*** dstanek_afk has quit IRC04:59
*** rcleere has quit IRC04:59
*** jorisroovers has joined #openstack-infra05:02
*** jroovers has joined #openstack-infra05:06
*** bingbu has quit IRC05:06
*** jorisroovers has quit IRC05:09
*** sarob has quit IRC05:13
*** sarob has joined #openstack-infra05:13
*** sshturm has quit IRC05:17
*** SergeyLukjanov is now known as _SergeyLukjanov05:19
*** _SergeyLukjanov has quit IRC05:19
*** bingbu has joined #openstack-infra05:20
*** jcooley_ has joined #openstack-infra05:26
*** talluri has joined #openstack-infra05:26
*** rongze has joined #openstack-infra05:27
*** jcooley_ has quit IRC05:34
*** jcooley_ has joined #openstack-infra05:34
*** nosnos_ has joined #openstack-infra05:35
*** bingbu has quit IRC05:35
*** rongze has quit IRC05:36
*** nosnos has quit IRC05:37
*** nosnos_ has quit IRC05:37
*** nosnos has joined #openstack-infra05:38
*** slong has joined #openstack-infra05:38
*** bingbu has joined #openstack-infra05:47
*** jcooley_ has quit IRC05:54
*** jcooley_ has joined #openstack-infra05:55
*** matsuhas_ has quit IRC05:55
*** matsuhashi has joined #openstack-infra05:56
*** rongze has joined #openstack-infra06:02
*** paul-- has quit IRC06:10
*** rongze has quit IRC06:11
*** rongze has joined #openstack-infra06:11
*** sarob has quit IRC06:17
*** sarob has joined #openstack-infra06:18
*** SergeyLukjanov has joined #openstack-infra06:20
*** sarob has quit IRC06:22
*** chandankumar is now known as ciypro06:35
*** bingbu has quit IRC06:37
*** liang has joined #openstack-infra06:42
*** nati_ueno has quit IRC06:42
*** DinaBelova has joined #openstack-infra06:44
liangHi, I have a patch merged for pbr last week, https://review.openstack.org/#/c/57892/.06:44
liangAnyone has an idea when this will be synced into the build environment(gate keeping)?06:44
pabelangerliang, I would imagine once they tag a new pbr release06:46
*** sarob has joined #openstack-infra06:48
*** jcooley_ has quit IRC06:48
*** jcooley_ has joined #openstack-infra06:49
mordredwe're getting close to that06:49
mordredI want a few more patches to land -then we'll cut a new release06:50
mordredliang: ^^06:50
liangyup06:50
liangpabelanger, mordred, Thanks for the info!06:50
liangmordred, Will there be an email announce for a new release (the next release)?06:51
*** jcooley_ has quit IRC06:52
*** jcooley_ has joined #openstack-infra06:52
*** sarob has quit IRC06:54
*** bingbu has joined #openstack-infra06:54
*** matsuhashi has quit IRC06:56
*** matsuhashi has joined #openstack-infra06:56
*** matsuhashi has quit IRC06:57
*** matsuhashi has joined #openstack-infra06:57
*** nosnos_ has joined #openstack-infra06:57
*** nosnos has quit IRC06:57
*** marun has joined #openstack-infra06:59
mordredliang: hrm. I should probably do that07:01
mordredliang: I normally just break everyone07:01
*** yolanda has quit IRC07:02
liangmordred, I see. Thanks!07:03
mordredliang: ;) yes - we'll do an announce for sure07:03
mordredzaro: WOOT07:06
mordredjust saw the bug update07:07
mordredzaro: that's the best thing EVAR07:07
*** ljjjustin has quit IRC07:09
*** lianglin has joined #openstack-infra07:11
jishaomeiHi,all.I met a question about gearman when I built our ci enviromen. log info ERROR zuul.Gearman: Job <gear.Job 0x7f919803ca90 handle: None name: build:gate-tempest-python27 unique: ab9175d0c2ba401f93ad76e9de134d97> is not registered with Gearman.Would you please do me a favor about how to fix this issue, thanks.07:11
*** ljjjustin has joined #openstack-infra07:11
*** jcooley_ has quit IRC07:12
*** jcooley_ has joined #openstack-infra07:12
*** jcooley_ has quit IRC07:17
*** sarob has joined #openstack-infra07:18
*** jcooley_ has joined #openstack-infra07:18
*** nati_ueno has joined #openstack-infra07:20
*** nati_ueno has quit IRC07:21
lianglinhi guys, anybody can answer shaomei's question above? we need a way to debug why Gearman can't invoke jenkins job07:21
lianglinit would be great if somebody can show step by step debug here :)07:23
*** jcooley_ has quit IRC07:23
*** jcooley_ has joined #openstack-infra07:23
lianglin@fifieldt, @fungi~  can you help?07:24
fifieldthi07:24
fifieldtunfortunately I'm not expert enough for this question :(07:24
lianglinha~ anybody you know may be the expert?07:25
*** DennyZha` has joined #openstack-infra07:25
*** DennyZhang has quit IRC07:25
fifieldtI suspect the fungi, mordred, jeblair might know - but they are likely asleep at the moment07:25
*** zenoway has joined #openstack-infra07:25
lianglingot it~ go through the code at first before the experts respons07:26
*** zigo has quit IRC07:27
*** zigo has joined #openstack-infra07:27
*** senk has quit IRC07:27
*** bingbu has quit IRC07:27
*** pblaho has joined #openstack-infra07:29
*** jcooley_ has quit IRC07:32
*** jcooley_ has joined #openstack-infra07:32
*** DennyZha` has quit IRC07:33
*** jcooley_ has quit IRC07:34
*** jcooley_ has joined #openstack-infra07:36
*** jcoufal has joined #openstack-infra07:40
*** bingbu has joined #openstack-infra07:42
*** yolanda has joined #openstack-infra07:42
*** avishayb has joined #openstack-infra07:43
*** jcooley_ has quit IRC07:43
*** jcooley_ has joined #openstack-infra07:44
*** sarob has quit IRC07:50
*** jcooley_ has quit IRC07:50
*** jcooley_ has joined #openstack-infra07:51
*** zigo has quit IRC07:55
*** zigo has joined #openstack-infra07:55
*** dstanek_afk has joined #openstack-infra07:57
*** jcooley_ has quit IRC07:58
*** jcooley_ has joined #openstack-infra08:00
*** dstanek_afk has quit IRC08:01
*** nati_ueno has joined #openstack-infra08:02
*** osanchez has joined #openstack-infra08:06
*** DinaBelova has quit IRC08:08
*** sergmelikyan has quit IRC08:08
*** jcooley_ has quit IRC08:08
*** SergeyLukjanov has quit IRC08:08
*** jcooley_ has joined #openstack-infra08:08
*** matsuhashi has quit IRC08:10
*** flaper87|afk is now known as flaper8708:15
*** SergeyLukjanov has joined #openstack-infra08:15
*** matsuhashi has joined #openstack-infra08:16
*** dizquierdo has joined #openstack-infra08:17
*** sarob has joined #openstack-infra08:18
*** hashar has joined #openstack-infra08:19
*** jcooley_ has quit IRC08:21
*** zenoway has quit IRC08:22
*** osanchez has quit IRC08:22
*** sarob has quit IRC08:23
*** nati_ueno has quit IRC08:23
*** osanchez has joined #openstack-infra08:23
*** DinaBelova has joined #openstack-infra08:24
*** nosnos_ has quit IRC08:27
*** nosnos has joined #openstack-infra08:28
*** dizquierdo has quit IRC08:39
*** avishayb has quit IRC08:40
*** ilyashakhat_ has joined #openstack-infra08:42
*** rongze has quit IRC08:44
*** SergeyLukjanov is now known as _SergeyLukjanov08:48
*** _SergeyLukjanov has quit IRC08:49
*** SergeyLukjanov has joined #openstack-infra08:53
*** che-arne has quit IRC08:55
*** nati_ueno has joined #openstack-infra08:55
*** yassine has joined #openstack-infra09:00
*** avishayb has joined #openstack-infra09:01
*** hashar has quit IRC09:03
*** markmc has joined #openstack-infra09:03
*** zigo has quit IRC09:06
*** zigo has joined #openstack-infra09:06
*** jpich has joined #openstack-infra09:09
*** matsuhashi has quit IRC09:14
*** bingbu has quit IRC09:16
*** sarob has joined #openstack-infra09:18
*** DinaBelova has quit IRC09:19
*** nati_ueno has quit IRC09:19
*** SergeyLukjanov has quit IRC09:20
*** fbo_away is now known as fbo09:20
*** derekh has joined #openstack-infra09:22
*** afazekas has joined #openstack-infra09:22
*** sarob has quit IRC09:23
*** SergeyLukjanov has joined #openstack-infra09:23
*** hashar has joined #openstack-infra09:25
*** SergeyLukjanov has quit IRC09:29
*** matsuhashi has joined #openstack-infra09:31
Adri2000hi09:41
Adri2000I recently changed my affiliation (updated my profile at openstack.org) but I noticed that my first merged contribution appears with my previous affiliation at http://activity.openstack.org/data09:43
Adri2000how can I fix this?09:44
*** DinaBelova has joined #openstack-infra09:51
*** dizquierdo has joined #openstack-infra09:51
openstackgerritJulien Danjou proposed a change to openstack-dev/hacking: Add a check for file with only comments  https://review.openstack.org/5909009:52
*** mancdaz has quit IRC09:53
*** SergeyLukjanov has joined #openstack-infra09:55
*** mkulkin has joined #openstack-infra09:58
*** dstanek_afk has joined #openstack-infra09:58
*** dstanek_afk has quit IRC10:03
*** jroovers has quit IRC10:05
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Enable #openstack-climate IRC channel logging  https://review.openstack.org/5767510:07
*** sarob has joined #openstack-infra10:18
*** lianglin has quit IRC10:20
*** avishayb has quit IRC10:20
katyaferventHello all10:28
*** Ryan_Lane has quit IRC10:28
*** jorisroovers has joined #openstack-infra10:30
katyaferventCould somebody help me please with branch creation? There is no opportunity to create new branch (from review.openstack -> Admin) any more.  What can be the problem?10:30
*** markmcclain1 has quit IRC10:33
*** andreaf has quit IRC10:35
*** andreaf has joined #openstack-infra10:35
*** jorisroovers has quit IRC10:36
*** avishayb has joined #openstack-infra10:37
*** guohliu has quit IRC10:40
*** markmcclain has joined #openstack-infra10:43
openstackgerritA change was merged to openstack-infra/storyboard: Adding six dependency  https://review.openstack.org/5923510:44
*** sarob has quit IRC10:51
*** rongze has joined #openstack-infra10:55
*** jishaomei has quit IRC10:56
*** adalbas has joined #openstack-infra10:57
*** che-arne has joined #openstack-infra10:58
*** salv-orlando has joined #openstack-infra10:59
*** jorisroovers has joined #openstack-infra11:00
*** yaguang has quit IRC11:02
*** rfolco has joined #openstack-infra11:07
*** pcm_ has joined #openstack-infra11:07
*** liang has quit IRC11:08
*** pcm_ has quit IRC11:10
*** pcm_ has joined #openstack-infra11:10
*** sdague_ has joined #openstack-infra11:11
*** jorisroovers has quit IRC11:11
*** wenlock has quit IRC11:13
*** DinaBelova has quit IRC11:14
*** sarob has joined #openstack-infra11:18
*** matsuhashi has quit IRC11:22
*** ArxCruz has joined #openstack-infra11:24
sdaguejog0: did you create an lp or somewhere for tracking er bugs / work?11:24
*** sarob has quit IRC11:27
*** ljjjustin has quit IRC11:29
*** HenryG has joined #openstack-infra11:35
*** nati_ueno has joined #openstack-infra11:42
*** michchap_ has joined #openstack-infra11:42
*** michchap has quit IRC11:44
*** nprivalova has joined #openstack-infra11:49
*** nprivalova is now known as nadya_11:50
*** jorisroovers has joined #openstack-infra11:53
*** aardvark has joined #openstack-infra11:53
*** yassine has quit IRC11:55
*** dtroyer_zz has joined #openstack-infra11:56
*** zigo_ has joined #openstack-infra11:56
*** nprivalova has joined #openstack-infra11:56
nadya_hi guys! I faced with a problem with nova+memcache. Tokens are marked as unauthorized in memcache. Does anyone know about this problem?11:57
sdaguenadya_: that's really a #openstack question11:57
*** cyeoh has quit IRC11:57
*** dtroyer has quit IRC11:57
*** zigo has quit IRC11:57
*** WarrenUsui has quit IRC11:57
*** cyeoh has joined #openstack-infra11:57
*** nadya_ has quit IRC11:57
*** sdake_ has quit IRC11:57
*** sdake_ has joined #openstack-infra11:57
*** markmc has quit IRC11:57
*** sdake_ has quit IRC11:58
*** sdake_ has joined #openstack-infra11:58
*** markmc has joined #openstack-infra11:58
*** dstanek_afk has joined #openstack-infra12:00
*** dstanek_afk has quit IRC12:04
*** markwash has quit IRC12:04
*** nprivalova is now known as nadya_12:07
*** ruhe has joined #openstack-infra12:08
*** nati_ueno has quit IRC12:08
*** hashar has quit IRC12:16
*** sarob has joined #openstack-infra12:18
*** markwash has joined #openstack-infra12:18
*** pcm_ has quit IRC12:22
*** nosnos has quit IRC12:22
*** sarob has quit IRC12:23
*** rongze has quit IRC12:23
*** hashar has joined #openstack-infra12:25
*** SergeyLukjanov is now known as _SergeyLukjanov12:26
*** heyongli has joined #openstack-infra12:32
*** smarcet has joined #openstack-infra12:36
*** saschpe_ has quit IRC12:41
*** saschpe has joined #openstack-infra12:43
*** fifieldt has quit IRC12:47
*** che-arne has quit IRC12:52
*** matsuhashi has joined #openstack-infra12:57
*** CaptTofu has joined #openstack-infra12:57
openstackgerritRussell Bryant proposed a change to openstack-infra/config: Add gate-solum-devstack job  https://review.openstack.org/5709813:00
*** pcm_ has joined #openstack-infra13:02
portantegood day, folks: is there somebody that could help track down why this swift container server did not start during this commit's last check job? https://review.openstack.org/#/c/53270/13:02
portantewhat log file has the screen command where the container server is started?13:02
portantethanks13:03
*** pcm_ has quit IRC13:03
*** pcm_ has joined #openstack-infra13:03
*** matsuhashi has quit IRC13:06
*** matsuhashi has joined #openstack-infra13:07
*** dkranz has quit IRC13:07
*** sandywalsh has joined #openstack-infra13:09
*** heyongli has quit IRC13:09
openstackgerritA change was merged to openstack-infra/reviewstats: E128 really isn't sane.  https://review.openstack.org/5890213:10
*** matsuhashi has quit IRC13:11
*** matsuhashi has joined #openstack-infra13:13
sdaguefungi: once you wake up I'm going to come after you with some reviews to look at, because in order to make a couple of switch overs I've got 3 config reviews that need to land with 2 er changes interleaved13:14
*** matsuhashi has quit IRC13:17
*** che-arne has joined #openstack-infra13:17
*** pcm_ has quit IRC13:18
*** sarob has joined #openstack-infra13:18
*** rongze has joined #openstack-infra13:25
*** SergeyLukjanov has joined #openstack-infra13:26
*** pcm_ has joined #openstack-infra13:28
*** pcm_ has joined #openstack-infra13:29
*** nati_ueno has joined #openstack-infra13:32
*** wenlock has joined #openstack-infra13:34
*** xchu has joined #openstack-infra13:36
*** pcm_ has quit IRC13:42
*** mancdaz has joined #openstack-infra13:44
*** ciypro is now known as chandankumar13:44
*** thomasem has joined #openstack-infra13:45
*** CaptTofu has quit IRC13:46
*** CaptTofu has joined #openstack-infra13:47
*** ruhe is now known as ruhe_13:47
*** sarob has quit IRC13:50
openstackgerritThierry Carrez proposed a change to openstack-infra/config: Add oslo.messaging to release status tracking  https://review.openstack.org/5937413:51
*** ruhe_ is now known as ruhe13:54
*** dkranz has joined #openstack-infra13:54
*** yamahata_ has joined #openstack-infra13:55
*** larsbutler has quit IRC13:56
*** nati_ueno has quit IRC13:57
*** alexpilotti has joined #openstack-infra13:57
*** nadya_ has quit IRC14:00
*** bpokorny has joined #openstack-infra14:01
*** jcoufal has quit IRC14:01
*** avishayb has quit IRC14:03
*** CaptTofu has quit IRC14:03
*** CaptTofu has joined #openstack-infra14:04
*** chandankumar is now known as ciypro14:05
*** julim has joined #openstack-infra14:06
*** marun has quit IRC14:08
*** marun has joined #openstack-infra14:09
*** nprivalova has joined #openstack-infra14:09
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Add gate-climate-devstack  https://review.openstack.org/5870214:10
*** dolphm has joined #openstack-infra14:11
*** mkulkin has left #openstack-infra14:12
*** mriedem has joined #openstack-infra14:12
*** pcm_ has joined #openstack-infra14:15
*** nati_ueno has joined #openstack-infra14:16
*** pcm_ has quit IRC14:17
*** pcm_ has joined #openstack-infra14:18
*** ruhe is now known as ruhe_14:18
*** reaper has quit IRC14:18
*** sarob has joined #openstack-infra14:18
*** ruhe_ is now known as ruhe14:20
*** mfer has joined #openstack-infra14:20
*** mfer has quit IRC14:20
*** mfer has joined #openstack-infra14:21
*** dprince has joined #openstack-infra14:21
*** sarob has quit IRC14:23
*** yaguang has joined #openstack-infra14:24
*** jergerber has joined #openstack-infra14:29
*** weshay has joined #openstack-infra14:31
*** marun has quit IRC14:32
*** marun has joined #openstack-infra14:32
sdaguehey jog0's awake again :) now I can hound him a little on reviews14:33
* jog0 ducks14:33
sdaguebasically I just want to ensure I didn't screw up anything too bad so I can land through this set of swizzles and refactoring quickly (especially as I want to not land new recheck bugs until we get over to the directory)14:35
sdagueand so when infra folks are awake, I can coordinate lands with them14:36
*** avishayb has joined #openstack-infra14:37
jog0sdague: yeah, normally I  don't like being harassed for reviews too much, but this is a  good exception to that rule14:37
fungisdague: hollow threats14:37
sdaguefungi: :)14:37
*** dcramer_ has joined #openstack-infra14:37
fungilooks like jishaomei and lianglin have disappeared, but i'm going to guess that the jenkins master which has gate-tempest-python27 configured on it either isn't running the jenkins-gearman plugin or can't communicate via gearman protocol with the gearman server zuul is using14:40
portanteclarkb, jog0: can I get some help debugging why swift container server did not start on a given job https://review.openstack.org/#/c/53270/, and how we can fix devstack to fail faster when that happens?14:41
*** alexpilotti has quit IRC14:42
fungiAdri2000: reed may be able to point you in the right direction, but the activity site isn't developed by the infra team (some company named bitergia)14:42
fungikatyafervent: what project in gerrit are you trying to create a branch in? it's probably just an acl permission you need added on it14:43
fungiportante: interesting... i see a log for it under http://logs.openstack.org/70/53270/11/check/check-grenade-devstack-vm/36429b6/logs/old/ but not http://logs.openstack.org/70/53270/11/check/check-grenade-devstack-vm/36429b6/logs/new/ (i wonder if it's missing from the devstack/grenade service list in new?)14:48
portantefungi: yes, the devstack.log file?14:49
portanteIf that is the one you are talking about, I have noticed that for a while14:49
fungiportante: well, in the old environment there's a screen-s-container.txt.gz but not in new14:50
portanteyes14:50
*** jcoufal has joined #openstack-infra14:50
sdaguemordred: well with no other known avenue to get support on the membership service, I'm going to tweet about it14:51
sdague... a lot14:51
mordredsdague: please do14:51
fungis-container is definitely in the services list in both old and new14:51
fungisdague: tweet in all-caps14:51
sdaguehttps://twitter.com/sdague/status/40752219079221657614:51
mordredsdague: if you would like to tweet about the fact that infra runs a collaborative and scalable development infrastructure14:51
fungialso, the wiki for the www site says contact todd@openstack.org for support14:51
portantefungi: where is the script output for the new servies?14:51
*** marun has quit IRC14:51
mordredsdague: and that it doesn't make any sense for the website and the memberdata systems to be there14:52
mordredto not be there14:52
sdagueagreed14:52
fungiportante: still looking. i was just comparing the localrc between them first to be sure14:52
portantefungi: k thanks14:52
sdaguemordred: is there some sort of TC level statement we could make to help fight that fight?14:53
sdaguebecause my foundation email the last time went no where14:53
fungisdague: mordred: isn't there an assertion statement in the governance repo being voted on about that?14:53
mordredfungi: there is, but that has not been submitted to the list for discussion yet14:54
mordredwe're still going back and forth on language :)14:54
fungiahh, i was only kinda lurking the last tc meeting14:54
*** jpeeler has joined #openstack-infra14:54
*** jpeeler has joined #openstack-infra14:54
mordredsdague: https://review.openstack.org/#/c/5807214:55
sdaguemordred: awesome14:56
*** nati_ueno has quit IRC14:56
mordredsdague: ok - updated to address typo feedback14:56
openstackgerritMatthew Treinish proposed a change to openstack-infra/devstack-gate: Up the default concurrency on tempest runs  https://review.openstack.org/5860514:57
*** dhellmann is now known as dhellmann_14:58
*** loq_mac has joined #openstack-infra14:58
sdagueso, https://review.openstack.org/#/c/59293/1 is a change that we need in infra before we can split up the queries.yaml into a directory of files14:58
sdagueif anyone has a moment14:59
*** loq_mac has quit IRC15:01
*** dstanek_afk has joined #openstack-infra15:03
jog0sdague: btw the issues dkranz is hitting in the gating on errors is scary15:07
sdaguejog0: agreed, that's why we want to enforce it15:07
jog0sdague: I think my takeaway from it is: ceilometer needs to up there QA game15:08
jog0and they may need some friendly pushing to get there15:09
sdagueyeh, ceilometer is kind of a mess15:10
mordredyeah - I was just thinking that if he's having too hard a time making the whitelist, that we should just start enforcing what we have right now15:10
sdaguemordred: yeh, that's the plan15:10
mordredawesome15:10
*** alcabrera has joined #openstack-infra15:10
mordredI saw something about adding a * to ceilometer because they were adding them too quickly15:11
sdaguedkranz: is going to push enforcing today15:11
sdagueso, honestly, ceilometer is probably in a worse state than neutron15:11
sdaguehowever, neutron I think it more important to get up to snuff15:11
sdagueso neutron in icehouse, ceilometer in j15:11
*** pcm_ has left #openstack-infra15:12
sdagueor consider de-integrating ceilometer and putting it back to an incubation state15:12
portantesdague: is this is the "fail gate jobs if ERROR messages encountered?"15:12
mordredsdague: above done, btw15:12
sdagueportante: yes15:12
portantek thanks15:12
sdaguewe're using a white list, so if there are valid reasons for the errors, we can have them15:13
sdaguethe problem is ceilometer is adding new errors so fast you can't whitelist them15:13
mordredsdague: are there really valid reasons for errors?15:13
sdaguemordred: I'm not sure yet15:13
mordredsdague: k. fair enough15:13
sdaguehowever, I'm not going to say for 100% that there aren't15:14
portantemordred: if you are testing how error conditions are handled, perhaps?15:14
mordredmy hypothesis is that a traceback in a log means that a bug happened15:14
mordredportante: don't think so - errors are different than exceptions15:14
sdagueand the whitelist model at least lets us have that discussion15:14
*** alexpilotti has joined #openstack-infra15:14
openstackgerritA change was merged to openstack-infra/config: also link in /etc/elastic-recheck/queries  https://review.openstack.org/5929315:14
mordredportante: an _exception_ caused by something runnins api commands means there is a bug in the code15:14
* jd__ hangs his head in shame15:14
mordredand error message printed to the logs mean that an error occured15:14
*** rwsu has joined #openstack-infra15:14
mordredbut - I believe sdague madkes a great point, and it's entirely possible I'm wrong on that15:16
portantemordred: not convinced yet, as this seems to be about definition of ERROR and how it is used currently across projects15:16
dkranzmordred, sdague : So my issue is that fail on error is ready to go but I don';t think dev folk are really aware of it even with all the messages sent about it.15:16
jd__dkranz: could you add a few links to logs with the errors in the bug report you reported btw?15:16
dkranzI would like to send a joint qa/infra announcement that this is being turned on15:16
dkranzjd__: I can. I saw your comments in the recent ticket.15:16
fungiportante: so... on the grenade failure for 53270, i can see in the syslog for the change you asked about that at 09:08:21 "container-server UNCAUGHT EXCEPTION..."15:17
jd__dkranz: thanks; FWIW I've added this subject to our next meeting agenda15:17
sdagueportante: right, it will help shape that conversation as well15:17
dkranzjd__: There is a possibility these errors came from a patch that was not yet merged15:17
sdaguedkranz: so, honestly, just turn it on15:17
mordredagree15:17
sdaguepeople have had plenty of warning15:17
jd__dkranz: ok, that's what I think, that would come from the tempest tests we are pushing and that are not merged yet15:17
mordredthey'll learn when they add new exceptions into the logs15:17
*** dolphm has quit IRC15:17
dkranzSo be it.15:17
portante mordred, sdague, dkranz, at the least wouldn't we want to fail jobs with exceptions (python tracebacks emitted)15:18
*** xchu has quit IRC15:18
mordredportante: I'm not sure what you mean?15:18
portantefungi: looking15:18
*** sarob has joined #openstack-infra15:18
mordredportante: (we're talking about being able to trap that exceptions emitted in logs should trigger a test failure - but to do that, it means that in normal successful operation, we need to have no exceptions in the log)15:19
portanteagreed15:19
fungiportante: unfortunately the 4k limit we put on syslog messages truncated it15:19
mordredfungi: doh15:19
dkranzsdague: https://review.openstack.org/#/c/58848/15:20
portanteyes, was that with old or new code?15:20
*** dkliban has joined #openstack-infra15:20
mordreddkranz: "except neutron"15:20
fungiportante: i think that's the old code. i see a couple minutes later in the syslog where container-server reports shutting down, but then i never see it mention starting again15:20
portanteI looks like the old code, which in grizzly, and possibly havana, did not handle the signal sent to kill the servers15:20
mordreddkranz: sigh15:20
fungiportante: okay, so not relevant then. i'll keep staring15:21
dkranzmordred: Do you have a better suggestion for now?15:21
mordreddkranz: nope. just sigh that it's needed - not sigh on you15:21
portantefungi: yes, but where is the command that starts the container seraver with the new code?15:21
dkranzmordred: I agree. This is clearly in the better late than never category15:22
mordredyah15:22
portantefungi: in the "old" directory there is a devstack.log.gz, do you know why there isn't one of those in the "new" directory?15:23
*** reaper has joined #openstack-infra15:23
*** ryanpetrello has joined #openstack-infra15:26
fungiportante: i could speculate, but the qa people probably have a lot more insight into grenade's inner workings there15:27
fungiportante: in the console.html i do see...15:27
fungi2013-12-02 09:09:08.525 | + screen -S stack -p s-container -X stuff 'cd /opt/stack/new/swift &amp;&amp; /opt/stack/new/swift/bin/swift-container-server /etc/swift/container-server/1.conf -v || echo "s-container failed to start" | tee "/opt/stack/new/status/stack/s-container.failure"^M'15:27
portanteyes, and I would like to konw if the s-container.failure exists?15:29
portante.15:29
fungiso it looks like it at least thinks it started, since i don't see the failure message echoed15:29
fungibut it doesn't seem to have created a screen-s-container.log for new15:30
fungiportante: i think maybe the grenade shell log is the equivalent of the devstack log for new15:31
*** CaptTofu has quit IRC15:32
fungisdague: is that ^ an accurate statement?15:32
*** dolphm has joined #openstack-infra15:33
ryanpetrelloso Monty merged the "gate pecan against Ceilometer and others" patch, but it looks like the ceilometer tests are failing trying to connect to mongo: http://logs.openstack.org/43/58643/1/check/gate-pecan-tox-ceilometer-stable/68bc334/console.html.gz15:33
ryanpetrelloanybody have ideas about this?  figured I'd ask rather than just running a recheck15:33
*** jcoufal has quit IRC15:33
*** loq_mac has joined #openstack-infra15:34
mordredryanpetrello: I blame ... uhm ...15:35
mordredryanpetrello: (I do not)15:35
fungi(EMONGO)15:35
mordredryanpetrello: I do agree with you that it looks like it's failing connecting to mongo15:35
jd__let's blame mordred since he merged the patch, no?15:35
* jd__ runs15:35
ryanpetrellohaha15:35
*** loquacities has joined #openstack-infra15:36
*** CaptTofu has joined #openstack-infra15:36
mordreddo we have mongo runing on unit test slaves now?15:36
ryanpetrelloyea, I was wondering if I was missing something that was causing this to run in a place where it shouldn't15:36
*** adalbas has quit IRC15:36
jd__mordred: no but Ceilometer's run-tests.sh launches it temporarily15:36
mordredoh - I see that. neat15:36
jd__while we're talking about that, is there a PostgreSQL and MySQL instances running on the slaves that can be used?15:37
mordredjd__, ryanpetrello: it looks lke it started up to me15:37
mordredjd__: yes there is15:37
*** dhellmann_ is now known as dhellmann15:37
mordredjd__: nova uses it in their unit tests15:37
jd__mordred: I want to do that in Ceilometer too15:38
mordredjd__: I want you to do that in ceilometer! :)15:38
*** loq_mac has quit IRC15:38
jd__mordred: is there a documentation on the URL we shall use or should I grep nova?15:38
*** datsun180b has joined #openstack-infra15:39
*** datsun180b is now known as datsun_F40PH15:40
mordredjd__: user openstack_citest passwd openstack_citest schema openstack_citest  :)15:40
jd__noted, thanks!15:40
mordredjd__: but you may want to look at nova/tests/db/test_migrations.py15:40
mordredjd__: it's got the testing to see if mysql/postgres are there and whatnot15:40
*** wenlock_ has joined #openstack-infra15:40
jd__nice15:41
*** sparkycollier has joined #openstack-infra15:42
fungiahh the ol' alma mater... my brother was supposed to give his undergrad research presentation this morning, but they had to cancel the entire symposium because a raccoon crawled into the power substation and shorted it out, so the entire campus is running on emergency generators15:42
fungihillbilly-u15:42
*** dims has joined #openstack-infra15:43
ryanpetrellofungi: only counts as hillbilly-u if they ate the raccoon afterwards15:43
anteayaha ha ha15:44
*** rcleere has joined #openstack-infra15:44
ryanpetrelloso re: the mongo ceilometer issue15:44
ryanpetrelloany ideas?  is anybody against me doing a recheck?15:45
*** markwash has quit IRC15:46
jog0ryanpetrello: if its a bug that isn't in your code, file a bug first and then ok to recheck bug x15:47
*** pcrews has joined #openstack-infra15:48
ryanpetrellosame procedure if there's a known bug involved that's already been fixed?15:48
*** adalbas has joined #openstack-infra15:48
ryanpetrelloand how's it work if the bug is in *another* launchpad project?15:48
*** senk has joined #openstack-infra15:48
*** dolphm has quit IRC15:50
jog0ryanpetrello: it jut works if its another launchhpad project15:50
jog0ryanpetrello: the bug has been fixed?15:50
ryanpetrelloin the case of one failure, yes15:51
*** sarob has quit IRC15:51
*** jroovers has joined #openstack-infra15:52
*** sparkycollier has quit IRC15:53
*** dolphm has joined #openstack-infra15:54
*** jorisroovers has quit IRC15:55
*** rnirmal has joined #openstack-infra15:55
*** loquacities has quit IRC15:55
*** jorisroovers has joined #openstack-infra15:55
*** ^d has joined #openstack-infra15:56
*** jroovers has quit IRC15:57
*** briancline has quit IRC15:58
*** SergeyLukjanov has quit IRC15:58
fungiportante: sdague: i'm at a loss for why no /opt/stack/new/screen-logs/screen-s-container.log15:58
fungiwas archived in http://logs.openstack.org/70/53270/11/check/check-grenade-devstack-vm/36429b6/console.html15:58
*** dkranz has quit IRC15:58
sdaguefungi: did it get that far?15:58
fungisdague: yep, should have been created. it got as far as stuffing the command to start the container service into the screen session15:59
fungiby then there should have been a screen log for that session15:59
fungiright?15:59
*** mgagne has joined #openstack-infra16:00
*** mgagne has quit IRC16:00
*** mgagne has joined #openstack-infra16:00
fungithere is later indication during testing that the container service was not actually running, and no mention in syslog that it was started at all after grenade upgraded it16:00
*** SergeyLukjanov has joined #openstack-infra16:01
sdagueso, we did experience straight up screen races before16:01
sdaguethat's the only thing I can think of with this16:02
fungihow did we work around that? i know at one point there was a suggestion that we simply disable running under screen in ci, but concerns were expressed that would deviate too far from what a typical dev runs16:02
*** DinaBelova has joined #openstack-infra16:03
sdaguehttp://logs.openstack.org/70/53270/11/check/check-grenade-devstack-vm/36429b6/console.html#_2013-12-02_09_09_08_52516:03
sdaguefungi: honestly, I don't remember atm16:03
*** jorisroovers has quit IRC16:03
*** DinaBelova has quit IRC16:03
fungiyup, that's the line i was looking at16:03
*** luisg has joined #openstack-infra16:04
*** jcooley_ has joined #openstack-infra16:04
jeblairgood morning16:04
fungimorning, jeblair16:04
*** SergeyLukjanov has quit IRC16:05
*** mancdaz has quit IRC16:06
jog0sdague: quick opinion question on https://review.openstack.org/#/c/5909016:06
jog0then  Iwill review your patches16:06
sdagueshoot16:06
jog0https://review.openstack.org/#/c/59090/7/doc/source/index.rst16:06
jog0that rule wasn't what was initially discussed on the ML therad16:07
jog0thread*16:07
jog0and although I like that rule and am ++ on it, do we need to get ML feedback on that?16:07
*** mgagne has quit IRC16:08
zaromordred, jeblair : w00t! WIP vote merged!16:08
jog0as that goes a little further then the no license header needed in an file with nothing else in it16:08
sdagueso do yuo see it as materially different than the thread?16:08
*** mgagne has joined #openstack-infra16:08
*** mgagne has quit IRC16:08
*** mgagne has joined #openstack-infra16:08
jog0 "empty files shouldn't16:08
jog0> have license headers, because their content of emptiness isn't16:08
jog0> copyrightable"16:08
jog0I see that as different16:08
sdagueok, fair16:08
jog0although the only case I Found in nova was nova/network/security_group/__init__.py16:09
mgagnehashar: would you mind rebasing this one? https://review.openstack.org/#/c/56397/16:09
sdaguedo you know of any actual hits in our source trees that would fail one versio the other?16:09
jog0which is pretty trivial16:09
*** pete5 has joined #openstack-infra16:09
jog0let me try another project16:09
fungisdague: portante: i guess if you see it again, recheck bug 1183449 ?16:09
uvirtbotLaunchpad bug 1183449 in grenade "Screen is causing non-deterministic behaviour" [High,New] https://launchpad.net/bugs/118344916:09
openstackgerritJulien Danjou proposed a change to openstack-dev/hacking: Update pep8 and flake8 requirements  https://review.openstack.org/5915216:09
jog0jd__: thanks btw ^^16:10
jd__jog0: np :)16:10
jog0so we are discussing your other patch, which  I like the idea of but ... see scrollback16:11
ttxfungi, mordred: would be nice to have https://review.openstack.org/#/c/58616/ accepted before EOD tomorrow, since we'll need to cut some savanna mp branches then16:12
*** ywu has joined #openstack-infra16:13
*** ciypro has quit IRC16:13
hasharmgagne: yeah still have to resurrect the cloverphp publisher change16:13
ttxfungi: thx!16:13
*** dkranz has joined #openstack-infra16:13
hasharmgagne: aka https://review.openstack.org/#/c/52940/316:13
mgagnehashar: mind rebasing on master instead?16:13
*** nprivalova has quit IRC16:14
mgagnehashar: your change will be able to get merged without cloverphp which is unrelated16:14
hashartrue16:14
hashar:D16:14
openstackgerritAntoine Musso proposed a change to openstack-infra/jenkins-job-builder: raise JenkinsJobsException instead of Exception  https://review.openstack.org/5639716:16
hasharmgagne: done16:16
hasharno more Exception :]16:16
mgagne:D16:16
*** pabelanger_ has joined #openstack-infra16:17
hasharthe other patch, I will resurect it whenever I have time16:17
*** sarob has joined #openstack-infra16:18
*** pabelanger_ has quit IRC16:18
*** pabelanger_ has joined #openstack-infra16:18
*** pabelanger has quit IRC16:18
openstackgerritA change was merged to openstack-infra/config: Add missed milestone-proposed rule  https://review.openstack.org/5861616:19
*** pabelanger has joined #openstack-infra16:19
*** pabelanger has quit IRC16:19
*** pabelanger_ is now known as pabelanger16:19
*** MIDENN_ has joined #openstack-infra16:20
*** pabelanger_ has joined #openstack-infra16:20
portantefungi thanks, will do16:21
*** pabelanger has quit IRC16:21
*** pabelanger has joined #openstack-infra16:22
*** mrodden has joined #openstack-infra16:22
*** MIDENN_ has quit IRC16:22
EmilienMcould someone from core have a review on https://review.openstack.org/#/c/59247/ ? it's for adding puppet-ironic into stackforge16:23
*** mdenny has quit IRC16:23
fungizaro: congrats on the gerrit wip patch!16:25
*** loq_mac has joined #openstack-infra16:25
ttxmordred: around?16:26
zarofungi: thx. almost gave up on it.16:27
*** mdenny has joined #openstack-infra16:27
*** loquacities has joined #openstack-infra16:28
*** pblaho has quit IRC16:28
*** pblaho has joined #openstack-infra16:28
*** loq_mac has quit IRC16:29
anteayazaro: well done on getting your patch into upstream gerrit16:29
anteayayay you16:29
anteayathat alone might be worthy of a talk :D16:30
zaromaybe just on the politics of upstream gerrit. :)16:30
*** markmc has quit IRC16:30
pleia2good morning16:31
openstackgerritJames E. Blair proposed a change to openstack-infra/zuul: Add a zuul client  https://review.openstack.org/5884116:31
anteayazaro: yeah, I think you would have good attendance16:32
anteayamorning pleia216:32
anteayaread the scrollback on last weeks meeing16:32
anteayasorry I wasn't here for the puppetboard item16:32
anteayayou covered it well16:33
*** adalbas has quit IRC16:33
anteayaI'm probably going to be absent from this week's meeting too :(16:33
*** jcooley_ has quit IRC16:33
*** yaguang has quit IRC16:33
*** sarob has quit IRC16:33
*** dstanek_afk has quit IRC16:36
*** hashar has quit IRC16:37
openstackgerritRussell Bryant proposed a change to openstack-infra/reviewstats: Add barbican.json project file  https://review.openstack.org/5947616:37
jog0sdague: I still don't see the resolved_at keyword logic16:37
openstackgerritA change was merged to openstack-infra/reviewstats: Add barbican.json project file  https://review.openstack.org/5947616:37
jog0in https://review.openstack.org/#/c/5925216:37
sdaguejog0: it isn't there yet16:38
jog0sdague: I thought this was ready to review16:38
sdaguethat was going to come in after the switch over16:38
jog0sdague: why?16:38
sdaguebecause sorting out the logic of what we want wasn't yet decided16:38
sdagueI can go the other path and remove references to it16:39
*** senk has quit IRC16:39
sdagueI guess we crossed wires about what the state of this patch would be16:39
jog0sdague: ahh, yeah I would prefer you mention that the lgoic isn't fully sortreed out and is coming soon16:39
*** mancdaz has joined #openstack-infra16:39
jog0maybe just an update to the readme and commig msg16:39
sdagueok, can do16:40
jog0cool16:40
*** danger_fo_away is now known as dangers16:40
*** avishayb has quit IRC16:42
*** mancdaz has quit IRC16:42
*** mancdaz has joined #openstack-infra16:43
pleia2so much email16:43
openstackgerritJustin Shepherd proposed a change to openstack-infra/config: Adding chef-style-rubocop job  https://review.openstack.org/5947816:43
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: Fix E122,E126,E128 items in codebase  https://review.openstack.org/5927216:44
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: move queries.yaml into a queries subdir  https://review.openstack.org/5925216:44
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: method extraction for readability  https://review.openstack.org/5926316:44
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: time to grow up and fit in 80 columns  https://review.openstack.org/5926216:44
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: wrapping the README.rst file to 80 cols  https://review.openstack.org/5926116:44
sdaguejog0: updated16:44
jog0sdague: now i have to go and +2 all your paches again ... sigh16:45
sdaguejog0: well, you're the the one that wanted the change :)16:45
sdagueI thought the +2 would come back16:45
jog0sdague: some will I think16:45
sdagueas they are rebases, except for the bottom one16:45
pleia2all hail the rebase detection script16:46
sdagueso let me know if you like the new 5925216:46
sdagueas fungi and mordred merged the thing in config that needed to land before it16:46
*** adalbas has joined #openstack-infra16:47
*** loquacities has quit IRC16:47
jog0so for https://review.openstack.org/#/c/59252 I assume you have actaully ran and tested your changes16:47
jog0as our tset coverage in gate for this is pretty so-so16:47
jog0especially for such a big change16:47
jog0sdague: ^16:47
sdaguethe check success scripts work and do the right thing16:47
sdagueso the loader path is tested16:48
*** loq_mac has joined #openstack-infra16:48
jog0that covers the changes here so cool16:49
sdagueyep16:49
*** DinaBelova has joined #openstack-infra16:49
*** senk has joined #openstack-infra16:51
ttxmordred: nm16:51
*** briancline has joined #openstack-infra16:52
*** gyee has joined #openstack-infra16:52
jog0sdague: dinner time so wil lbe AFK for a bit, all patches but the web one are reviewed16:53
sdaguecool16:53
jog0that one on a high level looks good to me, but haven't dug in yet.  also jim should review that one too16:53
sdagueyep, agreed16:53
sdagueand that's orthoginal enough16:54
jog0yup16:54
jog0have fun dealing with the new and imroved no stacktrace gate16:54
openstackgerritA change was merged to openstack-infra/config: Fix pep8 errors in modules dir  https://review.openstack.org/4764416:55
sdagueok, mtreinish did a +A on the dir change one16:55
*** mrodden has quit IRC16:55
*** rongze has quit IRC16:56
*** pabelanger has quit IRC16:56
sdaguewhich means we might actually need a manual restart of that bot... IIRC it doesn't do code changes CD16:56
sdaguethen https://review.openstack.org/#/c/59301/1 is next up16:56
sdagueto move the graphs over to it16:57
openstackgerritA change was merged to openstack-infra/elastic-recheck: move queries.yaml into a queries subdir  https://review.openstack.org/5925216:57
mordredsdague: ok- e-r waterfall of style patches above shoudl all fall in16:57
sdaguemordred: coolio16:57
mordredsdague: I also did not review anything complex or that did anything substantial16:58
sdagueyep16:58
sdaguethe patch series was called cleanup for a reason :)16:58
openstackgerritRossella Sblendido proposed a change to openstack-infra/devstack-gate: Test neutron full stack  https://review.openstack.org/5948416:58
*** dkranz has quit IRC16:59
openstackgerritA change was merged to openstack-infra/elastic-recheck: wrapping the README.rst file to 80 cols  https://review.openstack.org/5926116:59
openstackgerritA change was merged to openstack-infra/elastic-recheck: time to grow up and fit in 80 columns  https://review.openstack.org/5926216:59
openstackgerritA change was merged to openstack-infra/elastic-recheck: method extraction for readability  https://review.openstack.org/5926316:59
openstackgerritA change was merged to openstack-infra/elastic-recheck: Fix E122,E126,E128 items in codebase  https://review.openstack.org/5927216:59
sdaguenice16:59
sdagueok, time to heat up some lunch17:00
jeblairmordred: let me know if you want to talk about the -2 i just left on https://review.openstack.org/#/c/48064/17:01
jeblairmordred: i'm happy to help if you would like it17:01
*** boris-42 has quit IRC17:01
*** dstanek_afk has joined #openstack-infra17:02
mordredjeblair: morning! I believe that I have addressed your concerns, but it will take landing the patches that are listed as dependencies first before we can recheck that to verify that it is correct17:02
fungittx: added sergey as the initial savanna-milestone member (he doesn't seem to be around at the moment)17:03
mordredjeblair: I was intending on just pinging you here to talk about the 3-patch sequence anyway :)17:03
*** mrodden has joined #openstack-infra17:03
jeblairmordred: ok, i'll go look at the others (regardless, i don't think we should merge yours yet since it is not yet self-gating)17:04
mordredjeblair: I agree - I believe that once the other two land, this patch will be self-gating17:04
mikalSo... Who understands tempest more than me? Or perhaps just testr?17:05
jeblairmordred: you might try replying to major structural comments in the future.  :)17:05
mikalDoes testr promise to only run tests from one test class on a single worker?17:05
mikalOr could tests from a single class be of different workers at the same time?17:05
mordredjeblair: bah. that would be way too communicative :)17:06
mordredmikal: could be on different workers17:06
mikalHmmm17:06
mikalI'm looking at a tempest failure...17:06
mikalIts a delete instance that failed17:06
mordredmikal: testr makes no promises about adjacency or scheduling17:06
mikalBecause the instance was deleted17:06
mikalSo it feels a bit like a race in testr to me17:06
mordredmikal: nice. sounds like test interaction issues for sure17:06
mordredmikal: race in the tests17:07
*** dstanek_afk has quit IRC17:07
mordrednot in testr17:07
mikalWell, becuse of tyhe way testr runs the tests17:07
mordredthe tests are assuming a sequential ordering, most likely, and are not cleaning up / setting up properly17:07
mordreda test assuming and not creating a particular state is a bug in the test17:07
jeblairmordred: in https://review.openstack.org/#/c/59196/3 you say "Additionally, we need to stop updating17:07
jeblairrequirements.txt itself.17:07
jeblair"17:07
jeblairmordred: i don't understand that and the patch doesn't seem to do anything about that.17:07
mordredjeblair: crappit - I believe that is an aborted leftover sentence. lemme fix17:08
openstackgerritKhai Do proposed a change to openstack-infra/config: new maven plugin jobs to build and deploy maven plugins to maven nexus  https://review.openstack.org/4609917:08
openstackgerritMonty Taylor proposed a change to openstack/requirements: Align projects list with mirror-needing projects  https://review.openstack.org/5919617:11
mordredjeblair: there ya go - the english in that commit message probably makes more sense17:11
sdaguemikal: in testr we are running parallel only at the class level17:11
sdaguemikal: sorry, in tempest's use of testr17:11
mikalsdague: oh, interesting17:11
mikalsdague: so... I have a test that's failing because the instance is in an unexpected state17:12
mikalIt looks to me like its trying to delete and instance which is already deleted17:12
mikalWhich gives me the sads17:12
fungimordred: 59196 says "openstack/requirements itself also does not use the mirror, so remove17:12
fungiit from the list."17:12
sdaguemikal: you have the test fail?17:12
fungiwhich i think is not entirely accurate?17:12
mikalsdague: http://logs.openstack.org/99/55499/11/gate/gate-tempest-devstack-vm-full/5bffadf/console.html17:12
mordredfungi: it is accurate. we explicitly do not want changes to openstack/requirements to hit the mirror17:12
*** dkranz has joined #openstack-infra17:13
mordredbecause otherwise we can't test changes to it17:13
fungimordred: if you look at the mirror selection logic in devstack-gate, we skip mirror selection for the requirements integration job when the project is openstack/requirements17:13
mordredthat's right17:13
fungimordred: but other devstack-gate jobs which happen to run for changes to openstack/requirements are not skipped17:13
jeblairmordred: ok, whole thing makes sense; leaving -2 until everything else merges; you may need to ping me to lift that.17:13
mordredfungi: that's right17:13
fungier, rather, mirror overriding is not skipped17:14
mordredfungi: I'm not sure how that's different from what the above says?17:14
mikalsdague: http://paste.openstack.org/show/54301/17:14
fungii interpret your patch as "changes to openstack/requirements" don't use the openstack pypi mirror when being tested in devstack-gate17:15
mordredfungi: right. we can't ever use the mirror for changes to openstack/requirements17:15
mordredbecause they wouldn't work17:15
sdaguemikal: so that test is sending a hard reboot17:15
mordredif we think we're using it now, we're wrong17:15
fungiexcept they do, so long as the job is not the requirements integration test17:15
mordredno. it's not possible17:15
sdagueand expects the server to come back around to ACTIVE17:15
mordreda change to requirements can introduce a thing whic is not in the mirror17:15
*** dstanek_afk has joined #openstack-infra17:15
mordredthat means it cannot, by definition, be constrained by the mirror17:15
mordredno matter what the job is17:16
mordredor?17:16
sdagueit looks like the libvirt driver is racing, lost the server in some part of the transition, then left the server in the HARD_REBOOT state because it was confused17:16
fungimordred: no, you're right17:16
mordredfungi: however, this conversation has just pointed out to me another potential failure17:16
mordredsince requirements is in the integrated gate, changes behind it to non-requirements repos are going to want to be able to test in the context of that requirements change17:17
fungimordred: i misread. we skip mirror selection when running any devstack-agte job for openstack/requirements and *also* when running the requirements-integration jobs for any project17:17
mordredfungi: ok. cool. I was worried that I fundamentally misunderstood something17:17
mordredfungi: there is a weird case I just listed above, but since it hasn't bitten us yet, perhaps it's not as devastating as one might think :)17:17
fungiyeah, i agree that's a hole17:18
fungiand i'm not immediately seeing an easy solution to it17:18
mordrednope.17:19
sdaguethat being said, we could create a new server for the reboot test, as the issue might be that something else there left it in a silly state17:19
jeblairmordred: in what context would they fail to test what was expected?17:19
mordredfungi: not unless we create temporary mirrors for requirements changes and let subsequent gate jobs figure out how to use that17:19
*** dstanek_afk has quit IRC17:19
mordredjeblair: if a requirements change introduces a requirement that cannot be satisfied by the mirror17:20
jeblairmordred: (i understand the situation you are describing -- i'm wondering _exactly what thing_ they want to test won't be there)17:20
jeblairmordred: what thing will want to test that?17:20
jeblairmordred: what change will be behind it that will require that?17:20
mordredit's not that17:20
mordredjeblair: then a subsequent change to nova should, by zuul combination rules, pull in that requirements change and then run update.py in devstack17:20
mordredwhich means that it should just fail17:20
*** dstanek_afk has joined #openstack-infra17:21
jeblairmordred: except that that change won't even pass check tests until _after_ the requirement change lands17:21
fungimordred: so going back to the devstack-gate mirror selection logic for 59196, once the other two changes merge we should be able to simplify that conditional to just if [ "$DEVSTACK_GATE_REQS_INTEGRATION" -eq "0" ]17:21
mordredno - not a change to nova that wants to use that17:21
mordredjeblair: a change that's approved after the requirements change is approved17:21
mordredwill pick up the requirements chagne in the devstack update.py invocation17:21
jeblairmordred: then it will have been approved with its check tests failing.17:21
mordredno17:21
*** SergeyLukjanov has joined #openstack-infra17:21
mordredit's not a nova change that contains that requirements change17:21
sdaguemikal: actually, the test triggers a server rebuild if it's not active17:22
mordredits' a nova change that does not contain that requirements change - but is approved after it in sequence in the gate17:22
sdagueso that logic should already be there17:22
jeblairmordred: oh, but the requiremnts change should not affect it -- the requirements change itself tests that17:22
mordredwhich means that the state of the requirements repo at the point the nova change is tested will be the state of the pending requirements change17:22
mordredand since devstack syncs the requirements files of all of the projects with the checked out requirements repo17:22
jeblairmordred: oh, i think i see what you're saying17:23
*** senk has quit IRC17:23
mordredjeblair: it's a REALLY REALLY odd sequencing issue17:23
jeblairmordred: it will force an unsatisfiable dependency on the unsuspecting nova change17:23
mordredyup17:23
jeblairmordred: i'm not sure it's that odd17:23
jeblairmordred: it sounds quite likely.17:23
mordredwell, I mean, in practice, it's probably not likely to occur with great frequency17:23
mordredhrm. no - you're right, it shold happen more frequently - why are we not seeing the failure?17:24
jeblairmordred: won't it mean that any requirements change will completely break the gate until the mirror is updated?17:24
mikalHuh17:24
*** thedodd has joined #openstack-infra17:24
mikalThis error isn't an error in the driver17:24
mordredonly ones that introduce completely new boundaries that were previously not resolvable17:24
mikalIts a silly log level to use17:24
mordrednew requirements arent' a problem because nothing will be using them yet17:25
*** dstanek_afk has quit IRC17:25
*** pblaho has quit IRC17:25
mordrednew upper bounds probably usually work because they'll be constraining within the set of thigns we had already17:25
jeblairmordred: ah, ok.  that may be the answer to the question then.17:25
mordredbut if we had an upper bound, and we introduce a new change which raises and and also raises the lower bound past the previous upper bound17:25
mordredproblem is - landing THAT change is going to be almost impossible :)17:26
jeblairmordred: we probably don't ever merge requirements changes that are not resolvable with the previous set -- that would after all, break cd.17:26
mordredyah17:26
mordredso - neat. actually,let's call it a feature17:26
mordredjeblair: how's that for a mental exercise to start your monday?17:26
jeblairmordred: ++17:26
fungithough the iso8601>=0.1.8 change verges very close to that scenario17:26
jeblairmordred: well, back to clarkb's d-g refactor now.  :)17:27
mordredfungi: yes. I believ ethe iso8601 landing is a bug17:27
mordredfungi: that should never have occured17:27
*** moted has joined #openstack-infra17:27
*** dolphm has quit IRC17:27
mordredand that we did not prevent it from landing indicates a hole in our testing17:27
mordredwhich hopefully we'll fix soon17:27
*** mfer has quit IRC17:27
*** rongze has joined #openstack-infra17:28
sdaguemikal: well it actually is an error somewhere17:28
sdaguebecause after a server reboot, the server isn't coming back17:28
sdaguethat's actually why tempest is failing you17:28
sdagueif you hard reboot a server it should return to an active state17:28
*** reed has joined #openstack-infra17:28
sdagueafter some time, and we wait 196 seconds for it to do that17:29
*** afazekas has quit IRC17:29
*** dstanek_afk has joined #openstack-infra17:29
reedgood day everybody17:30
fungihowdy reed17:30
reedhow was the turkey?17:31
*** loq_mac has quit IRC17:31
reedmine was good, not wild this year... for the first time I could try the legs :)17:31
*** senk has joined #openstack-infra17:32
*** dizquierdo has quit IRC17:32
*** mfer has joined #openstack-infra17:33
reedspeaking of serious business, how can we get to an agreement on https://review.openstack.org/#/c/53644/ ?17:34
*** fbo is now known as fbo_away17:34
openstackgerritJulien Danjou proposed a change to openstack-dev/hacking: Add a check for file with only comments  https://review.openstack.org/5909017:35
*** hogepodge has joined #openstack-infra17:35
jeblairArxCruz: hi, i want to make sure i understand https://review.openstack.org/#/c/54386/17:36
ArxCruzjeblair: yes?17:36
jeblairArxCruz: you say kernel-headers is installed by "devstack_gate.sh"; do you mean by nodepool?17:36
*** bpokorny1 has joined #openstack-infra17:37
ArxCruzjeblair: yes17:37
ArxCruzElizabeth test in her infra for both precise and fedora17:37
jeblairArxCruz: so in ./modules/openstack_project/files/nodepool/scripts/prepare_devstack.sh17:37
*** jpich has quit IRC17:37
*** bpokorny2 has joined #openstack-infra17:37
ArxCruzjeblair: yup17:38
ArxCruzsudo yum -y install python-devel make automake gcc gcc-c++ kernel-devel redhat-lsb-core17:38
ArxCruzthis line install the kernel-headers17:38
*** bpokorny has quit IRC17:38
jeblairArxCruz, pleia2: i think what we want to do in the long run is make sure that devstack itself handles that...17:39
pleia2++17:39
jeblairArxCruz, pleia2: do you know if devstack itself handles any of those?17:39
ArxCruzjeblair: I can check, if don't I can submit a patch :)17:39
pleia2haven't tested that yet, but I can spin something up this afternoon17:40
pleia2or ArxCruz :)17:40
ArxCruz:)17:40
jeblairArxCruz, pleia2: cool, so i think this should be what we aim for:17:40
jeblairArxCruz, pleia2: 1) devstack itself installs as many openstack dependencies as it can17:40
jeblairArxCruz, pleia2: 2) anything that devstack-gate needs in order for it to run that isn't installed in 3) should be in devstack-gate17:41
jeblairArxCruz, pleia2: 3) anything that the nodepool setup scripts need to run should be installed by the nodepool setup scripts17:41
*** bpokorny1 has quit IRC17:42
jeblairArxCruz, pleia2: so 2 and 3 should be as small as possible; ideally, nothing.  :)17:42
pleia2sounds good17:42
jeblairArxCruz, pleia2: so i'll go ahead and +2 54386 then17:44
*** adalbas has quit IRC17:44
ArxCruzjeblair: beer++ for you :)17:44
ArxCruz:D17:44
pleia2I have a DMV appointment to run off to, hopefully it won't take all day ;)17:44
openstackgerritJustin Shepherd proposed a change to openstack-infra/config: Adding chef-style-rubocop job  https://review.openstack.org/5947817:49
*** hogepodge has quit IRC17:50
ArxCruzjeblair: whenever you have time, take a look at this one https://review.openstack.org/#/c/59473/ :)17:51
*** UtahDave has joined #openstack-infra17:52
jeblairmordred: -1 on https://review.openstack.org/#/c/59105/  (i note because it's blocking a critical bug)17:54
mordredjeblair: looking17:54
mordredjeblair: GAH. wow. how did I manage to mess up a change that small17:55
jeblairmordred: no "-Wall".  :(17:55
Alex_Gaynormordred: We've acquired a transitive dependency which doesn't have it's packages on pypi :(17:55
mordredAlex_Gaynor: dammit17:56
fungioh, wow. and how did i miss that it lacked a format string argument to the replicate command?17:56
mordredAlex_Gaynor: which one? and from where?17:56
openstackgerritMonty Taylor proposed a change to openstack-infra/gerritlib: Add replicate command  https://review.openstack.org/5910517:56
Alex_Gaynormordred: pytidylib, from sphinxcontrib-docbookrestapi-17:56
mordredAlex_Gaynor: I'm so sad17:56
Alex_GaynorMe too :-(17:56
*** adalbas has joined #openstack-infra17:58
openstackgerritA change was merged to openstack-infra/gerritlib: Add replicate command  https://review.openstack.org/5910517:58
fungimordred: jeblair: we wanted to tag a release of gerritlib now ^ that's in, right?17:58
mordredfungi: I'd like to do that - but there were a couple of other changes in master since our last release that don't have anything to do with this command17:59
*** alcabrera has quit IRC17:59
mordredfungi, jeblair: none of them look scary though17:59
*** alcabrera has joined #openstack-infra17:59
jeblairmordred, fungi: agreed; 0.3.0 ?18:00
*** nati_ueno has joined #openstack-infra18:00
mordredjeblair: sounds great to me18:00
jeblairi will do this thing.18:00
fungijeblair: yes, looks good18:00
fungisince it's not just bug fixes18:00
*** wusui has joined #openstack-infra18:01
*** krotscheck has joined #openstack-infra18:01
fungiif memory serves, we're consuming releases of gerritlib via pypi on review.o.o, so we can't land the related jeepyb changes until that happens18:01
openstackgerritA change was merged to openstack-infra/config: Added openstack-chef-repo to list  https://review.openstack.org/5858518:01
jeblairfungi, mordred: 0.3.0 pushed18:02
*** aardvark has quit IRC18:02
mordredjeblair: woot18:02
*** markwash has joined #openstack-infra18:02
clarkbmorning18:02
jeblairclarkb: hi there18:02
fungilooks like pypi upload completed for it18:02
fungiyup, 0.3.0 is on pypi now18:03
mordredjeblair, fungi: please check me very carefully on the manage-projects changes - given that I messed up the gerritlib change :)18:03
jeblairmordred: have you tested them?18:03
fungii tried to review them very carefully, but then again i messed up catching the error on that gerritlib change18:04
mordredjeblair: no. I'm not really sure how to test the things like the github failure (it's the logic about failure and resume I'm concerned about most)18:04
mordredjeblair: I'd love feedback or ideas on a good way to test that18:04
jeblairmordred: right, but have you tested that it still runs and does the right thing under normal circumstances?18:05
reedfungi, thanks for helping with the openid server patch... need to get that one moving18:05
*** pabelanger has joined #openstack-infra18:05
*** pabelanger has quit IRC18:05
*** pabelanger has joined #openstack-infra18:05
fungireed: no problem. testing it now to see if it's still functionally broken18:06
clarkbso much scrollback18:06
jeblairmordred: i take it that's a no as well.  :(18:07
*** senk has quit IRC18:07
mordredjeblair: no. I have not. let me go do that real quick (sorry, just got back to real internets last night - interactive remote things now possible)18:07
*** branen has quit IRC18:09
mordredanybody mind if I stop puppet on review-dev ?18:09
*** senk has joined #openstack-infra18:10
fungimordred: not i18:10
jeblairmordred: go for it18:10
*** mestery_ has joined #openstack-infra18:11
*** mestery has quit IRC18:11
*** senk has quit IRC18:13
clarkbArxCruz: did you see I had to roll back the change to add fedora hosts to nodepool? looked like permissions issues18:14
ArxCruzclarkb: no I didn't18:14
ArxCruzclarkb: what's happening ?18:15
*** senk has joined #openstack-infra18:15
clarkbArxCruz: at some point we try to write to /opt (a fiel copy iirc) and that fails18:15
ArxCruzclarkb: but that's related to my patch? I don't remember touch anything related to /opt18:15
mordredjeblair: hrm. has manage-projects ever worked on review-dev ?18:16
clarkbArxCruz: its related in that your patch wasn't sufficient to make it go on fedora18:16
clarkbArxCruz: we need more bootstrapping somewhere18:16
jeblairmordred: i think you did your initial testing there18:16
*** dhellmann is now known as dhellmann_18:16
mordredjeblair: heh. ok18:16
clarkbArxCruz: I am digging up the stacktrace now18:16
ArxCruzclarkb: okay, if you need help, let me know18:17
mordredjeblair: I am now getting key errors :)18:17
ArxCruzclarkb: I'm leaving now, but i'll be back in a few hours, have english class :)18:17
openstackgerritA change was merged to openstack-infra/config: Pass $mysql_password through, part two  https://review.openstack.org/5861818:17
clarkbArxCruz: np have fun18:17
*** openstackgerrit has quit IRC18:18
*** vipul is now known as vipul-away18:18
*** vipul-away is now known as vipul18:18
*** openstackgerrit has joined #openstack-infra18:18
clarkbArxCruz: sudo: sorry, you must have a tty to run sudo <- I think that may be the problem18:19
fungiclarkb: ArxCruz: yes, ssh -t usually18:19
ArxCruzclarkb: oh... I remember that in my Fedora image I change the sudoers to accept tty...18:19
clarkbfungi: I am guessing paramiko is given a tty without asking on ubuntu but not on fedora?18:19
*** sarob has joined #openstack-infra18:19
fungiquite possible18:19
ArxCruzclarkb: Defaults    requiretty18:20
openstackgerritDavid Caro proposed a change to openstack-infra/jenkins-job-builder: Added config options to not overwrite jobs desc  https://review.openstack.org/5208018:20
*** wusui has quit IRC18:20
ArxCruzclarkb: so, you want me to make the change and resubmit or a new patch?18:20
*** wusui has joined #openstack-infra18:21
clarkbArxCruz: is that in the sudoers file? I don't think we want to edit the sudoers file18:21
clarkbArxCruz: instead paramiko should request a tty18:21
ArxCruzssh -t works ?18:21
clarkbArxCruz: yeah the equivalent of ssh -t in nodepool with paramiko18:21
ArxCruzclarkb: can't remember, but it seems paramiko doesn't have this support18:22
fungiArxCruz: http://www.lag.net/paramiko/docs/paramiko.SSHClient-class.html#invoke_shell18:22
*** hogepodge has joined #openstack-infra18:22
ArxCruzfungi: thanks I will check18:22
ArxCruzclarkb: what's the change # ?18:22
*** mestery_ is now known as mestery18:23
ArxCruzhave to go now, ttyl18:23
clarkbArxCruz: this will be a different change18:23
clarkbthe reverted change was to openstack-infra/config18:24
fungiArxCruz: though poking around more, you may just want the get_pty() method of teh session object18:24
ArxCruzclarkb: weird, it was just a yum install patch18:24
jeblairmordred: non-urgent: i left an idea on https://review.openstack.org/#/c/58770/ that i think you should evaluate18:24
mordredjeblair: yes. I agree with you - and I have thoughts in that area18:25
*** ArxCruz has quit IRC18:25
dprincepleia2: question for you on infra access to TripleO nodes. Do we currently have (or plan to have) a way for infra to publically shell into the slaves, etc?18:26
dprincepleia2: Is the only public access for infra managed through the overcloud API accessible for the nodepool?18:27
mordredjeblair: https://github.com/emontyoncrack/wookie1818:28
openstackgerritMonty Taylor proposed a change to openstack-infra/jeepyb: Do github last  https://review.openstack.org/5910718:28
openstackgerritMonty Taylor proposed a change to openstack-infra/jeepyb: Catch errors centrally, and bail out more  https://review.openstack.org/5910618:28
mordredonly found one tiny thing18:28
jeblairmordred: w00t18:28
clarkbis wookie18 testing the new manage-projects?18:29
mordredclarkb: yes18:29
mordredclarkb: github project created. gerrit project created. repo populated in gerrit. local mirror populated18:29
mordredacls applied18:29
mordredso - best I can tell, it's no worse than current manage-projects18:29
clarkbwoot18:30
mordredit MAY be more resilient - but until github craps us, it's tough to say with 100% surity18:30
clarkbmordred: you could do something sinister like put an extry for github.com in /etc/hosts18:32
clarkbso that "github" fails18:32
mordredclarkb: speaking of - review-dev.openstack.org is in /etc/hosts on review-dev and the entry is wrong18:32
clarkbthat is awesome18:32
jeblairmordred, clarkb, fungi: so once that is all merged in, should we also start requiring new projects have 2 patches: first create the repo, then jjb/zuul?18:33
jeblair(to avoid the git.o.o/zuul race)18:33
clarkbjeblair: so that I am double sure. That race is where jobs can be started by zuul prior to content being in git.o.o which is where jobs expect the repos to be18:34
mordredright18:35
jeblairclarkb: that's my understanding from your description.  :)  i think you said when that happens you have to rm-rf the git repo on zuul18:35
mordredultimatley having a salt trigger that says "run puppet on git.o.o, then on review.o.o, then on zuul.o.o would be neat18:36
clarkbjeblair: no, that race is different18:36
mordredbut we do not have that yet18:36
clarkbjeblair: zuul isn't cloning from git.o.o, it is cloning from review.o.o. The problem was when manage-projects created an empty repo on review.o.o without pushing any content to it18:36
jeblairclarkb: oooh.18:36
jeblairclarkb: that only happened because of github failures?18:37
jeblairclarkb: so, ideally, fixed by mordred's change as well?18:37
clarkbjeblair: and it seemed to be doing that because github would fail between those two steps causing manage-projects to move to the next project18:37
clarkbjeblair: yup18:37
clarkbI don't think we need split changes yet. Would be good to see if the situation is much better after mordred's change without extra hand holding18:37
mordredclarkb: ++18:37
jeblairclarkb, mordred, fungi: ok, so based on that; we should be able to handle single-change creation patches, and we expect mordreds fixes to fix all known creation problems?18:38
clarkbI still need to review it, but yes :)18:38
mordredclarkb: enjoy the review18:39
mordredclarkb: I split it into three pieces, just for you!18:39
sdagueso can someone restart the openstackrecheck bot? it's not clear to me that it took the code updates18:39
fungijeblair: clarkb: mordred: that's my take on it, yes18:40
fungisdague: sure, just a sec and i'll check it out before i restart it too18:40
sdaguefungi: yeh, any log files would be appreciated18:40
clarkbmordred: thank you, I will look at it now since it seems to be the topic of discussion18:40
*** sandywalsh has quit IRC18:40
clarkbjeblair: fungi: mordred: should I approve things if they look good or do we want to do more testing first?18:40
jeblairclarkb: i think we can aprv jeepyb changes18:41
fungiclarkb: agreed, though i'm re-reviewing the one which just got a new patchset18:41
*** rongze has quit IRC18:42
fungiyup, all three lgtm18:43
*** senk has quit IRC18:43
clarkbok, I will review quickly and hit +A if they look good (I expect them to)18:43
*** hdd has joined #openstack-infra18:44
openstackgerrityolanda.robla proposed a change to openstack-infra/jenkins-job-builder: Enable ignore_cache flag on jenkins_jobs.ini  https://review.openstack.org/5951518:45
*** osanchez has quit IRC18:47
*** osanchez has joined #openstack-infra18:49
jeblairsdague: do we actually want to optionally set configdrive, or just always use metadata?  https://review.openstack.org/#/c/59303/118:50
*** dolphm has joined #openstack-infra18:50
sdaguejeblair: so... it feels like we should be able to test both18:53
jeblairmordred: one more thing on https://review.openstack.org/#/c/59107/18:53
sdaguefungi: so I figured out the issue18:53
jeblairsdague: ok, but are we actually going to test both in the gate?18:53
mtreinishsdague: well the way we've do it now is metadata18:54
mtreinishand then have specific config drive tests18:54
fungisdague: oh?18:54
mtreinishthat way we test both in the gate18:54
openstackgerritSean Dague proposed a change to openstack-infra/config: move to elastic-recheck web console in share  https://review.openstack.org/5929418:54
openstackgerritSean Dague proposed a change to openstack-infra/config: use queries directory instead of file for elastic-recheck  https://review.openstack.org/5930118:54
sdaguemtreinish: actually, you changed the devstack default to config drive18:55
sdagueso we are always doing config drive18:55
sdaguewe could just revert that I suppose18:55
sdaguefungi: https://review.openstack.org/5930118:55
sdagueso I missed one place in config that the queries dir was specified18:55
*** luisg has quit IRC18:56
mtreinishsdague: I did? Ok, cool18:56
fungisdague: yeah, i just started looking at it... seems there are two processes running, one started on the 15th and the other started when that last change merged18:56
*** luisg has joined #openstack-infra18:56
sdaguefungi: interesting18:56
clarkbmordred: I agree with jeblair's -1. We should do that to avoid confusion18:56
fungi(or shortly thereafter... 17:08)18:56
*** osanchez has quit IRC18:56
*** luisg has quit IRC18:56
*** luisg has joined #openstack-infra18:57
*** loq_mac has joined #openstack-infra18:57
clarkbfungi: nice catch on the grenade defaults thing in layout.yaml18:57
*** sandywalsh has joined #openstack-infra18:58
*** sarob has quit IRC18:58
sdaguemtreinish: oh, sorry it was dprince18:58
clarkbjeblair: fungi: If I fix that really quick should we take the plunge and push that change through? note it does make the job names much longer18:58
*** sarob has joined #openstack-infra18:58
dkranzclarkb: Is it possible to anything like a regexp match in a logstach query for a "message?18:58
sdaguehttps://github.com/openstack-dev/devstack/commit/24f6efadbdef558655abc5f1052bb61ae87b55f018:59
clarkbdkranz: no regexes, but you can do globs and simpler things like that. http://lucene.apache.org/core/2_9_4/queryparsersyntax.html18:59
jeblairclarkb: if the system is busy (it is), that would be safer as a 2-part change (jjb, then zuul).18:59
dkranzclarkb: Thanks. I want to write something that will parse the whitelist in tempest and check if the log error is still ocurring.19:00
clarkbjeblair: good point, I will split the change in two19:00
jeblairclarkb: i think part 1 will need to keep the old jobs in order to pass the layout.yaml check :(19:00
clarkbjeblair: hmm I don't like that19:01
jeblairclarkb: alternative: stop puppet on zuul until the jobs exist.19:02
*** sarob has quit IRC19:02
*** derekh has quit IRC19:03
clarkbjeblair: that avoids the problem. But will make the process much cleaner19:03
clarkbjeblair: I can go with that19:03
sdaguefungi: so regardless, it won't work until - https://review.openstack.org/#/c/59301/2 is merged19:04
sdaguebecause it's config is sending it down the wrong path19:04
clarkbjeblair: should I keep it in one change and stop puppet on zuul, or would you prefer we do the git dance?19:04
fungisdague: agreed, i was just reviewing that one19:04
sdaguebut it is weird that there are zombie processes19:04
jeblairclarkb: i'm okay with one change and you stopping puppet on zuul while it goes in.19:05
fungisdague: shall i go ahead and kill the old one?19:06
jeblairit's a big change19:06
clarkbjeblair: ok thanks, that simplifies a bunch of stuff19:06
sdaguefungi: yep19:06
*** loq_mac has quit IRC19:06
*** jcooley_ has joined #openstack-infra19:06
sdagueit's just spitting out messages that it doesn't find errors in channel19:06
*** loq_mac has joined #openstack-infra19:06
*** che-arne has quit IRC19:07
fungisdague: kill both pids in that case?19:07
*** nati_ueno has quit IRC19:07
clarkbwow git wants me to gc my config repo19:08
*** loq_mac has quit IRC19:08
openstackgerritClark Boylan proposed a change to openstack-infra/config: Create two classes of d-g jobs.  https://review.openstack.org/5837019:08
clarkbfungi: jeblair mordred ^ addresses fungi's comment19:09
*** loq_mac has joined #openstack-infra19:09
openstackgerritA change was merged to openstack-infra/config: use queries directory instead of file for elastic-recheck  https://review.openstack.org/5930119:09
*** loq_mac has quit IRC19:09
sdaguefungi: yes19:10
*** loq_mac has joined #openstack-infra19:10
sdaguekill them all19:10
*** nati_ueno has joined #openstack-infra19:10
sdaguelet puppet sort it out :)19:10
sdaguethe graphs should come back 15 minutes after the merge as well, which will be nice19:11
*** rongze has joined #openstack-infra19:12
*** dstanek_afk has quit IRC19:12
clarkbjeblair: we seem to have lost mordred, I will just address your comment on 59107 then we can push those changes through19:12
*** dstanek_afk has joined #openstack-infra19:13
*** jcooley_ has quit IRC19:13
*** loq_mac has quit IRC19:13
*** jcooley_ has joined #openstack-infra19:13
openstackgerritClark Boylan proposed a change to openstack-infra/jeepyb: Do github last  https://review.openstack.org/5910719:14
*** jcooley_ has quit IRC19:14
clarkbjeblair: fungi ^19:14
mordredclarkb: hey - sorry - I was filling out my board candidate application so that people could properly nominate me19:14
*** fungi has quit IRC19:15
mordredclarkb: thanks!19:15
*** hashar has joined #openstack-infra19:15
ekarlsoany news on this mordred ? https://bugs.launchpad.net/openstack-ci/+bug/124256919:18
uvirtbotLaunchpad bug 1242569 in openstack-ci "manage-projects error on new project creation" [Critical,In progress]19:18
ekarlsoor clarkb19:18
mordredekarlso: we're working on it19:18
jeblairekarlso: if you read the bug you'll note the link to a change that fixes it19:18
mordredclarkb: btw - did you see the note that the new HP cloud region is now GA - so we shoudl be able to start using it19:18
clarkbclarkb: I saw your note to me about it, OWA is being cranky so no email today yet19:19
clarkbwow19:19
*** rongze has quit IRC19:20
*** fungi has joined #openstack-infra19:20
*** loq_mac has joined #openstack-infra19:20
fungiugh. he.net is suddenly experiencing major ipv6 packet loss19:21
ekarlsofyi, it's GA in both West and East :)19:21
ekarlso@ clarkb and mordred19:21
*** dhellmann_ is now known as dhellmann19:22
clarkbI have approved the jeepyb changes19:22
mordredneat.19:23
openstackgerritA change was merged to openstack-infra/jeepyb: Refactor manage-projects for readability  https://review.openstack.org/5908419:23
openstackgerritA change was merged to openstack-infra/jeepyb: Catch errors centrally, and bail out more  https://review.openstack.org/5910619:23
clarkbfungi: if you rereview https://review.openstack.org/#/c/58370/ I can stop puppet on zuul.o.o and babysit that process (though packet loss may make that hard)19:23
ekarlsocan you push that change also ? ;)19:23
openstackgerritA change was merged to openstack-infra/jeepyb: Do github last  https://review.openstack.org/5910719:23
fungiclarkb: you got it19:24
*** nati_ueno has quit IRC19:24
*** nati_ueno has joined #openstack-infra19:24
clarkbstopping puppet on zuul now19:25
fungipainful, painful packet loss but almost done clicking19:26
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Setup devstack-gate tests for Savanna  https://review.openstack.org/5731719:27
fungithe soap/xml thing or whatever the gerrit does becomes very flaky under packet loss19:27
*** annegentle has quit IRC19:28
fungiclarkb: approved (finally)19:28
clarkbwoot thank you19:28
*** loq_mac has quit IRC19:28
*** jcoufal has joined #openstack-infra19:29
fungisdague: fwiw, the old e-r daemon process was ignoring sigterm (which is probably why it hung around), but woke up on sighup and went away19:30
fungisdague: and the other process was gone already, replaced by one which started 10 minutes ago when your latest change got applied19:30
*** NobodyCam is now known as NobodyCam_afk19:31
openstackgerritA change was merged to openstack-infra/config: Create two classes of d-g jobs.  https://review.openstack.org/5837019:33
*** DinaBelova has quit IRC19:33
mtreinishclarkb: have you seen this failure before?: http://logs.openstack.org/05/58605/2/check/check-tempest-devstack-vm-neutron/b8761ca/console.html19:33
clarkbmtreinish: yes, that seems to happen when zuul is under a lot of load19:34
*** markmcclain has quit IRC19:35
clarkbapache can't serve the file for some reason. There is a bug to have d-g retry like we retry in gerrit git prep19:35
clarkbI Think someone is owrking on it19:35
sdaguefungi: cool19:36
*** dstanek_afk has quit IRC19:36
clarkbmtreinish: let me find it19:36
mtreinishclarkb: ok, cool thanks19:36
ekarlsohttps://bugs.launchpad.net/openstack-ci/+bug/1242569 could that be approve ?19:37
uvirtbotLaunchpad bug 1242569 in openstack-ci "manage-projects error on new project creation" [Critical,Fix committed]19:37
ekarlsoehm19:37
ekarlsowrong one !19:37
*** dolphm has quit IRC19:37
ekarlsohttps://review.openstack.org/#/c/56927/19:37
*** _david_ has joined #openstack-infra19:38
sdaguefungi: are there any other log messages on the box?19:38
clarkbmtreinish: https://bugs.launchpad.net/openstack-ci/+bug/1255353 is the bug. I will update it to note that zuul has the same problem19:38
uvirtbotLaunchpad bug 1255353 in openstack-ci "during gate job  'error: Unable to get pack index https://review.openstack.org/p/openstack-infra/devstack-gate/objects/pack/pack-*.idx'" [High,Triaged]19:38
sdagueit still looks like the bot isn't parsing things19:38
mtreinishclarkb: thanks, I'll use that for recheck then19:39
clarkbekarlso: once we have had a chance to double check that all of the new jeepyb stuff is in place we can review the new project changes19:40
clarkbsdague: have a particular change you expected it to report on? I can use that to find relevant logs19:41
ekarlsonice clarkb :)19:42
*** dolphm has joined #openstack-infra19:42
fungisdague: you mean as far as logging why it wasn't killed? i think it may have not had a write lock on the log by then... trying to figure out if maybe it was still writing to one of the already-rotated logs instead19:44
sdagueclarkb: hmmm... well it's reported twice, but those might actually be unknown issues19:44
sdagueclarkb: ok, let me let it sit for a bit and see19:47
fungisdague: yeah, interestingly the old daemon seemed to have closed and reopened yesterday's log file and continued overwriting that today, but since it doesn't log its pid in the messages it's hard to confirm for certain that's what i'm looking at19:48
*** rongze has joined #openstack-infra19:49
sdaguefungi: odd19:49
sdagueso we only have the new bot running now, right?19:49
fungisdague: yes19:49
fungisdague: one process, which started at 19:1919:50
clarkbmordred: /opt/jeepyb is up to date on review.o.o but pip freeze reports an old version. Did you ahve to do anyhting on review-dev to make it happy?19:50
*** beagles is now known as beagles_brb19:50
clarkbmordred: looks like there are a bunch of jeepyb's installed to /usr/local/lib/python2.7/dist-packages19:51
fungiclarkb: i've assumed those are from before the entrypoints rework in pbr19:52
mordredclarkb: me goes to look19:52
clarkbjenkins01 has the new jenkins jobs, waiting for jenkins02 now19:53
*** rongze has quit IRC19:53
*** rwsu has quit IRC19:55
notmynamewhere's the bot config file (to set what alerts in channel)?19:56
*** rwsu has joined #openstack-infra19:56
*** yolanda has quit IRC19:58
clarkbnotmyname: openstack-infra/config/modules/gerritbot/gerritbot_channel_config.yaml19:59
clarkbjenkins01 and jenkins02 have the new d-g jobs, but not jenkins.o.o looking at that now20:00
*** annegentle has joined #openstack-infra20:00
clarkbrunning puppet by hand there to see what JJB is doing20:02
*** nati_uen_ has joined #openstack-infra20:02
clarkbpuppet runs without execing jjb, going to run it by hand20:03
notmynameis the gerritbot healthy? it's configured to report on something that I'm not seeing reports for.20:03
clarkbnotmyname: what is it not reporting on? the -2 events are known to not work20:03
notmynamehmm...or maybe it's just slow20:06
*** nati_ueno has quit IRC20:06
*** dprince has quit IRC20:06
*** loq_mac has joined #openstack-infra20:06
*** MarkAtwood has joined #openstack-infra20:06
*** mfer has quit IRC20:07
*** loq_mac has quit IRC20:07
*** loq_mac has joined #openstack-infra20:07
*** ^d has quit IRC20:08
clarkbfungi: jeblair: mordred: the way puppet copies files into /etc/jenkins_jobs/configs allows for old files to remain present20:09
clarkband those stale files may break JJB20:09
*** ^d has joined #openstack-infra20:09
*** sarob has joined #openstack-infra20:09
*** sarob has quit IRC20:09
*** sarob has joined #openstack-infra20:09
clarkbI am going to remove the stale files on jenkins.o.o and rerun JJB20:09
clarkbthen clean up jenkins01 and jenkins0220:10
*** mfer has joined #openstack-infra20:10
*** agordeev has joined #openstack-infra20:10
*** vkozhukalov has joined #openstack-infra20:10
vkozhukalovhere20:10
*** notmyname has quit IRC20:10
devanandamordred / clarkb / dansmith - is tripleo-ci still active?20:11
fungiclarkb: oh, should we be directly managing that config directory via puppet instead?20:11
*** fbo_away is now known as fbo20:12
clarkbfungi: we are, but I think the way we are enforcing the recursive config doesn't remove files that don't exist on the puppet master20:12
fungigot it20:12
devanandawoops, wrong dan -- meant to ping dan prince. who's not here?20:13
*** notmyname has joined #openstack-infra20:14
*** ruhe is now known as ruhe_20:15
*** ruhe_ is now known as ruhe20:15
clarkbdevananda: I have no idea20:16
devanandaclarkb: bah. you know everything :)20:16
*** esker has joined #openstack-infra20:21
*** mrmartin has joined #openstack-infra20:23
*** ruhe_ has joined #openstack-infra20:24
clarkbfungi: jeblair: mordred: All three jenkins masters should be happy now, I will propose a puppet change to fix this via puppet in a bit20:24
clarkbalso going to spot check that the jobs went in properly20:24
mordredclarkb: awesome20:24
*** alexpilotti has quit IRC20:25
clarkbthey look correct to me20:26
clarkbgoing to grab lunch quickly then continue through the job stuff20:26
*** ruhe has quit IRC20:26
clarkb(puppet is still not running on zuul.o.o_20:26
openstackgerritMonty Taylor proposed a change to openstack-dev/pbr: Trap failures due to old easy_install  https://review.openstack.org/5953220:27
clarkbmordred: are you working to fix that pbr bug on windows?20:27
mordredclarkb: no - I just got one reported in #openstack-dev and the fix was easy so I popped it in20:27
clarkbI think that fix should go into the next release of pbr. Also, we can in theory run tests on windows, can't believe I just said that20:28
mordred(like, less work than writing down the details in a bug)20:28
mordredI agree20:28
mordredI sort of think that running some tests on windows might not be a _terrible_ idea for things like pbr - but it also involves more things that I don't understand, so I keep deferring actually thinking about it20:28
*** dcramer_ has quit IRC20:29
fungiwe can run tests on windows? not in our infra presumably20:29
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Mention errors in service logs for irc messages  https://review.openstack.org/5953420:29
*** nati_uen_ has quit IRC20:30
*** nati_ueno has joined #openstack-infra20:30
openstackgerritMonty Taylor proposed a change to openstack-dev/pbr: Trap failures due to old easy_install  https://review.openstack.org/5953220:32
mordredfungi: clouds have windows guests (me feels unclean saying that)20:33
fungii'm sure they do... do we manage those with puppet? nodepool? what's the thought there? (not that i'm seriously entertaining the idea, but...)20:34
fungiand then there's the question of whether some member company is stepping up to provide free licenses for that operating system so we could legally use it20:35
mgagnefungi: somewhere in my TODO list is puppetizing a windows jenkins slave...20:36
fungifrightening20:36
mgagnefungi: I'm looking forward to it as much as you20:36
fungii have no doubt20:36
mgagnefungi: the challenge is mainly with packages/dependencies20:36
*** hogepodge has quit IRC20:37
*** beagles_brb is now known as beagles20:39
*** Ryan_Lane has joined #openstack-infra20:41
clarkbfungi puppet and I dont think we worry about license fees but could be wrong20:43
jeblairmordred, fungi, clarkb: I'm going to self-approve https://review.openstack.org/#/c/58888/1 unless someone else reviews it in the next 5 mins or so.20:43
jeblairit's blocking work on a production bug in nodepool, and i'm pretty sure it's older than many changes that have been reviewed today.  :(20:44
*** denis_makogon_ has joined #openstack-infra20:44
*** Ryan_Lane has quit IRC20:45
*** denis_makogon_ is now known as denis_makogon20:46
mordredjeblair: +A20:47
*** branen has joined #openstack-infra20:47
jeblairclarkb: yeah, we worry about license fees.  windows is not a target development platform for openstack.  i don't expect that we'd consider spinning up windows test nodes unless that changes.20:48
jeblairmordred: thx20:49
*** rongze has joined #openstack-infra20:49
*** sarob has quit IRC20:53
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Remove remote-directory from publishing in api-jobs  https://review.openstack.org/5953820:53
*** sarob has joined #openstack-infra20:53
openstackgerritBen Nemec proposed a change to openstack-dev/hacking: Add noqa support for H201 (bare except)  https://review.openstack.org/5733420:53
*** emagana has joined #openstack-infra20:54
*** rongze has quit IRC20:54
openstackgerritA change was merged to openstack-infra/nodepool: Improve image logging  https://review.openstack.org/5888820:56
*** sarob has quit IRC20:57
*** marun has joined #openstack-infra20:58
*** markmcclain has joined #openstack-infra20:59
openstackgerritA change was merged to openstack/requirements: Align projects list with mirror-needing projects  https://review.openstack.org/5919621:00
fungijust over three hours after it was approved ^ (things are starting to pile up in the gate again)21:00
fungiactually 3.5 hours after approval21:01
*** ^d is now known as ^demon|lunch21:01
*** alcabrera has quit IRC21:01
jeblairbug 1242569 seems to be happening a lot21:03
uvirtbotLaunchpad bug 1242569 in openstack-ci "manage-projects error on new project creation" [Critical,Fix committed] https://launchpad.net/bugs/124256921:03
jeblairga21:03
jeblairbug 125389621:03
uvirtbotLaunchpad bug 1253896 in neutron "test_minimum_basic_scenario fails with SSHException: Error reading SSH protocol banner" [High,In progress] https://launchpad.net/bugs/125389621:03
*** herndon_ has joined #openstack-infra21:05
mordredjeblair: oh good. I was worried that 1242569 was an issue21:05
*** ^demon|lunch has quit IRC21:05
jeblairmordred: ETOOMANYTABS21:05
*** ^demon|lunch has joined #openstack-infra21:06
lifelessdevananda: or here?21:08
lifeless:P21:08
devanandalifeless: all teh roomz!21:08
devanandalifeless: here is fine, too21:08
lifelessdevananda: actually, I need to power IRC off for a little21:08
lifelessdevananda: I need to move my IB card from the old server to the new server and do basic physical setup. I'll be 20-30m, then back.21:08
lifelessciao!21:08
*** lifeless has quit IRC21:08
devanandalifeless: question is vaguely around which parts of the testing story are reusable for ironic, what state those are in, and how to plug in other folks (eg some mirantis devs) who want to contribute to ironic CI21:09
devanandabah21:09
devanandahe'll be back21:09
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Mention errors in service logs for irc messages  https://review.openstack.org/5953421:10
*** loq_mac has quit IRC21:10
*** bpokorny has joined #openstack-infra21:11
*** SergeyLukjanov is now known as _SergeyLukjanov21:13
*** loq_mac has joined #openstack-infra21:14
*** nicedice has joined #openstack-infra21:15
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Add zuul rack-refs cron  https://review.openstack.org/5954321:16
*** branen_ has joined #openstack-infra21:16
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Add zuul pack-refs cron  https://review.openstack.org/5954321:16
*** jpeeler1 has joined #openstack-infra21:18
*** jcooley_ has joined #openstack-infra21:19
*** branen has quit IRC21:20
*** _david_ has quit IRC21:20
*** bpokorny2 has quit IRC21:20
*** dkranz has quit IRC21:20
*** hdd has quit IRC21:20
*** jpeeler has quit IRC21:20
clarkbjeblair: fungi: ready for me to start puppet on zuul.o.o?21:20
fungiclarkb: any time is fine by me21:21
jeblairclarkb: wfm21:21
clarkbstarting21:21
*** loquacities has joined #openstack-infra21:22
*** dcramer_ has joined #openstack-infra21:22
mordredO M G my review queue is evil21:23
openstackgerritClark Boylan proposed a change to openstack-infra/config: Purge the JJB config dir.  https://review.openstack.org/5955021:23
clarkbjeblair: fungi: mordred: ^ that will fix the JJB config thing I ran into earlier21:24
mordredclarkb: we probably want to do that everywhere we do recurse => true21:24
clarkbmordred: probably. I can do a quick grep for recurse => true and see21:25
*** MarkAtwood has quit IRC21:25
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Add zuul pack-refs cron  https://review.openstack.org/5954321:25
*** jcooley_ has quit IRC21:25
*** loq_mac has quit IRC21:25
*** jcooley_ has joined #openstack-infra21:26
*** sarob has joined #openstack-infra21:26
*** jcooley_ has quit IRC21:26
*** dcramer_ has quit IRC21:26
*** _david_ has joined #openstack-infra21:26
clarkbmordred: a few of them look safe, others definitely aren't21:27
*** hdd has joined #openstack-infra21:27
*** dkranz has joined #openstack-infra21:27
openstackgerritMonty Taylor proposed a change to openstack-dev/pbr: Use wheels for installation  https://review.openstack.org/4880321:27
*** dolphm has quit IRC21:30
*** dcramer_ has joined #openstack-infra21:30
openstackgerritA change was merged to openstack-infra/config: Purge the JJB config dir.  https://review.openstack.org/5955021:30
*** smarcet has left #openstack-infra21:32
openstackgerritClark Boylan proposed a change to openstack-infra/config: Use purge option where appropriate.  https://review.openstack.org/5955321:33
clarkbmordred: ^ those seem safe. Some of the others do not (they are used to apply permissions but not actually change files in many cases)21:33
*** jpeeler1 is now known as jpeeler21:33
*** jpeeler has quit IRC21:33
*** jpeeler has joined #openstack-infra21:33
clarkbzuul has the new config21:34
mordredclarkb: fantastic21:34
*** che-arne has joined #openstack-infra21:36
*** dolphm has joined #openstack-infra21:38
*** Ryan_Lane has joined #openstack-infra21:41
clarkbhttp://logs.openstack.org/33/59533/1/check/check-swift-devstack-vm-functional-default/5c62b7a/logs/devstack-gate-setup-host.txt.gz any idea why the pydistutils.cfg file cannot be created? that job failed and I am trying to figure out why21:41
clarkbhttp://logs.openstack.org/33/59533/1/check/check-swift-devstack-vm-functional-default/5c62b7a/logs/devstack-gate-setup-workspace-new.txt.gz is where the actual failure occured21:42
*** pabelanger has quit IRC21:43
openstackgerritAnne Gentle proposed a change to openstack-infra/config: Changing the output directory that gets picked up for the API Quick Start  https://review.openstack.org/5955621:44
clarkbgah I think I see the problem21:45
*** Ryan_Lane has quit IRC21:46
*** vkozhukalov has quit IRC21:47
portanteclarkb: have you seen this Syntax Error before fetching stuff from raw.github.com? http://logs.openstack.org/00/58500/4/check/check-swift-devstack-vm-functional/252a8d2/console.html21:47
openstackgerritClark Boylan proposed a change to openstack-infra/config: Set BRANCH_OVERRIDE in swift functional tests.  https://review.openstack.org/5955721:47
*** weshay has quit IRC21:47
clarkbjeblair: mordred: fungi: ^ can that get a quick review? I am going to make the change by hand on the jenkins masters really quick21:48
*** vkozhukalov has joined #openstack-infra21:48
notmynameclarkb: what does that do?21:48
*** loquacities has quit IRC21:48
clarkbnotmyname: we did a massive overhaul of the jobs to make them handle releases more easily21:49
jeblairclarkb: +2 aprv at will21:49
clarkbnotmyname: that allows us to override the branch for branch specific tests, but I forgot to put it in on the swift tests...21:49
fungiclarkb: safe to approve?21:49
notmynameclarkb: ok. so all non-master tests would fail?21:49
clarkbnotmyname: currently all of them are failing21:49
clarkbfungi: ya I approved21:50
fungiclarkb: okay, well lgtm21:50
*** loq_mac has joined #openstack-infra21:50
portanteclarkb: nevermind, https://bugs.launchpad.net/devstack/+bug/125707921:50
uvirtbotLaunchpad bug 1257079 in devstack "Fails occured with the retieval of devstack/files/get-pip.py in gate" [Undecided,New]21:50
*** sarob has quit IRC21:51
*** rongze has joined #openstack-infra21:51
*** adalbas has quit IRC21:51
*** sarob has joined #openstack-infra21:51
notmynameclarkb: is that ^^ the bug to use for rechecks?21:51
clarkbnotmyname: no, that is a different bug21:51
clarkbnotmyname: let me fix this really quick to limit damage then I will write a bug21:52
*** agordeev has quit IRC21:52
*** lifeless has joined #openstack-infra21:53
lifelessdevananda: ok, so hi21:53
devanandalifeless: question is vaguely around which parts of the testing story are reusable for ironic, what state those are in, and how to plug in other folks (eg some mirantis devs) who want to contribute to ironic CI21:53
*** vkozhukalov has quit IRC21:54
lifelessok so21:54
openstackgerritA change was merged to openstack-infra/config: Set BRANCH_OVERRIDE in swift functional tests.  https://review.openstack.org/5955721:54
lifelesswe have several different parallel efforts21:54
lifelesslets talk one at time, because though they have common bits, they are different enough that talking all at once is confusing21:55
lifelessthere's the emulated baremetal stuff21:55
lifelessand there's actual baremetal stuff21:55
*** sarob has quit IRC21:55
lifelessemulated baremetal is coming along very well21:56
*** dizquierdo has joined #openstack-infra21:56
lifelesshttps://etherpad.openstack.org/p/tripleo-test-cluster21:56
lifelessthe broker is written21:56
lifelessthere is a node definition in nodepool and jobs in the experimental queue21:56
*** rongze has quit IRC21:57
clarkbI think it should be mostly better now. Writing up a bug.21:57
clarkbany tests starting after this point in time should pass21:57
lifelesswe're within a week or so of spinning up some test environment machines and being able to start exercising things21:57
lifelessthat will be a little bad initially as the tripleo-gate instances would be on the old grizzly cloud (tripleo CD cloud is too unstable for infra as yet)21:57
lifelessbut we should be able to get up to silent, perhaps even gate with that21:58
lifelesserm21:58
lifelesssilent perhaps even check21:58
lifelessthough we'll need multiregion, but thats only a few weeks away before the RH region comes online21:58
*** sarob has joined #openstack-infra21:58
*** dolphm has quit IRC21:59
devanandalifeless: have the link to the nodepool bits?21:59
lifelessdevananda: git clone https://github.com/openstack-infra/nodepool, IIRC22:00
clarkbnotmyname: https://bugs.launchpad.net/openstack-ci/+bug/125708522:00
lifelessdevananda: Exactly how we map Ironic specific tests to this environment is extremely flexible22:00
uvirtbotLaunchpad bug 1257085 in openstack-ci "Swift functional tests fail to configure devstack." [Undecided,New]22:00
devanandalifeless: also, iiuc, all that is unrelated to the discussions i/we had with -infra folks ~6mo ago about how to get nova-bm tested inside of their existing cloud regions22:00
lifelessdevananda: we could just use devstack-gate nodes for you, for instance, if we install the broker client there22:00
lifelessdevananda: AFAIK all those discussions were predicated on hope.22:01
*** dolphm has joined #openstack-infra22:01
lifelessdevananda: which about 4 months ago we burnt to the ground.22:01
devanandalifeless: has that work been abandoned in favor of the separate-cloud-style stuff?22:01
devanandalifeless: what finally killer her?22:01
lifelessdevananda: I'm not aware of any other efforts by -infra in this area that are suitable for Ironic functional tests22:01
lifelessnested virt 30m boots22:02
lifelessLXC can't run tgtd22:02
devanandaah22:02
lifelesseither is a big issue; in combination boom22:02
lifelessso in principle we don't care where Ironic runs - it can run in the standard -gate image22:02
Alex_Gaynordhellmann: +1 from me22:02
Alex_Gaynor(that is, +1 to ignoring my -0 :-))22:03
lifelessproblem is that a qemu VM you test with, will take forever to get up to liveness to validate22:03
devanandayes22:03
lifelessof course, we were testing rather bigger immages that the bare minium you could get away with22:03
*** sarob has quit IRC22:03
lifelesseven still...22:03
devanandatoday's devstack-gate is running with 1 layer of nesting22:04
*** markmcclain has quit IRC22:04
devanandabut that isn't seeing 30m boots22:04
*** mfer has quit IRC22:04
lifelessit's running cirros22:04
devanandawe should, in theory, be able to do ironic functional tests with no more nesting than devstack-gate has today22:05
lifelessnot Ubuntu or Fedora22:05
devanandaright22:05
devanandasufficient for ironic. not for tripleo22:05
lifelesssure22:05
lifelessso - like I say, I'm not aware of anyone pushing on an effort to do that w/Ironic or Nova BM22:05
devanandagotcha22:05
lifelessI agree that it is in principle doable.22:05
devanandapleia2: were you looking into ^ at one point in time?22:06
devanandalifeless: so the question seems to be, is it worthwhile to do that?22:06
devanandagiven the progress ya'll are making on separate clouds for tripleo testing22:06
pleia2openstack on qemu is unusable22:07
lifelesspleia2: devananda is querying just the bm deployment layer22:07
devanandais there a benefit to getting ironic tested in a devstack-gate-like environment, with cirros, and the ssh driver22:07
lifelesspleia2: so booting cirros via pxe22:07
*** mrmartin has quit IRC22:07
jeblairdevananda: i think it makes sense if you want ironic to be tested in a gating environment by icehouse2.22:08
pleia2ah, I see22:08
devanandajeblair: which I do22:08
lifelessthere's a much smaller dependency chain to it22:08
openstackgerritA change was merged to openstack-dev/hacking: Add noqa support for H201 (bare except)  https://review.openstack.org/5733422:08
devanandajeblair: well. voting. smoke check. not gating yet b/c it's not integrated yet.22:08
*** sarob has joined #openstack-infra22:08
jeblairdevananda: right; though you are free to gate ironic on such tests.22:08
devanandalifeless: functional testing in the gate pipeline (even if it doesn't include actual physical machines) is required for ironic in a fairly short time frame22:09
devanandajeblair: right22:09
jeblairdevananda: (it would be asymmetric, so should be done with care, but anyway, that's a side track)22:09
devanandajeblair: we're 1 patch away from having devstack / tempest tests in our pipeline: https://review.openstack.org/#/c/5391722:10
devanandaand there's a similar patch for tempest's experimental pipe22:10
lifelessso, I think you should get devstack-gate up regardless22:10
*** dkranz has quit IRC22:10
jeblairdevananda: i'm excited and optimistic about the tripleo-based testing but would not want too many things to depend on such a large unknown at this point; and i'd like the tripleo work to proceed without the kind of pressure this would add22:10
jeblairlifeless: ++22:10
lifelessthere's a short path to it, and if we can do something better in future cool, but why add risk we don't need to22:10
devanandaawesome22:10
*** dolphm has quit IRC22:11
*** ArxCruz has joined #openstack-infra22:12
devanandapleia2: have a few minutes to brain dump where you got to with adding nova-bm to d-g?22:13
pleia2devananda: sure22:13
pleia2devananda: here or would you prefer a call?22:14
*** ^demon|lunch is now known as ^d22:14
devanandapleia2: which ever is easier22:15
mordreddhellmann: did we not ever release oslo.messaging 1.2 ?22:15
pleia2devananda: quick call might be easiest, terms get confusing :)22:15
openstackgerritClark Boylan proposed a change to openstack-infra/config: Use purge option where appropriate.  https://review.openstack.org/5955322:15
*** fifieldt has joined #openstack-infra22:16
clarkbsdague: dhellmann: for gating WSME/Pecan on devstack/tempest jobs. You can now overload the branch-designator to include a unique string for wsme/pecan22:16
mgagnewith hpcloud, how can I make use of the API key? (in place of password)22:16
clarkbsdague: dhellmann: that will give you an otherwise identical job to all of the other jobs, but the name will be different so zuul won't run them all in the same gate22:16
*** geekinutah has joined #openstack-infra22:18
*** zul has quit IRC22:18
*** zul has joined #openstack-infra22:20
*** pabelanger_ has quit IRC22:21
*** pabelanger has joined #openstack-infra22:21
*** dkliban has quit IRC22:25
*** dizquierdo has quit IRC22:27
*** rwsu has quit IRC22:27
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Make job names shorter  https://review.openstack.org/5956422:28
jeblairclarkb: you may be interested in that ^22:30
*** sarob has quit IRC22:30
ruhe_jeblair, should this https://review.openstack.org/#/c/57317 be modified to comply with new naming style? ^22:32
sdaguemordred: apparently yelling on twitter about broken openstack things get's johnathan to email you :)22:33
mordredsdague: nice!22:33
mordredsdague: what a great and open process22:33
sdagueI put you into that email thread as well22:33
mordredplease do22:33
*** ruhe_ is now known as ruhe22:34
openstackgerritClark Boylan proposed a change to openstack-infra/config: Gate current clients on stable branches  https://review.openstack.org/4193122:35
clarkbmordred: ^ updated that change to be more like what we are doing with the other jobs22:36
clarkbjeblair: looking22:36
*** loq_mac has quit IRC22:38
*** mriedem has quit IRC22:39
clarkbjeblair: I like that. I will rebase 41931 on it probably22:39
*** CaptTofu has quit IRC22:40
*** CaptTofu has joined #openstack-infra22:40
mordredclarkb: looks sexy22:42
*** rwsu has joined #openstack-infra22:43
clarkbI am going to try and do some reviews now22:45
*** MarkAtwood has joined #openstack-infra22:45
clarkbjeblair: fungi: mordred: once the d-g job name scheme settles we should do a massive job cleanup on the jenkins masters22:46
*** _david_ has quit IRC22:49
*** dims has quit IRC22:51
jeblairclarkb: yes22:53
*** rongze has joined #openstack-infra22:53
*** fbo is now known as fbo_away22:53
*** thomasem has quit IRC22:53
*** leifmadsen has joined #openstack-infra22:54
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Make job names shorter  https://review.openstack.org/5956422:56
*** bpokorny has quit IRC22:56
*** rongze has quit IRC22:58
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Make job names shorter  https://review.openstack.org/5956422:59
*** leifmadsen has quit IRC22:59
fungiclarkb: it's way, way, way easier now that jjb has an option for that22:59
clarkbjeblair: I am going to hold off on additional d-g change until we can get your rename in23:00
*** flaper87 is now known as flaper87|afk23:00
jeblairclarkb: k.  i think my patch may need a similar puppet stoppage23:00
*** denis_makogon has quit IRC23:00
*** denis_makogon has joined #openstack-infra23:01
*** pete5 has quit IRC23:02
clarkbjeblair: yup23:02
clarkbjeblair: https://review.openstack.org/#/c/54331/9/nodepool/provider_manager.py wouldn't it be better to pass pool in where addPublicIP is called?23:03
*** sarob has joined #openstack-infra23:04
jeblairclarkb: yes23:05
clarkbI am leaving a review23:05
*** dolphm has joined #openstack-infra23:08
*** ryanpetrello has quit IRC23:08
*** denis_makogon has quit IRC23:09
*** jcooley_ has joined #openstack-infra23:09
*** esker has quit IRC23:09
clarkbjeblair: jenkins still isn't happy with the rename change23:11
jeblairclarkb: thx; will fix23:12
openstackgerritYathiraj Udupi proposed a change to openstack/requirements: Added a requirement to COIN PULP LP Modeler module  https://review.openstack.org/5344623:13
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Make job names shorter  https://review.openstack.org/5956423:16
jeblairclarkb, mordred, fungi: fyi i'm doing some testing with a manually created job called 'dev-nova-pep8' on jenkins0123:18
fungiokay23:18
*** jergerber has quit IRC23:18
*** hashar has quit IRC23:18
fungii23:19
fungier23:19
fungii'll refrain from deleting it23:19
jeblairfungi: i won't be upset if it is deleted; i know what i'm getting into.  :)23:19
*** dkliban has joined #openstack-infra23:20
ekarlsoany chance to get this merged now ? https://review.openstack.org/#/c/56927/23:20
clarkbekarlso: mordred was looking into why newer jeepyb wasn't getting installed onreview.o.o23:21
clarkbekarlso: once we hear back on that that things are good we can start approving new project changes23:21
clarkblooks like the version is newer now than what it was before23:22
clarkbmordred: did you fix that? what did you have to do?23:22
clarkbjeblair: fungi: mordred: we can guinea pig with https://review.openstack.org/#/c/56107/1 to see it work on review.o.o23:24
fungiclarkb: checking it out now, but looks like it's probably a safe one to test with23:26
clarkbjeepyb version reported by pip on review.o.o is current. I am guessing I missed what mordred did to fix that in scrollback?23:28
fungiclarkb: i don't think he said what he did23:29
fungi(or else it self-corrected)23:29
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add support for installing the web dashboard  https://review.openstack.org/5928923:31
*** jcooley_ has quit IRC23:32
*** Ryan_Lane has joined #openstack-infra23:33
*** yamahata_ has quit IRC23:34
*** herndon_ has quit IRC23:35
*** krotscheck has quit IRC23:35
*** FallenPegasus has joined #openstack-infra23:36
*** FallenPegasus has quit IRC23:36
*** Ryan_Lane has quit IRC23:37
*** sandywalsh has quit IRC23:37
*** jcoufal has quit IRC23:38
clarkbjeblair: https://review.openstack.org/#/c/59564/4/modules/openstack_project/files/jenkins_job_builder/config/python-bitrot-jobs.yaml I don't see the change in projects.yaml to change the branch names to prefix with -23:38
clarkboh wait, we instantiate the templates in the job group23:38
*** yamahata_ has joined #openstack-infra23:39
jeblairclarkb: all set?23:39
*** krotscheck has joined #openstack-infra23:39
clarkbjeblair: I think so. I gave it a +223:39
*** marun has quit IRC23:40
*** jerryz has joined #openstack-infra23:44
*** sarob has quit IRC23:45
*** sarob has joined #openstack-infra23:46
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Add a bare precise node to nodepool  https://review.openstack.org/5957223:47
clarkbjeblair: can you review https://review.openstack.org/#/c/56107/ and approve (remove the -2) if it looks good to you?23:48
clarkbnow for some hpcloud new region nodepooling /me digs up that change23:48
jeblairclarkb: you babysitting it?23:48
clarkbjeblair: I can23:49
jeblairclarkb: aprvd23:49
fungii will also babysit that if nedded23:49
clarkbty23:49
fungier, needed23:49
*** ruhe has quit IRC23:50
clarkbfungi: "nedded": verb, to be greeted with a "Hey Diddly Ho neighbor"23:50
*** emagana has quit IRC23:50
fungihidey-ho, neighbor!23:50
fungi[you have been nedded]23:51
clarkbmordred: can you confirm on https://review.openstack.org/#/c/56260/ that we should go ahead and start trying the new region?23:51
clarkbmordred: maybe mark it ready for review23:52
*** sandywalsh has joined #openstack-infra23:53
mordredclarkb: done23:53
clarkbwoot, I will approve that once I am done babysitting the new project change23:54
fungiclarkb: jeblair: https://review.openstack.org/#/c/59564/4/modules/openstack_project/files/jenkins_job_builder/config/python-bitrot-jobs.yaml we're ending up with job names like "periodic-nova-python27havana" (threw away a hypnen... intended?)23:54
*** bpokorny has joined #openstack-infra23:55
*** rongze has joined #openstack-infra23:55
fungi(hyphen too. damned fingers)23:55
clarkbfungi: nope, that was my question too. see the end of that file, we set the parameter values there and not in projecs.yaml23:56
fungiso right23:56
*** thedodd has quit IRC23:57
*** UtahDave has quit IRC23:57
fungibranch-name: '-havana'23:57
fungiignore me23:57
fungii should hold my questions until the end of the presentation23:57
openstackgerritJames E. Blair proposed a change to openstack-infra/elastic-recheck: Update job names  https://review.openstack.org/5957323:58
*** CaptTofu has quit IRC23:58
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: raise JenkinsJobsException instead of Exception  https://review.openstack.org/5639723:58
*** CaptTofu has joined #openstack-infra23:58
jeblairsdague, mtreinish, jog0: what's the deal with the query bug 1250168 ? it doesn't seem like including a job name in a query is a good idea..?23:58
uvirtbotLaunchpad bug 1250168 in nova "gate-tempest-devstack-vm-neutron-large-ops is failing" [High,Fix committed] https://launchpad.net/bugs/125016823:58
jeblairs/the query bug/the query for bug/23:59

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