rick_h | timClicks: ping, I'm trying to load the omnivector redis charm in the store but it's failing for me. Can you confirm? | 04:37 |
---|---|---|
rick_h | https://jaas.ai/u/omnivector/redis just hangs but I can load https://jaas.ai/u/omnivector/redis-singleton/bundle/1 just fine | 04:38 |
timClicks | rick_h: looking | 04:43 |
timClicks | rick_h: not working for me either | 04:44 |
rick_h | :( | 04:44 |
rick_h | timClicks: might be worth a check if your redis demo works atm. I was going to use it as a demo for a lightning talk but moving forward with something else atm | 04:44 |
timClicks | i wonder if he meant to upgrade and forgot to push to stable | 04:45 |
timClicks | am receiving a 502 Bad Gateway, which implies that there is an error with the charm store code | 04:45 |
rick_h | timClicks: yea, not sure what's up. Seems odd others work | 04:51 |
rick_h | bdx: did you break something :P | 04:51 |
manadart | achilleasa: https://github.com/juju/juju/pull/10899 | 10:32 |
achilleasa | manadart: should I bump the machiner API for my proposed change? If not I suspect it could break if you upgraded the models but not the controller | 11:18 |
manadart | achilleasa: I think we should, yes. | 11:19 |
nammn_de | manadart stickupkid: in for a second review on this integration test fix for "vsphere" https://github.com/juju/juju/pull/10897/files ? | 11:43 |
manadart | nammn_de: Will look in a sec. | 11:44 |
nammn_de | manadart i already got some feedback from stickupkid and rick_h which I added. I incorporated ricks additional feedback from the comment. Could you give me a sanity check? Espeically because I renamed the featureflag to be branch instead of generations from the feebdack | 12:03 |
manadart | nammn_de: Just approved it., | 12:04 |
nammn_de | manadart: thanks! Sry i meant this one as well https://github.com/juju/juju/pull/10867 little bigger, but stickupkid and rick_h already gave there a review and almost +1. Might need to have a sanity check for the feature flag change | 12:05 |
nammn_de | *little* | 12:05 |
nammn_de | *its big nvm me | 12:05 |
achilleasa | manadart: so... that machinewatcher thingie in the instancepoller? this is why it does not react to document changes: https://github.com/juju/juju/blob/develop/state/watcher.go#L501 | 12:16 |
achilleasa | manadart: quick HO? | 12:18 |
manadart | achilleasa: Just firing down lunch. Gimme a few. | 12:41 |
nammn_de | manadart: your comment here: https://github.com/juju/juju/pull/10867/files/ff32067866004a0a04d5dba8e281dea7857f33ea#r345173545 with renaming you mean reuse the existing one? Because we already have them | 12:43 |
manadart | nammn_de: You could actually use a single type with fields common to all of those transports. At the client we transform them into specific types for display anyway. | 13:02 |
manadart | achilleasa: I'm in Daily. | 13:03 |
nammn_de | manadart: resuse the results? I can do that, I was "afraid" to add new fields as omnitempty | 13:03 |
nammn_de | to existing ones | 13:03 |
nammn_de | manadart: added your changes. I replaced my own "commit" types by the "generation" types and just added fields if thats you meant https://github.com/juju/juju/pull/10867 | 13:13 |
stickupkid | who knows about description and optional fields? | 15:22 |
stickupkid | achilleasa, whilst i've got your attention :D https://github.com/juju/description/pull/65 | 17:26 |
achilleasa | stickupkid: looking | 17:26 |
stickupkid | achilleasa, ty | 17:36 |
babbageclunk | hmm, having vpn issues | 21:07 |
babbageclunk | anastasiamac, hml: vpn working for you? | 21:07 |
hml | babbageclunk: attempting now | 21:08 |
hml | babbageclunk: yes, i’m using the US based | 21:09 |
babbageclunk | hml: thanks - seems all better now | 21:09 |
anastasiamac | babbageclunk: m usually not strating a day with vpn ... when i don't have enough caffeine in me, i do nothave patience for it... but it looks like it's sorted for u ;) | 21:16 |
babbageclunk | anastasiamac: I basically always connect to it - otherwise if I need to later I have to drop on irc and reconnect | 21:17 |
babbageclunk | + to get to vsphere I need to be on the vpn | 21:18 |
anastasiamac | babbageclunk: u've foud silver lining for vsphere - so beatiful :D | 21:18 |
babbageclunk | uh, it's like a grey lining | 21:22 |
anastasiamac | grey with sprinkles or sequin will still make it shiny | 21:40 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!