| airship-irc-bot1 | <xing.zhou> @james.gu Thank you for your quick reply. That's what I want. Prior to this, I have also guessed the meaning of the treasuremap v2 from airshipctl/manifests source code. I think the airshipctl may make the project convert to a container cloud. So it can not deploy openstack infra from stratch any more. | 02:54 |
|---|---|---|
| *** uzumaki has joined #airshipit | 03:55 | |
| *** uzumaki has quit IRC | 08:27 | |
| *** rezroo has joined #airshipit | 08:55 | |
| *** rezroo has quit IRC | 09:25 | |
| *** uzumaki has joined #airshipit | 11:21 | |
| *** jhesketh has quit IRC | 14:02 | |
| *** jhesketh has joined #airshipit | 14:04 | |
| airship-irc-bot1 | <sirishagopigiri> Hi Team, I would like to request some core reviews on this PS https://review.opendev.org/c/airship/hostconfig-operator/+/779176 Thank you in advance! | 14:08 |
| airship-irc-bot1 | <hr858f> Team, looking for one more +2: https://review.opendev.org/c/airship/airshipctl/+/780394. Thanks. | 17:27 |
| airship-irc-bot1 | <james.gu> Hello team, could I have some quick core views on this PS: https://review.opendev.org/c/airship/treasuremap/+/781903. We need it for stl3 baremetal reference site testing using the tools/deployment script. | 17:42 |
| airship-irc-bot1 | <james.gu> A sibling PS in airshipctl for the same purpose: https://review.opendev.org/c/airship/airshipctl/+/781909 | 19:10 |
| *** Anticime1 is now known as Anticimex | 19:29 | |
| *** uzumaki has quit IRC | 19:53 | |
| *** thiagop has joined #airshipit | 21:06 | |
| thiagop | G'day folks! A couple of weeks ago I asked about bugfixing for Armada due to a nasty bug that we got on StarlingX with evicted containers. Only now I got to work on the tests to ensure that my fix is covered, so here it is: https://review.opendev.org/c/airship/armada/+/777011 | 21:08 |
| *** thiagop is now known as outbrito | 21:08 | |
| outbrito | Let me know if that's the correct approach | 21:08 |
| *** rezroo has joined #airshipit | 21:33 | |
| airship-irc-bot1 | <mattmceuen> Would appreciate reviews on Manoj's change to add VRRP HA to treasuremap ingress: https://review.opendev.org/c/airship/treasuremap/+/778953 | 21:54 |
| airship-irc-bot1 | <mattmceuen> Ratnopam has another thing he'd like to add into the new catalogue defined there, so it would be helpful to get it merged (and it looks good to me) | 21:55 |
| airship-irc-bot1 | <pb269f> @matt - what was the rationale here for doing this 'the hard way' rather than say using metal-lb? | 21:57 |
| *** uzumaki has joined #airshipit | 21:57 | |
| airship-irc-bot1 | <mattmceuen> I think it was just a known quantity. Is it harder than adding metallb? | 21:58 |
| airship-irc-bot1 | <pb269f> i honestly dont know | 22:00 |
| airship-irc-bot1 | <pb269f> this is much 'lower' down the stack | 22:00 |
| airship-irc-bot1 | <mattmceuen> yeah | 22:00 |
| airship-irc-bot1 | <pb269f> so - in yaml terms - probably slightly more loc | 22:00 |
| airship-irc-bot1 | <pb269f> and you loose the 'api' of metallb | 22:00 |
| airship-irc-bot1 | <mattmceuen> metallb would certainly be a cleaner interface - I think we should look into it as a potential alternative | 22:01 |
| airship-irc-bot1 | <pb269f> but not sure if its bteer, or worse | 22:01 |
| airship-irc-bot1 | <pb269f> *better | 22:01 |
| airship-irc-bot1 | <pb269f> ok | 22:01 |
| airship-irc-bot1 | <mattmceuen> Just added it to the ptg agenda :slightly_smiling_face: | 22:02 |
| airship-irc-bot1 | <mattmceuen> thanks for the idea Pete | 22:02 |
| airship-irc-bot1 | <pb269f> if not already done - may be a good idea to back this up with some docs | 22:02 |
| airship-irc-bot1 | <pb269f> ++ | 22:02 |
| airship-irc-bot1 | <pb269f> metal-lb, ipvs, keepalived - all have pros and cons | 22:03 |
| airship-irc-bot1 | <pb269f> so probably not clear cut, but would be good to have our bases covered with a 'why we did this' even if its just a couple of sentences | 22:03 |
| airship-irc-bot1 | <mattmceuen> Yeah, agree | 22:03 |
| airship-irc-bot1 | <pb269f> and we should proably include haproxy | 22:04 |
| airship-irc-bot1 | <pb269f> just so we can have a line that says 'no' next to it ;) | 22:04 |
| airship-irc-bot1 | <mattmceuen> haha | 22:05 |
| airship-irc-bot1 | Action: pb269f should note this is personal irrationaility clouding his judgement - all are equal, untill they are not ;D | 22:05 |
| airship-irc-bot1 | Action: mattmceuen technology that make Pete mad bad by definition | 22:05 |
| airship-irc-bot1 | <pb269f> all technology? | 22:06 |
| airship-irc-bot1 | Action: mattmceuen ergo all technology bad? | 22:06 |
| airship-irc-bot1 | <mattmceuen> lol | 22:06 |
| airship-irc-bot1 | <pb269f> right - im off to make fire with sticks, back in a bit. | 22:06 |
| airship-irc-bot1 | <mattmceuen> thiagop/outbrito: thanks for sharing the Armada fix! LGTM -- @sean.eagan could you take a peek when you get a moment? https://review.opendev.org/c/airship/armada/+/777011 | 22:16 |
| airship-irc-bot1 | <sidney.shiba> Hello team, | 22:30 |
| airship-irc-bot1 | <sidney.shiba> Hello team, does anybody know if the latest `airshipit/images` merge was removed? The reason I ask is because when I clone it, the changes I made for `helm-chart-operator` (which included `dex-aio` charts) is not part of master anymore. | 22:34 |
| airship-irc-bot1 | <pb269f> dont see any reverts here: https://review.opendev.org/q/project:airship%252Fimages | 22:36 |
| airship-irc-bot1 | <pb269f> are you cloning from github, or opendev? | 22:36 |
| airship-irc-bot1 | <pb269f> i think github cloning is still broken? | 22:36 |
| airship-irc-bot1 | <pb269f> s/cloning/replication | 22:36 |
| airship-irc-bot1 | <pb269f> oh | 22:38 |
| airship-irc-bot1 | <pb269f> im sorry: https://opendev.org/airship/images/commit/8a3f4265817d307a244938a5782753b378a44a2a | 22:38 |
| airship-irc-bot1 | <pb269f> looks like it was reverted | 22:38 |
| airship-irc-bot1 | <pb269f> https://review.opendev.org/c/airship/images/+/781581 | 22:38 |
| airship-irc-bot1 | <pb269f> looks like some work to build behind a corp proxy would get it back in | 22:38 |
| airship-irc-bot1 | <sidney.shiba> when I clone using git clone "ssh://esidshi@review.opendev.org:29418/airship/images" && scp -p -P 29418 esidshi@review.opendev.org:hooks/commit-msg "images/.git/hooks/" and my latest merge | 22:39 |
| airship-irc-bot1 | <sidney.shiba> What needs to be done to get it back and who can do it? | 22:40 |
| airship-irc-bot1 | <pb269f> i think you would be best positioned to do so | 22:40 |
| airship-irc-bot1 | <pb269f> if you have a look at the other images in there | 22:40 |
| airship-irc-bot1 | <pb269f> if you need help @craig.anderson should be able to assist | 22:41 |
| airship-irc-bot1 | <sidney.shiba> @pb269f I saw the message. Will contact him. Thanks. | 22:44 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!