Wednesday, 2014-03-12

fungiclarkb: i think we never did anything for 1253774, no. probably first you want to hop onto bare-precise, py3k-precise and devsatck-precise node examples and see what's lingering in their sources00:00
sdagueyeh, salv-orlando if there were different configs that you liked, we could do them. But I definitely think running more than one tempest job helped the project00:00
clarkbfungi: ya just held a devstack-precise node to do that00:00
clarkbfungi: there is the puppetlabs sources list in sources.list.d00:00
*** aysyd has quit IRC00:01
fungiclarkb: noting that each of them may have different stray things (for example the py3k nodes have ppa links for python3.3 and pypy)00:01
clarkbfungi: quick question though. will apt-get freak out if it has installed a thing from a source (puppet) which results in a newer version of the normal repo when it goes to install other things?00:01
salv-orlandosdague: I think it's ok this way. I'm not sure we need to enable different configs at this stage. I just saw the doubles and wondered if it was by design or not.00:01
fungiclarkb: it will not care in the least00:01
clarkbfungi: I think my plan is to rm /etc/sources.list.d/*00:01
clarkbfungi: awesome. I will just propose a change that does ^00:01
fungiclarkb: it leaves packages it doesn't know about alone unless it sees newer versions available somewhere it knows about00:02
clarkbfungi: rm'ing sources.list.d/* is sufficient right?00:02
fungiclarkb: sounds perfectly safe00:02
fungiclarkb: note that if that were not safe, it would also muck with packages you install via dpkg -i somefile.deb00:02
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1291108  https://review.openstack.org/7981200:02
fungiclarkb: you *do* want to do another apt-get update after that though, just to be on the safe side (in case something later tries to do an apt-get install without first doing another update)00:03
salv-orlandosdague: as a side question, do you remember if before the integrated gate template we were running exercises for neutron jobs?00:03
krotscheckjeblair, clarkb: We test our migrations against mysql and postgres - see TestMigrations, it overrides the necessary properties to make the cifail switch not flip. You're right though, we don't set up a DB per test.00:03
sdaguesalv-orlando: a long time ago00:03
sdaguewe phased out exercises basically entirely last summer, except for grenade00:04
*** gokrokve has quit IRC00:04
sdagueand we got rid of them in grenade a month ago00:04
krotscheckjeblair, clarkb: hence the yelling.00:04
clarkbfungi: oh right thanks00:05
*** yamahata__ has joined #openstack-infra00:06
*** thuc_ has joined #openstack-infra00:07
*** thuc_ has quit IRC00:07
*** thuc_ has joined #openstack-infra00:08
salv-orlandosdague: ok.. so exercise should not be running for check-tempest-dvsm-neutron?00:08
jogomriedem:   unit test  are called gate in check queue as well00:09
*** thuc_ has quit IRC00:09
mriedemjogo: well kiss my grits00:09
jogore: https://review.openstack.org/#/c/79812/00:09
clarkbwe lost gerrit bot00:10
clarkbfungi: https://review.openstack.org/7981400:10
* jogo likes his grits cheesy00:10
openstackgerritClark Boylan proposed a change to openstack-infra/config: Remove unneeded apt sources from nodepool nodes.  https://review.openstack.org/7981400:10
clarkboh nvermind, just slow00:10
mriedemjogo: anyway, if we drop build_name from the query that's one less wildcard search00:10
mriedemand sdague has me all paranoid about wildcards in the queries now00:10
mriedemjogo: or is that somehow offset by filtering by build_name, even with the wildcard?00:10
mriedemclarkb: ^?00:10
mriedemi had grits in tennessee last year for the first time, grossed me out00:11
jogomriedem: not sure which way is better, I agree lets ask clarkb00:11
clarkbmriedem: instead of using the wildcard be specific00:11
clarkband OR them all together00:11
mriedemyeah, that's the alternative00:11
fungimriedem: what's gross about grits? you just soak dry corn in wet lye for a while and then...00:11
mriedempew00:12
clarkbgate-cinder-python27 OR gate-cinder-python26 OR gate-cinder-pypy andso on (I don't think we are testing pypy or py3k with cinder yet, 26 and 27 are the values you need)00:12
mriedemclarkb: we don't even really need build_name in the query though,00:12
sdaguesalv-orlando: correct00:12
mriedemthis only shows up in python UT jobs anyway00:12
fungimriedem: but really, if you grow up with them, it's no stranger than, say, lutefisk00:12
mriedemfungi: also gross, and my mom's side is swedish00:13
mriedemso every xmas was lutefish or oyster stew, both nasty00:13
salv-orlandosdague: I'm off to bed now, but I'll leave you with this -> http://logs.openstack.org/43/79443/3/check/check-devstack-dsvm-neutron/6118ec2/console.html00:13
mriedemuntil uncle don put his foot down and brought a ham one year, xmas was saved!00:13
fungimriedem: you just have an aversion to caustic alkaline-preserved foods, obviously ;)00:13
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1291108  https://review.openstack.org/7981200:13
sdaguesalv-orlando: so.... that job exists for grenade00:13
*** mgagne has joined #openstack-infra00:13
sdagueexcept we don't need it any more00:13
clarkbmriedem: right you aren't doing that to make sure you only see those jobs you are doing it so that the poor ES cluster doesn't have to search everything00:14
openstackgerritDavanum Srinivas (dims) proposed a change to openstack-infra/devstack-gate: WIP : Try patched libvirt from @hallyn - Not for review  https://review.openstack.org/7981600:14
clarkbmriedem: filters restrict the search space00:14
mriedemclarkb: ok, that's the answer i was looking for00:14
*** dstufft has quit IRC00:14
salv-orlandosdague: so I can avoid worrying about those failures, and move on to the next one?00:14
sdagueclarkb: so in reality, do you think wildcards in filters like that is actually helpful or not00:15
*** dstufft has joined #openstack-infra00:15
sdaguesalv-orlando: yeh, I think we can actually probably delete that job honestly00:15
salv-orlandook thanks then00:15
salv-orlandoand goodnight!00:15
clarkbsdague: I think it is less helpful. wildcards are a search. non wildcards are a filter00:15
sdaguegoodnight00:15
clarkbsdague: so by having that as a wildcard we have to serach through that space. having it as a filter reduces any remaining search space00:16
sdagueclarkb: so if that was SQL, it would actually be slower with the wildcards00:16
sdaguebecause the message field is full text indexed, so it's pretty fast00:16
sdaguebut that ends up being string compares with regex, which is slow00:17
clarkbsdague: thats what I am saying00:17
clarkbwildcards are slower00:17
sdagueclarkb: I mean slower with wildcards, then making no reference to that field at all00:17
mriedem^ that's what i was wondering00:17
sdagueas in deleting the filter would make the query faster00:17
*** yamahata has joined #openstack-infra00:18
clarkbsdague: making no reference in this case is probbaly slower because then the console.html for every single job in the last 2 weeks will be searched00:18
clarkbsdague: rather than searching through only the those for gate-python-cinder2X00:18
sdagueright, but the way it will be searched is actually fast00:18
sdaguerelatively00:18
*** atiwari has quit IRC00:18
clarkbsdague: right but we are talking orders of magnitude different data that must be combed through (granted the index helps so it isn't that terrible)00:18
sdagueit would be interesting to ask an ES person about things like these actually00:18
clarkbsdague: I did00:19
clarkbat a local meetup00:19
clarkbsdague: filtering is good00:19
clarkbfiltering != wildcards00:19
sdagueok, awesome +100:19
sdagueright, filtering definitely good00:19
sdagueI guess the wildcard thing would be interesting to understand00:19
clarkbso explicitly spell it out. build_name:"gate-cinder-python26" OR build_name:"gate-cinder-python27"00:19
sdagueright that would be faster00:19
clarkbsdague: I think if you flip the order of the searches it may be faster00:20
clarkbconsole.html is about half our log data00:20
clarkbso on a busy day a quarter of a billion documents00:20
sdagueit doesn't do query reordering automatically?00:20
sdaguemost of those do00:20
*** thuc has joined #openstack-infra00:20
clarkbif you can restrict that first with a relatively cheap wildcard serach to go from .25 billion to .01 billion I think you win00:21
clarkbsdague: I am not sure. the behavior I have seen is that order matters but I haven't heard that from someone that actually know00:21
sdagueso it might be really nice to add another tag for "unittests"00:21
*** thuc has quit IRC00:21
clarkbsdague: we don't have a reliable way to do that00:21
clarkbsame problem as slave az and image00:21
sdaguebecause then we could tag:unittests AND project:openstack/cinder00:21
*** thuc has joined #openstack-infra00:22
sdagueclarkb: *python* ?00:22
*** thuc has quit IRC00:22
sdaguewhen we create that field00:22
clarkbsdague: but that misses pypy00:22
clarkband py3k00:23
clarkband all the funny django tests00:23
clarkband so on00:23
clarkb:/00:23
*** thuc has joined #openstack-infra00:23
sdagueok, well we can add them over time00:23
sdagueit would still be much better to filter on00:23
sdaguejust a thought00:23
mriedemsdague: clarkb: so the take away for me is be really strict now on wildcard usage when i see it in reviews00:23
*** thuc has quit IRC00:23
mriedemi'd like to get something in the readme at least on wildcards00:23
clarkbsdague: I agree, I just think we need to be better about the information upfront rather than approximating00:23
mriedemwe used to say it wasn't supported/didn't work, but it does...00:23
clarkbsdague: because with approximations we will fail and confusion will happen00:24
*** thuc has joined #openstack-infra00:24
sdaguebah, approximations are the spice of life :)00:26
*** matsuhashi has joined #openstack-infra00:26
*** mgagne has quit IRC00:26
fungithe approximations must flow00:26
mriedemand he who controls the spice controls the galaxy00:26
fungid'oh!00:26
mriedemha00:27
mriedemwhere is my wife, she loves dune references...00:27
openstackgerritA change was merged to openstack-infra/jeepyb: Load project settings from ini file, if present.  https://review.openstack.org/7132300:27
* fungi whistles and walks away from this mess00:27
openstackgerritSean Dague proposed a change to openstack/requirements: allow sqla 0.9.x  https://review.openstack.org/7981700:27
*** bhuvan_ has quit IRC00:27
*** bhuvan has quit IRC00:27
mriedemsdague: ^ typo in there i think00:28
*** bhuvan has joined #openstack-infra00:28
*** bhuvan_ has joined #openstack-infra00:28
sdaguemriedem: probably, we need test results anyway00:29
mriedemfungi: clarkb: do you know where the code is that sends out the automatic global-reqs updates to the other projects?00:30
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add fingerprint for bug 1291108  https://review.openstack.org/7981200:30
clarkbmriedem: ya I will get a link00:31
clarkbmriedem: it needs to not do that when there are changes in the gate otherwise they get kicked out00:31
fungimriedem: just a sec, let me let clarkb get that for you ;)00:31
mriedemclarkb: what needs to not do what now?00:31
*** mgagne has joined #openstack-infra00:32
fungimriedem: when the gate is long, a project might approve their requirements updates change while there is a requirements change ahead of them in te gate00:32
clarkbmriedem: https://git.openstack.org/cgit/openstack-infra/config/tree/modules/jenkins/files/slave_scripts/propose_requirements_update.sh those jobs push updates immediately after a requirements change merges00:32
clarkbthen what fungi says happens and we kick stuff out of the gate due to new patchsets00:32
*** markmcclain has joined #openstack-infra00:32
fungimriedem: then the merger of the global requirements change triggers an update job, and snipes any currently approved per-project requirements changes pending merger00:32
*** HenryG_ has quit IRC00:32
clarkbalso maybe it hsan't been advertized enough but requirements should be feature frozen too for the most part00:32
clarkbso now is a good time to get everything synced00:33
mriedemclarkb: reqs freeze is kind of what i was thinking too00:33
fungiclarkb: yes, i think feature freeze and requirements freeze should go hand-in-hand personally00:33
*** gokrokve has joined #openstack-infra00:33
*** mgagne has quit IRC00:33
clarkbI wonder if we can make that a ttx release check, for project in $PROJECTS; do check_reqs_syncage ; done00:34
clarkbiirc nova's havana release was immediately broken because we didn't do ^00:34
fungithough fwiw, in havana there was a lot less time to get all that matched up, but we definitely hoped that for icehouse everyone would be in sync00:35
clarkbthis felt like a productive bug day00:35
clarkbpleia2: thank you for putting it together (and reminding us that we should do one)00:35
fungiit was bug-productive, but i didn't get to most of the other stuff i need to do00:35
pleia2clarkb: sure thing :)00:35
fungipleia2: awesome job!00:35
*** Sukhdev has quit IRC00:36
pleia2now I get to go to a hockey game00:36
mriedemjogo: looks like the propose_requirements_update.sh script actually does try to match the change-id to the requirements project: https://git.openstack.org/cgit/openstack-infra/config/tree/modules/jenkins/files/slave_scripts/propose_requirements_update.sh#n5100:36
StevenKpleia2: Ice or grass?00:36
pleia2StevenK: ice, with canadians!00:36
*** mgagne has joined #openstack-infra00:36
clarkbmriedem: thats matching the change id to any existing unmerged requirement updates for that change00:37
clarkbmriedem: this way we push new patchsets not new changes00:37
mriedemclarkb: so what i wanted was to link something like this https://review.openstack.org/#/c/79338/ back to the original global-requirements change00:38
mriedemso we can see the original commit message to know why it's being updated, b/c that's what i do when i review these00:38
mriedemi.e. if it's a bug fix or something00:38
jogo2014-03-12 00:34:09.832 TRACE nova Stderr: 'Traceback (most recent call last):\n  File "/usr/local/bin/nova-rootwrap", line 6, in <module>\n    from oslo.rootwrap.cmd import main\nImportError: No module named rootwrap.cmd\n'00:38
jogois that one of those weird oslo namespace issues?00:39
*** thuc has quit IRC00:39
jogothat was when running grenade locally00:39
clarkbmriedem: you can do that if you can build the state in that script00:39
clarkbmriedem: I would need to think about that more to know how difficult it would be00:39
*** thuc has joined #openstack-infra00:39
clarkbmriedem: so honestly, I think people should be reviewing openstack/requirements if they care at that level00:39
clarkbmriedem: you are already being tested with the requirements in the global requirements. The only reason we havethose files in nova is so that you can pip install nova without an additional step00:40
mriedemclarkb: well and packagers rely on them right?00:40
mriedemi know we do internally, we use that to update rpm specs00:40
clarkbmriedem: maybe, openstack/requirements is largely for the packagers00:40
clarkbthey package the stuff in the list then don't have to worry about stuff on a case by case basis00:40
mriedemright00:41
*** ociuhandu has quit IRC00:41
clarkbmy point is adding that info to the nova requirements update isn't a bad thing but it is the wrong place to be reviewing that data00:41
fungimriedem: packagers probably rely on the requirements names listed in each project, but the versions in global-reqs are what's going to be needed if they want to package more than one project for teh same distro00:41
clarkbit needs to be reviewed when proposed to openstack/requirements00:41
*** ryanpetrello has joined #openstack-infra00:41
mriedemfungi: yeah, and that's what we do00:41
mriedemfungi: and so like when you have some projects capped on different versions of sqla/migrate, it gets weird00:42
clarkbpleia2: good point about if debian. do you know how yum works when we add the puppet repos? we should remvoe them from yum too if possible00:42
clarkbmriedem: not really, they are all tested with the same version00:42
clarkbmriedem: which is why openstack/requirements exists00:42
fungiclarkb: well, i suspect he meant it gets weird trying to package them00:43
*** mrodden1 is now known as mrodden00:43
fungifor the same distro00:43
mriedemclarkb: so as of yesterday i know that all projects run against global-requirements,00:43
mriedemfungi: yes00:43
*** thuc has quit IRC00:43
mriedemso is it pip/pbr that allows version caps to be ignored?00:43
locke105wut?00:44
clarkbmriedem: fungi but it isn't weird00:44
clarkbyou just package what oepnstack/requirements says and ignore what nova says because nova is wrong00:44
fungimriedem: well, not entirely true. what clarkb neglected to mention is that it's just the integration tests which get run reset to the global requirements lists. not individual project unit tests00:44
fungiclarkb: you have to pay attention to the list of dependencies nova declares, just not necessarily the versions of them00:44
mriedemok, "you just package what oepnstack/requirements says and ignore what nova says because nova is wrong" is news to me00:45
fungiclarkb: you wouldn't package nova and claim it depends on every single package listed in the global requirements00:45
clarkbfungi: right but we aren't talking about that we are talking about 50 different versions of sqlalchemy00:45
mriedemright, like pecan and the other 20 things that ceilometer uses but no one else does, for exapmle00:45
mriedembut i'm getting the point00:45
mriedemand yeah it's what happens anyway, we just end up updating the nova rpm spec using nova's requirements.txt, but the values in there match global requirements more or less anyway00:46
*** rms_13 has left #openstack-infra00:46
fungiclarkb: true... though if nova doesn't keep up to date on global requirements sync, it may run with the versions in global-reqs per our integration tests, but may not be able to pass its own unit tests00:46
mriedemwe're just usually a day or so behind b/c someone in nova has to merge those changes00:46
fungicorner case, but i expect it to happen at some point00:47
mriedemso is there value in updating the sync commit message to explain some of this for reviewers?00:47
clarkbfungi: it is certianly possibly00:47
clarkb*possible00:47
mriedemlike 'if you're worried about testing, you're too late because it's already tested, this is more or less for downstream CD'00:47
clarkbmriedem: IMO it doesn't hurt, but I also think that if you care about that at that level you need to review openstack/requirements changes instead00:47
fungiopenstack proves the infinite monkey theorem on a regular basis00:47
mriedemclarkb: my caring is really only in doing some kind of review on these changes when they show up in nova00:48
clarkbmriedem: I don't want nova to get the impression that they can refuse an update because they don't like the commit from requirements00:48
mriedem*on doing00:48
clarkbit should've been refused in requirements00:48
*** thuc has joined #openstack-infra00:48
mriedemclarkb: no i get that00:48
mriedemnot my point00:48
* locke105 google inifinite monkey theorem00:48
locke105googles* bah00:48
mriedemso i'm sure this has been asked, but then why don't we just auto-merge those changes in the projects if we shouldn't care about them there?00:48
*** thuc has quit IRC00:48
*** bhuvan_ has quit IRC00:49
*** bhuvan has quit IRC00:49
fungilocke105: basically saying that if there is a corner case we can define, we will almost certainly encounter it at some point due to the sheer size of the project00:49
mriedemalthough if i didn't poke on the sqla nova sync change yesterday, we wouldn't have known this was broken in how things were tested...00:49
*** thuc has joined #openstack-infra00:49
clarkbmriedem: we really don't want jenkins to have merge rights :)00:49
clarkbfor code that it proposes00:49
mriedemfor when it becomes self-aware00:50
fungimriedem: it provides some oversight, but it's on the same level as translation proposals00:50
clarkbright now we mostly have that separated out with zuul being the organizer00:50
mriedemyeah, just playing devil's advocate00:50
*** ryanpetrello has quit IRC00:50
*** ryanpetrello has joined #openstack-infra00:50
jeblairit's dangerously close to self-aware; don't give it any more weapons00:50
*** Alexandra has quit IRC00:50
*** isviridov has quit IRC00:50
*** derekh has quit IRC00:50
mriedemalright, well not sure if i have something useful to propose for the sync commit message or not00:50
*** ryanpetrello has quit IRC00:51
* locke105 proposes a change: New Jenkins job: gate-become-self-aware00:51
*** CaptTofu has joined #openstack-infra00:51
fungimriedem: also keep in mind that the next patchset on a change is going to be updated to include an adjustment from an entirely different global requirements change, so you may not be providing much context unless you never let it get past one patchset on the change00:51
fungimriedem: or unless you keep rereading and appending to the commit message00:52
mriedemfungi: yeah, i've abandoned that idea00:52
fungiahh, okay00:52
mriedemi'm thinking more general guidance for reviewers00:52
locke105what about running nova unit tests against global req changes?00:52
fungiit's probably doable, but i would expect it to get real ugly real quick00:52
fungilocke105: we've got some things going on with oslo.test which may make that viable00:53
mriedemlocke105: nova UT/grenade/tempest is run against the nova sync, but not global-requirements00:53
fungilocke105: basically a pattern for changes to one project getting gated on the unit tests of a list of other projects00:54
*** gondoi has quit IRC00:56
*** hallyn has quit IRC00:56
*** vishy has quit IRC00:56
*** mugsie has quit IRC00:57
*** zhiyan_ has quit IRC00:57
*** kiall has quit IRC00:57
*** pabelanger has quit IRC00:57
*** StevenK has quit IRC00:57
*** sdake has quit IRC00:57
*** bogdando has quit IRC00:57
*** jraim has quit IRC00:57
*** branen__ has quit IRC00:57
*** mancdaz has quit IRC00:57
*** nijaba_ has quit IRC00:57
*** odyi has quit IRC00:58
*** exz has quit IRC00:58
*** zigo has quit IRC00:58
*** sc68cal has quit IRC00:58
*** fungi has quit IRC00:58
mriedemif a sync breaks a project, the project fixes themselves i think - that's what happened when sqla-migrate 0.8.2 was released i believe and synced to nova00:58
locke105sure but we have a downstream job that is similar00:58
*** isviridov has joined #openstack-infra00:58
*** ryanpetrello has joined #openstack-infra00:58
mriedemnova had some sqla-migrate version capped code in it that needed to be removed00:58
locke105some extension gets updated, jenkins pulls in nova, pulls in latest extension, runs tests, reports back on extension change00:58
openstackgerritSean Dague proposed a change to openstack/requirements: allow sqla 0.9.x  https://review.openstack.org/7981700:58
sdaguemriedem: oh.... that kind of typo....00:58
* sdague facepalms00:58
sdaguejeblair: we should only give it cupcakes00:58
sdaguethen it will spare us in the revolution00:58
mriedemsdague: yeah, i wasn't sure what you were saying earlier00:58
sdaguemriedem: I assumed you were dinging me for a commit message00:58
openstackgerritClark Boylan proposed a change to openstack-infra/config: Remove unneeded package sources from nodepool node  https://review.openstack.org/7981400:58
sdaguebecause I mess those up plenty00:58
mriedemsdague: i assumed you were way over my head on something00:58
clarkbpleia2: ^ I think that fixes the problem you pointed out00:58
*** alexandra_ has joined #openstack-infra00:58
*** branen_ has joined #openstack-infra00:58
clarkbpleia2: I looked at a centos machine but figure all redhat ish distros use yum.repos.d since they use yum00:58
sdagueso yeh, bad rebase, which I totally didn't resolve right00:58
sdagueanyway, it's right now00:58
mriedemyeah, +100:58
*** mugsie_ has joined #openstack-infra00:58
*** CaptTofu has quit IRC00:58
sdaguenow we just need to figure out if it works00:58
sdaguealso... if it runs the right versions00:58
* mriedem ponders signing up for all new openstack/requirements patches00:58
*** zhiyan` has joined #openstack-infra00:58
*** jgriffith has quit IRC00:58
*** akscram has quit IRC00:58
*** GheRivero has quit IRC00:58
*** ProfFalken has quit IRC00:58
*** nijaba has joined #openstack-infra00:58
sdagueit's pretty light load00:58
mriedemyou've convinced me00:58
mriedemi have to deal with these things when they merge anyway so i might as well know they are coming00:58
*** VijayTripathi1 has quit IRC00:58
*** sc68cal_ has joined #openstack-infra00:58
dims_gah! netslpit00:58
*** vish1 has joined #openstack-infra00:58
*** vish1 is now known as vishy00:58
*** GheRivero has joined #openstack-infra00:58
*** fungi has joined #openstack-infra00:58
*** jraim has joined #openstack-infra00:58
*** thuc has quit IRC00:59
*** sileht has quit IRC00:59
*** openstack has joined #openstack-infra01:05
*** zns has quit IRC01:05
*** mattymo has joined #openstack-infra01:05
jogosdague: so  I am a little concerned that we have 100% nodepool utilization most ofthe time01:05
fungijeblair: heh, great... i'll be done cleaning that up by lunch time if i'm lucky ;)01:05
jogoit slows down checks and whatnot01:05
jogothe concern is purely about velocity01:06
jeblairjogo: you could contribute to the effort to move into hpcloud 1.101:06
jeblairjogo: where apparently hpcloud has more resources01:07
jeblairalso, there are lots of public openstack clouds we aren't using01:08
*** Daisy_ has joined #openstack-infra01:08
jeblairpeople have said they are interested in contributing clouds, but we've seen very little followup01:08
jogojeblair: sure, although I question the value of the 24 recheck (when not before gate) ... I wonder what 36 or 48 hours would look like01:09
jeblairjogo: it's 72 hours when not approved (24 after approved)01:10
jogojeblair: ohh in that case ...01:10
jogo having to wait an extra hour for check queue definitly slows things down01:11
jogoI do like the get more resoures solution in the long run01:11
jeblairjogo: i think we should do all the things.  get more resources, be smarter about the ones we have (sdague is working on that as you know)01:12
*** jnoller has joined #openstack-infra01:13
*** ryanpetrello has quit IRC01:13
clarkbI think rechecking was crucial to gate health during feature freeze01:13
clarkbit was so smooth compared to havana01:14
jogoclarkb: while I would like to agree with you, I don't think we really have enough data to say *what* helped -- too many changes01:14
jeblairwe can also do mean things like fast-fail (abort jobs on the first failure), or hang everything off of a group of jobs (people suggest pep8, but i also think requirements check failures should get the same treatment)01:15
SpamapSYeah, could just be Neutron sucking less. ;)01:15
jeblairthose would be zuul changes01:15
*** weiwei has quit IRC01:15
*** jhesketh_ has quit IRC01:15
jogoso I am not sure how much of the resource issue is gate jobs vs check jobs01:15
jeblairalso, if developers are impatient... they _could_ run tests locally before uploading :)01:15
jogomy hunch is we now just have tons of check jobs  (for several reasons)01:16
jogoand not enough nodes01:16
clarkbcheck most == gate01:16
*** gokrokve has quit IRC01:17
clarkbjeblair ++01:17
jogoclarkb: ?01:17
fungii still think we don't allow the gate window floor to dip low enough when there's high volume and a lot of gate resets going on, wasting resources over and over while more or less starving out the check pipeline for a while (which slows new changes entering gating too)01:17
sdaguejeblair: we need that as the new openstack hoodie "we should do all the things"01:17
jogofungi: interesting idea01:17
jeblairfungi: really?  i think 20 is too low but a good compromise01:18
jogosdague: haha01:18
locke105sdague: ++ would buy01:18
jogosdague: quick grenade question01:18
jogohow do I run locally from havna to trunk?01:18
jeblairfungi: considering how important the gate is, i think it's a tiny amount of resources... :)01:18
jogostock doesn't seem to do that01:18
sdaguejogo: stock should01:18
jogosdague: unless I missed a step (which I bet I did)01:18
sdaguebut thingee said he was running into an issue the other day as well01:19
*** krotscheck has quit IRC01:19
fungijeblair: well, granted the gate is important, but we've also effectively made the check pipeline part of the gate now01:19
sdagueso.... I should verify tomorrow01:19
*** mkerrin has joined #openstack-infra01:19
*** talluri has joined #openstack-infra01:19
*** zhiwei has joined #openstack-infra01:19
sdaguefungi: and I'd be -1 on a floor < 20. Because we just as often as reset, complete all the work in the 20 nodes except for on slow job on the tip01:20
sdague20 jobs ...01:20
fungi20 seems plenty low if we're merging 5-10 changes before a reset, but today the job failures in the gate pipeline were a good bit more plentiful than that01:20
jeblairfungi: i would be okay with a more _accurate_ heuristic01:20
fungias would i01:20
jeblairfungi: but with the current model, i think going below 20 would hurt us more than help01:21
sdagueyeh, we'd starve01:21
fungimmm, perhaps so01:21
jeblairfungi, clarkb: i think we can make a more accurate heuristic too...01:21
sdagueand, honestly, I think it's a low item to optimize01:21
sdaguebecause the floor was about not thrashing01:21
sdaguenot really about waste01:21
sdagueand we've managed to stop thrashing01:21
*** tteggel has quit IRC01:22
sdaguewhich is goodness01:22
fungiwe're not thrashing hard, but we are getting to a point where we're maxed out on quotas waiting for deletes01:22
sdaguefungi: sure01:22
openstackgerritJoshua Harlow proposed a change to openstack/requirements: Update the keyring dependency  https://review.openstack.org/7979601:22
sdaguethe other option would be put stackforge into a lower priority queue01:22
fungibut we seem to be riding high enough that the deletes aren't hurting us too bad either, so maybe i'm just over-thinking the current tuning01:23
sdagueyeh, I think we're actually ok there01:23
*** tteggel has joined #openstack-infra01:23
jeblairopenstack could make deletes faster. ;)01:24
sdaguefrom a development optimization perspective, I think the cross project dependencies in zuul is going to be something important to get in zuul01:24
sdaguein juno01:24
mordredI still think we shouldn't do that01:24
fungiit looks like at any one point in time we have roughly 300 nodes in use running jobs, the rest are building or deleting01:24
sdaguemordred: did you see how the event patch to fix neutron / nova interaction had to get tested?01:24
*** gokrokve has joined #openstack-infra01:25
jeblairsdague, mordred: i'm okay with the feature, though i do think it's worth considering how it relates to CD01:25
mordredit's the CD thing I'm worried about01:25
sdaguewith 10+ integrated projects, having to test changes in multiple projects is important01:25
sdaguemordred: so I'm not saying we land the changes atomically01:25
mordredI don't think we should land it until we've got somethign that tests rolling upgrades01:25
sdagueI'm saying we can define ordering01:26
mordredsdague: oh ... you're saing01:26
mordredok01:26
mordredyeah01:26
sdagueyeh, never atomic lands01:26
jeblairmordred, sdague: i think sdague is describing the one-way variant of cross-project dependencies01:26
mordredlike, "I need this unlanded change in neutron to land before testing this unlanded change in nova"01:26
mordredyeah01:26
sdagueyes01:26
sdaguethat01:26
mordredthat I think is a great idea01:26
jeblaira much easier problem than the full-cycle variant01:26
sdaguethat I think is going to be critical with 10+ integrated projects01:27
*** zigo has quit IRC01:27
locke105CD =?01:27
jeblairlocke105: continuous deployment01:27
locke105by that you mean devstack-gate?01:27
jeblairlocke105: (deploy every comment, or as near as you can make it)01:27
sdagueotherwise we end up with "oh, land this neutron change, guess it's the right one, wait until it's landed, then start working on the nova change"01:27
clarkbwhy is atomic required?01:27
jeblairlocke105: it's specifically referring to operators, though the devstack-gate process is designed to make it easier for them01:27
jeblairclarkb: it's not01:28
clarkbok didnt think it was01:28
mordredclarkb: in fact, it's not desired01:28
*** jhesketh_ has joined #openstack-infra01:28
mordredthe coordinated change dependency version requires atomic landing01:28
*** weiwei has joined #openstack-infra01:28
sdagueclarkb: no, I don't want it01:28
jeblairclarkb: that's only required for full-cycle cross-repo deps, which are considered to be hazardous to CD and possibly upgrade testing01:28
sdagueI think atomic is bad01:28
sdagueI agreed with mordred on that01:28
mordredsdague: I figured we proably agreed01:28
sdaguehe did't realize that :)01:28
locke105oh so CD isn't something that infra does... its just consideration of how operators would be live consuming and deploying openstack changes01:29
locke105?01:29
sdaguethis other thing, we really do need01:29
jeblairlocke105: correct01:29
jeblairlocke105: (though we actually do cd some of our own tools)01:29
jeblairlocke105: (but that's beside the point)01:29
locke105like gerrit-dev01:29
*** yaguang has joined #openstack-infra01:29
locke105err01:29
locke105review-dev01:29
jeblairlocke105: trying to get there with gerrit01:29
sdaguejeblair: so that I might hit you up about post release to hack on in zuul01:30
sdaguejust to figure out what's needed and see if I could take a first pass01:30
*** reed has quit IRC01:30
jeblairsdague: sure thing; i think many of the bits are there.  i think the main design review thing is how to specify the c-r deps; i think we should look at the defunct gerrit patch to see their syntax and possibly adopt it01:31
sdaguejeblair: I was assuming we'd use an Idempotent id01:31
jeblairsdague: i think that may be what they did.01:32
sdaguecool01:32
clarkbit would be goid to make it not gerrit specific01:32
sdagueyeh, honestly, I've been doing Depends-On: Idempotent id01:32
locke105if they merge in-order, and if its shortly one after another, whats stopping the CD framework from say, picking up both the nova and neutron update and not realizing they need to be done in-order?01:32
sdaguejust manually for signaling in the past01:32
sdagueclarkb: it needs to be an idempotent id01:32
*** salv-orlando_ has joined #openstack-infra01:32
jeblairsdague: but istr something about using the change-id and s/I/D/... but i don't recall all details.  main thing is -- a lot of thinking has already been done there01:32
sdagueotherwise it breaks on rebasing01:32
*** zigo has joined #openstack-infra01:33
sdaguecoolio, if you have pointers, would appreciate them01:33
jeblairclarkb: it's necessarily not gerrit-specific because it's not in gerrit.  :)01:33
jeblairsdague: not off hand, but i should be able to find the patch series when it's time01:33
*** dstanek_afk is now known as dstanek01:34
jeblairi need to run now, ttyl01:34
clarkbwhats wromg with change id + project?01:34
clarkboh branches01:34
*** salv-orlando has quit IRC01:37
*** salv-orlando_ is now known as salv-orlando01:37
clarkbchange id's may not be best because gerrit pairs them with project:branch to get the unique key01:37
*** gokrokve has quit IRC01:38
*** nosnos has joined #openstack-infra01:39
clarkbsdague: https://gerrit-review.googlesource.com/#/q/topic:cross-repo-dependencies is the code jeblair is talking about I think01:39
mordredsdague: so - it doesn't affect the gate per-se ... but why is n-novnc enabled by default in devstack?01:40
fungiokay, the project plan for tomorrow morning's maintenance is here for anyone who wants to give it a once-over... https://etherpad.openstack.org/p/project-rename-2014-03-1201:40
mordredsdague: we do not have a project option for it01:40
clarkbfungi: looking01:40
sdaguemordred: for raisons?01:42
mordredsdague: awesome01:42
fungiclarkb: also note that SergeyLukjanov's more details series of savanna-specific stuff is here... https://etherpad.openstack.org/p/savanna-renaming-process01:42
sdaguehonestly, that so predates me, I have no idea01:42
sdaguecould be linear-b01:43
clarkbmordred: probably because people like vnc consoles by default?01:43
mordredclarkb: for their devstack working env?01:43
clarkbmordred: ya01:43
mordredhrm01:43
clarkbfungi: plan looks mostly good to me, couple of notes01:44
sdagueI thought we had working spice now though01:44
clarkbmordred: remember devstack is the most common deployment method :)01:44
mordredwell, just pointing out, spurred by looking at it again because of the mailing list thread - that it's the only thing we clone from github with no other optino01:44
mordredoption01:44
*** gokrokve has joined #openstack-infra01:44
clarkbmordred: there is an option01:44
clarkbyou can override the location, seem my response to the thread01:45
sdagueanyway, time to call it a night. See you guys in the morning.01:45
mordredsdague: have fun!01:45
clarkbmordred: I mean its not necessarily a great option but you can clone from elsewhere01:47
clarkbfor most elsewhere is probably a fork of the github project on github though :)01:48
*** sarob has joined #openstack-infra01:48
fungiclarkb: suggestions integrated--thanks!01:48
fungiSergeyLukjanov: when you wake up, you'll probably want to review the project plan for the bits i'm doing at 12:00 utc (except step 14, which is up to you for your projects)01:54
*** khyati_ has quit IRC01:56
*** arosen has joined #openstack-infra01:57
*** gokrokve has quit IRC01:58
*** Ryan_Lane has quit IRC01:58
*** matsuhashi has quit IRC01:58
*** gokrokve has joined #openstack-infra02:01
*** changbl has joined #openstack-infra02:02
*** dkliban has joined #openstack-infra02:06
*** nicedice has joined #openstack-infra02:06
*** vkozhukalov has joined #openstack-infra02:07
*** talluri has quit IRC02:07
*** arosen is now known as arosen-home02:08
*** nicedice_ has quit IRC02:08
*** thuc has joined #openstack-infra02:09
*** sarob has quit IRC02:11
*** sarob has joined #openstack-infra02:12
*** HenryG has joined #openstack-infra02:12
*** gokrokve has quit IRC02:14
*** thuc has quit IRC02:14
*** gokrokve has joined #openstack-infra02:15
*** sarob has quit IRC02:16
fungiSergeyLukjanov: oh, the link would probably help... https://etherpad.openstack.org/p/project-rename-2014-03-1202:16
fungianyway, that's enough fun for me for the night. time to get ready for a nap02:16
*** matsuhashi has joined #openstack-infra02:17
zhiweithank you very much for your work.02:17
fungizhiwei: you're welcome! don't thank me until it's done though ;)02:18
zhiwei:)02:18
*** gokrokve has quit IRC02:19
*** zigo has quit IRC02:19
*** bhuvan has joined #openstack-infra02:21
*** bhuvan_ has joined #openstack-infra02:21
*** sweston has joined #openstack-infra02:23
*** zigo has joined #openstack-infra02:23
*** caleb_ has quit IRC02:24
*** khyati has joined #openstack-infra02:27
*** VijayTripathi has joined #openstack-infra02:31
*** rcarrillocruz has joined #openstack-infra02:32
*** vkozhukalov has quit IRC02:36
*** rcarrillocruz has quit IRC02:37
*** rpodolyaka has quit IRC02:38
*** harlowja is now known as harlowja_away02:48
*** sarob has joined #openstack-infra02:48
*** ryanpetrello has joined #openstack-infra02:49
*** CaptTofu has joined #openstack-infra02:52
*** jepoy has joined #openstack-infra02:54
*** gyee has quit IRC02:54
*** CaptTofu has quit IRC02:57
*** akscram has joined #openstack-infra02:57
*** amcrn has quit IRC03:05
*** talluri has joined #openstack-infra03:09
*** david-lyle has joined #openstack-infra03:09
*** dcramer__ has joined #openstack-infra03:12
*** mriedem has quit IRC03:15
*** rwsu has quit IRC03:19
*** sarob has quit IRC03:21
*** pcrews has quit IRC03:21
*** CaptTofu has joined #openstack-infra03:23
*** matsuhashi has quit IRC03:23
*** bhuvan_ has quit IRC03:25
*** bhuvan has quit IRC03:25
*** dkliban has quit IRC03:28
clarkbfungi: https://developer.github.com/v3/oauth/ we need to add the read:org scope to our token. this may require a new token03:29
clarkbfungi: turns out I am in a channel with someone that contributes to pygithub and they were able to help. seriously though why have api versions if when you change stuff like this you don't create a new version03:29
openstackgerritJoshua Hesketh proposed a change to openstack-infra/config: Add zuul overlay to review page  https://review.openstack.org/7983603:31
openstackgerritJoshua Hesketh proposed a change to openstack-infra/config: Add zuul overlay to review page  https://review.openstack.org/7983603:32
*** SumitNaiksatam has joined #openstack-infra03:32
*** jnoller has quit IRC03:33
*** jnoller has joined #openstack-infra03:33
openstackgerritJoshua Hesketh proposed a change to openstack-infra/config: Add zuul overlay to review page  https://review.openstack.org/7983603:33
jhesketh__jeblair, clarkb: ^ what do you think of this for an improvement to gerrit?03:34
jhesketh__(more or less proof of concept at this point)03:34
*** wchrisj has joined #openstack-infra03:37
clarkbjhesketh__: I am not the greatest person to review css and javascript, but I am super willing to throw that on review-dev and give it a spin03:37
*** sarob has joined #openstack-infra03:37
clarkbI think it is a great idea03:37
*** reed has joined #openstack-infra03:37
jhesketh__clarkb: heh, I'm not the greatest person at writing javascript/css!03:37
jhesketh__particularly the css part03:37
clarkbits basically zuul status in the change for that change right?03:37
jhesketh__yep03:37
clarkbthen yes I think we should put this on review-dev asap :)03:38
jhesketh__woo :-)03:38
clarkbI am currently taste testing a couple IPAs though so will have to wait for tomorrow before I can help :)03:38
arosen-homeone change i'd like to make is to add the patch owner to that page by the id. I  find myself having to look up the  patch id and search the zuul page over and over when i only care about my patches :)03:39
jhesketh__clarkb: mmm, nice03:39
jhesketh__enjoy! :-)03:39
jhesketh__arosen-home: you mean to the zuul status page?03:39
arosen-homejhesketh_ i mean here: http://status.openstack.org/zuul/03:40
*** oda-g_ has joined #openstack-infra03:40
jhesketh__arosen-home: sure. I would want to get clarkb's and jeblair's opinion on that. I'd be cautious of not adding too much clutter. The change I proposed embeds those boxes on the associated patches. So to see the status you'd just have to open your patches.03:41
jhesketh__(in gerrit that is)03:41
mattoliveraujhesketh__: awesome idea :)03:42
*** sarob__ has joined #openstack-infra03:43
*** sarob has quit IRC03:44
*** sarob__ has quit IRC03:45
*** sarob has joined #openstack-infra03:46
*** lcheng has joined #openstack-infra03:48
openstackgerritRyan Petrello proposed a change to openstack-infra/config: Add pypy jobs to pecan.  https://review.openstack.org/7983903:50
*** rwsu has joined #openstack-infra03:50
openstackgerritRyan Petrello proposed a change to openstack-infra/config: Add pypy jobs to pecan.  https://review.openstack.org/7983903:50
*** fifieldt has joined #openstack-infra03:51
*** Ryan_Lane has joined #openstack-infra03:53
*** dstanek has quit IRC03:54
*** sileht has quit IRC03:56
*** jnoller_ has joined #openstack-infra03:56
clarkbdhellmann: the oslo.test chagnes lgtm03:56
*** jnoller has quit IRC03:56
*** Daisy_ has quit IRC03:57
*** sileht has joined #openstack-infra03:58
dstufftmordred: I almost have pip taking control of setup_requires04:00
*** SumitNaiksatam has quit IRC04:01
dstufftscratch that04:01
dstufftI do have it04:01
dstufftat least a working PoC04:01
*** zhiyan` is now known as zhiyan04:02
*** SumitNaiksatam has joined #openstack-infra04:02
*** dstanek has joined #openstack-infra04:02
*** gokrokve has joined #openstack-infra04:05
*** markmcclain has quit IRC04:05
clarkbdstufft: that half does something pbr did do and half makes it safer to install pbr yeah?04:05
clarkbslowly one day pbr will die and I will be happy04:05
*** thuc has joined #openstack-infra04:06
dstufftclarkb: well even if you ``pip install foo``, if foo uses setup_requires in it's setup.py, then setuptools will be used to download and install wahtever is in the setup_requires04:06
dstufftmy PoC uses pip to do that instead04:07
*** thuc_ has joined #openstack-infra04:07
dstufftso that if you're using pip, setuptools should never touch the network04:07
dstufftit's horrible though04:07
dstufftI'm monkeypatching distutils (which will then be monkeypatched by setuptools) inside of a python -c "" which reads the setup.py file and evals it04:08
dstuffthttps://github.com/dstufft/pip/compare/pypa:develop...dstufft:eldritch-horror04:08
*** sarob has quit IRC04:08
clarkbwow04:08
clarkbappropriate branch name04:08
*** sarob has joined #openstack-infra04:09
dstufftmostly this means there won't be a second place to make settings for setup_requires04:10
*** thuc has quit IRC04:10
clarkbmikal: I want you to know that if someone were to read your twitter stuff they would probably think you run an ISP04:12
clarkbmikal: I am sure you would run a great isp04:12
*** sarob has quit IRC04:13
*** wchrisj has quit IRC04:15
*** jepoy_ has joined #openstack-infra04:16
*** jepoy has quit IRC04:18
mordreddstufft: nice!04:22
*** talluri has quit IRC04:23
*** CaptTofu has quit IRC04:24
*** Daisy_ has joined #openstack-infra04:25
*** matsuhashi has joined #openstack-infra04:25
*** psedlak has quit IRC04:28
*** jepoy_ has quit IRC04:28
*** nati_ueno has quit IRC04:28
*** pcrews has joined #openstack-infra04:29
*** psedlak has joined #openstack-infra04:31
*** arosen-home has quit IRC04:33
*** jnoller_ has quit IRC04:34
*** pcrews has quit IRC04:34
*** jnoller has joined #openstack-infra04:37
*** ryanpetrello has quit IRC04:37
lifelessmordred: btw, if you have a few cycles, a patch to testrepository to use pbr (assume that i have a candidate git tree) would be awesome04:39
mordredlifeless: sure thing04:40
mordredlifeless: (do you have a candidate git tree somewhere I can clone it?)04:41
lifelessmordred: haven't pushed it anywhere yet, will do so04:41
*** alexandra_ is now known as alex-luncheon04:42
lifelessmordred: https://github.com/testing-cabal/testrepository04:44
lifelessmordred: basically I need to make sure sdist does the right thing in a git tree, and in a non-git tree, including sucking in README and version handling, like it did in our old bespoke code.04:44
lifelessmordred: then I can do a release and make folk happy04:44
mordredlifeless: can you tell me what you want from version handling?04:46
mordredlifeless: like - how much like openstack things do you want it - vs. how much like what your current code does?04:47
lifelessmordred: current codebase is semver with a special doohicky for unreleased.04:47
lifelessmordred: what openstack does for that is just fine04:47
mordredok. cool04:47
mordredthat's less work04:48
*** VijayTripathi has quit IRC04:48
lifelessyeah, that bit didn't concern me04:48
lifelessits the different doc meta etc handling that I was unsure where to poke at.04:48
lifelessI'm fine in principle with moving other files around to line up with pbr's expectations04:48
mordredk - you look mostsly fine04:48
lifelesssubject to the right to go OMG no on a specific patch :>04:48
mordredlemme make patch04:48
*** talluri has joined #openstack-infra04:49
*** talluri has quit IRC04:53
*** jepoy has joined #openstack-infra04:55
*** jepoy has quit IRC04:56
*** thuc_ has quit IRC04:57
*** thuc has joined #openstack-infra04:58
mikalclarkb: no, just filled with rage at the govt04:59
*** sarob has joined #openstack-infra05:00
*** jnoller has quit IRC05:00
*** xchu_ has joined #openstack-infra05:01
*** thuc has quit IRC05:02
mordredmikal: what's the difference?05:05
*** talluri has joined #openstack-infra05:05
*** nicedice has quit IRC05:05
*** nicedice has joined #openstack-infra05:06
*** talluri has quit IRC05:06
*** talluri has joined #openstack-infra05:07
*** Daisy__ has joined #openstack-infra05:09
mordredlifeless: your tags aren't signed, just fyi05:09
*** Daisy_ has quit IRC05:09
mordredlifeless: I do not know how much you care05:09
*** mgagne has quit IRC05:09
mordredlifeless: but version creation will be weird until you have one signed tag05:09
mordredlifeless: https://github.com/testing-cabal/testrepository/pull/105:11
*** talluri has quit IRC05:11
*** SumitNaiksatam has quit IRC05:11
*** jepoy has joined #openstack-infra05:12
StevenKmordred: Which is partly bong05:13
StevenK+# Copyright (c) 2013 Hewlett-Packard Development Company, L.P.05:14
StevenK-# Copyright (c) 2009-2013 Testrepository Contributors05:14
mordredStevenK: what' partly bong? what did I miss?05:14
StevenKIn setup.py05:14
mordredStevenK: yeah05:14
mordredit's a completely different file05:14
mordredStevenK: I'm pretty sure that "Testrepository Contributors" don't have any copyright in the file05:15
StevenKmordred: Well, your PR makes you a contributor, so ... :-)05:15
mordredah. well. now I get to decide if I'm going to disagree with this read on how the Berne Convention works - or just say screw it and go along05:16
clarkbmordred: honesly that file is so small  you could probably argue it isn't copyrightable :P05:17
mordredclarkb: that's the better argument, tbh05:17
StevenKmordred: It's up to lifeless. Not sure if I even have a valid opinion, since I can't recall if I have any code in testr05:17
mordredto me - Copyright (c) 2009-2013 Testrepository Contributors implies that "Testrepository Contributors" have copyright over this particular work, which, if it is copyrightable in the first place, is invalid05:18
mordredas neither I nor HP have granted any copyright to any of the other contributors05:18
mordredbut tbh - clarkb's opinion makes this much easier05:18
*** markwash has joined #openstack-infra05:20
mordredStevenK: fixed05:21
mordredStevenK, lifeless: actually - the history in that branch was silly - re-submitted https://github.com/testing-cabal/testrepository/pull/205:23
*** markwash has quit IRC05:24
*** markwash_ has joined #openstack-infra05:24
*** xchu_ has quit IRC05:26
jhesketh__Hey guys, looks like zuul is having trouble communicating to gerrit: http://status.openstack.org/zuul/05:26
jhesketh__ah, ignore me, just a delay05:26
lifelessmordred: btw you can push --overwrite ;)05:27
lifelesserm05:27
lifeless-f05:27
lifelessmordred: on github05:27
lifelessmordred: are05:28
lifeless+[files]05:28
lifeless+scripts =05:28
lifeless+[files]05:28
lifelessbah05:28
lifelessis that needed?05:29
lifelessoh hmm, tox. I think I'll disect this patch a little, since tox isn't a thing in this context.05:29
mordredok05:30
mordredfeel free - the bits you need should be there- feel free to do whatever you want05:30
lifelessthanks05:30
*** rpodolyaka has joined #openstack-infra05:31
mordredlifeless: the [files] scriptts = is how you do the equiv of scripts=['testr'], from the old setup.py05:31
lifelesson the (C) thing, see COPYING; e.g. HP was fine (its NAME)05:31
*** oda-g_ has quit IRC05:31
lifelessand its simpler to avoid confusion in lawyers to have it, so I'll reinstate the HP (C) you had05:31
*** sarob has quit IRC05:32
mordredlifeless: ok. works for me05:32
*** reed has quit IRC05:32
mordredlifeless: also- I checked python setup.py sdist and python setup.py build_sphinx - I did not check other potential uses05:34
lifelessthat + pip install is all there is05:35
lifelesssince setup.py test is so awful05:35
*** nicedice has quit IRC05:35
lifelessbut thats a different rant05:35
mordredyah05:35
*** alex-luncheon is now known as Alexandra05:35
lifelessmordred: you have no problem with BSD + Apache2 I presume05:36
mordredlifeless: I do not05:36
*** SumitNaiksatam has joined #openstack-infra05:36
openstackgerritJoshua Hesketh proposed a change to openstack-infra/config: Add zuul overlay to review page  https://review.openstack.org/7983605:37
mordredI also really am not sure there is much that's truly copyrightable in that patch I just sent you05:37
lifelessmordred: I know05:37
lifelessmordred: its pro forma05:37
lifelessmordred: since e.g. debian gets weird about shit, and I can't be bothered ....05:38
mordredyah05:38
mordredlifeless: ok. I'm off to bed - as I said - my patch is your patch - i hope it's helpful - if it's not, you know, no ego attachment to any of it here05:39
lifelessmordred: its super helpful05:39
mordredawesome05:39
lifelessmordred: I'm just doing qa of it05:39
lifelessmordred: looking for surprises, things to document, etc05:39
* mordred does a little jig05:39
lifelessmordred: also tweaking the commit a little :)05:39
mordredoh - I did not change version consumption in testrepository/__init__.py at all05:40
mordredI figure using pbr.version is simple enough if you want to do that - so I left it alone05:40
lifelessmordred: signed tags - any version, or the most recent?05:40
mordredmost recent is the most important05:40
lifelessmordred: I presume e.g. nova has an example of pbr.version use ?05:40
mordredthe calculated version comes from latest signed tags05:40
mordredyup05:40
mordredactually, nova is a bad example - they're weird...05:41
lifelessversion_info = pbr.version.VersionInfo('nova')05:41
mordredyah05:41
lifelessok, what's a good example :)05:41
mordrednah -that's fine05:41
mordredI was just scanning the file and saw all the other stuff05:41
lifelessmordred: so remind me how to tell pbr about the next version # to use ?05:43
dstufftmordred: oh05:44
dstufftand lifeless05:44
dstufftspeaking of setup.py test being awful05:44
dstufftthis stuff i'm doing might make it pretty easy to make seup.py test use pip05:45
lifelessdstufft: how do you get from VersionInfo to a __version__ style version tuple ?05:47
lifelessdstufft: e.g. the same as sys.version_info05:48
dstufftI have no idea what a __version__ style tuple is, there is no standard for that :)05:48
lifelessdstufft: I rather think there is :) - sys.version_info is it, no ?05:48
dstufftAll packages I have have a str __version__ tbh05:48
lifelesssys.version_info05:48
lifelesssys.version_info(major=3, minor=3, micro=1, releaselevel='final', serial=0)05:48
dstufftyea that's something special in python, I don't think that's exposed anywhere05:49
lifelessdstufft: well, its just a tuple05:49
dstufftnamedtuple!05:49
dstufftmaybe what you want is pkg_resources.parse_version05:49
lifelessdstufft: so not very special, but anything that can process said tuple can process other versions in the same structure, so I have a strong interest in using it05:49
dstufftI don't know what CPython itself uses05:50
lifelessaieee no :)05:50
lifelesspkg_resources.parse_version(testrepository.new_version.version)05:50
lifeless('00000000', '00000000', '00000018', '*final')05:50
dstufftlol05:52
*** vkozhukalov has joined #openstack-infra05:53
lifeless0.0.18.5, hmm05:55
lifeless5 commits after 0.0.1805:55
lifelessmordred: how do I get alpha version strings out ?05:56
lifelessmordred: 0.0.18 is now signed05:57
clarkblifeless: you have to tag alphas iirc05:57
*** zhiwei has quit IRC05:57
*** CaptTofu has joined #openstack-infra06:00
jhesketh__is anybody else having a very slow response time from gerrit queries? (ie via the commandline)06:00
jhesketh__or are any root/core infra's around to help me debug?06:00
clarkbjhesketh__: gerrit ls-projects is responsive06:01
clarkbhttp://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=25&rra_id=all there si a bit of iowait there06:02
jhesketh__okay, so I can query successfully from my local machine, but it seems slow06:02
clarkbwonder if that is backups06:02
jhesketh__clarkb: more to the point though our zuul is failing to get a response altogether06:02
* clarkb is about to go to bed and doesn't trust himself to debug06:02
jhesketh__right, possibly06:02
jhesketh__that's okay06:02
*** sarob has joined #openstack-infra06:11
*** unicell has quit IRC06:11
*** unicell has joined #openstack-infra06:14
*** ildikov_ has quit IRC06:15
openstackgerritDavid Pursehouse proposed a change to openstack-infra/jenkins-job-builder: Minor fixes in the trigger documentation  https://review.openstack.org/7985406:16
openstackgerritlifeless proposed a change to openstack-dev/pbr: Fix up some docstrings.  https://review.openstack.org/7985506:19
openstackgerritlifeless proposed a change to openstack-dev/pbr: Return the real class in VersionInfo __repr__.  https://review.openstack.org/7985606:19
*** dstufft|laptop has joined #openstack-infra06:20
*** oubiwann has quit IRC06:26
*** khyati has quit IRC06:29
*** Alexandra is now known as alex-out06:29
lifelessso, I think get_version(pkg, pre_version...) is dead code, no ?06:29
*** lcheng has quit IRC06:30
*** sarob has quit IRC06:43
*** lukego has joined #openstack-infra06:44
*** gokrokve has quit IRC06:46
*** dangers_away has quit IRC06:48
openstackgerritSergey Kolekonov proposed a change to openstack-infra/jenkins-job-builder: ZMQ Event Publisher plugin support added  https://review.openstack.org/7756106:50
*** denis_makogon has joined #openstack-infra06:50
openstackgerritSergey Kolekonov proposed a change to openstack-infra/jenkins-job-builder: Added send-to options support to email-ext plugin  https://review.openstack.org/7360106:50
lifelessyeah, I'm going to need to patch pbr. Tomorrow. mordred :) - pbr isn't compatible with it's own semver doc :>06:51
*** saju_m has joined #openstack-infra06:52
*** rpodolyaka has quit IRC07:00
SergeyLukjanovfungi, clarkb, mordred, jeblair, are you sleeping folks?07:05
SergeyLukjanovwe need a force push for savanna to revert 0.9 sqla07:05
*** pblaho has joined #openstack-infra07:08
*** ildikov_ has joined #openstack-infra07:10
*** mrda is now known as mrda_away07:11
SergeyLukjanovfungi, thank you for the renaming doc, looking on it07:15
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Initialize transifex data for openstack-manuals  https://review.openstack.org/7986607:16
SergeyLukjanovfungi, everything looks ok, I'll do #14 by myself07:17
AJaegerSergeyLukjanov, good morning! If you find some time between renaming to sahara, could you review 79866, please?07:20
openstackgerritValeriy Ponomaryov proposed a change to openstack-infra/config: Fix of using concurrency for tempest in manila job  https://review.openstack.org/7986707:21
SergeyLukjanovAJaeger, morning07:21
SergeyLukjanovAJaeger, sure, I'll take a look on it07:21
*** AaronGreen has quit IRC07:21
*** AaronGr has joined #openstack-infra07:23
*** vponomaryov1 has joined #openstack-infra07:23
*** bknudson has quit IRC07:24
*** flaper87|afk is now known as flaper8707:24
AJaegerSergeyLukjanov, thanks07:25
*** yolanda has joined #openstack-infra07:26
*** bknudson has joined #openstack-infra07:26
SergeyLukjanovAJaeger, could you please capitalize and remove ',' in comment in https://review.openstack.org/#/c/79866/1/modules/jenkins/files/slave_scripts/upstream_translation_update_manuals.sh07:30
AJaegerSergeyLukjanov, will do - thanks07:33
*** skraynev_afk is now known as skraynev07:34
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Initialize transifex data for openstack-manuals  https://review.openstack.org/7986607:34
*** jlibosva has joined #openstack-infra07:35
*** CaptTofu has quit IRC07:35
SergeyLukjanovAJaeger, thank you07:35
AJaegerSergeyLukjanov, thanks for spotting!07:36
*** iv_m_ has joined #openstack-infra07:38
*** vkozhukalov has quit IRC07:38
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard: Rename migration revisions  https://review.openstack.org/7986907:40
*** iv_m_ is now known as iv_m07:40
*** vponomaryov1 has quit IRC07:41
*** sarob has joined #openstack-infra07:41
*** vponomaryov1 has joined #openstack-infra07:41
*** gokrokve has joined #openstack-infra07:42
*** gokrokve has quit IRC07:47
*** sarob has quit IRC07:48
*** rcarrillocruz has joined #openstack-infra07:50
*** rlandy has joined #openstack-infra07:50
AJaegerSergeyLukjanov, did you forgot to review 79866 with a vote?07:51
*** dstufft|laptop has quit IRC07:57
*** jcoufal has joined #openstack-infra07:59
SergeyLukjanovAJaeger, /me searching for coffee :) will do08:02
*** rossella_s has joined #openstack-infra08:06
*** gokrokve has joined #openstack-infra08:13
*** morganfainberg is now known as morganfainberg_Z08:17
*** locke105 has quit IRC08:17
*** vponomaryov1 has left #openstack-infra08:18
*** gokrokve has quit IRC08:18
*** eharney has joined #openstack-infra08:20
*** matsuhashi has quit IRC08:21
*** jgallard has joined #openstack-infra08:24
*** bada_ has joined #openstack-infra08:26
*** bada has quit IRC08:30
openstackgerritFlavio Percoco proposed a change to openstack-infra/devstack-gate: Archive config files along with logs  https://review.openstack.org/6934408:32
*** ociuhandu has joined #openstack-infra08:35
*** amcrn has joined #openstack-infra08:35
*** jooools has joined #openstack-infra08:35
*** jungleboyj has quit IRC08:38
*** e0ne has joined #openstack-infra08:40
*** saju_m has quit IRC08:41
*** hashar has joined #openstack-infra08:41
*** gokrokve has joined #openstack-infra08:43
*** sarob has joined #openstack-infra08:44
*** ekarlso has quit IRC08:47
*** ekarlso has joined #openstack-infra08:47
*** gokrokve has quit IRC08:47
*** e0ne has quit IRC08:50
*** amcrn_ has joined #openstack-infra08:50
*** amcrn has quit IRC08:53
*** iv_m has left #openstack-infra08:53
*** amcrn_ is now known as amcrn08:53
*** yassine has joined #openstack-infra08:54
*** e0ne has joined #openstack-infra08:56
*** jepoy_ has joined #openstack-infra08:57
*** e0ne_ has joined #openstack-infra08:57
*** jepoy has quit IRC08:59
*** e0ne has quit IRC09:01
*** zhiyan is now known as zhiyan_09:06
*** jpich has joined #openstack-infra09:07
*** dizquierdo has joined #openstack-infra09:08
*** saju_m has joined #openstack-infra09:09
*** derekh has joined #openstack-infra09:12
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard: Fill the last_login field  https://review.openstack.org/7988709:18
*** sarob has quit IRC09:18
*** Daisy__ has quit IRC09:20
*** jcoufal has quit IRC09:20
*** jcoufal has joined #openstack-infra09:22
*** jhesketh_ has quit IRC09:23
*** jhesketh__ has quit IRC09:24
*** enikanorov_ has joined #openstack-infra09:24
*** CaptTofu has joined #openstack-infra09:32
*** johnthetubaguy has joined #openstack-infra09:33
*** jepoy_ has quit IRC09:35
*** yaguang has quit IRC09:36
*** jhesketh__ has joined #openstack-infra09:36
*** afazekas has joined #openstack-infra09:36
*** jhesketh_ has joined #openstack-infra09:36
*** CaptTofu has quit IRC09:37
*** amcrn has quit IRC09:38
*** dizquierdo has quit IRC09:40
*** gokrokve has joined #openstack-infra09:43
*** gokrokve has quit IRC09:48
*** denis_makogon has quit IRC09:51
*** hashar has quit IRC09:58
*** pbelanyi has joined #openstack-infra10:01
*** zhiwei has joined #openstack-infra10:06
*** yamahata__ has quit IRC10:10
*** hashar has joined #openstack-infra10:10
*** jp_at_hp has joined #openstack-infra10:13
*** sarob has joined #openstack-infra10:14
*** yamahata has quit IRC10:17
*** rlandy has quit IRC10:18
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard: Missing security decorators added  https://review.openstack.org/7989510:18
*** branen_ has quit IRC10:18
*** jerryz has quit IRC10:20
*** andreaf has joined #openstack-infra10:26
*** rlandy has joined #openstack-infra10:31
*** dizquierdo has joined #openstack-infra10:32
*** jamielennox is now known as jamielennox|away10:34
*** mugsie_ has quit IRC10:36
*** dcramer__ has quit IRC10:41
*** gokrokve has joined #openstack-infra10:43
openstackgerritRussell Bryant proposed a change to openstack-infra/config: Create a nova-docker repo  https://review.openstack.org/7990010:44
openstackgerritRussell Bryant proposed a change to openstack-infra/config: Create a nova-docker repo  https://review.openstack.org/7990010:46
*** sarob has quit IRC10:47
*** gokrokve has quit IRC10:48
openstackgerritVictor Stinner proposed a change to openstack/requirements: Allow trollius 0.2  https://review.openstack.org/7990110:48
*** saju_m has quit IRC10:49
*** eharney has quit IRC10:50
*** lcostantino has joined #openstack-infra10:51
jlibosvahello, is there a way how can I re-trigger check on gate even though Jenkins gave +1?10:53
StevenKjlibosva: 'recheck no bug' as a review comment10:54
jlibosvaStevenK: I tired that but it didn't trigger recheck10:54
openstackgerritRussell Bryant proposed a change to openstack-infra/config: Create a nova-docker repo  https://review.openstack.org/7990010:54
* fungi missed the eldritch-horror pip branch discussion... that's what i get for taking a nap10:56
fungijlibosva: do you have a link to the review where that didn't work?10:57
fungiSergeyLukjanov: where's the savanna reqs change you need forced in?10:57
SergeyLukjanovfungi, morning10:58
SergeyLukjanovfungi, all our d-g jobs are failing now10:58
SergeyLukjanovand I can't understand why10:58
SergeyLukjanovfungi, potential reason is that we've merged sqla 0.910:58
*** bada_ has quit IRC10:58
SergeyLukjanovfungi, and IIRC due to the devstack issue with requirements revert failing10:59
jlibosvafungi: yes, https://review.openstack.org/#/c/58695/2710:59
fungiSergeyLukjanov: there was a logic error in devstack which was causing changes not to get tested with the global requirements list, and instead tested them with the combined requirements of projects being integrated. the fix for that finally merged yesterday but it's possible there were some requirements changes in projects which ended up being unsupportable and need to get adjusted11:00
*** Ryan_Lane has quit IRC11:00
SergeyLukjanovfungi, oh, got it11:00
SergeyLukjanovfungi, so, could you, please, forse push https://review.openstack.org/#/c/79647/11:00
SergeyLukjanovto revert11:00
SergeyLukjanovand probably take a look on how d-g jobs failing for savanna11:00
fungilooking over the failure log on that one now...11:01
*** spm has joined #openstack-infra11:01
SergeyLukjanovfungi, thank you11:01
SergeyLukjanovin fact I see "ImageKilledException: Image 8c68628a-3cdd-4e4a-87aa-9ed2562a6cd4 'killed' while waiting for 'active'" that's libvirt issue IIRC and horizon 50011:02
*** Ryan_Lane has joined #openstack-infra11:02
*** AaronGr has quit IRC11:02
fungiyeah, on the check-tempest-dsvm-savanna-full failure, that looks like one i've seen before11:02
cyeohdoes anyone know how I can get the ical feed for the openstack meetings updated with a new meeting?11:04
fungithe other two look like error responses from a service api11:04
fungicyeoh: you update the wiki. ttx subscribes to updates to that article and manually updates the ical feed11:05
cyeohfungi: ah ok, thanks! Have updated the wiki so I'll just wait :-)11:05
fungicyeoh: http://git.openstack.org/cgit/openstack-infra/gerrit-powered-agenda/ hopes to fix that11:05
*** AaronGr has joined #openstack-infra11:05
cyeohfungi, ah that'd be nice11:06
SergeyLukjanovfungi, the only change we merged in the last few days is bump sqla, so, revert should help11:06
cyeohwas also wondering if the various commands like #topic, #startmeeting etc are documented anywhere?11:07
fungicyeoh: a little here http://ci.openstack.org/irc.html#id311:08
cyeohfungi: thanks again11:08
fungicyeoh: and more at https://git.openstack.org/cgit/openstack-infra/meetbot/tree/doc/Manual.txt11:08
fungiwe probably ought to publish that in a slightly more discoverable place11:09
*** mugsie has joined #openstack-infra11:09
SergeyLukjanovfungi, before repo rename we need the to have landed https://review.openstack.org/#/c/79063/ and https://review.openstack.org/#/c/79347/ to add aliases11:09
cyeohheh, thx - yea I did google a bit first for it, but I didn't know that it was called meetbot so didn't find anything11:09
SergeyLukjanovbut both blocked by failing gate11:09
*** jgallard has quit IRC11:10
fungiSergeyLukjanov: well, there's actually no way of knowing what else might have changed in global-requirements.txt which is breaking savanna devstack jobs now that it's being applied again. in actuality i don't think any change to your requirements.txt or test-requirements.txt files in savanna should affect devstack jobs now that global-requirements.txt is being used to override those11:11
SergeyLukjanovfungi, savanna-related tests passed successfully11:12
SergeyLukjanovhttp://logs.openstack.org/47/79647/2/check/check-tempest-dsvm-savanna-full/877f1ca/console.html#_2014-03-11_19_16_43_14011:12
fungiSergeyLukjanov: so i don't have a lot of faith that 79647 will make any difference there at all11:12
fungibut i'll go ahead and force submit it to merge11:12
SergeyLukjanovfungi, thx11:13
jlibosvathanks Jeremy :) seems like Jenkins is just ignoring my name11:13
SergeyLukjanovfungi, there were only oslo sync in savanna and sqla version bump for the last 5 days11:13
SergeyLukjanovfungi, and savanna tests pass11:13
SergeyLukjanovfungi, so, I just don't know how it could affect other tests...11:13
fungiSergeyLukjanov: right, i mean the devstack jobs for savanna should be completely ignoring versions listed in your requirements files in savanna now that devstack is fixed, so i'm pretty sure changing your requirements in savanna isn't going to change the behavior for those. it would change behavior for your unit tests though11:15
SergeyLukjanovfungi, yup... if so, then it looks like Savanna addition to the d-g breaks something...11:15
fungiSergeyLukjanov: probably11:16
fungiSergeyLukjanov: okay, the three you requested are merged11:19
SergeyLukjanovfungi, thank you11:19
SergeyLukjanovfungi, I've recheck https://review.openstack.org/#/c/78789/11:19
SergeyLukjanovit's just a doc change, so, we should wee how it works11:19
fungiSergeyLukjanov: okay. i'm going to start putting jenkins masters in shutdown mode now11:20
SergeyLukjanovfungi, thank you!11:20
fungiwhich means currently running jobs will finish but new jobs will wait in a pending state in zuul11:20
*** hashar has quit IRC11:20
SergeyLukjanovfungi, I've reviewed renaming pad and it's lgtm (one path fixed)11:21
SergeyLukjanovand I'm volunteering to update .gitreviews on all updated repos11:21
fungiSergeyLukjanov: thanks--i guess i had an extra letter in that line or something?11:22
SergeyLukjanovfungi, double paste path11:22
SergeyLukjanovI mean doubled11:22
fungiahh, overzealous middle button ;)11:22
SergeyLukjanov:)11:22
*** e0ne_ has quit IRC11:22
fungithanks11:22
SergeyLukjanovthanks to *you* :)11:23
*** e0ne has joined #openstack-infra11:23
*** CaptTofu has joined #openstack-infra11:24
*** masayukig has joined #openstack-infra11:25
*** bada has joined #openstack-infra11:25
fungiokay, all jenkins masters are quiescing now11:25
fungitime to see if statusbot catches fire and burns to the ground11:25
*** e0ne_ has joined #openstack-infra11:26
fungi#status alert test/gate jobs are queuing now in preparation for gerrit maintenance at 12:00 utc (eta to resume is 12:30 utc)11:27
openstackstatusfungi: sending alert11:27
SergeyLukjanovaaand looks like we still alive11:27
SergeyLukjanovbut topic is unchanged11:27
openstackstatusNOTICE: test/gate jobs are queuing now in preparation for gerrit maintenance at 12:00 utc (eta to resume is 12:30 utc)11:27
*** ChanServ changes topic to "test/gate jobs are queuing now in preparation for gerrit maintenance at 12:00 utc (eta to resume is 12:30 utc)"11:27
fungipatience ;)11:27
SergeyLukjanovoh11:27
SergeyLukjanov:)11:27
openstackstatusfungi: finished sending alert11:28
SergeyLukjanovawesome11:28
SergeyLukjanovand the same with other channels11:28
SergeyLukjanovexcept both savanna / sahara ;)11:28
*** CaptTofu has quit IRC11:28
fungiwith the latest changes, it's now in >50 official irc channels, so it has a built-in throttle to prevent it from getting killed by the irc server for abuse11:28
fungiwhich is also why it now replies to you when it starts and finishes updating11:29
*** e0ne_ has quit IRC11:29
SergeyLukjanovyeah, I've reviewed it, but forget about throttle11:29
*** e0ne has quit IRC11:29
*** e0ne_ has joined #openstack-infra11:29
*** bada has quit IRC11:30
sdaguefungi: man, what are you doing up so early :)11:31
fungisdague: breaking things of course11:31
*** masayukig has quit IRC11:31
fungisdague: i'm usually up this early, i just don't normally jump in irc this early11:31
sdagueheh11:31
fungii typically wake up around 10:00 utc11:32
sdagueah, I jump straight to irc in the morning. with my wife still sleeping might as well get some work done early.11:32
sdaguefungi: regardless of dst?11:32
* fungi doesn't subscribe to his government's notions of clock-related fascism11:33
sdaguedamn fiat clocks!11:33
fungii find that with the interrupt-driven nature of what i work on, if i don't make time for non-openstack things first, i frequently find that it's suddenly evening and i never showered and ate breakfast11:34
SergeyLukjanovsdague, morning11:34
fungiwhich i'm going to go do quickly now while i wait for these running jobs to wrap up11:34
* fungi will brb11:34
sdagueSergeyLukjanov: morning11:35
sdagueSergeyLukjanov: so what are the changes you need landed, and are they before or after the rename?11:36
SergeyLukjanovsdague, fungi, heh, I have inversed day schedule ;) wake up around 5am UTC (9am local) and creep to the bad around 10pm UTC11:36
SergeyLukjanovsdague, all CRs needed to be landed before the rename already landed11:37
sdagueok11:37
SergeyLukjanovsdague, after repos rename - d-g update + devstack update11:37
SergeyLukjanovand I hope it'll just work11:37
sdaguesounds good, point me to them once we are done11:37
SergeyLukjanovprobably several force pushed commits :)11:37
SergeyLukjanovsdague, sure, thanks11:37
*** CaptTofu has joined #openstack-infra11:38
*** johnthetubaguy1 has joined #openstack-infra11:39
*** johnthetubaguy has quit IRC11:42
*** fifieldt has quit IRC11:42
*** gokrokve has joined #openstack-infra11:43
*** sarob has joined #openstack-infra11:44
*** mbacchi has joined #openstack-infra11:46
*** gokrokve has quit IRC11:47
*** talluri has joined #openstack-infra11:48
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard: Missing security decorators added  https://review.openstack.org/7989511:52
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: Add n-novnc and n-cauth  https://review.openstack.org/5572111:52
*** talluri has quit IRC11:52
*** mkerrin has quit IRC11:53
*** lcestari has joined #openstack-infra11:55
*** mwagner_lap has quit IRC11:59
*** jamespage has joined #openstack-infra11:59
fungiokay, bacvk11:59
fungilooks like there are some savanna-related changes which ended up approved into the gate. not sure what will happen to those when the jobs no longer exist... guess we'll find out12:00
*** spm has left #openstack-infra12:00
SergeyLukjanovfungi, they aren't important, I'll reapprove them if needed12:01
*** mkerrin has joined #openstack-infra12:01
fungii'm ready to stop gerrit, but i'm going to rename your gerrit groups first so that manage-projects doesn't recreate them when it imports the updated acls12:01
*** rlandy_ has joined #openstack-infra12:02
*** aysyd has joined #openstack-infra12:02
SergeyLukjanovfungi, ok12:03
*** rlandy has quit IRC12:04
*** mburned_out is now known as mburned12:04
*** eharney has joined #openstack-infra12:05
funginow to test whether statusbot will still correctly update topics when changing an alert without cancelling it first12:05
fungi#status gerrit on review.openstack.org is down for maintenance (eta to resume is 12:30 utc)12:06
openstackstatusfungi: unknown command12:06
fungiduh12:06
fungi#status alert gerrit on review.openstack.org is down for maintenance (eta to resume is 12:30 utc)12:06
openstackstatusfungi: sending alert12:06
fungiit's not actually down yet, but the last jobs are about wrapped up12:08
*** jgallard has joined #openstack-infra12:09
*** talluri has joined #openstack-infra12:12
fungiand statusbot seems not to have updated correctly (or it's still trying)12:12
*** e0ne has joined #openstack-infra12:13
SergeyLukjanovfungi, I see "[16:06:38]  <openstackstatus> NOTICE: gerrit on review.openstack.org is down for maintenance (eta to resume is 12:30 utc)" in some channels12:14
fungiokay, hopefully it's just taking its time12:14
*** julim has joined #openstack-infra12:17
*** sarob has quit IRC12:17
*** dprince has joined #openstack-infra12:18
*** yassine has quit IRC12:19
*** yassine has joined #openstack-infra12:19
*** yassine has quit IRC12:19
*** yassine has joined #openstack-infra12:19
*** yassine has quit IRC12:20
*** yassine has joined #openstack-infra12:20
*** enikanorov_ has quit IRC12:21
fungii'm killing these last few long-running jobs because they're all in check and either non-voting or associated with changes which have a bunch of other failed jobs12:22
fungiand now we're ready12:23
fungi#status alert gerrit on review.openstack.org is down for maintenance (revised eta to resume is 13:00 utc)12:23
openstackstatusfungi: sending alert12:23
*** david-lyle has quit IRC12:24
openstackstatusNOTICE: gerrit on review.openstack.org is down for maintenance (revised eta to resume is 13:00 utc)12:24
*** ChanServ changes topic to "gerrit on review.openstack.org is down for maintenance (revised eta to resume is 13:00 utc)"12:24
openstackstatusfungi: finished sending alert12:24
*** enikanorov_ has joined #openstack-infra12:26
fungidb updates in step #5 done. working on filesystem modifications now12:27
* SergeyLukjanov crossed fingers12:30
*** bogdando has quit IRC12:33
fungifilesystem updates done. going to skip steps 8-12 and do step 13 first since i just realized that otherwise i may end up with a mess courtesy of manage-projects12:33
*** saju_m has joined #openstack-infra12:37
*** esker has joined #openstack-infra12:39
afazekasOn https://review.openstack.org/ I got 50312:39
fungiafazekas: see /topic12:40
afazekas:)12:40
*** jasondotstar has joined #openstack-infra12:40
fungiokay, #13 done, dropping back to 8 now12:41
*** ArxCruz has joined #openstack-infra12:42
fungigerrit is back up, working on remaining steps niow12:42
*** gokrokve has joined #openstack-infra12:43
*** jhesketh__ has quit IRC12:44
*** jhesketh_ has quit IRC12:45
fungistep 9 done, zuul shows jobs running again, cancelling alert before proceeding12:45
fungi#status ok12:45
openstackstatusfungi: sending ok12:45
tristanCGood work!12:45
SergeyLukjanov fungi, yay! I'm going to send .gitreview updates12:45
openstackstatusfungi: finished sending ok12:45
fungiSergeyLukjanov: not done yet12:46
fungiSergeyLukjanov: your projects still aren't done being renamed until i get to the end12:46
*** hashar has joined #openstack-infra12:46
fungii merely ended the alert because gerrit and jenkins are back online again12:46
fungiokay, 79097 and 64788 have been approved12:46
SergeyLukjanovfungi, yup12:47
*** CaptTofu has quit IRC12:47
*** zhiyan_ is now known as zhiyan12:47
*** CaptTofu has joined #openstack-infra12:47
fungito speed this along, i'm preparing to pull an early update on ci-puppetmaster once they get through the gate in a moment12:48
*** gokrokve has quit IRC12:48
fungiotherwise we'll be waiting on the 13:00 cron pulse before i can update the puppet config on review.o.o12:49
*** johnthetubaguy1 has quit IRC12:49
openstackgerritA change was merged to openstack-infra/config: First step of renaming Savanna to Sahara  https://review.openstack.org/7909712:49
SergeyLukjanovfungi, ok12:49
*** rlandy__ has joined #openstack-infra12:50
fungiSergeyLukjanov: also, i expect at least some of your jenkins jobs will be broken for the next few minutes too until jenkins-job-builder runs on the jenkins masters12:50
SergeyLukjanovfungi, git it12:51
*** thuc_ has joined #openstack-infra12:51
SergeyLukjanov:)12:51
fungibut it should sort itself out within about 15 minutes12:51
SergeyLukjanovgot*12:51
openstackgerritA change was merged to openstack-infra/config: Rename a StackForge project  https://review.openstack.org/6478812:51
fungiokay, updating the puppetmaster so i can continue12:51
*** johnthetubaguy has joined #openstack-infra12:51
*** ociuhandu has quit IRC12:52
*** CaptTofu has quit IRC12:52
*** esker has quit IRC12:52
*** esker has joined #openstack-infra12:53
*** rlandy_ has quit IRC12:53
*** rlandy has joined #openstack-infra12:53
*** dkranz has joined #openstack-infra12:54
*** openstackgerrit has quit IRC12:54
*** openstackgerrit has joined #openstack-infra12:54
*** rlandy__ has quit IRC12:54
*** bogdando has joined #openstack-infra12:55
fungialso looks like statusbot never updated the topics back, so something like 54 channels we need to reset topics on12:55
*** johnthetubaguy has quit IRC12:55
*** weshay has quit IRC12:55
*** johnthetubaguy has joined #openstack-infra12:55
SergeyLukjanovfungi, heh :(12:57
*** changbl has quit IRC12:57
fungimanage-projects is running for the 9 renamed repositories now12:57
fungilooks like it ran clean... checking behind it12:58
AJaegerfungi, IRC worked on openstack-doc - so only 53 ;)12:59
fungiAJaeger: nice!12:59
fungispot checks of renamed projects' acls look correct13:01
fungialso puppet agent is restarted on review.o.o now13:02
SergeyLukjanovfungi, looks like chanserv start returning old topics, is it you?13:02
fungiSergeyLukjanov: oh, is it still trying?13:02
SergeyLukjanovfungi, yup13:02
fungihasn't updated in here yet13:02
*** dcramer__ has joined #openstack-infra13:02
SergeyLukjanovfungi, in #heat for e13:02
SergeyLukjanov[16:45:28] ChanServ changed the topic to support @ https://ask.openstack.org | developer wiki @ https://wiki.openstack.org/wiki/Heat | development @ https://launchpad.net/heat | logged @ http://eavesdrop.openstack.org/irclogs/%23heat/13:02
*** rlandy_ has joined #openstack-infra13:03
fungiSergeyLukjanov: yeah, but back at 12:45 utc it also responded to me that it was done sending updates to channels13:03
fungiso i don't think it fixed any more of them after that13:03
fungifor example, the topic in here hasn't been restored to normal yet13:04
fungii haven't checked its logs, but will once we're settled on the final renaming tasks13:04
SergeyLukjanovfungi, ok, thank you13:04
fungiSergeyLukjanov: i suppose you can send your .gitreview updates to those projects now and we'll make sure they run jobs and whatnot13:05
SergeyLukjanovfungi, already done13:05
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard-webclient: Hide "Create Project" for non-superusers  https://review.openstack.org/7993913:05
*** rlandy has quit IRC13:05
fungizuul reports "Last reconfigured: Wed Mar 12 2014 12:57:46 GMT+0000 (UTC)" so it should know what to do with them if they got submitted to gerrit after that13:05
SergeyLukjanovfungi, yup13:07
fungiif not, recheck no bug13:07
SergeyLukjanovfungi, I've send them ~13:0013:07
SergeyLukjanovfungi, I'll track them13:07
fungigreat13:07
fungithanks!13:07
AJaegerfungi, 79932,1 has one "NOT_REGISTERED"13:07
SergeyLukjanovfungi, could you, please, approve https://review.openstack.org/#/c/78960/ too13:08
AJaegerfungi, same with 79333 - the ones afterwards look fine so far13:08
SergeyLukjanovfungi, to make us able to move to the channel13:08
fungiAJaeger: yeah, that probably means that jjb hadn't run on any of the jenkins masters to create it yet13:08
SergeyLukjanovfungi,yeah, sounds like jjb isnt created jobs yet13:08
*** openstackgerrit has quit IRC13:08
*** openstackgerrit has joined #openstack-infra13:08
fungidone. doesn't look like there were any meetings going on yet13:09
SergeyLukjanovfungi, thx13:10
fungii'm going to start analyzing the statusbot problems13:10
*** rcarrillocruz1 has joined #openstack-infra13:12
*** rcarrillocruz has quit IRC13:12
fungi2014-03-12 12:45:39,086 INFO statusbot.bot: Setting topic on #openstack-infra to Discussion of OpenStack Project Infrastructure | Docs http://ci.openstack.org/ | Bugs https://launchpad.net/openstack-ci | Code https://git.openstack.org/cgit/openstack-infra/13:13
fungiexcept... it didn't13:13
*** talluri has quit IRC13:13
SergeyLukjanovfungi, heh13:14
*** dkliban has joined #openstack-infra13:14
funginothing useful in the debug log either. we may need to add a belt and braces so it checks behind itself and retries topic changes multiple times13:15
*** sarob has joined #openstack-infra13:15
fungiin the meantime, at least the log says what it *thought* it did, so i'll check behind it and fix what it didn't13:15
*** rlandy__ has joined #openstack-infra13:15
*** jeckersb_gone is now known as jeckersb13:15
*** esker has quit IRC13:16
SergeyLukjanovfungi, are you ok if I self-approve such cleanup https://review.openstack.org/#/c/79543/ for sahara renaming when env will be ready?13:16
*** ociuhandu has joined #openstack-infra13:16
fungiSergeyLukjanov: i'll review it real quick13:17
fungiSergeyLukjanov: keep in mind that if the group name is the same as the project name (after the /) then it's not actually needed13:17
*** rlandy_ has quit IRC13:18
SergeyLukjanovfungi, yup, just for visibility13:18
fungiokay13:18
SergeyLukjanovfungi, or you think it's better to remove?13:18
*** mriedem has joined #openstack-infra13:18
fungii think it's likely to cause a lot of other people to not realize it's an override and start doing the same thing on all new project additions13:18
fungibut we can fix it in a cleanup pass. i'd like to clear out a lot of the old cruft in that file anyway (remove unneeded upstream parameters too)13:19
fungii also want to normalize the gerrit acls and collapse some of them, but i've not been finding the time13:19
openstackgerritDavanum Srinivas (dims) proposed a change to openstack-infra/devstack-gate: WIP : Try patched libvirt from @hallyn - Not for review  https://review.openstack.org/7981613:19
fungibecause right now a lot of people cargo cult what they see in existing files into their new projects, which just adds new cruft13:20
SergeyLukjanovfungi, yeah, it's a big issue for acls13:20
*** johnthetubaguy1 has joined #openstack-infra13:21
SergeyLukjanovfungi, I'll take a look on projects list and try to clenup groups/upstream13:21
*** johnthetubaguy has quit IRC13:21
fungiSergeyLukjanov: apprecuiated! there's no hurry on that though... you've got your hands full with your renaming13:21
SergeyLukjanovfungi, now, I should wait for some time to see how jobs are running :)13:22
SergeyLukjanovfungi, I've retriggered https://review.openstack.org/#/c/79099/13:23
AJaegerSergeyLukjanov, time for a coffee or two - and send a virtual one to fungi ;)13:23
SergeyLukjanovAJaeger, yeah :)13:23
fungithanks!13:24
*** sweston has quit IRC13:25
SergeyLukjanovfungi, heh, looks like we'll wait for a quite long time before sahara will be unblocked :( queue isn't empty :)13:25
*** rlandy_ has joined #openstack-infra13:25
*** ryanpetrello has joined #openstack-infra13:27
sdaguefungi: the fails in the gate right now are weird - https://jenkins07.openstack.org/job/gate-tempest-dsvm-full/2000/console13:27
sdaguesomething wrong with nodepool?13:27
sdagueor the images13:27
sdaguehttps://jenkins07.openstack.org/job/gate-tempest-dsvm-full/2000/console13:27
fungisdague: not that anybody's told me yet. care to enlighten?13:27
sdague2014-03-12 13:12:35.413 | /opt/stack/new/devstack-gate/devstack-vm-gate.sh: line 269: cd: /opt/stack/new/devstack: No such file or directory13:27
fungilooking13:28
*** rlandy__ has quit IRC13:28
SergeyLukjanovsdague, fungi, probably it's because of job id 2000 (millennium failure)? :)13:28
sdagueso I think review 80k will get allocated today13:29
sdaguewill be interesting to see what it is13:29
fungihttp://logs.openstack.org/53/79053/2/gate/gate-tempest-dsvm-full/4e8262b/logs/devstack-gate-setup-workspace-new.txt.gz13:29
fungi"fatal: https://git.openstack.org/openstack/savanna-dashboard/info/refs not found: did you run git update-server-info on the server?13:29
sdagueah, so everything is borked until we merge those things13:29
fungii guess we need to cram the d-g change through SergeyLukjanov13:30
sdagueok, where are the rename patches?13:30
SergeyLukjanovhttps://review.openstack.org/#/c/79099/13:30
SergeyLukjanovsdague, fungi ^^13:30
*** thuc_ has quit IRC13:30
fungiremaining patches in that topic are https://review.openstack.org/#/q/status:open+branch:master+topic:savanna-sahara,n,z13:30
*** thuc has joined #openstack-infra13:31
sdaguefungi: so I think you shuold just ninja merge - https://review.openstack.org/#/c/79099/13:31
fungisdague: yep13:31
sdagueyou have my +2 on it13:31
sdaguefor backup13:31
SergeyLukjanovyeah, d-g is non-conditional13:31
sdaguefungi: then can we clear dtroyer's -2 via ninja means in https://review.openstack.org/#/c/79353/3 ?13:32
openstackgerritA change was merged to openstack-infra/devstack-gate: Rename Savanna to Sahara  https://review.openstack.org/7909913:32
fungisdague: i've spirited him away13:32
*** CaptTofu has joined #openstack-infra13:34
SergeyLukjanovfungi, sdague, I think that we shouldn't approve https://review.openstack.org/#/c/79353/ before rechecking - it'll not affects anyone except sahara13:34
fungialso reshuffled the gate to start retesting anything which hadn't already been kicked out13:34
*** esker has joined #openstack-infra13:34
*** rlandy__ has joined #openstack-infra13:34
*** thuc has quit IRC13:35
sdagueSergeyLukjanov: sure, but clean check handles than anyway13:35
fungiback to checking channel topics, but ping me again if anything else looks horribly broken13:36
sdagueoh, I guess it doesn't quite, because saraha is nv13:36
fungimmm, #openstack-bacon13:36
*** thomasem has joined #openstack-infra13:36
SergeyLukjanovheh, the one positive thing for me that we catch name issue earlier... renaming being in the sync gate is much painful13:36
*** rlandy_ has quit IRC13:37
*** smarcet has joined #openstack-infra13:37
*** dcramer__ has quit IRC13:37
sdagueagreed13:39
*** jgrimm has quit IRC13:40
fungigiven that all devstack jobs probably want to clone savanna (well, now sahara), i should probably at a minimum start rebuilds of the devstack-precise images in all providers so we don't ddos our git servers again13:41
SergeyLukjanovfungi, that's correct13:41
sdaguefungi: good call13:41
sdagueman, who needs git servers anyway13:42
sdaguewe should just do this with smoke signals13:42
*** rlandy__ is now known as rlandy13:42
dims_lol13:43
*** nosnos has quit IRC13:43
*** gokrokve has joined #openstack-infra13:43
openstackgerritAntoine Musso proposed a change to openstack-infra/zuul: cloner to easily clone dependent repositories  https://review.openstack.org/7037313:44
*** adalbas has joined #openstack-infra13:47
*** sarob has quit IRC13:47
fungi"Failed to fetch http://nov.ec2.archive.ubuntu.com/ubuntu/pool/universe/q/qpid-python/python-qpid_0.12-2_all.deb ..." huh?13:48
fungihttp://logs.openstack.org/65/77965/9/gate/gate-tempest-dsvm-large-ops/87dc2df/logs/devstacklog.txt.gz13:48
*** weshay has joined #openstack-infra13:48
*** gokrokve has quit IRC13:48
fungidoes hpcloud seriously fetch debian packages from amazon?13:48
fungiwell, rather does archive.ubuntu.com seriously serve their packages from amazon?13:49
*** talluri has joined #openstack-infra13:49
openstackgerritArx Cruz proposed a change to openstack-infra/nodepool: Adding the possibility to pass a cleanup timeout to nodepool  https://review.openstack.org/7126113:49
fungibut yeah, i can't resolve nov.ec2.archive.ubuntu.com either13:49
*** matsuhashi has joined #openstack-infra13:50
*** wchrisj has joined #openstack-infra13:51
openstackgerritA change was merged to openstack-infra/config: Add logging to the new #openstack-sahara channel  https://review.openstack.org/7896013:52
sdaguefungi: it wouldn't surprise me13:53
sdagueor at least that it's a valid mirror13:53
openstackgerritArx Cruz proposed a change to openstack-infra/config: Initial support to a custom puppet master server  https://review.openstack.org/7758713:53
fungiapparently cloud-init picks your package mirror for you13:53
*** talluri has quit IRC13:54
*** matsuhashi has quit IRC13:56
sdagueI wonder where it picks it from13:56
*** yamahata has joined #openstack-infra13:57
fungithat's one of those ubuntu mysteries i try not to look to deeply into13:57
fungibecause it will probably just send me running screaming if i discover the truth13:58
*** vhoward- has joined #openstack-infra13:58
*** zhiwei has left #openstack-infra13:59
*** matsuhashi has joined #openstack-infra14:00
*** mwagner_lap has joined #openstack-infra14:01
*** markmcclain has joined #openstack-infra14:01
openstackgerritA change was merged to openstack-infra/config: Rename savanna group to sahara  https://review.openstack.org/7954314:02
*** dcramer__ has joined #openstack-infra14:02
*** e0ne_ has joined #openstack-infra14:03
*** dims_ has quit IRC14:03
*** openstack has joined #openstack-infra14:04
*** e0ne has quit IRC14:05
*** ChanServ sets mode: +o fungi14:06
*** fungi changes topic to "Discussion of OpenStack Project Infrastructure | Docs http://ci.openstack.org/ | Bugs https://launchpad.net/openstack-ci | Code https://git.openstack.org/cgit/openstack-infra/"14:06
*** ChanServ sets mode: -o fungi14:06
*** dkranz has quit IRC14:07
*** rcleere has joined #openstack-infra14:09
*** talluri has joined #openstack-infra14:10
*** dims_ has joined #openstack-infra14:15
*** jasondotstar has quit IRC14:16
*** jepoy has joined #openstack-infra14:16
openstackgerritAntoine Musso proposed a change to openstack-infra/zuul: cloner to easily clone dependent repositories  https://review.openstack.org/7037314:17
*** mfer has joined #openstack-infra14:17
*** talluri has quit IRC14:20
jeblairfungi: sitrep?14:22
*** chandan_kumar has quit IRC14:22
fungistill updating topics in channels, but otherwise all sitnorm14:22
sdaguejeblair: I think things are generally good on the rename14:22
*** dkranz has joined #openstack-infra14:22
fungibiggest hiccup was that we forgot that devstack-gate would die if it couldn't update savanna repos from git.o.o after the rename14:23
fungiso had to cram the d-g change sooner in to stop the bleeding14:23
*** matsuhashi has quit IRC14:23
*** talluri has joined #openstack-infra14:24
*** chandan_kumar has joined #openstack-infra14:25
SergeyLukjanovyup14:25
jeblairfungi: ok.  i see you did decide to road test statusbot :)14:25
*** gokrokve has joined #openstack-infra14:25
fungiyep, it... mostly worked?14:25
fungioh, and i have nodepool image updates running for devstack-precise in every provider so that we don't ddos ourselves with every single devstack job cloning a fresh sahara repo14:26
jeblairfungi: good idea14:26
*** krotscheck has joined #openstack-infra14:26
*** homeless has quit IRC14:27
*** dkranz has quit IRC14:28
*** johnthetubaguy1 is now known as johnthetubaguy14:28
*** e0ne_ has quit IRC14:30
*** matsuhashi has joined #openstack-infra14:30
*** david-lyle has joined #openstack-infra14:30
jeblair2014-03-12 12:45:55,077 DEBUG irc.client: FROM SERVER: :ChanServ!ChanServ@services. NOTICE openstackstatus :+You have triggered services flood protection.14:31
jeblair2014-03-12 12:45:55,077 DEBUG irc.client: FROM SERVER: :ChanServ!ChanServ@services. NOTICE openstackstatus :+This is your last warning. You will be ignored for 30 seconds.14:31
jeblairthere we go14:31
SergeyLukjanovheh14:31
hasharhopefully freenode can whitelist your bot14:32
*** zns has joined #openstack-infra14:32
jeblairhashar: oh they can do that?14:32
fungioh! i overlooked that because i was digging for failure responses on the /topic commands14:32
hasharjeblair: we have a bunch of very spammy bots and I am pretty sure we are not encountering flood silencing.14:33
hasharjeblair: so either we have our bot throttle their output or we got them whitelisted somehow.14:33
fungihashar: do they try to update topics on >50 channels at once? that might be a separate flood protection setting for them14:34
hasharah that might be too spammy indeed :]14:34
jeblairfungi: i think you should be able to set the topics via chanserv too, which might be faster/easier (you don't have to op yourself)14:34
*** e0ne has joined #openstack-infra14:35
fungijeblair: ooh, good idea14:35
*** nati_ueno has joined #openstack-infra14:36
*** eharney has quit IRC14:37
*** sweston has joined #openstack-infra14:37
openstackgerritJames E. Blair proposed a change to openstack-infra/statusbot: Sleep 1 second  https://review.openstack.org/7995614:37
*** chandankumar_ has joined #openstack-infra14:40
*** dkranz has joined #openstack-infra14:40
*** chandankumar_ has quit IRC14:40
*** jgrimm has joined #openstack-infra14:44
*** sarob has joined #openstack-infra14:45
*** rfolco has joined #openstack-infra14:49
*** eharney has joined #openstack-infra14:50
*** thedodd has joined #openstack-infra14:50
*** wenlock has joined #openstack-infra14:51
*** andreaf has quit IRC14:52
znsilyashakhat: thank you for merging my stackalytics update.14:54
ilyashakhatzns: you are welcome :)14:55
hasharjeblair: I might have ported devstack fallback system to python with latest patch of https://review.openstack.org/70373 :]14:56
hasharjeblair: looking for a way to test the zuul cloner know:]14:56
znsWe got PEP-257 updated and patched pep257 and the flake8 plugins so we can check and gate on that. We're ready to try that out on satori, but we need https://review.openstack.org/#/c/79054 merged in to update the requirements files. Any chance we can get that in?14:57
sdaguezns: so actually, I'm confused, satori is in stackforge, why would we change global requirements for that14:59
*** changbl has joined #openstack-infra15:00
*** sc68cal_ has quit IRC15:00
*** sc68cal has joined #openstack-infra15:00
*** gokrokve has quit IRC15:02
znssdague: I think there's two part answer to that. For OpenStack in general, adding those in will allow us to start checking existing projects for PEP-257. For satori, I can't get https://review.openstack.org/#/c/79013/ to pass... but from your comment it sounds like maybe we are gating on something we should not be? Excuse my lack of familiarity with some of this setup..15:02
openstackgerritMalini Kamalambal proposed a change to openstack-infra/config: Add experimental job for Marconi  https://review.openstack.org/7839215:03
*** gokrokve has joined #openstack-infra15:03
sdagueyeh, I have no idea why a stackforge project is gatting on global requirements15:03
*** atiwari has joined #openstack-infra15:03
*** saju_m has quit IRC15:03
*** malini has joined #openstack-infra15:03
*** bogdando has quit IRC15:03
sdagueI think there was an overreach there in job setup15:03
*** bogdando has joined #openstack-infra15:03
sdaguewhen an openstack project wants to add this, we should consider it15:03
sdaguebut not for stackforge projects15:03
*** saju_m has joined #openstack-infra15:04
*** gokrokve_ has joined #openstack-infra15:04
znssdague: that would be my mistake in copying another project's setup. I didn't understand what the requirements gate was for. I'll remove that from our setup.15:04
fungiright, satori has a gate-satori-requirements job in zuul's layout.yaml, but does not actually need that15:04
sdagueyeh, I think that's the crux of things15:05
sdaguezns: sorry I didn't catch this on the first round of reviews15:05
sdagueit only sunk in15:05
fungiit's okay if you want to force your project to work with only openstack's specified requirements list, but don't expect that to mean that you get to add requirements to openstack if you do15:05
jeblairit's a fine idea if a stackforge project wants to make sure they are compatible15:05
*** krotscheck has quit IRC15:06
jeblairbut yeah, if you need something openstack doesn't (yet) need, then it's best to turn off the job and try to keep things in sync manually15:06
sdaguejeblair: sure, but we need to not extend the scope of global requirements to include content that's not in openstack projects15:06
sdaguespeaking of which, it might be worth and audit to figure out if there are things in g-r that aren't in any project15:06
znssdague: np. That unblocks satori. What about having the pep257 tools available for other projects to add to other style checks? Per this thread: https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg17780.html15:07
*** mfer has quit IRC15:07
sdaguezns: so if an integrated project proposes a patch that needs it, then that's different15:07
*** gokrokve has quit IRC15:08
sdaguebut lacking that, that list is basically a "must package" list for distributors15:08
sdagueso we don't want to add things not actually used15:08
znssdague: OK. I can approach it from that end, then.15:09
ArxCruzclarkb: I'm getting https://git.openstack.org/openstack/savanna-dashboard/' not found15:09
fungiArxCruz: http://lists.openstack.org/pipermail/openstack-dev/2014-March/029770.html15:09
*** pcrews has joined #openstack-infra15:10
ArxCruzfungi: thanks! I need to pay more attention on mail lists15:10
znsWhere can I find out what the jobs actually do?15:10
*** changbl has quit IRC15:11
fungiArxCruz: we've updated devstack-gate to clone the right repos, but there was a brief window where it was looking for the old ones which were no longer there, so if you're using d-g and deploying it continuously, you might have hit a patch there where it broke15:11
ArxCruzfungi: I'm using d-g but with a few changes by my own, I'm planning to send patches to work more generic15:12
fungizns: i think we currently lack good documentation describing the jobs themselves, but you can see the yaml-based definitions for them all at http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config15:13
*** changbl has joined #openstack-infra15:14
russellbSergeyLukjanov: I did edit the file you mentioned, https://review.openstack.org/#/c/79900/315:14
fungizns: the job in question is defined at http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/python-jobs.yaml#n21815:14
fungizns: and the script you see it calling there is http://git.openstack.org/cgit/openstack-infra/config/tree/modules/jenkins/files/slave_scripts/project-requirements-change.py15:15
znsfungi: ahhhh..... that helps!15:16
*** jasond has joined #openstack-infra15:16
jasonddoes this situation call for a recheck or a reverify? https://review.openstack.org/#/c/77677/15:16
*** sarob has quit IRC15:17
*** branen has joined #openstack-infra15:17
znsDo you have a job that checks the coverage or pylint change for a patch? I.e. did it add pylint errors or decrease coverage? That would be a cool check (we have one on an internal project that maybe I could port over). Any interest?15:17
fungijasond: yes, you were one of the unlucy (hopefully) few mentioned in http://lists.openstack.org/pipermail/openstack-dev/2014-March/029770.html15:17
*** VijayTripathi has joined #openstack-infra15:18
fungijasond: i'll open a bug for that real quick you can reverify against15:18
jasondfungi: thank you15:18
*** matsuhashi has quit IRC15:18
*** rcarrillocruz has joined #openstack-infra15:18
fungizns: we do have jobs which run coverage reports (we stopped running pylint mostly in favor of the pyflakes backend to flake8) but nothing which calculates deltas for those i don't think15:19
fungiit's been discussed before, and i think there were at least a few people interested in seeing something along those lines15:19
*** miqui has joined #openstack-infra15:19
*** afazekas has quit IRC15:19
*** homeless has joined #openstack-infra15:20
*** rcarrillocruz1 has quit IRC15:20
*** sarob has joined #openstack-infra15:22
*** mgagne has joined #openstack-infra15:24
fungijasond: reverify bug 129144015:24
openstackgerritArx Cruz proposed a change to openstack-infra/devstack-gate: Correcting the right path to mysql log in Fedora 19  https://review.openstack.org/7996715:24
SergeyLukjanovrussellb, I'll take a look on it soon15:24
jasondfungi: thanks!15:25
*** rpodolyaka has joined #openstack-infra15:25
russellbSergeyLukjanov: ok no rush, thanks15:25
*** chandan_kumar has quit IRC15:26
*** sarob has quit IRC15:28
*** chandan_kumar has joined #openstack-infra15:28
*** chandan_kumar has quit IRC15:28
*** sarob has joined #openstack-infra15:28
rcarrillocruzjeblair: hi there, I'm looking at low-hanging-fruit https://bugs.launchpad.net/openstack-ci/+bug/1061929 , but I can't seem to find where in the code we check for the recheck/reverify review comments, i'd be grateful if you could point me in the right track15:30
rcarrillocruzactually i meant https://bugs.launchpad.net/openstack-ci/+bug/1235335 , they are similar15:30
*** gyee has joined #openstack-infra15:31
*** johnthetubaguy1 has joined #openstack-infra15:31
*** sarob has quit IRC15:33
*** johnthetubaguy has quit IRC15:33
jeblairrcarrillocruz: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n1515:35
*** johnthetubaguy1 has quit IRC15:35
jeblairrcarrillocruz: the recheck/reverify triggers are in regexes like that ^15:35
jeblairrcarrillocruz: i'm not sure i would consider that a low-hanging-fruit bug15:35
jeblairrcarrillocruz: i have no good ideas about how to implement that :/15:35
jeblairrcarrillocruz: ...maybe....15:36
jeblairrcarrillocruz: it could be done like the 'require-approval' was done...15:37
jeblairrcarrillocruz: where we say "in order for this trigger to match, we also require that the commit message match this regex"15:37
jeblairrcarrillocruz: but then we'de have to actually get regex group data from the comment filter regex into the commit message regex...15:38
*** mfer has joined #openstack-infra15:38
jeblairrcarrillocruz: it seems very complicated, a bit messy, and probably too much work for the benefit...15:38
rcarrillocruzahh...i was looking straight into the trigger python code...15:39
*** johnthetubaguy has joined #openstack-infra15:39
*** saju_m has quit IRC15:40
*** rpodolyaka has quit IRC15:40
rcarrillocruzthat explains, heh15:40
jeblairrcarrillocruz: i key aspect of zuul's design is that it's all _very_ simple building blocks.  we don't put any openstack-specific stuff in there -- instead, we break problems down until we can describe them in simpler terms.15:40
jeblairso zuul has no idea what a reverify or recheck comment is -- it's all just configuration15:41
*** eharney has quit IRC15:41
*** wenlock has quit IRC15:41
rcarrillocruzso that's why you say  in the bug 'This would be a good thing to add to the recheck magic. Unfortunately, it may require adding some openstack specific sauce to zuul' . I infere from that sentence that you want to be zuul openstack independent... so no python checks in there for stuff that pertains to an openstack issue, like reported on bug/123533515:42
rcarrillocruz>15:42
rcarrillocruz?15:42
*** smarcet has left #openstack-infra15:42
rcarrillocruzi will think about your suggestion15:43
jeblairright; we'd need to think of a way to describe the problem abstractly, like the two-regex idea above (which would work but is very complicated)15:43
rcarrillocruzif it gets hard to do, i'll move on to other low hanging fruit15:43
rcarrillocruzthx jeblair15:43
maliniSergeyLukjanov, sdague: Can you please take look at this https://review.openstack.org/#/c/78392/  when you have a minute ?15:44
SergeyLukjanovmalini, added to the giant length backlog15:46
maliniSergeyLukjanov: hope the other giants are small :)15:47
*** esker has quit IRC15:47
maliniif any other core reviewers need good karma, here is a one line change https://review.openstack.org/#/c/78392/ !!15:48
*** johnthetubaguy1 has joined #openstack-infra15:50
jeblairrcarrillocruz, clarkb: i left my suggetion in https://bugs.launchpad.net/openstack-ci/+bug/1235335 for the record, but i don't think trying to implement that is low-hanging-fruit.15:50
rcarrillocruzcool, thx sir, i'll try it and depending on how hard it gets i'll go forward or ditch15:51
*** yamahata has quit IRC15:52
*** johnthetubaguy has quit IRC15:53
jeblairzaro, hashar: is current jjb HEAD a good place to tag a release?15:53
*** salv-orlando_ has joined #openstack-infra15:54
* hashar wishes Gerrit could let us review tagging15:54
jeblairwe all wish that :/15:55
openstackgerritZiad Sawalha proposed a change to openstack-infra/config: satori: Drop requirements, pypi, and old upstream  https://review.openstack.org/7997515:55
mordredhashar: ++15:56
hasharjeblair: the current jjb is nice.  We got a few nasty changes pending such as using threads and pbr15:56
*** salv-orlando has quit IRC15:56
*** salv-orlando_ is now known as salv-orlando15:56
hasharso maybe cutting 0.7.0 before the nasty changes land in is a good idea15:56
*** sweston has quit IRC15:56
mordredhashar: I am highly supportive of any patches to gerrit which add tag reviewing15:56
hasharplus last release was back in November15:56
hasharmordred: I am java illiterate and Wikimedia mostly tag mediawiki/core which is already pair reviewed via a release process. So unlikely to happen on our side15:57
hashar:-(15:57
*** wenlock has joined #openstack-infra15:57
znssdague, fungi: thanks for the help. I'm updating satori's config accordingly: https://review.openstack.org/#/c/79975/15:57
*** rpodolyaka has joined #openstack-infra15:58
mordredhashar: sniff. sniff15:58
hasharjeblair: maybe ping openstack-infra list about jjb releasing?  I am sure all core maintainers are subscribed to it.15:58
*** che-arne has joined #openstack-infra15:58
jeblairhashar: zaro asked for it last week (he got a request from a packager i think)15:58
jeblairso i'm pretty sure he's on board15:58
*** gyee has quit IRC15:59
*** rpodolyaka has quit IRC16:00
*** reed has joined #openstack-infra16:00
*** rpodolyaka has joined #openstack-infra16:00
hasharfor logging purpose: +1 on releasing a new JJB version :-]16:00
*** CaptTofu has quit IRC16:00
*** pblaho has quit IRC16:00
hasharone day we would have to craft a nice ChangeLog for end users consumption16:01
*** CaptTofu has joined #openstack-infra16:01
jeblairsent email16:01
*** Sukhdev has joined #openstack-infra16:02
*** salv-orlando has quit IRC16:02
*** hogepodge has quit IRC16:02
jeblairalso, i -1'd the idea of a jjb-ptl group that can tag before, but that was when i was able to keep more current on what's going on with it.  it's clear that i'm not as up to date now, so i'm okay with expanding that group now.16:03
*** yassine has quit IRC16:03
hasharKhai is doing an awesome job at reviewing/approving the changes16:04
jeblairhttps://review.openstack.org/#/c/56823/ has my +2 now16:04
*** CaptTofu has quit IRC16:05
*** SumitNaiksatam has quit IRC16:07
*** talluri has quit IRC16:07
hasharjeblair: added the rest of JJB approver to the change and +1ed it :-]16:11
hasharI like seeing delegation16:11
sdaguejeblair: what is your feeling on making a functional test in the gate which only applied to a single project which did active clones outside of zuul?16:16
*** yolanda has quit IRC16:16
sdaguebecause it turns out that grenade ends up breaking for non gate cases quite a bit16:16
sdaguebecause we do so much overwriting of config16:17
sdaguerealizing that we'd have a failure rate based on real cloning associated with it16:17
sdaguebut it would apply to only the grenade project16:17
jeblairsdague: can you describe the problem a bit more?  i'm not sure i understand16:17
sdaguegrenade master doesn't work16:18
sdaguebecause it actually sets up a set of git clones that don't work16:18
sdaguebecause in the gate all of that is done by d-g16:18
*** vkozhukalov has joined #openstack-infra16:18
jeblairhow don't they work?16:19
sdaguethey turn out to be completely invalid git urls16:19
*** dstanek_afk has joined #openstack-infra16:19
sdaguebecause novnc isn't on git.openstack.org, for instance16:20
*** dstanek has quit IRC16:20
sdagueand all of devstack trees are controled by a GIT_BASE16:20
sdaguewhich got changed to git.openstack.org in devstack master16:21
sdagueand grenade master16:21
sdaguebut defaults to github.com in devstack stable/havana16:21
sdagueso things go funny16:22
sdaguealso, turns out TARGET_BRANCH was defaulting to stable/icehouse16:22
*** pbelanyi has quit IRC16:22
sdaguewhich, also, never tested, because d-g sets all that16:22
jeblairsdague: so in order to test what you want, the test would need to clone from github, since you want to make sure the novnc url works?16:23
sdaguejeblair: yep16:23
*** nati_ueno has quit IRC16:23
sdaguewhich is why it would be limited to grenade, because the failure rate that would introduce would be non zero16:23
jeblairsdague: having been on the receiving end of "why did this test break" thanks to github, i have to admit, i'm not excited about that.16:23
sdaguesure, however the alternative is "why hasn't grenade worked for normal people in 4 months"16:24
jeblairsdague: could we unit test it instead?  (make sure these urls or git commands look sane?)16:24
*** dstanek_afk has quit IRC16:24
sdaguejeblair: probably, but that just means we know the right assumptions, not that things actually work16:25
sdagueso if upstream trees change, we break16:25
*** melwitt has joined #openstack-infra16:25
*** gokrokve_ has quit IRC16:25
sdagueI think the crux of this is a non negligle part of grenade is to actually set up the tree structures16:25
sdaguewhich overlaps with d-g16:25
sdagueso is untested on it's own16:26
jeblairso maybe a non-voting test?16:26
sdaguesure16:26
jeblairso that we're not signing up the gate to be broken by github16:26
sdaguejeblair: well, it wouldn't be in the gate16:26
jeblair(i mean, basically, you would not have been able to land _any_ changes to grenade yesterday)16:26
sdagueit would be on check only for grenade16:27
jeblairsdague: check ~= gate now, at least for this purpose16:27
*** rpodolyaka has quit IRC16:27
sdaguewell, check doesn't kill someone else16:27
SergeyLukjanovHouston, we have a problem....16:27
SergeyLukjanovhttps://git.openstack.org/cgit/openstack/sahara is empty O_O16:27
jeblairi mean, continuing to use yesterday as an example, no grenade check tests would have passed if this were voting16:27
sdagueso it's more limitted pain16:27
*** wenlock has quit IRC16:27
sdaguejeblair: this job would not have passed, agreed16:28
SergeyLukjanovfungi, jeblair, clarkb, could you please take a look on gerrit mirroring? https://git.openstack.org/cgit/openstack/sahara is empty...16:28
mgagneSergeyLukjanov: showing fine to me here16:28
jeblairSergeyLukjanov: i will trigger gerrit replication16:28
jeblairSergeyLukjanov: you might want to check each of "git01" .. "git05" to narrow down the problem16:28
*** rpodolyaka has joined #openstack-infra16:29
*** wenlock has joined #openstack-infra16:29
SergeyLukjanovjeblair, yup, /me checking16:29
fungithat's definitely weird. i did mv the backing directory for it on all the git servers16:29
jeblairSergeyLukjanov: like http://git01.openstack.org:8080/cgit/openstack/sahara16:29
fungioh!16:29
fungii forgot git0516:29
fungii forgot _we had_ git05 :/16:29
* fungi sighs16:29
jeblairfungi: yep that's the one16:30
jeblairhttp://git05.openstack.org:8080/cgit/openstack/sahara16:30
SergeyLukjanovthere are no sahara repos on git0516:30
*** CaptTofu has joined #openstack-infra16:30
*** alexpilotti has joined #openstack-infra16:30
SergeyLukjanovand savanna too16:30
SergeyLukjanov:)16:30
fungiat this point the repos have all been created on there, but if you haven't triggered replication yet i can remove them and move the old ones to the right nams16:31
funginames16:31
jeblairfungi: i have not triggered replication16:31
fungidoing it now--thanks!16:31
SergeyLukjanovfungi, jeblair, thank you!16:31
*** Ryan_Lane1 has joined #openstack-infra16:33
fungiall removed/moved. retriggering replication for them now16:35
*** dstanek_afk has joined #openstack-infra16:35
*** jlibosva has quit IRC16:36
*** Ryan_Lane has quit IRC16:36
*** Ryan_Lane1 is now known as Ryan_Lane16:36
*** Ryan_Lane has joined #openstack-infra16:36
*** Ryan_Lane1 has joined #openstack-infra16:37
fungilooks like it should all be replicated and up to date now?16:39
pleia2clarkb: just had one comment re: 79814 (not actually a blocker, but might consider it)16:40
*** bhuvan has joined #openstack-infra16:41
*** bhuvan_ has joined #openstack-infra16:41
*** khyati has joined #openstack-infra16:41
*** amcrn has joined #openstack-infra16:42
openstackgerritKhai Do proposed a change to openstack-infra/publications: update slides  https://review.openstack.org/7999316:44
*** reed has quit IRC16:44
*** dizquierdo has quit IRC16:45
*** gyee has joined #openstack-infra16:49
*** jasond has quit IRC16:50
*** gokrokve has joined #openstack-infra16:50
*** rcarrillocruz1 has joined #openstack-infra16:51
*** SumitNaiksatam has joined #openstack-infra16:51
openstackgerritSergey Lukjanov proposed a change to openstack/requirements: Rename Savanna to Sahara  https://review.openstack.org/7999416:51
SergeyLukjanovfungi, clarkb, jeblair, sdague, folks, could you please fast approve https://review.openstack.org/79994 to make sahara jobs running using pypi.o.o16:52
*** thuc has joined #openstack-infra16:52
*** lcostantino has joined #openstack-infra16:53
*** rcarrillocruz has quit IRC16:53
fungiSergeyLukjanov: approved16:53
*** thuc_ has joined #openstack-infra16:53
SergeyLukjanovfungi, thank you!16:53
clarkbmorning16:54
SergeyLukjanovclarkb, good morning16:54
fungiSergeyLukjanov: i'm just sorry i didn't think to suggest a patch to that file too16:54
clarkbpleia2: lookig thanks16:54
*** e0ne has quit IRC16:54
SergeyLukjanovfungi, my bad, absolutely forget about the dep on it for choosing mirrors16:54
fungiSergeyLukjanov: i think it would probably have also broken requirements update proposals too16:55
SergeyLukjanovfungi, yeah16:55
*** thuc has quit IRC16:56
*** thuc has joined #openstack-infra16:57
*** cody-somerville has quit IRC16:58
*** thuc_ has quit IRC17:00
*** cody-somerville_ has joined #openstack-infra17:03
*** cody-somerville_ has quit IRC17:03
*** cody-somerville_ has joined #openstack-infra17:03
openstackgerritClark Boylan proposed a change to openstack-infra/config: Remove unneeded package sources from nodepool node  https://review.openstack.org/7981417:03
clarkbpleia2: ^ I think that should do it17:03
*** harlowja_away is now known as harlowja17:04
openstackgerritJames E. Blair proposed a change to openstack-infra/statusbot: Sleep 1 second  https://review.openstack.org/7995617:05
*** nati_ueno has joined #openstack-infra17:06
jeblairoh wow, i just learned you can "yum install /path/to/binary/you/want"17:07
jeblairand that will install the package that provides it17:07
clarkbjeblair: and you don't even need to do a package list update first :)17:08
SergeyLukjanovjeblair, oh, interesting17:08
SergeyLukjanovclarkb, bah, fantastic17:09
*** afazekas has joined #openstack-infra17:09
*** afazekas has quit IRC17:09
fungineat. better than using apt-file search17:09
*** gokrokve_ has joined #openstack-infra17:09
*** jasondotstar has joined #openstack-infra17:09
*** sweston has joined #openstack-infra17:09
*** reed has joined #openstack-infra17:10
dstufftjeblair: that is awesome17:11
*** andreaf has joined #openstack-infra17:11
clarkbdtroyer: Alex_Gaynor for the openstack sdks' ptl group who should be the initial member17:12
*** gokrokve has quit IRC17:12
fungiclarkb: added a few more comments on 7981417:12
Alex_Gaynorclarkb: Jesse Noller I suppose17:13
*** morganfainberg_Z is now known as morganfainberg17:13
clarkbAlex_Gaynor: thanks17:13
Alex_Gaynorclarkb: sure17:13
fungidstufft: did you get my ping the other night that i put your keys on those sample servers where we're seeing discrepancies in pip behavior wrt https://github.com/pypa/pip/issues/163217:13
dstufftfungi: oh I missed it17:14
dstufftand then forgot17:14
dstufft:[17:14
*** VijayTripathi has quit IRC17:14
dstufftsorry!17:14
clarkbfungi: responded. I can fix the comment but we actually need sudo there because distros17:14
Alex_Gaynordstufft: lol. Have fun debugging that17:15
fungidstufft: 'ssh jenkins@23.253.82.112' is where we're seeing it break (rackspace) and 'ssh jenkins@15.185.123.189' is where we're seeing it work normally (hpcloud)17:15
clarkbfungi: can you take a look before I push a new patchset?17:15
fungiclarkb: looking17:15
*** esker has joined #openstack-infra17:16
zaroclarkb: i tried verifying this on my personal VM but i don't know how to setup a reverse proxy.  https://review.openstack.org/#/c/60893/17:16
*** saju_m has joined #openstack-infra17:16
clarkbzaro: just install gerrit with that puppet change17:17
dstufftfungi: Ok, i'm fixing my devenv since homebrew and apple decided to conspire to destroy my productivity today17:17
clarkbzaro: that will setup apache and everything17:17
dstufftI'll look at it after that17:17
fungidstufft: appreciated--and good luck with the fruit17:17
dstuffthmm17:18
dstufftyou know what though17:18
dstufftI could totally use my eldritch-horror branch on pip and make it so pip just assumes setuptools is a setup_requires and installs it in a temp location :V17:18
*** kmartin has quit IRC17:18
dstufftthen this shouldn't be possible to happen17:18
fungiclarkb: lgtm17:19
zaroclarkb: i don't think i even have apache installed.  don't even have the /etc/apache folder.17:19
fungidstufft: i wanted to compliment you on your choice of branch naming there, if nothing else. very nice ;)17:19
clarkbfungi: thanks17:19
dstufftfungi:  the first revision was worse than the current revision :V17:20
zaroclarkb: is it just a straight forward install of apache with those changes?17:20
dstufftthe current one is not entirely super horrible17:20
fungidstufft: i assume while it's still named eldritch-horror, i should expect to be driven mad simply by a mere glimpse of the source code there17:20
clarkbzaro: mostly. but you can have puppet do it all for you17:21
*** afazekas has joined #openstack-infra17:21
dstufftfungi: https://github.com/dstufft/pip/compare/pypa:develop...eldritch-horror monkeypatching via python -c "" is the best ever17:21
zaroclarkb: i forgot how.  i'll be in tomorrow, just i'll just postpone until then.17:22
fungidstufft: you know now that i've clicked, i'm going to live out the remainder of my days gibbering in a corner, mumbling incoherently about ancient computer gods from other dimensions17:23
zaroclarkb: have you taken a look at jeblair comment on https://review.openstack.org/#/c/61321/17:23
andreafsdague: ping17:24
clarkbzaro: not yet, still digging out of the remains of bug day17:24
* clarkb is going to fix release two more bugs17:24
fungidstufft: also always a fan of commit messages like "Make this less bad"17:24
dstufftfungi: :317:25
dstufftfungi: it's cool though, it actually works (not options like --cert are passed through yet), no more setuptools touching the network with that branch17:25
funginice that pip source is growing a "hacks" module too17:26
fungisays a lot right there about the fright level of monkey-patching there17:27
dstuffta lot of pip could justifably be moved into that module17:27
openstackgerritA change was merged to openstack-infra/statusbot: Sleep 1 second  https://review.openstack.org/7995617:27
clarkbjeblair: fungi SergeyLukjanov mordred I have two nodepool script changes that need an additional +2 and +A. I can babysit some image builds if I can get reviewers17:28
SergeyLukjanovclarkb,  you get one17:29
fungiclarkb: i think i already +2'd them both17:29
*** sarob has joined #openstack-infra17:30
clarkbfungi: you did thanks17:30
clarkbzaro: I think jeblair is right. We need to make sure that line 30 of https://review.openstack.org/#/c/61321/12/modules/openstack_project/templates/nodepool/nodepool-dev.yaml.erb runs a script that will install the correct key for jenkins-dev/nodepool-dev17:31
*** jcoufal has quit IRC17:31
*** krotscheck has joined #openstack-infra17:32
jeblairclarkb: aprvd17:32
clarkbjeblair: awesome thank you17:32
openstackgerritA change was merged to openstack-infra/config: Remove unneeded package sources from nodepool node  https://review.openstack.org/7981417:32
*** dprince has quit IRC17:34
*** kmartin has joined #openstack-infra17:34
*** sarob has quit IRC17:34
openstackgerritNikola Dipanov proposed a change to openstack-dev/hacking: Add an import whitelist helper factory  https://review.openstack.org/8001817:37
*** jepoy has quit IRC17:37
*** jepoy has joined #openstack-infra17:38
*** openstackstatus has quit IRC17:38
*** openstackstatus has joined #openstack-infra17:38
*** julienvey has quit IRC17:39
*** jepoy has quit IRC17:42
malinithanks SergeyLukjanov for the review17:44
malinifungi: Can you take a look at this if you have a minute https://review.openstack.org/#/c/78392/ ?17:45
openstackgerritNikola Dipanov proposed a change to openstack-dev/hacking: Add an import whitelist helper factory  https://review.openstack.org/8001817:47
fungimalini: getting sdague or dtroyer to weigh in on that would be good too, before we possibly approve it17:48
malinisure fungi!17:48
malinidtroyer, sdague: Can you take a look please https://review.openstack.org/#/c/78392/ ?17:49
fungii can evaluate the (very trivial) change on its technical correctness, but they're the ones who know whether that's going to cause any inconvenience for them17:49
malinisure..makes sense17:49
malinidtroyer, sdague: This is to make the marconi devstack troubleshooting easier17:50
dtroyermalini, fung:  lgtm17:51
malinithanks dtroyer!!17:51
malinifungi: ^17:51
*** jpich has quit IRC17:52
*** rpodolyaka has quit IRC17:52
*** bhuvan__ has joined #openstack-infra17:53
*** bhuvan___ has joined #openstack-infra17:53
*** dstanek_afk is now known as dstanek17:53
openstackgerritA change was merged to openstack-infra/storyboard: Rename migration revisions  https://review.openstack.org/7986917:54
*** bhuvan_ has quit IRC17:54
*** bhuvan has quit IRC17:54
clarkbuh17:54
*** hashar has quit IRC17:54
clarkbso you may as well use migrate if you are going to do ^17:54
openstackgerritA change was merged to openstack-infra/config: Add experimental job for Marconi  https://review.openstack.org/7839217:55
clarkbiirc alembic uses proper uuid like ids because it can build arbitrary graphs17:55
clarkbrather than doing the "human readable" thing that migrate does17:55
*** rpodolyaka has joined #openstack-infra17:56
*** rpodolyaka has quit IRC17:57
*** reed has quit IRC17:57
*** reed has joined #openstack-infra17:57
*** rlandy_ has joined #openstack-infra17:57
fungiclarkb: jeblair: mordred: you'll be amused to know that i'm getting eoferror (or sometimes connection reset) repeatedly retrying to update the devstack-precise image in rax-iad now ;)17:58
clarkbfungi: ya I ran into that when sudo went sideways17:58
*** vkozhukalov has quit IRC17:58
*** bhuvan_ has joined #openstack-infra17:58
*** bhuvan has joined #openstack-infra17:58
fungigot it once or twice in dfw and ord too but they eventually completed after once or twice17:59
fungicompleted after one or two tries i mean17:59
jeblair:/17:59
*** bhuvan___ has quit IRC17:59
*** bhuvan__ has quit IRC17:59
clarkbfungi: I think we may have leaked nodes too. I meant to look at that toay18:00
fungii was actually about to do the same, and noticed that my iad image still wasn't updated18:00
*** rlandy has quit IRC18:00
*** rpodolyaka has joined #openstack-infra18:01
clarkbthere are a bunch of ~140 hour old ready nodes18:01
openstackgerritA change was merged to openstack-infra/config: Limit non tripleo nodepool nodes to 8GB of RAM  https://review.openstack.org/7844018:01
*** bhuvan___ has joined #openstack-infra18:01
*** bhuvan__ has joined #openstack-infra18:01
fungiclarkb: yeah, i'm looking at those. some in other states past 3 hours as well18:01
fungii'll give 'em all a quick death18:01
clarkbnodepool list | grep ready | sort -t'|' -k11 -n18:02
fungithe 140-hour-old ones are dsvm-precise-krnl18:03
*** bhuvan_ has quit IRC18:03
*** bhuvan has quit IRC18:03
fungimore of those than i would have expected18:03
fungii omitted any in a hold state and am deleting any others which are >3 hours in any given state18:05
*** Sukhdev has quit IRC18:06
clarkbwoot ram change merged. I am going to spin up a new hpcloud region b devstack-precise-check image18:06
clarkbfungi: that doesn't interfere with what you are doing does it?18:06
fungiclarkb: not in the least18:06
*** derekh has quit IRC18:07
*** jgallard has quit IRC18:09
SergeyLukjanovfolks, I've just find that we need to have etc/sahara/sahara.conf in repo for d-g, will symlink work well or I need to copy-paste conf sample?18:09
SergeyLukjanovhttps://review.openstack.org/8003818:09
clarkbSergeyLukjanov: I think copy paste then you can delete the old one after a release or two, but then you have to double account which sucks18:10
clarkbSergeyLukjanov: also symlinks aren't a thing on windows. not sure if that matteers18:10
sdaguegiven how pegged we are on dsvm nodes, thoughts on merging this - https://review.openstack.org/#/c/78687/18:11
fungiclarkb: windows has "shortcuts" instead (don't ask why i know this)18:11
sdaguewhich drops jobs we don't want18:11
sdaguejogo: can you rebase that?18:11
SergeyLukjanovclarkb, we're planning to not keep backward compat for such stuff, so, I'll add temp copy-paste18:11
SergeyLukjanovclarkb, thanks18:11
*** zns has quit IRC18:11
SergeyLukjanovclarkb, we just need it to merged updated devstack integration and then rename all internals18:12
clarkbSergeyLukjanov: wfm18:12
SergeyLukjanovclarkb, could I ask you to forse push it? https://review.openstack.org/8003818:13
clarkbSergeyLukjanov: is it not possible to run it through the gate normally? (would be good to see it actually fix the problem)18:14
SergeyLukjanovclarkb, devstack integration update isn't merged yet :(18:14
SergeyLukjanovclarkb, here is an issue - http://logs.openstack.org/53/79353/3/check/check-tempest-dsvm-sahara-full-nv/f4580cf/logs/devstacklog.txt.gz#_2014-03-12_17_27_25_56018:15
*** bhuvan has joined #openstack-infra18:15
*** bhuvan_ has joined #openstack-infra18:15
clarkbSergeyLukjanov: right but won't 80038 fix ^ without needing a force push?18:15
clarkbjgriffith: you got change 80000 I am jealous18:16
openstackgerritNikola Dipanov proposed a change to openstack-dev/hacking: Add an import whitelist helper factory  https://review.openstack.org/8001818:16
*** wenlock has quit IRC18:16
*** bhuvan___ has quit IRC18:16
*** bhuvan__ has quit IRC18:16
jeblairsdague: lgtm18:16
jeblairSergeyLukjanov: where's the devstack change?18:16
SergeyLukjanovclarkb, I can't merge it to sahara because d-g job is voting18:16
SergeyLukjanovhttps://review.openstack.org/#/c/79353/18:16
SergeyLukjanovjeblair, ^^18:16
clarkboh I see 79353 depends on 80038 and vice versa18:17
openstackgerritNikola Dipanov proposed a change to openstack-dev/hacking: Add an import whitelist helper factory  https://review.openstack.org/8001818:17
*** bhuvan_ has quit IRC18:17
*** bhuvan has quit IRC18:17
clarkbI think I am ok force pushing the sahara change since it is just copy pasta. jeblair fungi mordred any dissent?18:17
jeblairhttps://review.openstack.org/#/c/79353/ depends on 80038?18:17
SergeyLukjanovclarkb, yeah, we've added all needed aliases except conf file I hope18:18
clarkbjeblair: that is my understanding since it is trying to setup sahara etc files not savanna etc files18:18
*** bhuvan has joined #openstack-infra18:18
*** bhuvan_ has joined #openstack-infra18:18
clarkbjeblair: we could probably split it out into a couple changes that use savanna etc fiels with sahara then sahara etc files with sahara18:18
jeblairis 79353 overreaching then?18:18
jeblairclarkb: yeah18:18
clarkbjeblair: ya it may be ^18:18
SergeyLukjanovclarkb, that's an option too18:19
fungiclarkb: seems fine, though note that this morning before we started the rename SergeyLukjanov had already discovered that the devstack fix to start properly overriding from global-requirements.txt may have put savanna in a broken state, so until it's determined what is broken there there may be more forced merges to come18:19
clarkbSergeyLukjanov: I think we should go that route then18:19
clarkbSergeyLukjanov: its more CD friendly and all that18:19
SergeyLukjanovclarkb, ok, it's fair18:19
SergeyLukjanovheh, looks like +10h of looking on zuul status page ;)18:19
*** sarob has joined #openstack-infra18:20
*** rpodolyaka has quit IRC18:20
clarkbhpcloud region b image update is running now18:21
jeblairSergeyLukjanov: ?18:22
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: Minor fixes in the trigger documentation  https://review.openstack.org/7985418:22
SergeyLukjanovjeblair, you're about "+10h of looking on zuul status page ;)"?18:22
jeblairyeah18:22
jeblairi didn't understand that18:22
*** rpodolyaka has joined #openstack-infra18:23
SergeyLukjanovjeblair, I mean that I'll need to wait for several more CRs to be checked by Jenkins :)18:23
*** bhuvan has quit IRC18:24
*** bhuvan_ has quit IRC18:24
jeblairSergeyLukjanov: yeah; jobs at the top of the check queue look like they waited about 30 mins to get nodes18:24
*** ildikov_ has quit IRC18:25
mtreinishjogo: so on https://review.openstack.org/#/c/79123/ should I respin it to just be nova, infra, and qa?18:28
*** SumitNaiksatam has quit IRC18:28
*** e0ne has joined #openstack-infra18:29
*** saju_m has quit IRC18:30
*** esker has quit IRC18:30
*** bhuvan has joined #openstack-infra18:30
*** afazekas has quit IRC18:30
*** esker has joined #openstack-infra18:30
*** sarob has quit IRC18:30
SergeyLukjanovjeblair, done, could you please take a look if you have some time https://review.openstack.org/7935318:31
*** zns has joined #openstack-infra18:32
*** bhuvan has quit IRC18:32
jeblairSergeyLukjanov: lgtm; sdague ^18:33
reedwhere can I find a list of all the web properties managed by the CI team?18:33
clarkbreed: openstack-infra/config/manifests/site.pp is probably the closest thing18:34
reedclarkb, wonderful, that's what I was looking for18:34
jeblairclarkb: a possibly incomplete list is in our docs (it omits some smaller things): http://ci.openstack.org/18:34
jeblairreed: ^18:34
*** homeless_ has joined #openstack-infra18:35
openstackgerritJames E. Blair proposed a change to openstack-infra/devstack-gate: WIP: test caching-forwarding nameserver  https://review.openstack.org/8004518:35
reedyep, thanks18:35
*** homeless has quit IRC18:35
*** sarob has joined #openstack-infra18:36
SergeyLukjanovjeblair, thx18:36
SergeyLukjanovrussellb, re https://review.openstack.org/#/c/79900/318:36
clarkbhttp://paste.openstack.org/show/73279/ huh18:37
clarkbI am going to retry the image build to see if this is consistent. May just be that puppet forge said go away18:37
SergeyLukjanovrussellb, you should add section about nova-docker to modules/openstack_project/files/jenkins_job_builder/config/porjects.yaml to generate jobs in jenkins18:37
jeblairclarkb: yeah18:38
*** wenlock_demo has joined #openstack-infra18:38
jogomtreinish: that was my thinking18:38
*** sweston has quit IRC18:39
*** cody-somerville_ is now known as cody-somerville18:39
*** wenlock_demo has quit IRC18:39
*** rlandy__ has joined #openstack-infra18:39
*** SumitNaiksatam has joined #openstack-infra18:39
jeblairjogo, mtreinish: do other channels want it?18:39
*** wenlock has joined #openstack-infra18:40
*** jasondotstar has quit IRC18:40
mtreinishjeblair: dunno, jogo asked me to push out the patch18:40
mtreinishjeblair: do you want it for infra?18:40
jeblairi ask because we could have had gerritbot announcing failures for years, but no one has fixed that... that makes me think people aren't interested in failure announcements18:40
*** lcostantino has quit IRC18:40
*** sdake has quit IRC18:41
jeblairi mean, i do think it's extremely useful for people tending e-r, _especially_ the unclassified announcements18:41
clarkbwell someone did try, but then it got more complicated than we realized18:41
jamespagedims_, was really hoping to have libvirt 1.2.2 + patch into the icehouse CA today18:41
mtreinishjeblair: well this is a bit more specific it will only announce the failures with a known bug and only if the bug matches the projects listed in the yaml for the channel18:41
jamespagehowever debugging a unit test failure in it right now18:41
jamespageso lagging a bit18:41
*** wenlock has quit IRC18:42
*** sdake_ has quit IRC18:42
jeblairmtreinish: oh, so it's like "hey this nova bug keeps hitting stuff"18:42
*** lcostantino has joined #openstack-infra18:42
fungipaste.o.o is grinding. i bet it's swapping hard again18:42
mtreinishjeblair: yeah18:42
*** sweston has joined #openstack-infra18:43
*** rlandy_ has quit IRC18:43
jogojeblair: my thinking was others will but push this out on phases18:43
fungiactually it looks fine18:43
clarkbfungi: I think we should make it read only and let it die18:43
jogoand I as nova want it18:43
clarkbfungi: I think drizzle is just slow18:43
*** krotscheck has quit IRC18:43
clarkbbecause old drizzle and so on18:43
mtreinishjogo: well I was thinking the first patch it might make sense to include neutron and cinder too18:44
jeblairmtreinish: i could see that being useful then; though that makes me think we might not want it in infra -- we have a lot of unfixable bugs, so much of what it would say to us is unactionable18:44
jogomtreinish: your call18:44
*** sdake_ has joined #openstack-infra18:44
jeblairmtreinish: (example: ubuntu mirror is wonky)18:44
clarkbthere are so many paste services out that that all support basically the same functionality I am not sure we need our own. The only reason to keep it would be to avoid a paste provider going away causing everyone to not have paste but there are a lot of options (I suppose running an open version of a thing is valuable too)18:44
jogojeblair: good point about infra bugs. mtreinish ^18:44
mtreinishjeblair: you don't want e-r to spam the channel for every patch  when there is a mirror outage :)18:45
*** sdake has joined #openstack-infra18:45
clarkbjeblair: the puppet module thing happened again. I think I am going to get lunch and try again after18:45
*** andreaf has quit IRC18:45
jeblairclarkb: ack.  also, i think it's worth spending a few minutes fixing paste.18:45
*** weshay has quit IRC18:45
clarkbmordred: ^ have you seen hpcloud region b connectivty issues to puppet forge18:45
mordreduhm. no18:45
clarkbjeblair: definitely. I don't think we should kill that service without some headroom18:45
mtreinishjogo: yeah I didn't think about that, we probably should exclude infra then18:45
clarkbjeblair: just throwing it out there that I think the value of having our own is minimal18:46
clarkbmordred: jeblair actually I got more error message this time. I think we may have conflicting requirements somewhere18:46
clarkb:( I will look into fixing that after lunch18:46
*** rpodolyaka has quit IRC18:47
openstackgerritMatthew Treinish proposed a change to openstack-infra/config: Elastic recheck irc spam everywhere  https://review.openstack.org/7912318:48
mtreinishjogo: ^^^ just nova, cinder, and neutron18:48
fungithose most deserving of its ire?18:48
mtreinishrussellb, markmcclain, jgriffith: ^^^ do you guys have an issue with that?18:48
*** e0ne has quit IRC18:49
markmcclain+118:49
dims_jamespage, very cool18:49
mtreinishfungi: heh, yeah something like that :)18:49
dims_jamespage, the 0.9.8+ deb(s) are holding up very well. need to run it another 5-8 times (i have 5-6 good runs so far)18:49
mtreinishfungi: that or most likely to respond to an e-r comment in irc18:50
fungimtreinish: ooh, burn ;)18:50
jeblairfungi, clarkb: so my thinking is recursive resolvers on all servers, and forwarding resolvers on nodepool nodes.  how does that sound?18:52
jeblairfungi, clarkb: https://review.openstack.org/#/c/80045/ is an attempt to see if that would mess up devstack18:52
fungijeblair: *caching* forwarding resolvers on nodepool nodes? if so, yes perfect18:53
jeblairfungi: yes18:53
*** johnthetubaguy1 has quit IRC18:53
jeblair(bind will cache in both cases)18:53
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Remove group for the projects with the same name  https://review.openstack.org/8004918:53
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Remove unneeded upstream links  https://review.openstack.org/8005018:53
fungijeblair: though i think we need a little extra bootstrapping in jobs once that's in place. some initial retries to lookup things we know we'll use and "prime the pump" so to speak18:53
SergeyLukjanovfungi, ^^18:53
fungijeblair: which hopefully will help guard against stray initial cache misses timing out18:54
clarkbjeblair: out of curiousity why bind?18:54
jgriffithmtreinish: reading back18:55
jeblairclarkb: "yum install caching-nameserver" and "apt-get install bind9" are all that are needed to get a caching recursive resolver on both os's.18:55
*** thuc has quit IRC18:55
fungii personally am fine with bind. at least since 9.x it's not the bloated monstrosity of spaghetti it was in the post-4 days of 8.x18:55
jeblairclarkb: i'd be pretty coll with unbound though.18:55
jeblaircool.18:55
jgriffithmtreinish: I'm fine with trying it18:55
*** thuc has joined #openstack-infra18:55
jgriffithmtreinish: if it's overwhelming we can revisit18:56
*** SumitNaiksatam_ has joined #openstack-infra18:56
clarkbjeblair: I don't have any thing against bind it just seems overcomplicated for what we need18:56
jgriffithmtreinish: it could get pretty distracting18:56
fungiwhen you run bind as a recursive resolver, most of the sharp edges and moving parts are safely out of harm's way18:56
fungibut i'm also fine with something even more lightweight and designed not to act as an authoritative resolver at all18:57
jeblairunbound is available on both os's.18:57
fungii do agree that if we pick something, it should be the same something on all platforms with consistent configuration18:57
clarkb++ to consistency18:57
*** SumitNaiksatam has quit IRC18:57
clarkblooks like unbound will only do recursive18:58
jeblairclarkb: unbound can forward18:58
*** SumitNaiksatam_ is now known as SumitNaiksatam18:58
clarkbgreat. So either one works for me18:58
clarkbwas curious if there was a specific reason for bind18:58
dims_jamespage, @hallyn had provided a url that i am wget-ing for the 0.9.8+ debs. how much work/time is it move to proposed?18:59
fungii take it nobody's a huge fan of nscd (i've used it very little myself, so it very well may be a hideous idea)18:59
*** thuc has quit IRC19:00
clarkbfungi: nscd is that service you restart anytime something else breaks19:00
jeblairunbound does use like 1/5 the ram of named...19:00
fungiclarkb: got it19:00
fungiclarkb: i thought that was pulseaudio19:00
*** e0ne has joined #openstack-infra19:00
sdaguemtreinish: I think you need russellb sign off before doing it19:00
jeblairmore like 1/319:01
*** weshay has joined #openstack-infra19:01
clarkbjeblair: that might be a good reason to use it19:01
*** mark0n has joined #openstack-infra19:01
fungiwhat percentage of our 8gb of ram does that work out to? ;)19:01
clarkbfungi: jeblair: does centos not do nscd? that might be the simplest to start with19:02
fungibut yeah, unbound sounds like a great choice19:02
jeblairdoes nscd actually cache dns records?19:03
clarkbya it caches host lookups19:03
openstackgerritSean Dague proposed a change to openstack-infra/config: Don't run large-ops test on repos that it doesn't touch  https://review.openstack.org/7868719:03
sdagueok, that's the rebase19:03
mark0nclarkb: remember my problem with JJB and multiple Git remotes? I prepared a patch for that. I also want to add a simple unit test. But I have problems running "tox -epy27". Are you familiar with tox?19:03
clarkbmark0n: yup19:03
*** thuc has joined #openstack-infra19:03
fungithe trick with nscd, as i understand it, is that it hooks into the local resolver library and then forwards and caches based on your resolv.conf19:04
fungirather than putting your loopback in resolv.conf19:04
fungibut i always thought the c in nscd stood for "cache"19:04
jeblairhttp://stackoverflow.com/questions/11020027/dns-caching-in-linux19:05
mark0nclarkb: I'm running it on Ubuntu 13.10. It crashes with an exception: https://gist.github.com/mark0n/951398719:05
*** rpodolyaka has joined #openstack-infra19:05
jeblairhttps://bugs.debian.org/cgi-bin/bugreport.cgi?bug=33547619:05
clarkbhuh its disabled19:05
clarkbgood to know19:05
fungioho, "You should install this package only if you use slow services like LDAP, NIS or NIS+."19:06
jeblairyah, cause it's broken19:06
clarkbjeblair: I wonder if unscd has the same problem. but unbound seems like a great options if we want simple and bind if we need to do more complicated things19:06
jeblairi'll hack out a change to use unbound after lunch19:06
fungiawesome19:07
clarkbmark0n: huh, let me try reproducing19:07
jeblairthat d-g change should still give us the answers we need about running a caching nameserver locally and pointing /etc/hosts at it in general19:07
clarkb++19:07
fungiagreed19:07
*** sweston has quit IRC19:08
*** sweston has joined #openstack-infra19:09
clarkbmark0n: works for me. Can you try `tox -repy27` that will rebuild the virtualenv for you. I wonder if the virtualenv is missing something19:09
mark0nclarkb: that gives me the same output19:10
mark0nclarkb: what OS are you using?19:10
clarkbmark0n: the same19:10
clarkbbut my tox is version 1.6.119:11
*** caleb_ has joined #openstack-infra19:11
mtreinishjgriffith: yeah I could see it getting distracting, but how many gate failures does e-r attribute to cinder bugs?19:11
clarkbmark0n: is pip installed?19:12
mark0nclarkb: did you install tox from the Ubuntu repo or using pip?19:12
clarkbmark0n: using pip19:12
clarkbmark0n: I think what is happening there is tox is trying to fork and exec pip19:13
clarkbbut not finding pip19:13
*** salv-orlando has joined #openstack-infra19:14
mark0nclarkb: I tried 1.7.0 - seems to have a different issue, see gist19:18
clarkbmark0n: yea 1.7.0 won't work19:18
clarkbmark0n: they broke backward compat19:18
*** thuc_ has joined #openstack-infra19:18
*** thuc has quit IRC19:18
clarkbmark0n: I would install 1.6.1, pip install --upgrade tox==1.6.119:18
*** thuc has joined #openstack-infra19:19
*** hogepodge has joined #openstack-infra19:19
*** zhiyan is now known as zhiyan_19:20
mark0nclarkb: hmm, with 1.6.1 I have the first issue again...19:20
*** mburned is now known as mburned_out19:21
clarkbmark0n: weird19:21
clarkbmark0n: are you doing this on tip of master or on your patch? can you try doing it on tip of master to see if that is different?19:21
clarkbI wonder if something in the local tree is confusing tox19:21
mark0nclarkb: without my patch19:21
*** thuc_ has quit IRC19:22
*** jooools has quit IRC19:23
*** mburned_out is now known as mburned19:24
mark0nclarkb: I also purged all auto-generated files (including git-ignored files)19:24
clarkbmark0n: ok, let us try debugging this step by step using the virtualenv tox creates. `source .tox/py27/bin/activate ; pip install -r/home/marko/work/automated builds/vagrant/jenkins-job-builder/tools/pip-requires -r/home/marko/work/automated builds/vagrant/jenkins-job-builder/tools/test-requires ; python setup.py testr --slowest`19:24
clarkboh after pip install -r's you should pip install $PATH_TO_JENKINS_JOB_BUILDER19:25
*** krotscheck has joined #openstack-infra19:26
*** melwitt has quit IRC19:26
clarkbthat is basically what tox should be doing for us, and hopefully gives us an idea of what is breaking19:26
*** melwitt has joined #openstack-infra19:27
*** malini is now known as malini_afk19:27
clarkbbut I think tox can't find pip for some reason19:27
clarkbI need to grab lunch now, but will be back in a bit others in here should be able to help if you find something new before I get back19:28
mordredkiall: we should actually probably have that conversation here19:30
*** zns has quit IRC19:30
mordredkiall: because even if you do all the work over the weekend because you think it's fun, we'll want jeblair and clarkb and fungi and SergeyLukjanov to be on board19:30
*** SumitNaiksatam_ has joined #openstack-infra19:31
*** SumitNaiksatam has quit IRC19:31
*** SumitNaiksatam_ is now known as SumitNaiksatam19:31
*** rpodolyaka has quit IRC19:31
*** rpodolyaka has joined #openstack-infra19:31
fungii would hate for someone to waste their time doing work we can't make use of, if a little input from us would result in work we *can* make use of. what's the work?19:32
mordredfungi: the idea is to be able to support non-irc output for some of our bots who are generall useful -such as gerritbot19:33
*** afazekas has joined #openstack-infra19:33
kiallmordred: was AFK19:33
funginon-irc = rss?19:33
mordredfungi: kiall has pointed out errbot, which supports both xmpp and irc backends19:33
kiallXMPP .. HipChat19:33
kiallwe use HipChat at HP19:33
fungioh, you mean aol instant messenger!19:33
fungii have seen that before19:34
mordredfungi: so as best I can tell, options would be either adding jabber support to gerritbot, or porting gerritbot to errbot, or writing a duplicate of the gerritbot functionality for something else19:34
kiallfungi: lol.. I wish HP Cloud would get onboard with IRC .. But.. The animated gif support got too many people19:34
*** ildikov_ has joined #openstack-infra19:34
clarkbcan it do ssl connections?19:34
mordredI'm a fan of options 1 and 2 because they lead to less forking of things19:34
fungihonestly, i think gerritbot may be too fragile for much of that, and suspect it would be a porting/rewriting effort19:35
kiallIt uses the same underlying IRC lib as gerritbot does near as I can see19:35
dstufftor abstract away chat protocols and make that pluggable :D19:35
kiallso .. SSL is probably a yes19:35
kialldstufft: like this? https://github.com/gbin/err/tree/master/errbot/backends ;)19:35
mordredhaha19:36
mordredclarkb: errbot usees irc.bot as its irc stuff19:36
mordredso - I'd say it supports the exact same irc features :)19:36
kiallwell.. irc.connection, not irc.bot.. same lib then19:36
mordredhttps://github.com/gbin/err/blob/master/errbot/backends/irc.py#L1119:36
kiallhumm19:37
kiallI only noticed L5 ;)19:37
kiallWhich .. Funny enough.. Looks like a bug. That try import will neverh it19:37
kiallnever hit*19:38
mordredit won't?19:38
fungii'll admit i haven't stared at gerritbot's code much, so it might be amenable to some refactoring to make that abstraction happen... looking it over with an eye for that now19:38
*** yolanda has joined #openstack-infra19:38
kiallyea - L5 will raise an ImportError before the L11 import runs19:38
mordredkiall: oh right- if the irc lib was not there19:39
kiallYep19:39
*** locke105 has joined #openstack-infra19:39
mark0nclarkb: a white-space in my path broke it...19:40
mark0nclarkb: thanks19:40
clarkbnp19:40
dstufftkiall: possibly!19:41
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard: Go back to alembic migrations  https://review.openstack.org/7920819:42
mordredfungi: so - in any case, I didn't want kiall to go have fun this weekend porting gerritbot to be an errbot plugin only to have us tell him to go away19:42
fungiit might not actually be too terribly ugly to pass GerritBot() to inherit from something other than irc.bot.SingleServerIRCBot19:42
mordredfungi: if, otoh, we tell him to go away before hand and he does that work anyway - it's at least a conscious choice19:43
fungicome up with some sort of translation for the common methods supported by both that and the xmpp backend...19:43
*** thuc has quit IRC19:43
clarkbI'd be onboard with it if we can add ssl support19:43
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard: Missing security decorators added  https://review.openstack.org/7989519:44
*** thuc has joined #openstack-infra19:44
fungimordred: well, my biggest concern, as with a lot of these sorts of contributions of features we don't use, is that gerritbot has no testing so we could easily break that feature and not know about it if it didn't break our very specific use case19:44
kiallclarkb: SSL is baked into it as of 4 months ago.. https://github.com/gbin/err/commit/93ff296483d13c0097341101637bd3c5d177648919:44
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard: Fill the last_login field  https://review.openstack.org/7988719:44
mordredkiall: well that certainly seems handy19:45
fungiand one thing gerritbot *does* have going for it, is that it's not using twisted-python19:45
dstufftlifeless: halp, what does "db type could not be determined" mean besides the obvious, I assume it's testrepository but it might also be subunit or some other thing19:45
dstufftfungi: twisted is pretty good for writing network things19:45
fungidstufft: yes i know. but it also comes with a lot of baggage19:46
mordredand I, for one, don't know how to debug twisted applications. I hear great thigns about them though19:46
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard: Logging in with a pre-existing user saves details  https://review.openstack.org/7970519:46
clarkband no ipv619:47
fungidstufft: and things like, you can't use ipv6 and ssl/tls together19:47
fungithat19:47
kiallfungi: I don't see twisted as a dep? At least for the IRC parts (no clue what the XMPP etc backends rely on)19:47
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard: Added paging to list endpoints  https://review.openstack.org/7975719:47
fungikiall: it isn't for gerritbot (it is for supybot which we use in other bots we run)19:48
*** SumitNaiksatam_ has joined #openstack-infra19:48
*** nicedice has joined #openstack-infra19:48
kiallOh.. Fair enough. supybot always felt like eggdrop to me.. Extended beyond it's design without a re-factoring.. ;)19:48
fungii was just taking the opportunity to make an undeserved dig at twisted19:49
fungieggdrop's fine as long as you have a time machine which can take your brain back to tcl syntax19:49
clarkbfungi did you catch the ghub fix?19:49
dstufftmordred: heh19:49
clarkbwe need to add read:orgs perms to the token19:50
dstufftmordred: debugging twisted apps is like 100x easier thand ebugging eventlet/gevent at least :|19:50
*** SumitNaiksatam has quit IRC19:50
*** SumitNaiksatam_ is now known as SumitNaiksatam19:50
fungiclarkb: oh! did they add that as a new permission?19:50
clarkbapparently19:51
fungiclarkb: because we haven't touched the perms for that account on the github side in, like, well and embarrassingly long time i think19:51
clarkbdstufft lol19:51
clarkbour brains must work differently19:51
fungidstufft: in twisted-python, apps debug you!19:51
dstufftclarkb: the problem with eventlet/grevent is that the context switch can happen at any point, so you get a combinatorial number of states the software can be19:51
dstufftwith twisted the context switch happens at very defined points19:52
clarkbdstufft thats true of any threads19:52
dstufftyes19:52
dstufftthreads are horrible to debug too19:52
clarkbwhich means I have lots of practice19:52
dstufftthat just means your brain is pre-broken :D19:52
clarkbthey arent that bad. hard yes but not terrible19:52
* fungi thought it was all so much easier in the pvm/mpich days with explicit message-passing between processes19:53
dstufftIt's probably a result of my primary language being Python, and before that PHP but i've done next to nothing with threads19:53
mordreddstufft: you know we write a bunch of threaded python code aroudn here, yeah?19:53
clarkbdstufft ah. /me did C lots of it19:54
dstufftmordred: crazy people19:54
dstufftpip had some threads19:54
mordreddstufft: works pretty well...19:54
dstufftuntil I ripped them out19:54
*** jp_at_hp has quit IRC19:54
mordredI don't know if you know about the scalable gating system controller called zuul we use...19:54
fungipip does seem like the sort of tool which doesn't need any threading19:54
mordred++19:54
clarkbthere are airplanes flying around with a radio I helped build in them. scary19:54
fungiclarkb: as long as someone else built the black box, they should eventually be able to blame your work19:55
dstufftwell19:55
dstuffthistorically pip used threading because19:55
kiallfungi: I'm not sure that's comforting ;)19:55
dstufftit has an unbounded number of urls it would hit to discover stuff19:56
* kiall starts using the ferry more often19:56
clarkbdont worry its not certified for commercial jets19:56
dstufftor did previously19:56
fungidstufft: right, eww19:56
kialldstufft: Sure, but pip etc should have stopped what they were doing. Immediately. Once they realized they were building a crawler.19:57
dstufftkiall: pip is by defintion a crawler :]19:57
kiall;)19:57
dstufftthat's how you install stuff19:57
dstufftwell find stuff to install19:57
clarkbfungi we have another case of unfamiliar with gerrit but stackforging...19:57
dstufft:/19:57
clarkbfungi I can respond to it19:58
kiallclarkb: no canned response ready to go? ;)19:58
fungiclarkb: thank you for leaping onto that grenade. remind me to beer you in atlanta or something19:58
clarkbfungi let me know if you need more eyes on github token stuff19:58
fungiclarkb: pulling up the bug now, but if i can confirm it, patch on the way19:58
jeblairkiall: can we make gerritbot and statusbot errbot plugins and run them in the same bot?19:58
jeblairkiall: and then, does errbot have a meetbot plugin?  if not, could we make one of those too?19:59
kialljeremyb: yea.. they could be melded into 2 plugins of 1 bot19:59
fungimeetbot too, while we're talking pie in the sky19:59
jeblairkiall: and finally, can errbot do eavesdrop-style channel logging?19:59
kialllol - that's more than a weekend right there jeblair ;)19:59
jeblairkiall: there's lots of weekends!19:59
*** unicell has quit IRC20:00
clarkbfungi patch? you should be able to update the token in github I think20:00
fungii agree it would be beyond awesome not to be managing bots from several disparate codebases20:00
jeblairkiall: because other irc related problems that would be great to solve are: we have a lot of bots, each of which requires configuration and account management...20:00
mordredjeblair: from what I see - I betcha all of those things could be put on a roadmpa - so perhaps this would turn in to the beginning of projet "one bot to rule them all"20:00
fungiclarkb: oh, on the github end. got it. will take a look20:00
jeblairkiall: and also 'we can't restart meetbot (which is also eavesdrop) to pick up a new configuration during meetings'20:00
kiallI'm going with No - not because it's not possible.. But because it's way more work than I can offer ;) But.. Yea.. You can load 100 plugins doing different things into it20:00
*** unicell has joined #openstack-infra20:00
mordredjeblair: also - apparently - each of those could be plugins, and they coudl all still live in diffrent repos if we wanted20:00
fungii think they would almost certainly live in separate repos20:01
jeblairkiall: let's pretend i didn't say "could you" in those questions, and instead am just asking for your insight to decide if this is a good road to head down20:01
jeblair(which should be easy because i didn't say that ;)20:01
mordredas the unit of plugin installation in errbot is  a git repo20:01
fungiheh20:01
*** VijayTripathi has joined #openstack-infra20:01
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Remove logging of sahara projects to #savanna  https://review.openstack.org/7954520:02
jeblairi would have suggested moving our other bots into supybot a while ago, but for the ipv6/ssl thing20:02
jeblairtwisted seems not at all interested in fixing that, so i kind of think that makes supybot a dead end20:02
kialljeblair: Sure ;) So .. yea, it probably could.. It'd boil down to it's internals of if it's capable of watching N channels at once and actually doing something with the input I guess20:02
jeblairkiall: okay, so that's a question we should probably answer early on...20:02
mordredjeblair: I _believe_ we'll have to answer that to do gerritbot at all20:03
kialli.e. It's got a bunch of threading stuff in there, but I'm not sure what it's used for (never looked..)20:03
mordredsince gerritbot needs to watcha  stream20:03
jeblairkiall, mordred: i'm much more interested in a technology shift here if we move to something that puts us in a better place overall20:03
mordredjeblair: I agree20:03
*** Sukhdev has joined #openstack-infra20:04
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: only care about unclassifieds in gate queue  https://review.openstack.org/8006620:05
mordredfrom what I can tell - since gerritbot needs to run a thread to watch the gerrit stream - it means that using errbot for gerrit at all will necessarily have to figure out if it can do multiple things in threads20:06
*** sarob has quit IRC20:06
*** SumitNaiksatam has quit IRC20:07
jeblairmordred: yeah, it doesn't necessarily have to have a huge thread infrastructure, but just having things like the a "send privmsg" method be thread-safe (even better if it has thread-safe flood control) would be good.20:07
jeblairmordred, kiall: so open questions are: a) thread safety; b) on-the-fly reconfiguration; c) sensible configuration for multiple plugins; d) existence of meetbot/eavesdrop plugins20:09
sdagueso honestly, what the suggestion that fungi made off hand earlier20:09
sdagueto gerrit -> rss20:09
jeblairsdague: like openstackwatch?20:09
fungiclarkb: found the permission on the token and added it, rerunning the script now... hasn't died yet either20:10
sdagueI guess, I don't know what that is :)20:10
*** caleb_ has quit IRC20:10
sdaguebut regardless then bots pull that and off we go20:10
jeblairsdague: it's a 95% completed project in the config repo to make an rss feed for gerrit.  afaik it just needs some finishing touches20:10
jeblairsdague, fungi: maybe i don't understand the suggestion...20:10
*** sarob has joined #openstack-infra20:11
sdagueand would solve having everyone talking to gerrit20:11
kialljeblair: noted .. Will eye up the code to see if it's capable of handle those20:11
sdagueor was that not part of what was trying to be solved20:11
jeblairsdague: you want to go from a real-time push notification system to a delayed polling based pull system?20:11
fungijeblair: i was indeed making an illusion to openstackwatch20:11
*** jasond` has left #openstack-infra20:11
fungiallusion20:11
jeblairsdague: afaik the problem kiall wants to solve is that his company doesn't use irc for their internal copy of infra, so they want a gerritbot that talks to their proprietary thing20:12
*** weshay has quit IRC20:12
kialljeblair: Yep - bang on20:12
kiallBut if the choice of bot/lib/etc has other benefits, cool.20:13
jeblairkiall: ++20:13
kialljeblair: .. seeing code like this suggests on-the-fly reconfig won't be an issue.. https://github.com/gbin/err/blob/master/errbot/plugin_manager.py#L140-15420:15
*** esker has quit IRC20:15
sdagueyeh, it just makes me wonder about the other bots as well. because I imagine the desire for another elastic recheck at some point.20:16
kiallseems to handle reloading the plugin's code without a restart.. So, promising it can handle re-config20:16
jeblairsdague: yeah, i'd love it if all our bots could be plugins to one bot; including er; to reduce the overhead of what we really want to do which is "add functionality to irc"20:16
sdaguejeblair: yeh that would be nice20:17
kialljeblair: s/irc/irc+hipchat/ ;))20:18
sdagueI expect the complexities build up pretty quickly here though20:18
*** resker has joined #openstack-infra20:18
fungisdague: the openstackwatch module in openstack-infra/jeepyb makes http://rss.cdn.openstack.org/nova.xml20:19
fungijust basically a query of open reviews20:20
fungiand uploads to a swift container periodically20:20
sdaguecool20:20
*** dprince has joined #openstack-infra20:20
fungichmouel wrote it and anteaya puppeted it a while back20:20
*** rpodolyaka has quit IRC20:20
*** rpodolyaka has joined #openstack-infra20:21
*** rpodolyaka has quit IRC20:21
*** rpodolyaka1 has joined #openstack-infra20:21
fungii think the idea was that once we have more control over gerrit themes, it could get linked as an rss feed from an out-of-the-way part of the ui20:21
*** Sukhdev has quit IRC20:21
*** mrda_away is now known as mrda20:21
fungiclarkb: close-pull-requests completed its run with no errors20:22
sdaguecoolio20:23
*** thuc has quit IRC20:24
*** thuc has joined #openstack-infra20:24
*** sweston has quit IRC20:25
jeblairyeah, forwarding everything in unbound is trivial20:26
*** changbl has quit IRC20:26
sdaguejeblair / fungi: I rebased jogo's patch - https://review.openstack.org/#/c/78687/ to free up some dsvm nodes20:27
*** mrodden has quit IRC20:28
clarkbfungi woot20:28
jeblairlgtm20:28
fungiclarkb: spammed us with about 30-60 pr closures20:29
*** rossella_s has quit IRC20:29
clarkbfungi was the token thing through their webui?20:29
fungiclarkb: yep!20:29
fungiunder settings->applications->personal token access20:29
*** rossella_s has joined #openstack-infra20:29
fungithere's a whole array of permissions you can add/remove in there now20:29
jeblairlooks like check node wait time is about 1.5 hours20:29
fungibut that was the only one i needed to add20:29
*** afazekas has quit IRC20:29
jogosdague: thanks!20:30
*** MarkAtwood has joined #openstack-infra20:30
*** che-arne has quit IRC20:30
fungijogo: lgtm too... aprv'd20:31
*** mburned is now known as mburned_out20:32
jeblairof course the dnssec trust file is /var/lib/unbound/root.anchor on centos and /var/lib/unbound/root.key on ubuntu.20:32
openstackgerritA change was merged to openstack-infra/config: Don't run large-ops test on repos that it doesn't touch  https://review.openstack.org/7868720:33
fungithey're almost the same... what's a file suffix between frenemies?20:33
lifelessdstufft: it mens python changed defaults for dbm between minor versions and screwed everyone20:34
lifelessdstufft: *means*20:34
dstufftlifeless: how can I fix it20:34
dstufftit happens on the first run even if I delete the .testrepository dir :[20:35
lifelessdstufft: rm .testrepository/timing.db20:35
lifelessdstufft: I'm going to guess that you're using tox to run several versions of python at once20:35
dstufftlifeless: yes tox, just 1 version20:35
jogohttp://logs.openstack.org/67/57967/23/gate/gate-tempest-dsvm-neutron/0de62a7/logs/devstack-gate-setup-workspace-new.txt.gz20:35
jogois there a bug filed for that20:35
lifelessdstufft: just one version? really?20:35
lifelessdstufft: colour me skeptical :)20:36
*** e0ne has quit IRC20:36
dstufftlifeless: actually I remember the symtpoms slightly wrong, it works the first time but fails every other time after that20:36
*** e0ne has joined #openstack-infra20:36
dstufftunless I delete20:36
dstufftlifeless: well i'm using tox -e py2620:37
dstufftso if it's executing something other than py26 ima be angry20:37
lifelessdstufft: you may need to be angry; anyhow what I'd do is just not use testr within the venv; use it from outside.20:37
lifelessdstufft: then you won't be subject to python version vaguaries20:37
jogofungi: ^^20:38
*** emagana has joined #openstack-infra20:38
jogosaw a whole slew of failures like this http://logs.openstack.org/59/62859/1/gate/gate-tempest-dsvm-neutron/7025cba/console.html20:38
mordredlifeless: there's no good way to do that in a tox context really20:38
jogownat to add a e-r  fingerprint to remove them from the unclassificed list20:38
fungijogo: 129144020:38
dstufftlifeless: did I ever mention that software is the worst20:39
jeblairOverLimit: Quota exceeded for ram: Requested 16384, but already used 944128 of 960000 ram (HTTP 413) (Request-ID: req-368f5ce0-8211-4acd-9acb-ba002baea11b)20:39
jeblairfungi, clarkb, lifeless: ^20:39
*** VijayTripathi has quit IRC20:39
clarkbjeblair: which provider is that from?20:39
fungijeblair: in tripleo?20:39
jeblairdoes anyone understand why it is requesting 16384 when it's configured for 819220:39
clarkboh right becaseu 16G nodes20:39
fungi(i assume because you pinged lifeless)20:39
jeblairtripleo, yes20:39
clarkbjeblair: there is no 8192 flavor, so it picks the next bigger20:39
jeblairthat's q.120:40
jeblairok20:40
*** e0ne has quit IRC20:40
jeblairso q.2:  960000/16384 = 58.59375020:40
*** ociuhandu has quit IRC20:40
*** thuc_ has joined #openstack-infra20:40
lifelessmordred: just remove testrepository from the test-requirements, no?20:40
lifelessmordred: use binddep instead20:40
jeblairour actual limit is 58 servers; lifeless: should we adjust nodepool's config to match, or do you want to increase the quota?20:41
jogofungi: how does 'message:"savanna"  AND message: "fatal\: https://git.openstack.org/openstack" AND filename:"logs/devstack-gate-setup-workspace-new.txt"' sounds for a fingeprint20:41
lifelessjeblair: memory quota? let me check hypervisor stats20:41
jeblairlifeless: yes20:41
lifelessfree_ram_mb          | 10762420:41
fungiclarkb: on that last update to 1291479, you'd think they'd be working more closely with the heat developer community (they did request sbaker as a member of their ptl group)20:42
lifeless| memory_mb            | 966760 |20:42
lifeless| memory_mb_used       | 859136 |20:42
clarkbfungi: you would think...20:42
clarkbfungi: I am going to respond now20:42
clarkbthen figure out puppet module installs20:42
*** e0ne has joined #openstack-infra20:42
ttxfungi, jeblair: howdy! Is the world still turning ?20:42
fungittx: still turning20:42
lifelessroom for 6 more in there20:43
mordredlifeless: what?20:43
lifelessso I think reduce nodepool config20:43
lifelessmordred: what what20:43
fungilifeless: have we maybe leaked nodes in that tenant?20:43
ttxfungi: damn, that hurts my narcissism20:43
*** thuc has quit IRC20:43
fungittx: become a solipsist and then it won't matter ;)20:44
lifelessfungi: there's 859GB used, which is ~53 servers20:44
ttxfungi, jeblair: it nothing urgent is needed from me, i'll go back to ignoring what happens this week20:44
lifelessfungi: so hitting 58 seems about right20:44
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Adjust tripleo nodepool config to actual values  https://review.openstack.org/8007820:44
lifelesssince 53 + 6 = 5920:44
*** jasondotstar has joined #openstack-infra20:45
jeblairttx: go back to skiing.  or start skiing if you've been snowboarding.  ;P20:45
lifelessjeblair: dunno if you saw https://review.openstack.org/#/c/79091/20:45
*** wenlock has joined #openstack-infra20:45
lifelessjeblair: which I put up when I changed the flavor20:45
jeblairlifeless: nope, hadn't gotten to that yet, though it looks like it's still to high, yeah?20:45
sdaguettx: everythings pretty solid, nothing crazy on fire20:46
jeblairlifeless: i went with 50 since i calculate the max at 5820:46
lifelessjeblair: it is, I can abandon happily enough20:46
*** yolanda has quit IRC20:46
*** mrodden has joined #openstack-infra20:46
fungijogo: "savanna-dashboard/info/refs not found" maybe20:46
fungijogo: i think they all failed on the same point in the list20:46
fungijogo: and that's one which definitely won't recur exactly the same way again20:46
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1291440  https://review.openstack.org/8008020:47
clarkbttx: enjoy the snow20:47
*** Sukhdev has joined #openstack-infra20:47
* fungi is popping out for dinner... bbl20:47
jogofungi: thanks20:47
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1291440  https://review.openstack.org/8008020:48
SergeyLukjanovjogo, agreed with fungi20:48
jogoSergeyLukjanov: ^20:48
SergeyLukjanovjogo, thanks for fingerprint20:48
SergeyLukjanovjogo, are there many affected changes?20:49
jogoSergeyLukjanov: once that lands I think we should have a pretty high classifcation rate http://status.openstack.org/elastic-recheck/data/uncategorized.html20:49
jogoSergeyLukjanov:  there were a good number in gate20:49
jogo258 hits in gate20:50
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: fix formatting on 'and'  https://review.openstack.org/8008320:50
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: only care about unclassifieds in gate queue  https://review.openstack.org/8006620:51
ttxalrighty then20:51
* ttx signs off20:51
*** ArxCruz has quit IRC20:52
openstackgerritMartin Konrad proposed a change to openstack-infra/jenkins-job-builder: SCM module: Add support for multiple Git remotes.  https://review.openstack.org/8008420:52
SergeyLukjanovjogo, oh, quite a lot20:52
fungia lot of those didn't report though20:52
jogofungi: that makes sense20:53
clarkbnibalizer:  Module 'puppetlabs-inifile' (v1.0.3) is already installed20:53
fungias soon as sdague pointed them out, we force-merged the fix immediately and i reshuffled the gate to get everything which was still in there but about to die to retest20:53
jogoI think that caused the recent spike http://jogo.github.io/gate/20:53
*** thuc__ has joined #openstack-infra20:53
clarkbnibalizer: puppet module install should install the modules that it needs as dependencies right? but you want ot install it as no deps?20:53
clarkbnibalizer: I am slowly debugging this but can't install git on my VM because my apt repo is gone apparently20:53
*** thuc_ has quit IRC20:54
clarkbnibalizer: I will know a lot more in about 5 minutes20:54
nibalizerclarkb: okay20:54
fungijogo: probably so. shouldn't have happened outside of 12:45-13:30 if memory serves20:54
* fungi is really at dinner now ;)20:54
nibalizerpuppet module install brings in deps, yes20:54
SergeyLukjanovhttps://review.openstack.org/#/c/80080/ lgtm20:54
nibalizerclarkb: is this changes that have already landed or a change that I can look at?20:54
clarkbnibalizer: so I think something is installing 1.0.3 then the specific 1.0.0 version dies20:55
nibalizeralso i'm about to go into a work meeting so will be of limited use for an hour or two20:55
clarkbnibalizer: its tip of master20:55
nibalizergreaaat20:55
clarkbnibalizer: no worries I am debugging now20:55
nibalizeruh well just rollback to 1.0.0 and screw puppetdb, i can fix it later :P20:55
clarkbI can't rollback that is what it is trying to do :P20:55
clarkband failing20:55
*** khyati has quit IRC20:57
*** malini_afk is now known as malini20:57
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Periodic purge of fixed and unseen bugs  https://review.openstack.org/8008620:59
*** skraynev is now known as skraynev_afk21:00
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for nova unit test bug 1291605  https://review.openstack.org/8008721:02
*** vkozhukalov has joined #openstack-infra21:02
*** rlandy_ has joined #openstack-infra21:04
*** rpodolyaka1 has quit IRC21:05
*** smarcet has joined #openstack-infra21:05
*** thuc has joined #openstack-infra21:05
*** gyee has quit IRC21:05
*** ociuhandu has joined #openstack-infra21:06
*** nati_ueno has quit IRC21:07
*** rlandy__ has quit IRC21:07
*** mburned_out is now known as mburned21:08
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: fix formatting on 'and'  https://review.openstack.org/8008321:08
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: only care about unclassifieds in gate queue  https://review.openstack.org/8006621:08
*** thuc__ has quit IRC21:08
*** nati_ueno has joined #openstack-infra21:08
*** rpodolyaka has joined #openstack-infra21:09
*** weshay has joined #openstack-infra21:09
*** dprince has quit IRC21:11
mattoliverauMorning21:11
*** mrodden has quit IRC21:11
*** mrodden has joined #openstack-infra21:12
*** rlandy__ has joined #openstack-infra21:13
*** jeckersb is now known as jeckersb_gone21:13
jeblairmattoliverau: good morning21:13
openstackgerritRandy Coulman proposed a change to openstack-infra/jenkins-job-builder: Make email-ext's AlwaysTrigger the default.  https://review.openstack.org/7933721:14
*** denis_makogon_ has joined #openstack-infra21:14
*** rlandy_ has quit IRC21:15
*** e0ne has quit IRC21:16
*** dkranz has quit IRC21:17
*** e0ne has joined #openstack-infra21:17
clarkbnibalizer: huh it worked fine on my test machine21:19
*** rfolco has quit IRC21:19
nibalizerweird21:19
clarkbnibalizer: is there a reson for explicitly listing 1.0.0 in install_modules? is that so you can install puppetdb module without deps?21:20
clarkbI am going to give it a try with nodepool again21:20
*** mfer has quit IRC21:20
clarkb(I used the same region and base image and everything to test)21:20
*** nati_ueno has quit IRC21:20
*** denis_makogon_ is now known as denis_makogon21:21
*** zns has joined #openstack-infra21:23
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for nova unit test bug 1291620  https://review.openstack.org/8009121:23
*** hogepodge has quit IRC21:23
*** rlandy__ is now known as rlandy21:24
clarkbI will try a different provider too21:24
*** hogepodge has joined #openstack-infra21:24
nibalizerclarkb: no inifile can have any 1.x release21:25
nibalizeras far as puppetdb is concerned21:25
*** sweston has joined #openstack-infra21:25
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Use unbound  https://review.openstack.org/8009221:26
nibalizerclarkb: you can bump inifile to 1.0.3 if that makes problems go away21:26
clarkbnibalizer: ok it just worked21:26
openstackgerritRandy Coulman proposed a change to openstack-infra/jenkins-job-builder: Fix typo in ircbot's default matrix-notifier.  https://review.openstack.org/7932821:26
clarkbnibalizer: I think puppet forge may haev been cranky with us21:26
*** cody-somerville- has joined #openstack-infra21:26
nibalizerpuppetdb just wants 1.something21:26
nibalizerhttps://github.com/puppetlabs/puppetlabs-puppetdb/blob/master/Modulefile21:26
*** jasondotstar has quit IRC21:26
clarkbnibalizer: so I think we are good, thank you21:26
jeblairclarkb, fungi, pleia2, mattoliverau: https://review.openstack.org/80092 could use some scrutiny as it has the capability to break everything.  everywhere.  all at once.  :)21:26
clarkbjeblair: looking21:27
nibalizerokay21:27
clarkbjeblair: is that the sort of thing we merge friday afternoon?21:27
jeblairclarkb: i'm thinking so.21:27
jeblairafter some dry runs21:28
*** rcarrillocruz has joined #openstack-infra21:28
*** caleb_ has joined #openstack-infra21:28
*** caleb_ has quit IRC21:29
openstackgerritA change was merged to openstack-infra/storyboard: Fill the last_login field  https://review.openstack.org/7988721:29
*** caleb_ has joined #openstack-infra21:29
*** caleb_ has quit IRC21:29
*** rcarrillocruz1 has quit IRC21:29
*** sweston has quit IRC21:29
*** sweston has joined #openstack-infra21:30
*** caleb_ has joined #openstack-infra21:30
*** caleb_ has quit IRC21:30
*** sweston has quit IRC21:30
*** sweston has joined #openstack-infra21:31
*** caleb_ has joined #openstack-infra21:31
*** nati_ueno has joined #openstack-infra21:33
clarkbjeblair: reviewed21:35
openstackgerritRandy Coulman proposed a change to openstack-infra/jenkins-job-builder: Fix typo in ircbot's default matrix-notifier.  https://review.openstack.org/7932821:38
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard: Logging in with a pre-existing user saves details  https://review.openstack.org/7970521:42
* clarkb wait for vmdk to finish downloading so that we can have new hpcloud region b image21:43
clarkbsdague: ^ about that21:44
*** thedodd has quit IRC21:44
clarkbsdague: I think the discussion yesterday was that anyone can have bigger images if they want them21:44
*** CaptTofu has quit IRC21:44
sdagueclarkb: I think that whatever parses devstack should be smarter about that21:44
clarkbsdague: and we do download an ubuntu image. None of that takes forever21:44
clarkbthe vmdk takes forever21:44
sdaguebecause there is no reason that vmdk is in that image at all21:45
clarkbsdague: its there because devstack may useit21:45
*** mriedem has quit IRC21:45
clarkbbecause its in the devstack config21:45
sdaguebut you know which hypervisors you are going to support in that image21:45
sdagueand all of those are hidden behind hypervisor ifs21:46
*** smarcet has left #openstack-infra21:46
sdagueso instead of parsing that, eval it21:46
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add fingerprint for bug 1291440  https://review.openstack.org/8008021:46
openstackgerritA change was merged to openstack-infra/elastic-recheck: Periodic purge of fixed and unseen bugs  https://review.openstack.org/8008621:46
clarkbsdague: the image creator can't eval it with a high degree of certainty about which case will be hit21:46
clarkbsdague: because we could xen or kvm cirros or ubuntu21:46
openstackgerritA change was merged to openstack-infra/storyboard: Missing security decorators added  https://review.openstack.org/7989521:47
*** khyati has joined #openstack-infra21:47
mark0nI'm using gerrit for the first time. I submitted a patch and jenkins discovered a minor style issues. How am I supposed to resubmit? git commit --amend && git review or git commit && git review?21:47
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard: Logging in with a pre-existing user saves details  https://review.openstack.org/7970521:47
*** thomasem has quit IRC21:48
clarkbmark0n: if you need to update a change you already pushed do git commit --amend && git review21:48
clarkbmark0n: git commit && git review will give you a second change21:48
mark0nclarkb: ok, thanks21:48
mattoliveraujeblair: looking now, sorry, had a lot of scroll back to read :)21:50
jeblairsdague: yeah, so i suggested mordred propose that change to spark a conversation about that...21:51
*** rlandy has quit IRC21:51
jeblairsdague: my understanding is that devstack has been shrinking to core functionality with good support for plugins/addons21:51
*** rpodolyaka has quit IRC21:52
jeblairsdague: so i thought with that in mind, it was worth considering whether vmware support is still something that should be in core21:52
jeblairsdague: or if it makes better sense as add-ons21:52
sdaguejeblair: so that's probably a can of worms in politics that will take us a while to get there21:53
sdaguebecause it's an in tree driver for nova21:53
sdaguewhich is fair game21:53
sdagueclarkb: and I disagree with the high degree of certainty issue21:54
clarkbsdague: how can nodepool image builder know if it needs cirros or ubunut image or xen21:54
sdaguehttp://paste.openstack.org/show/73317/21:54
clarkbsdague: it can't21:54
*** fifieldt has joined #openstack-infra21:54
clarkbthe distinguishing features between cases aren't strong enough21:54
sdagueyou should have a config for nodepool to decide what hypervisor list you want to support21:54
sdagueand just drop vmware from that list21:55
clarkbwhcih means we need to udpate devstack-gate anytime you want a new hypervisor21:55
clarkbwhich we don't need today21:55
sdaguesure21:55
jeblairclarkb: not devstack-gate, nodepool config21:55
clarkbjeblair: oh right this is nodepool config now21:55
clarkbI am thinking of the old thing21:55
sdaguebut your alternative is to impose policy into devstack because you don't want something in the gate21:56
sdaguewhich I think is the wrong thing21:56
jeblairsdague: whoah whoah21:56
jeblairsdague: no one is imposing policy21:56
*** dkliban has quit IRC21:56
*** rpodolyaka has joined #openstack-infra21:56
sdagueI really think it should be fine for nodepool to decide the list of hypervisors it's going to support and eval devstack to get all those images. Then the vmware thing won't impact things.21:57
sdagueor whatever part of infra wants to make that decision21:57
jeblairsdague: we're just having a discussion about where the configuration of this kind of environment belongs in the developer deployment tool...21:58
openstackgerritMartin Konrad proposed a change to openstack-infra/jenkins-job-builder: SCM module: Add support for multiple Git remotes.  https://review.openstack.org/8008421:58
sdaguesure21:58
jeblairsdague: mordred suggested that we just ignore the vmdk21:59
*** rossella_s has quit IRC21:59
sdagueI strongly believe that having a functional base image for your hypervisor that you installed is an important starting point21:59
jeblairsdague: i said i thought it would be nice if we actually talked to some devstack developers about it before we just worked around the problem21:59
sdaguehonestly I think devstack should provide more of those21:59
jeblairsdague: i'm starting to regret that now that i've been accused of trying to 'impose policy'21:59
sdaguejeblair: ok, well mordred's patch was basically delete the vmdk because it's slow :)22:00
sdaguewhich is what I heard from clarkb as well22:00
sdagueso maybe I misheard22:00
mordredsdague: two different patches22:00
mordredsdague: I made a patch to devstack which was "delete the ref to the vmdk" - and I did that to start the discussion22:00
mordredsdague: I also made a patch to devstack-gate which trims the vmdk out of the list of images we pre-download22:01
mordredsdague: which would work, but puts more not less logic in d-g22:01
clarkbmordred: nodepool22:01
mordredclarkb: yah22:01
clarkbmordred: I keep confusingthem too because d-g was the cacher. One day I will remember22:01
mordredsdague: my view, fwiw, is that if it's not tested, it's broken - and we don't test the vmdk in the gate, so I dont' know why it's in devstack22:02
lifelessfinally!22:02
lifelessCloud Images is currently in Limited Availability. It will allow you to import properly prepared images. We expect to open it to all customers before the end of March.22:02
lifelessYou can take a look at the FAQ to see exactly what’s coming: http://www.rackspace.com/knowledge_center/article/frequently-asked-questions-about-using-glance-and-the-images-api22:02
mordredlifeless: yup. excited about that22:02
*** jhesketh__ has joined #openstack-infra22:02
*** jhesketh__ has quit IRC22:02
*** jhesketh_ has joined #openstack-infra22:02
*** jhesketh__ has joined #openstack-infra22:03
*** jhesketh_ has quit IRC22:03
*** rpodolyaka has quit IRC22:03
*** jhesketh has joined #openstack-infra22:03
jhesketh__Morning22:03
mordredmornign jhesketh__22:03
*** mwagner_lap has quit IRC22:04
*** zul has joined #openstack-infra22:04
clarkbsdague: we noticed the thing because it is slow to cache22:04
clarkbI think more than half our image build time is downloadingthat image22:05
*** zul has quit IRC22:05
clarkbnot saying you can't have large images. (and the ubuntu image is evidence of that unless I am missing something)22:06
jeblairwhy is it a different image in the first place?22:07
jeblairi mean, can you not run cirros on esx?22:08
jeblair(and sure, if you want to have a full os instead of cirros that's fine -- couldn't you run the same full os on kvm and esx?)22:09
mtreinishfungi, clarkb, jeblair: have you guys seen this before?: http://logs.openstack.org/72/79572/1/check/gate-tempest-python27/f099c44/console.html#_2014-03-11_10_02_35_37222:09
jeblairmtreinish: unit tests aren't allowed to sudo22:09
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: If not posting IRC comment for unrecognized error, log something  https://review.openstack.org/7880122:10
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Make IRC bot list which failures were seen in which job.  https://review.openstack.org/7879022:10
jogomtreinish: this was for you ^22:10
*** rpodolyaka has joined #openstack-infra22:10
mtreinishjeblair: ahh ok, I missed that the proposed patch was actually trying to run something with sudo instead of mocking it22:12
* mtreinish leaves a -1 comment22:12
jeblairmtreinish: yay the system works! :)22:13
mtreinishjogo: it was't my thing I was fine without the tests. I was just trying to get mriedman to stand up for himself :)22:13
*** maxbit has joined #openstack-infra22:14
*** rcleere has quit IRC22:15
clarkbhow do we feel about merging mordred nodepool change? (I am still waiting for this new image, makes it really hard to iterate on this stuff)22:15
clarkbI don't understand if garyk is saying they are using nodepool or only devstack22:15
jogomtreinish: well it was a valid point, that code needed tests22:15
mtreinishjogo: yeah but we've kind of just ignored it for most things in the past22:16
jogomtreinish: heh i noticed22:17
jeblairclarkb: wfm22:17
*** lcostantino has quit IRC22:18
sdaguesorry, got pulled away22:19
clarkbjeblair: were you able to parse garyk's statement? I can't tell if the nodepool change will breka him or not22:19
sdaguemordred: I think it's in devstack because the vmware system does test it22:19
sdaguein their CI22:19
sdaguewe can confirm and call them on that22:20
mordredsdague: do they use devstack in their CI?22:20
sdaguefor nova, probably22:20
*** aysyd has quit IRC22:20
sdagueI know the hyperv guys do22:20
mordredok. then  I can buy it being decently valid - and I'd vote for us filtering in d-g - since they are one of the first ones who stepped up to the 3rd party gate strongly22:20
mordredif it was "a dev might want to do vmware stuff" my opinion is different22:21
sdaguemordred: ok, fair.22:21
clarkbis garyk usually on irc? /me would like to clarify comment on https://review.openstack.org/#/c/79537/1 before approving22:22
*** yassine has joined #openstack-infra22:22
sdagueso I guess it begs a second question, which is I'd like to add fedora20 and ubuntu(something) cloud images into devstack to make it more useful to people for dev (as that's one of the first things I add)22:23
sdagueso we should figure out if that's going to cause issues here, or if there is some other way we want to mark the images22:23
clarkbsdague: I don't think it is going to cause issues. see my earlier statements22:23
sdagueok22:24
clarkbthe existing ubuntu image is fine22:24
sdaguethe lxc one?22:24
*** lukego has quit IRC22:24
clarkbopenvz22:24
sdagueok22:24
sdaguenow, openvz, that I could get behind deleting entirely :)22:24
clarkb++22:24
sdaguebecause, no one is using it in the gate22:24
sdaguebut we'll wait until the docker drop22:25
*** e0ne has quit IRC22:25
*** maxbit_ has joined #openstack-infra22:25
*** e0ne has joined #openstack-infra22:25
*** e0ne has quit IRC22:25
*** jgrimm has quit IRC22:26
clarkbermagherd22:26
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: If not posting IRC comment for unrecognized error, log something  https://review.openstack.org/7880122:26
clarkbthe image cacher is downloading the same image over and over22:26
sdaguehah22:26
* clarkb looks at fixing that22:26
sdagueso, I do think it is probably worth while to figure out why vmware needs a custom image. I will put that on a list, because I'd really like to get us to people pulling fairly standard upstream cloud images (fedora and ubuntu)22:27
sdagueas that seems more useful22:27
*** mgagne has quit IRC22:27
clarkb++22:27
*** rpodolyaka has quit IRC22:28
sdagueclarkb: do you have the vmware bot id ?22:29
sdagueso I could find some runs in gerrit to confirm what they are doing22:29
*** rpodolyaka has joined #openstack-infra22:30
mordredclarkb, sdague how about we figure out a way to flag images that the gate doesn't need to download some how?22:30
*** harlowja has quit IRC22:30
jgriffithfungi: that wasn't nice BTW :)22:31
sdaguemordred: right, well I was thinking about that in terms of evaling with hypervisor22:31
*** harlowja has joined #openstack-infra22:31
jgriffithfungi: re the comment about "most deserving of it's ire"22:31
sdaguemordred: http://paste.openstack.org/show/73317/22:31
sdagueevery image add in devstack is behind some conditional22:31
sdagueso if you just set those conditionals, you could eval out all the images22:32
jeblairsdague: keep in mind that we have to do this for every branch of devstack, so we need a fairly stable api22:32
sdaguejeblair: that's been pretty stable22:32
sdagueVIRT_DRIVER has been there forever22:32
jeblairsdague: yeah, but 'functions' hasn't22:33
sdaguewe'll make sure to make it more stable if we know it's something that will be externally called. I don't have an issue with that22:33
jeblairsdague: cool22:33
sdaguejeblair: sure, well we could also provide the script in devstack22:33
mordredhell - I'd say we just do that snippet above ^^22:33
mordredand just do it for libvirt - since d-g doesn't do xenserver either22:33
clarkbmordred: no22:34
sdaguemordred: yeh, that was my suggestion. There is probably one other variable for heat that's needed as well22:34
mordredand as soon as d-g does xenserver or lxc or anything else, we can add a thing to VIRT_DRIVERS22:34
jeblairmordred: not yet; it might possibly could, and actually that could get a lot easier with glance22:34
mordredyah22:34
clarkbI don't like it I don't want to update nodepool each time we need a new hypervisor22:34
jeblairthere are people working on that22:34
mordredbut as long as it's just a list22:34
clarkbdocker, xen, etc are all waiting22:34
clarkband if they have to wait for new images thats a huge delay22:34
mordredjeblair: I'm quite excited about glance22:34
clarkbif we can just download some sane images then we are set22:34
sdaguejeblair: so what if we provided a devstack tool that let you declare a list of hypervisors you care about on the command line22:34
sdagueand it dumps image urls yuo need22:34
jeblairsdague: and make that tool name/command like argument the stable api?22:35
sdagueinstead of the parser, which I know I broke you guys22:35
sdaguejeblair: yep22:35
mordred++22:35
jeblairsdague: sounds lovely if we can get clarkb on board with the idea :)22:35
sdaguewe'll put it in devstack and backport it22:35
clarkbthat still requires a huge turn around delay on getting support for new stuff22:35
mordredclarkb: what don't you like it?22:35
sdagueso, I've got something else which will make clarkb happy immediately22:35
clarkbmordred: beacuse you are talking days of waiting, get image in devstack, get nodepool to update, wait for images22:35
*** jergerber has joined #openstack-infra22:35
jeblairclarkb: it's worth noting that devstack does work with a cache miss in this system22:36
sdagueturns out vmware doesn't use this image in their CI - http://208.91.1.172/logs/77736/5/console.txt22:36
clarkbwith the current system docker/xen/whoever could just start doing stuff22:36
sdaguethey use a cirros image from the same url22:36
sdaguewhich will be tons smaller22:36
mordredclarkb: what?22:36
mordredclarkb: I'm not trying to troll - I am not following what you are talking about at all22:36
jeblairclarkb: so you can actually start doing your devstack work etc before updating the image cacher to add your hypervisor22:36
clarkbjeblair: with a potentially massive delay as we are seeing22:36
jeblairclarkb: and, tbh, in 3 years we still haven't added a hypervisor.22:37
clarkbjeblair: thats fair22:37
clarkbjeblair: but, docker and xen are wanting to22:37
clarkband I expect them to in juno22:37
jeblairclarkb: we could go ahead and put them in our initial list22:37
sdagueclarkb: https://review.openstack.org/8010122:39
*** malini is now known as malini_afk22:39
sdagueso that will make you immediately more happy.22:39
jeblairclarkb: moreover, with glance we might end up having 'devstack-foo-xen' 'devstack-foo-kvm' images anyway, with the nodepool scripts having a bit more knowledge of hypervisors...22:39
*** jamielennox|away is now known as jamielennox22:40
sdagueok, so I need to run away. if we want a tool in devstack that takes hypervisors and spits out images, I am happy to do that22:41
*** jergerber has quit IRC22:41
sdaguebut I'll let folks decide22:41
sdagueregardless we should be able to fix the giant debian image issue in the short term22:41
sdagueper patch above22:41
openstackgerritClark Boylan proposed a change to openstack-infra/config: Download each cached image once.  https://review.openstack.org/8010222:42
clarkbjeblair: mordred ^ you will like that22:42
clarkband I think that fixes a majority of the problem here22:43
sdagueclarkb: well dtroyer just approved the switch to cirros as well22:45
sdagueso that will also help22:45
clarkbwoot22:45
*** MarkAtwood has quit IRC22:45
sdagueok, running away for real, have a good night22:45
*** sarob__ has joined #openstack-infra22:46
*** Daviey has quit IRC22:46
*** Sukhdev has quit IRC22:47
*** changbl has joined #openstack-infra22:47
*** mrodden has quit IRC22:48
*** sarob has quit IRC22:49
*** sarob__ has quit IRC22:50
*** emagana has quit IRC22:51
*** hogepodge has quit IRC22:53
*** MarkAtwood has joined #openstack-infra22:53
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Use unbound  https://review.openstack.org/8009222:55
clarkbjeblair: mordred fungi SergeyLukjanov I held 15.126.230.111 it booted off of the hpcloud-region-b image. It came up with 8GB of RAM and no non standard apt lists22:55
clarkbfigured I would give folks an opportunity to take a look but it seems happy22:55
jeblairclarkb, pleia2, mattoliverau: ^ i responded to your comments; maybe we should have further discussion here?22:55
clarkbjeblair: let me look22:55
*** alex-out is now known as Alexandra22:56
jeblairoh, i guess gerrit doesn't order those things...22:56
*** Daviey has joined #openstack-infra22:56
jeblairum, let me know if you have trouble matching my response to your comment.  :)22:57
clarkbjeblair: for the centos thing, puppet will set resolve.conf to 127.0.0.122:57
mordredclarkb: woot22:57
jeblairclarkb: my response to you was "I tested this on a centos node. Can you point me at where you're talking about?"22:57
clarkbjeblair: then the nodepool script addition will run and make the unbound talk to localhost instead of external resolver22:58
jeblairclarkb: oh, you're talking about my own change...22:58
jeblairthat makes sense.  :)22:58
clarkbjeblair: https://review.openstack.org/#/c/80092/1/modules/unbound/manifests/init.pp line 5022:58
clarkbyup22:58
*** _sweston_ has joined #openstack-infra22:58
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Use unbound  https://review.openstack.org/8009222:59
jeblairclarkb: thank you, that was a very nice "doh!" catch.  :)22:59
*** sweston has quit IRC22:59
clarkbnp23:00
openstackgerritTom Fifield proposed a change to openstack-infra/config: Enable Welcome Message to post review comments  https://review.openstack.org/8011123:00
jeblairclarkb: so as to your suggestion about getting a list of nameservers... where should we get them?23:00
jeblairmattoliverau: ^ goes to your comment too...23:00
clarkbnow the other thing to worry about23:01
clarkbwill dhcp update the nameserver for us?23:01
jeblairclarkb: what dhcp?23:01
clarkbbecause that would be annoying23:01
clarkbjeblair: the dhcp run by the cloud provider to set things when VMs come up23:01
clarkbiirc nova has a really low lease time so we would be picking up new leases at least a few times during a test23:01
openstackgerritA change was merged to openstack-infra/config: Download each cached image once.  https://review.openstack.org/8010223:02
*** dims_ has quit IRC23:02
clarkbjeblair: this is easy to test, I will change the nameserver on one of my hpcloud instances23:02
clarkbthen wait and see if a new lease updates it (can rpobably also look in /var/log/dhclient/leases*23:02
jeblairclarkb: well, rackspace writes /etc/network/interfaces23:02
jeblairclarkb: and uses a static config23:02
clarkbhuh no dhclient, maybe that is how hpcloud does it too23:03
*** hogepodge has joined #openstack-infra23:03
clarkbah /var/lib23:03
*** adalbas has quit IRC23:03
jeblairclarkb: ?23:03
clarkbjeblair: option domain-name-servers 8.8.4.4,8.8.8.8; is set in the hpcloud lease23:03
clarkbjeblair: we can do something to ignore that option. I am trying to remember what; haven't had to do it in years23:04
*** melwitt has quit IRC23:04
jeblairclarkb: so hpcloud does run dhclient?23:04
clarkboption dhcp-rebinding-time 151200; is not as short as I thought it may be23:05
*** mark0n has quit IRC23:05
clarkbjeblair: yup23:05
jeblairclarkb: also, hpcloud uses google for their dns servers ?!?!?!23:05
pleia2jeblair: so I just manually edited /etc/resolv.conf on an hpcloud ubuntu instance, when I rebooted it had reset back to the old one23:05
clarkbjeblair: apparently23:05
clarkbpleia2: ya because dhcp23:05
clarkbpleia2: check your leases in /var/lib/dhcp/23:05
pleia2yep, as I mentioned in my comment23:05
jeblairpleia2: your comment was about a bit of code that didn't write to /etc/resolv.conf23:06
jeblairpleia2: nothing i did in that patch writes to /etc/resolv.conf on ubuntu23:06
*** atiwari has quit IRC23:06
jeblairpleia2: instead, unbound tells resolvconf about itself and so resolvconf writes /etc/resolv.conf with the localhost value23:07
pleia2jeblair: oh, ugh, I misread :(23:07
pleia2yeah23:07
clarkbjeblair: /etc/dhcp/dhclient-enter-hooks.d/resolvconf pushes this information to resolvconf. So resolvconf may do the right thing if unbound wins over dhcp23:08
clarkbwhich honestly I think dhcp should win in a properly designed env23:08
*** maxbit_ has quit IRC23:08
jeblairclarkb: you think dhcp should override a locally configured resolver?23:08
jeblairclarkb: i strongly disagree23:08
clarkbjeblair: yes, because dhcp's job is to configure stuff23:09
jeblairclarkb: dhcp doesn't have the right context to make that decision23:09
*** maxbit has quit IRC23:09
clarkbyou start having rogue devices otherwise23:09
clarkbjeblair: sure it does, you control dhcp server and clients23:09
*** mriedem has joined #openstack-infra23:09
clarkbusually23:09
jeblairclarkb: if a sysadmin installs a local recursive resolver, it _clearly_ should be used23:09
jeblairclarkb: you may not have noticed that we do not control the dhcp server23:09
jeblairclarkb: and the people who do are making really bad decisions about dns23:09
clarkbjeblair: right hence my comment about proper env23:09
clarkbcloud is special23:09
jeblairso, anyway... back to getting this working23:10
jeblairclarkb: when i look at an hpcloud centos node, i see nameserver 10.6.0.1 in resolv.conf23:10
*** mrodden has joined #openstack-infra23:10
clarkbjeblair: I can isntall unbound on my test box and reboot it to see what happens23:10
jeblairclarkb: that does not seem to equal 8.8.8.823:10
clarkbjeblair: right I think the lease time is long enough we will only see trouble at boot23:10
jeblairclarkb: i'm not following23:11
*** nati_ueno has quit IRC23:11
clarkbjeblair: when we boot off of the image you have made with that nameserver, it will ask for a dhcp lease23:12
clarkbif resolvconf lets dhcp win then we have a problem23:12
jeblairclarkb: if you see 8.8.8.8 in the lease and i see 10.6.0.1 in resolv.conf, that suggests to me that resolv.conf is not affected by dhcp at all :/23:12
*** nati_ueno has joined #openstack-infra23:12
clarkbjeblair: but have you rebooted that machine23:12
jeblairclarkb: i'm not talking about an image with any of the new unbound stuff23:12
jeblairclarkb: i'm just talking about what we have now23:12
clarkboh23:12
clarkbhuh23:12
*** melwitt has joined #openstack-infra23:13
clarkbjeblair: my resolv.conf is google23:13
clarkbthis is a node in hpcloud-region-b23:13
jeblairclarkb: oh i used azx23:13
jeblairi can't tell you how excited i am that we have 6 combinations to test.23:14
jeblairclarkb: is dhclient running on your region b node?23:14
clarkbjeblair: yes23:15
* clarkb sets to dyndns and reboots23:15
*** dims_ has joined #openstack-infra23:16
clarkbjeblair: yup dhcp reset my nameserver. Now I am going to try unbound23:17
jeblairokay, so it seems like hpcloud has dhclient running in both clouds, and sends 10.6.0.1 in old and google in new.23:17
jeblairclarkb: the answer is probably different with centos and ubuntu too -- there's a chance resolvconf will do the right thing and leave it on ubuntu; but i'm assuming dhclient writes resolv.conf directly on hpcloud centos.23:18
fungijgriffith: cinder isn't deserving my *my* ire at all! i can't help if some irc bot decides to take out a vendetta ;)23:18
jeblairi <3 cinder23:19
fungijeblair: clarkb: most dhcp clients allow you to configure which options to request, which options to ignore, and which list options you might want to apprend/prepend to23:19
fungii will just about guarantee i can find you a config option to let you override the dhcp-suggested resolver list23:20
jeblairfungi: yeah, it seems like we might need to reconfigure dhclient on hpcloud, and do nothing on raxspace (which uses static config)23:20
* fungi looks to see which client hpcloud is installing23:20
jeblair/sbin/dhclient -1 -q -cf /etc/dhcp/dhclient-eth0.conf -lf /var/lib/dhclient/dhclient-eth0.leases -pf /var/run/dhclient-eth0.pid eth023:20
clarkbfungi: yes I know I was suggesting we do that23:20
jeblairdhclient3 -e IF_METRIC=100 -pf /var/run/dhclient.eth0.pid -lf /var/lib/dhcp/dhclient.eth0.leases -1 eth023:21
fungioh, that's what i get for skimmingh23:21
fungiskimming23:21
jeblaircentos and ubuntu, respectively23:21
clarkbI have unbound resolving on my test box23:21
clarkbrebooting now to see what happens to resolv.conf23:21
*** atiwari has joined #openstack-infra23:21
clarkbjeblair: looks like unbound won on precise hpcloud23:22
jeblairyeah, so i think we're set on precise everywhere thanks to resolvconf.  i never thought i'd say that.23:23
*** yassine has quit IRC23:23
jeblairthen on rax-centos, we just write /etc/resolv.conf, and on hpcloud-centos, we reconfigure dhclient?23:23
clarkbjeblair: should be safe to reconfigure dhclient on all nodes and not think to hard about it23:24
clarkbnot sure if we want to be specific or not if one node type doesn't deal with this properly23:24
*** gyee has joined #openstack-infra23:24
jeblairclarkb: would be good to avoid it since currently puppet has no idea if we're on hpcloud or rax.23:24
fungidhclient-4.1.1-38.P1.el6.centos.x86_64 on centos6, config is in /etc/dhcp/dhclient-eth0.conf with a /etc/dhcp/dhclient.d run-parts dir23:24
*** atiwari has quit IRC23:24
jeblairfungi, clarkb: that same version is also installed on rax23:25
jeblairclarkb: so yeah, seems like the idea of 'just reconfigure dhclient everywhere' should work23:26
jeblairor at least, for all centos23:26
clarkbya23:26
*** wchrisj has quit IRC23:26
fungiisc-dhcp-client on precise with /etc/dhcp/dhclient.conf and dhclient-enter-hooks.d and dhclient-exit-hooks.dm run-parts dirs23:27
fungiwe want to 'prepend domain-name-servers 127.0.0.1;'23:27
*** sweston has joined #openstack-infra23:28
fungiat least that's what i've usually done with dhclient23:28
jeblairfungi: i'd like the other servers not to show up at all, is that possible?23:28
fungiprecise has a commented-out example of that in the conffile23:28
jeblair(rather than prepend)23:28
fungijeblair, pretty sure that's 'supersede domain-name-servers 127.0.0.1;' then23:29
*** _sweston_ has quit IRC23:29
fungigotta love the twenty-five-cent words the dhclient maintainers decided on for their config options23:30
jeblairi'm going broke23:30
*** dcramer__ has quit IRC23:30
* fungi spent all his twenty-five cent words on cheap booze23:31
jeblairfungi, clarkb: do we need to append that to /etc/dhcp/dhclient-eth0.conf ?23:31
*** denis_makogon has quit IRC23:32
fungithat should work fine. anywhere in the global scope, just not inside other config scope blocks like lease{}; or alias{};23:32
*** maxbit has joined #openstack-infra23:32
jeblairsend vendor-class-identifier "anaconda-Linux 2.6.32-431.el6.x86_64 x86_64";23:32
jeblairtimeout 45;23:32
*** mbacchi has quit IRC23:32
jeblairthat's the entirety of the existing config ^23:32
jeblairdifferent numbers on rax v hpcloud...23:32
*** maxbit_ has joined #openstack-infra23:33
jeblairi'm not sure what we should do with puppet in that case?23:33
fungiyeah, the centos one is wonderfully terse. the precise one has a lot more to ir23:33
fungito it23:33
clarkbjeblair: dump a file in the .d dir?23:33
clarkbwith one line overriding nameservers?23:33
fungifor centos there's just the one run-parts dir so that ought to work23:33
jeblairi thought that just ran scripts23:33
jeblairi didn't think it was included config23:33
fungioh, wait. looking at the ntp example there...23:34
jeblairdhclient.d/ntp.sh23:34
fungiyep, script23:34
fungiso we do need to append to the config23:34
clarkbah23:34
clarkb:/23:34
*** wenlock has quit IRC23:34
*** wenlock_ has joined #openstack-infra23:34
*** wenlock_ is now known as wenlock23:34
jeblairso we need to do this? http://projects.puppetlabs.com/projects/puppet/wiki/Simple_Text_Patterns/523:35
*** jhesketh has quit IRC23:36
fungithat's why debubtu properly has the run-parts dirs called foo-hooks.d23:36
*** maurosr has quit IRC23:37
*** mtreinish has quit IRC23:37
*** mtreinish has joined #openstack-infra23:37
*** maurosr has joined #openstack-infra23:37
fungijeblair: i had never seen that facility of puppet before23:38
fungithat looks way better than some of the places we've just slurped in entire conffiles23:38
jeblairfungi: i don't think it is a facility23:38
jeblairfungi: i think it's way better to slurp in conffiles; having config management edit files is almost always pain23:38
fungis/facility/insane hack/23:39
*** david_lyle_ has joined #openstack-infra23:39
fungii've probably just never gotten to witness some of the myriad of ways that can go horribly wrong23:39
*** VijayTripathi has joined #openstack-infra23:40
clarkbthere is parsed file23:40
clarkbwhich is clunky23:41
*** david-lyle has quit IRC23:41
fungii wonder whether completely normalizing the dhclient config between platforms/providers would present new problems23:41
jeblairthis is another "multi cloud is hard" moment.23:42
*** sweston has quit IRC23:42
*** dcramer__ has joined #openstack-infra23:43
clarkbdoes centos not do resolvconf?23:45
fungii thought resolvconf was an ubuntuism which got up-ported into debian, though i could be wrong23:46
*** CaptTofu has joined #openstack-infra23:46
jeblairyum search resolvconf comes up empty23:46
fungistarting as basically a hackish dhcp client hook script which made in-place edits to your resolv.conf file23:46
clarkbgotcha23:47
fungione of those innovations to ease linux adoption in the average joe desktop market23:47
*** SumitNaiksatam has joined #openstack-infra23:48
*** wchrisj has joined #openstack-infra23:48
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Use unbound  https://review.openstack.org/8009223:49
jeblairfungi: turns out it inadvertently eased it in the server market as well..23:49
jeblairokay, so that change should do the dhclient thing (i think)...23:49
jeblairone more thing while we're brainstorming though...23:50
fungiindeed23:50
jeblairclarkb and mattoliverau: both suggested having puppet configure the forward ips23:50
*** CaptTofu has quit IRC23:50
*** yamahata has joined #openstack-infra23:50
jeblairi chose not to do that because i figured reading /etc/resolv.conf would give us the ips on all os/provider combinations...23:50
jeblairi think that's still the case, yeah?23:50
clarkbjeblair: yes, that should still be the case23:51
openstackgerritArnaud Legendre proposed a change to openstack/requirements: Update oslo.vmware to be >= 0.3  https://review.openstack.org/8012023:51
jeblairgiven what we know, would it make sense to do it in puppet?23:51
mattoliverausorry, been in a meeting.. reading scroll back23:51
*** jhesketh_ has joined #openstack-infra23:51
jeblairi mean, i think the fact would have to read the dhcp lease on hpcloud and the /etc/network/interfaces or /etc/system/whatever files on rax23:51
jeblairright?23:51
jeblairso that's at least 3 different files (with different syntaxes) that we would need to parse23:52
clarkbjeblair: probably. I think we can start with what you have there. Mostly my concern is/was that if you run puppet on a machine without the nodepool script you will get a half configured node23:52
clarkbjeblair: for things related to image building thats fine (boot parameters and so on) but the unbound service is configured half in pupet and half in nodepool23:53
jeblairclarkb: actually we will run the puppet on all machines23:53
*** Alexandra is now known as alex-afk23:53
jeblairclarkb: that's going to run on servers too, and configure unbound, and the nodepool bit is not going to run23:53
jeblairclarkb: which is fine, because that will give us unbound in the default recursive resolver config, which is what we want on servers23:54
*** resker has quit IRC23:54
clarkboh right, nodepool is just doing forwarding config for us23:54
*** gokrokve_ has quit IRC23:54
jeblairclarkb: so it's sort of like every host (nodepoor or not) gets unbound, then nodepool hosts drop in a forwarding config23:54
clarkbwfm23:54
fungithat feels safe enough23:55
*** esker has joined #openstack-infra23:55
jeblair(nodepoor is what we are right now)23:55
*** gokrokve has joined #openstack-infra23:55
fungibut soon, noderich!23:55
fungiprofit!23:55
*** SumitNaiksatam has quit IRC23:55
*** SumitNaiksatam has joined #openstack-infra23:56
*** gokrokve has quit IRC23:59
jogoclarkb: 1291440 isn't showing up in the  e-r even though the pages have regenerated23:59
*** esker has quit IRC23:59

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