mup | PR operator#324 opened: model: don't use --app for relation-get/-set before 2.7.1 <Created by chipaca> <https://github.com/canonical/operator/pull/324> | 02:24 |
---|---|---|
=== stub` is now known as stub | ||
davigar15 | Any updates on this? https://bugs.launchpad.net/juju/+bug/1882127 I have asked in the #juju channel too. I'm not sure which team should look into it. | 06:12 |
Chipaca | moin moin | 08:44 |
jam | morning Chipaca | 08:44 |
Chipaca | jam: had a bit of insomnia yesterday | 08:45 |
jam | hi davigar15, I don't have any updates, I'll raise it in standup today with the Juju team to hopefully increase visibility. At a minimum there is a Juju bug if it *ever* panics. | 08:46 |
Chipaca | jam: so there's a pr up | 08:46 |
jam | Chipaca, yeah, I saw about --app. I was trying to set up different Juju versions to test it for you. | 08:46 |
Chipaca | jam: (and email with context on snapcrafters) | 08:46 |
Chipaca | jam: <3 | 08:46 |
Chipaca | incidentally we have no tests with is-app true | 08:46 |
jam | Chipaca, well, you can fix that one :) | 08:47 |
Chipaca | should i fix it in the same pr i'm touching the code tho | 08:47 |
jam | Chipaca, do you know how far back we care about? eg, do we care about 2.6? | 08:47 |
jam | Chipaca, adding tests when touching the code sounds like a good thing, no? | 08:47 |
Chipaca | jam: I wouldn't care about 2.6 unless somebody asks for it | 08:47 |
Chipaca | jam: but i would like to know what would break in 2.6 :-) | 08:48 |
Chipaca | at some point | 08:48 |
Chipaca | wrt tests, yeah, on it | 08:48 |
jam | Chipaca, k. I can at least spin one up and be somewhat aware. | 08:48 |
Chipaca | brain not fully engaged | 08:48 |
jam | Chipaca, we should (eventually) look to have a "test the features" charm that we can run against various Juju versions. Maybe we can polish 'test-main' to grow into that. | 08:50 |
Chipaca | d'oh there's a bug in my pr | 08:51 |
* Chipaca fixes | 08:51 | |
* Chipaca writes tests first | 08:51 | |
Chipaca | today is going to be a long day | 09:11 |
Chipaca | spent 10 minutes trying to figure out why 1.8.0 was saying it didn't have app data | 09:12 |
jam | Chipaca, 1.8 ? ah, in testing | 09:44 |
jam | I don't think Juju ever had a 1.8 (due to odd numbering with pyjuju vs go juju, etc.) | 09:44 |
Chipaca | jam: as opposed to 2.8 | 09:44 |
Chipaca | ah, yeh | 09:45 |
Chipaca | that | 09:45 |
Chipaca | i think i'm going to take a break, go outside, see if the cobwebs go away | 09:46 |
Chipaca | can't be this /lelo/ for the hangouts | 09:47 |
jam | Chipaca, feeling any better? | 10:50 |
jam | i'm trying to figure out what the actual issue is that they're running into, because --app was introduced in 2.7.0 | 10:50 |
mup | Issue operator#223 closed: Operator fails on Juju 2.7.0 and below <bug> <Created by chris-sanders> <Closed by jameinel> <https://github.com/canonical/operator/issues/223> | 10:56 |
mup | Issue operator#223 opened: Operator fails on Juju 2.7.0 and below <bug> <Created by chris-sanders> <https://github.com/canonical/operator/issues/223> | 10:57 |
Chipaca | jam: omw | 11:00 |
facubatista | Muy buenos días a todos! | 11:37 |
Chipaca | facubatista: [citation needed] | 11:38 |
Chipaca | facubatista: :) hola | 11:38 |
facubatista | Chipaca, hola! sorry I skipped the first meeting, had a rough night | 11:39 |
Chipaca | facubatista: same here (but 4 time zones) | 11:57 |
Chipaca | jam: ping | 12:03 |
jam | Chipaca, pong | 12:03 |
jam | ah, sorry, brt | 12:04 |
Chipaca | and now i'm going to have lunch | 12:13 |
Chipaca | i blame james for that | 12:13 |
jam | davigar15, can you confirm which charm it is where you run into https://bugs.launchpad.net/juju/+bug/1882127 ? | 12:56 |
davigar15 | Zookeeper one | 12:57 |
jam | Chipaca, reading operator#223 I'm pretty sure it is just "below 2.7.0" where we fail | 12:58 |
mup | Issue #223: Operator fails on Juju 2.7.0 and below <bug> <Created by chris-sanders> <https://github.com/canonical/operator/issues/223> | 12:58 |
davigar15 | When relating to Kafka | 12:58 |
Chipaca | jam: but note in email '~2.7.4' is mentioned | 13:29 |
jam | Chipaca, so if it is 2.7.4 then it is about bugs in reading application data in K8s charms, not about --app not being available as an arg | 13:31 |
jam | but I have the feeling they saw something that kind-of sounded like what they were seeing and decided it was the same | 13:31 |
jam | Chipaca, sorry, having a serious discussion in daily, will be to standup soon | 13:31 |
Chipaca | jam: yay | 13:31 |
Chipaca | need to step out, bbiab | 15:23 |
Chipaca | facubatista: i'll be 5 minutes late | 16:31 |
facubatista | Chipaca, no worries | 16:31 |
facubatista | (I'm there, just step in) | 16:31 |
Chipaca | OK | 17:03 |
Chipaca | I am EOD, peeps | 17:03 |
Chipaca | i didn't think i'd make it this far after barely sleeping, but hey :-) | 17:03 |
* facubatista eods and tomorrow is off | 20:58 | |
facubatista | see you all on Thursday | 20:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!