Wednesday, 2017-09-13

openstackgerritzhangyangyang proposed openstack-infra/zuul master: change assert(Not)Equals to assert(Not)Equal  https://review.openstack.org/50350102:12
openstackgerritzhangyangyang proposed openstack-infra/nodepool master: change assert(Not)Equals to assert(Not)Equal  https://review.openstack.org/50350302:19
openstackgerritzhangyangyang proposed openstack-infra/zuul master: change assert(Not)Equals to assert(Not)Equal  https://review.openstack.org/50350702:49
openstackgerritzhangyangyang proposed openstack-infra/zuul master: change assert(Not)Equals to assert(Not)Equal  https://review.openstack.org/50350102:52
*** robled has quit IRC03:05
*** robled has joined #zuul03:07
openstackgerritzhangyangyang proposed openstack-infra/zuul master: change assert(Not)Equals to assert(Not)Equal  https://review.openstack.org/50350103:36
*** yolanda has quit IRC03:50
*** robled has quit IRC03:58
*** robled has joined #zuul03:59
*** robled has joined #zuul03:59
*** robled has quit IRC04:00
*** robled has joined #zuul04:01
*** robled has joined #zuul04:01
*** bhavik1 has joined #zuul05:00
*** hashar has joined #zuul06:01
*** xinliang has quit IRC07:15
*** xinliang has joined #zuul07:28
*** xinliang has joined #zuul07:28
*** electrofelix has joined #zuul08:36
*** bhavik1 has quit IRC09:32
*** hashar is now known as hasharAway09:45
*** yolanda has joined #zuul11:09
*** jkilpatr has joined #zuul13:19
*** hasharAway has quit IRC13:49
*** yolanda has quit IRC14:58
*** pabelanger_ is now known as pabelanger15:05
*** jkilpatr has quit IRC15:05
*** yolanda has joined #zuul15:15
dmsimardhttps://review.openstack.org/#/c/502147/15:24
dmsimardnot sure why the v3 jobs in ^ are returning POST_FAILURES15:27
openstackgerritMerged openstack-infra/nodepool feature/zuulv3: Fix typo in variable name  https://review.openstack.org/50342215:29
jeblairdmsimard: i don't see those builds on disk; we must have not had 'keep' enabled when they ran15:30
jeblairdmsimard: i'll need to recheck them15:30
dmsimardjeblair: sure, go ahead, it's part of the multi distro work -- all of them should be passing now that the ci-mirror.sh thing merged for base-test. After they all pass we'll be able to go ahead and merge the base ci-mirror.sh review15:31
clarkbmordred: I am beginning to wonder if your shade tests aren't killing themselves somehow?15:31
clarkbis exit called anywhere?15:32
openstackgerritTristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: Add ref_url column to the buildset reporter table  https://review.openstack.org/50371415:38
pabelangerdmsimard: I think the failure on 502147 is happening on logs.o.o, since the last post playbook is return exit code 215:38
dmsimardpabelanger: how do you tell ? the end of the logs are truncated15:39
pabelangerdmsimard: I looked on executor debug.log15:40
dmsimardpabelanger: well that's convenient when you can do that :)15:40
pabelangerdmsimard: so, possible rsync failed for some reason15:40
pabelangerdmsimard: ya, only way to fix that would be have zuul directly copy logs outside of ansible15:41
dmsimardlet's see what the recheck ends up doing15:41
pabelangeryes15:41
pabelangerdmsimard: maybe it is zuul_return that is failing for some reason, because I see all the log I expect uploaded to logs.o.o15:46
dmsimardpabelanger: perhaps there could be a minimal message in the gerrit comment about the reason for the post failure if we have a clue15:47
pabelangerdmsimard: yes, I think that is something jeblair has been working on over time15:48
dmsimardfor syntax failures, yeah15:48
dmsimardnot sure about post15:48
pabelangerwell, right now I only see exit code: 2 in debug.log, so we'd have nothing to post right now15:49
pabelangeralso, right now15:50
dmsimardI just rebased that patch, just in case15:50
*** yolanda has quit IRC15:50
openstackgerritJesse Keating proposed openstack-infra/zuul-jobs master: Improve the check for sudo not working  https://review.openstack.org/50344815:53
*** yolanda has joined #zuul15:54
pabelangerclarkb: mgagne: so, we haven't been able to upload a new DIB into inap for 8 days. We are still getting glance proxy errors15:58
pabelangerOpenStackCloudHTTPError: (504) Server Error for url: https://image.api.mtl01.cloud.iweb.com/v2/images?marker=c327d32e-694d-45a7-b3f9-39127f3741eb Gateway Time-out (Inner Exception: Expecting value: line 1 column 1 (char 0))15:59
*** jkilpatr has joined #zuul15:59
clarkbpabelanger: probably want to try upload a small image like cirros by hand15:59
jlkmordred: https://developer.openstack.org/api-ref/networking/v2/index.html15:59
pabelangerclarkb: mgagne sorry, this should have been in #openstack-infra15:59
mgagnepabelanger: wow, thanks for pointing that out, will look right now15:59
jlkugh that doesn't work.15:59
jlkmordred: https://developer.openstack.org/api-ref/networking/v2/index.html#quotas-extension-quotas15:59
jlkjeblair: can you confirm what I think is true? If a project config lists a job for a project, and that job is defined in a repo that is not the same as the project, Zuul is smart enough to clone the repo that the job exists in and runs the job from the pwd context of the repo / path where the job lives?16:10
* rbergeron sends a billion hugs from los angeles16:11
jlkohai rbergeron!16:13
jeblairjlk: it will clone the project for the purpose of having the playbooks available, but will not check that project out in the src/ dir unless it is specified it required-projects16:15
*** yolanda has quit IRC16:16
mordredtobiash: http://git.openstack.org/cgit/openstack/os-client-config/tree/os_client_config/vendor-schema.json16:22
Shrewsjlk: jeblair: maybe it's supposed to work that way, but https://review.openstack.org/491937 seems to point to something about that being off16:23
Shrewsnote the zuul error "Job zuul-migrate not defined"16:24
Shrewsyeah, i think the it was just not merged yet. seems to be ok now16:31
*** yolanda has joined #zuul16:58
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Update time database to v3  https://review.openstack.org/50375016:59
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Migration: handle shared queues  https://review.openstack.org/50317017:02
jlkmordred: jeblair: et al -> https://review.openstack.org/503751  adds the linting.17:08
* SpamapS currently trying to figure out how to get his zuul to pull in zuul-jobs17:09
jlkput it in your project config17:10
jlkwait, no17:11
jlkput it in your tenant.yaml17:11
jlkadd it as a config-projects17:11
jlkor untrusted-projects17:12
mordredSpamapS: I recommend untrusted-projects17:13
SpamapSWell I also have to add a gerrit connection17:13
SpamapSright?17:13
mordrednope17:13
mordredwe replicate to github you konw17:14
tobiashSpamapS: if you fetch it from a mirror (you might have to as the git driver is not complete yet) you might have to exclude the project config17:14
SpamapSbut I'm on GHE17:14
mordredSpamapS: ah. then yes - there ar ea few options ...17:14
mordredSpamapS: one is to add a gerrit connection - one is to make a local clone in your ghe that you point to and update from time to time17:14
mordredSpamapS: one is to add a public GH connection17:14
jlkyou could add a public GH connection, without an auth.17:15
jlkthinking about this though17:15
jlkUnless you had a webhook on that repo, you'd never see changes to that repo, which could trigger reconfiguration17:15
SpamapSa mirror might make the most sense17:16
SpamapSsince I can control updating17:16
jeblaireventually we'll have the git driver for this17:18
jlkjeblair: even with the git driver, how do you become aware of changes to the repo? Periodic polling?17:18
jeblairjlk: yeah.  i think that's probably okay for zuul-jobs.17:19
mordredSpamapS: also, whilst you do that - it's required that a deployer make their own base job currently - but all the roles we use in ours are in zuul-jobs - what is currently missing is a doc on "make a base job that does X Y and Z and if you wnat to re-use our roles how to, use these 5 and then make sure you have these 4 site-variables set"17:19
mordredSpamapS: I think having such a doc would be AWESOME of course, so if you happen to learn things about that area, collecting notes at leats would be neat17:19
SpamapSmordred: oh yeah I can def write that doc once I get through this.17:20
jlkhttps://github.com/j2sol/project-config may help crib from17:21
pabelangerjlk: mordred: jeblair: see https://review.openstack.org/495463 also for linting things on project-config. Was my first attempt17:26
jlkpabelanger: ah good, I'll try to incorporate some of that into my change.17:30
jlk(if you don't mind)17:30
*** electrofelix has quit IRC17:41
*** yolanda has quit IRC17:42
*** jkilpatr has quit IRC17:43
jlkpabelanger: what is the point of the tests/ dir in your change and the inventory/vars ?17:45
jlkoh hah n/m17:45
dmsimardjeblair: more post failures in https://review.openstack.org/#/c/502147/17:58
pabelangerjlk: ya, feel free to use what you need17:58
jeblairdmsimard: is that derived from base-test?18:01
dmsimardjeblair: yes18:02
dmsimardjeblair: the point is to test the ci-mirror role that is in base-test18:02
jeblairdmsimard: ah.  it collided with my tests of subunit, sorry.18:02
jeblairdmsimard: they work now18:03
jeblairso a recheck should take care of it18:03
dmsimardjeblair: hm, actually someone's merged the base change too now so I can try base instead18:03
dmsimardjeblair: any hope of getting some hint of the problem showing up on a post failure ?18:04
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Don't set nonvoting based on name suffix  https://review.openstack.org/50376918:05
*** yolanda has joined #zuul18:35
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Expand templates for project-specific matchers  https://review.openstack.org/50314318:43
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Expand templates for project-specific matchers  https://review.openstack.org/50314318:46
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Expand templates for project-specific matchers  https://review.openstack.org/50314318:46
*** yolanda has quit IRC18:46
*** yolanda has joined #zuul18:47
Shrewsignore ^^^18:48
*** hashar has joined #zuul18:57
*** maxamillion has quit IRC18:58
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Update time database to v3  https://review.openstack.org/50375019:00
*** maxamillion has joined #zuul19:00
*** yolanda has quit IRC19:17
* SpamapS wanders to DownTown LA to find rbergeron and other Ansiblers19:19
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Expand templates for project-specific matchers  https://review.openstack.org/50314319:20
Shrewsmordred: jeblair: well, it's not pretty (because the logic is ugly), but we do have irrelevant-files being set in projects now with ^^^19:20
Shrewsno idea if it's actually correct19:20
* Shrews breaks for abit19:20
*** yolanda has joined #zuul19:41
*** hashar has quit IRC19:51
mordredERROR:zuul.Migrate:Template python35-jobs not found for expansion into project openstack/patrole20:08
mordredjeblair: don't forget WORKSPACE in the ZUUL_ vars thing20:20
jeblairmordred: ack20:22
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Expand templates for project-specific matchers  https://review.openstack.org/50314320:31
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380620:33
qwcHey, guys where did you move the v2.5.3 documentation to ?!20:33
qwcWe need it!20:33
dmsimardqwc: I don't have an answer for you20:34
dmsimardqwc: but if necessary you can build the docs yourself from the master branch: https://github.com/openstack-infra/zuul20:35
dmsimardtox -e docs will get you the html docs20:35
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Expand templates for project-specific matchers  https://review.openstack.org/50314320:35
Shrewshrm, thought we fixed the docs thing20:36
Shrewspabelanger: ^^^ ???20:36
qwcdmsimard: k, will check tomorrow. all of a sudden the docs were gone, just 404 errors on all open tabs I had ...20:36
Shrewshttps://docs.openstack.org/infra/zuul/ still showing v320:36
qwcbit of a shock20:37
Shrewsqwc: that was an accident20:37
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380620:39
qwcaccidents happen, just get them back somehow, or link on the new location for v2.5.320:39
pabelangerShrews: it's should be fix, but we need to trigger a master job20:39
pabelangerShrews: we should be able to trigger it manually in post to do the doc builds20:40
qwcThe new v3 docs are a bit confusing when you are used to the old variant, btw.20:41
qwcyou check for triggers and you end up at the development docs not the configuration docs ... O.o20:41
qwcWell, thanks for looking and fixing, have a good night! :)20:43
openstackgerritJesse Keating proposed openstack-infra/zuul-jobs master: Fix linting issues found via project-config  https://review.openstack.org/50381720:57
*** mordred has quit IRC20:57
*** yolanda has quit IRC21:01
*** yolanda has joined #zuul21:04
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Update time database to v3  https://review.openstack.org/50375021:12
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380621:21
jlkhrm.21:24
*** mordred has joined #zuul21:24
jlkjeblair: I'm slightly confused. I proposed a change to project-config (https://review.openstack.org/503751) which updates zuul.yaml, and adds a new job to the check pipeline. Zuul appears to be processing that and trying to USE it in the change, which I thought it wouldn't since it's a config repo.21:26
jlkor is project-config not a config repo?21:26
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Support 'files' matcher logic  https://review.openstack.org/50383221:26
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Gate on zuul-stream-functional  https://review.openstack.org/50286421:29
Shrewsjlk: responded to your question21:33
Shrewsin 503832, that is21:34
jlkthanks21:34
jlkI'lll wait for that job to run21:34
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: WIP: Honor cloud quotas before launching nodes (second approach)  https://review.openstack.org/50383821:41
tobiashjeblair, mordred: that's an approach for quota calculation21:42
tobiash^^21:42
tobiashmissing a few tests and some polishing yet21:42
*** mordred has quit IRC22:00
*** yolanda has quit IRC22:03
dmsimardjeblair: https://review.openstack.org/#/c/503806/ multinode-test finger://ze01.openstack.org/b20e765e8b9344d88d6fa62b2a8cc479 : RETRY_LIMIT in 42s22:05
jeblairdmsimard: http://paste.openstack.org/show/621084/22:08
*** mordred has joined #zuul22:09
jeblairdmsimard: http://paste.openstack.org/show/621085/22:11
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add zuul legacy vars filter  https://review.openstack.org/50385222:20
jeblairmordred, Shrews: ^22:20
tobiashtristanC: comment on https://review.openstack.org/#/c/503714/122:21
Shrewsjeblair: do you need to make the zuul_legacy_vars() mention in FilterModule an actual call?22:30
*** yolanda has joined #zuul22:35
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Use connection type supplied from nodepool  https://review.openstack.org/50197622:43
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380622:44
jlkShrews: no, it does not appear so22:45
jlkat least from Ursula's example22:45
pabelangerjlk: comments on 502208 when you have time22:46
jlkwoo22:46
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380622:51
openstackgerritJamie Lennox proposed openstack-infra/zuul-jobs master: A nodejs install role  https://review.openstack.org/50320122:58
openstackgerritJamie Lennox proposed openstack-infra/zuul-jobs master: A nodejs install role  https://review.openstack.org/50320123:00
jlkIs there a clear definition of what goes in zuul-jobs vs what goes in openstack-zuul-jobs vs what goes in project-config ?23:05
openstackgerritTristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: Ensure build.start_time is defined  https://review.openstack.org/48084323:06
Shrewsjlk: https://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/README.rst and https://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/README.rst23:10
jlkthat works23:10
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380623:11
*** jkilpatr has joined #zuul23:14
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380623:16
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380623:29
openstackgerritTristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/builds route  https://review.openstack.org/46656123:29
openstackgerritTristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{source}/{project}.pem route  https://review.openstack.org/50253023:29
openstackgerritTristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /tenants route  https://review.openstack.org/50326823:29
openstackgerritTristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/status route  https://review.openstack.org/50326923:29
openstackgerritTristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/jobs route  https://review.openstack.org/50327023:29
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Remove zuul-cloner shim  https://review.openstack.org/50175023:30
openstackgerritTristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/builds route  https://review.openstack.org/46656123:34
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380623:35
openstackgerritJesse Keating proposed openstack-infra/zuul feature/zuulv3: change assert(Not)Equals to assert(Not)Equal  https://review.openstack.org/50387023:35
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: DNM: Test v3 multinode things  https://review.openstack.org/50380623:43
*** jkilpatr has quit IRC23:47
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Don't set nonvoting based on name suffix  https://review.openstack.org/50376923:48
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Use feature/zuulv3 for zuul-migrate test  https://review.openstack.org/50387323:48
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Emit job definitions  https://review.openstack.org/50387423:48
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Use feature/zuulv3 for zuul-migrate test  https://review.openstack.org/50387323:51
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Don't set nonvoting based on name suffix  https://review.openstack.org/50376923:52
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Emit job definitions  https://review.openstack.org/50387423:52
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: Ensure build.start_time is defined  https://review.openstack.org/46673223:54
*** jkilpatr has joined #zuul23:57

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