[04:37] timClicks: ping, I'm trying to load the omnivector redis charm in the store but it's failing for me. Can you confirm? [04:38] https://jaas.ai/u/omnivector/redis just hangs but I can load https://jaas.ai/u/omnivector/redis-singleton/bundle/1 just fine [04:43] rick_h: looking [04:44] rick_h: not working for me either [04:44] :( [04:44] 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:45] i wonder if he meant to upgrade and forgot to push to stable [04:45] am receiving a 502 Bad Gateway, which implies that there is an error with the charm store code [04:51] timClicks: yea, not sure what's up. Seems odd others work [04:51] bdx: did you break something :P [10:32] achilleasa: https://github.com/juju/juju/pull/10899 [11:18] 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:19] achilleasa: I think we should, yes. [11:43] manadart stickupkid: in for a second review on this integration test fix for "vsphere" https://github.com/juju/juju/pull/10897/files ? [11:44] nammn_de: Will look in a sec. [12:03] 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:04] nammn_de: Just approved it., [12:05] 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] *little* [12:05] *its big nvm me [12:16] 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:18] manadart: quick HO? [12:41] achilleasa: Just firing down lunch. Gimme a few. [12:43] 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 [13:02] 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:03] achilleasa: I'm in Daily. [13:03] manadart: resuse the results? I can do that, I was "afraid" to add new fields as omnitempty [13:03] to existing ones [13:13] 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 [15:22] who knows about description and optional fields? [17:26] achilleasa, whilst i've got your attention :D https://github.com/juju/description/pull/65 [17:26] stickupkid: looking [17:36] achilleasa, ty [21:07] hmm, having vpn issues [21:07] anastasiamac, hml: vpn working for you? [21:08] babbageclunk: attempting now [21:09] babbageclunk: yes, i’m using the US based [21:09] hml: thanks - seems all better now [21:16] 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:17] anastasiamac: I basically always connect to it - otherwise if I need to later I have to drop on irc and reconnect [21:18] + to get to vsphere I need to be on the vpn [21:18] babbageclunk: u've foud silver lining for vsphere - so beatiful :D [21:22] uh, it's like a grey lining [21:40] grey with sprinkles or sequin will still make it shiny