| m_3 | I think I'm just gonna cat the unit logs on to the end of the build log upon failure | 00:01 |
|---|---|---|
| m_3 | that way it's readable from the web | 00:01 |
| SpamapS | m_3: there's a workspace dir that you can stick them in too | 00:10 |
| SpamapS | m_3: would be good to dump the whole charm in there too | 00:10 |
| m_3 | yeah, I was just dumping the log, but I guess there's no harm in more | 00:10 |
| m_3 | SpamapS: what do you think jenkins would do if jobs/ was a remote mount of some sort? | 00:11 |
| m_3 | it really just looks like logging, so it shouldn't choke it up too much | 00:12 |
| SpamapS | m_3: I think it copies the work dir into a build-id specific place after the run | 00:12 |
| m_3 | right, but the work dir isn't really doing anyting atm | 00:13 |
| SpamapS | m_3: I'd RTFM .. but I'm too busy reviewing nijaba's drupal charm :) | 00:13 |
| m_3 | just thinking of multiple units... charmtester-P charmtester-Q charmtester-billyjoe-jimbob | 00:14 |
| m_3 | and like mounts rather than rsyncs for persistence | 00:14 |
| m_3 | but I'll let you get back to charm reviews :) | 00:15 |
| * m_3 thinks thin... maybe nobody will notice how far behind the review queue is | 00:15 | |
| SpamapS | m_3: we need to poke the Incompletes for status | 00:17 |
| m_3 | roger | 00:19 |
| SpamapS | man, I'm sorry, but the new look for bug listings *sucks* | 00:30 |
| SpamapS | pastels? really? | 00:30 |
| * SpamapS sends apologies to those responsible for the design | 00:31 | |
| m_3 | I'm confused... it's legal yaml to leave quotes off of strings | 00:33 |
| m_3 | why are we barfing them if they're not? | 00:33 |
| SpamapS | m_3: where are we barfing on them? | 00:33 |
| SpamapS | btw.. I wish I could say I haven't asked that on a drunken night of merriment | 00:34 |
| m_3 | the charm-store... gustavo sent out an "Updated list of charm problems" | 00:35 |
| m_3 | when you declare a config param of type: string | 00:35 |
| m_3 | the default value can't be just 0 | 00:35 |
| m_3 | it's gotta be "0" | 00:35 |
| m_3 | but that's not strictly required of yaml | 00:35 |
| m_3 | I think we're trying to add stronger typing on top of yaml | 00:43 |
| m_3 | but that's not what users will expect at all... it's called '.yaml' and they'll expect yaml | 00:44 |
| m_3 | maybe I misunderstand and it's not just quoting | 00:50 |
| m_3 | crap, yeah... it's forcing int and bool... that sucks | 00:54 |
| m_3 | it's much easier in shell to use strings for everything | 00:55 |
| m_3 | ok, nevermind... I'm being dumb | 00:56 |
| SpamapS | m_3: I agree with Gustavo on this one. If its "string", put a yaml *string* there. 0 is an int.. and the way things are converted is hard to predict. | 01:10 |
| SpamapS | m_3: I doubt yaml defines the algorithm that one must use to convert between types. :) | 01:11 |
| SpamapS | m_3: btw, you can also do reviews on charm-tools. :) | 01:12 |
| === flaviamissi_ is now known as flaviamissi | ||
| === ejat- is now known as ejat | ||
| === Furao_ is now known as Furao | ||
| === fenris is now known as Guest60220 | ||
| === fenris_ is now known as Guest303 | ||
| === Guest303 is now known as ejat | ||
| === ejat is now known as fenris | ||
| === fenris is now known as Guest33989 | ||
| === Guest33989 is now known as ejat | ||
| === francisco is now known as fsouza | ||
| === flaviamissi_ is now known as flaviamissi | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!