| smoser | Odd_Bloke, i think i said vivid to you | 14:21 |
|---|---|---|
| smoser | but i meant wily. wrt webhooks reporter | 14:21 |
| Odd_Bloke | Oh, OK. | 14:21 |
| Odd_Bloke | Cool. :) | 14:21 |
| Odd_Bloke | smoser: You're going to confuse zuul again by doing workflow +1 without two CR +2s. :p | 14:57 |
| smoser | ? | 14:59 |
| smoser | i did +2 | 14:59 |
| smoser | i guess i didnt understand what i did wrong. | 15:00 |
| smoser | clearly. | 15:00 |
| smoser | explain ? | 15:00 |
| Odd_Bloke | smoser: Will do in a mo; in a meeting ATM. | 15:03 |
| Odd_Bloke | Oh, confusing, I would have expected it to do nothing because there was only one +2. | 15:04 |
| Odd_Bloke | smoser: My understanding is that zuul only examines a change for gate+merge when a workflow +1 happens. | 15:05 |
| Odd_Bloke | When that happens, it checks if there are two code review +2s. | 15:05 |
| Odd_Bloke | If there are, it queues for gate. | 15:05 |
| Odd_Bloke | If not, it does nothing. | 15:05 |
| smoser | why would it want *2* code review +2 ? | 15:05 |
| Odd_Bloke | I thought that was what was required. | 15:05 |
| smoser | hm.. | 15:06 |
| smoser | harlowja_, why does cgerrit just show "Connecting" for me ? | 15:08 |
| Odd_Bloke | smoser: Looking at project-config, I see no requirements for code review at all. | 15:14 |
| Odd_Bloke | So I'm confused in one of two ways. :p | 15:14 |
| Odd_Bloke | harlowja_: Can you explain how stuff gets in to gate? | 15:14 |
| openstackgerrit | Merged stackforge/cloud-init: Basic implementation of a reporting framework. https://review.openstack.org/191144 | 15:22 |
| Odd_Bloke | \o/ | 15:24 |
| smoser | harlowja_, why does cgerrit just show "Connecting" for me ? | 16:13 |
| Odd_Bloke | harlowja_: WAKE UP | 16:14 |
| harlowja_ | whatttttttttt | 17:11 |
| harlowja_ | smoser hmmm, cgerrit showing connecting, hmmm, might not work anymore, lol, or a bug or something :-P | 17:12 |
| harlowja_ | smoser ssh keys setup and stuff? | 17:13 |
| harlowja_ | cgerrit connects into the gerrit ssh event stream | 17:13 |
| harlowja_ | smoser run -v (which should enable more logging that may show something) | 17:16 |
| harlowja_ | https://github.com/harlowja/gerrit_view/blob/master/scripts/cgerrit#L800 should get configured then | 17:17 |
| harlowja_ | -v output.log (since the screen isn't useful) | 17:17 |
| smoser | harlowja_, can you give me an example that works for you? | 17:19 |
| harlowja_ | let me get it running again, sure | 17:20 |
| harlowja_ | see if it works :-P | 17:20 |
| * harlowja_ hasn't ran cgerrit in a while | 17:20 | |
| smoser | its petting cpu right now. | 17:20 |
| smoser | ah. it came back. now waiting for events. | 17:20 |
| harlowja_ | ya, so that means whenever a review or review comment comes in via the ssh event stream | 17:20 |
| smoser | so it just sits there and polls ? | 17:21 |
| harlowja_ | negative | 17:22 |
| harlowja_ | gerrit pushes | 17:22 |
| harlowja_ | *pushes events | 17:22 |
| harlowja_ | so its push based | 17:22 |
| harlowja_ | *vs the web ui, which is poll based | 17:22 |
| harlowja_ | hmmm, i got 'socket.error: [Errno 113] No route to host' let me see, ha | 17:31 |
| harlowja_ | ok finally got it to work, ha | 18:08 |
| harlowja_ | smoser "cgerrit -v output.log -i 500 -p 100 -k "/home/harlowja/.ssh/id_rsa" -u harlowja -s "review.openstack.org" -p "29418" " | 18:08 |
| harlowja_ | that worked for me | 18:08 |
| harlowja_ | so guess it still works lol | 18:09 |
| smoser | it pegs cpu ? | 18:09 |
| smoser | its eating 100% of one of my cpu | 18:09 |
| harlowja_ | not for me | 18:09 |
| harlowja_ | cpu 1.0% | 18:09 |
| harlowja_ | make sure the key works | 18:12 |
| harlowja_ | by trying 'ssh review.openstack.org -p 29418' | 18:12 |
| harlowja_ | that should emit something like http://paste.openstack.org/show/355953/ | 18:12 |
| harlowja_ | i've got reviews up to make https://github.com/openstack-infra/gerritlib/ better, but they just sit there for months | 18:13 |
| harlowja_ | ^ which is the lib thats used to talk to gerrit | 18:13 |
| harlowja_ | cgerrit imho not super-useful, just neat to 'see' (c) stuff, but u can't review stuff on it (gertty was made for this, after cgerrit was made) | 18:17 |
| harlowja_ | it was a fun mini-project no-matter (and inspired the guy who made gertty to make that program, lol) | 18:18 |
| === rangerpb is now known as rangerpbzzzz | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!