| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Basic tests for airship-seaworthy deployment https://review.openstack.org/604210 | 00:04 |
|---|---|---|
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Uplift latest charts/images https://review.openstack.org/604941 | 00:06 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Basic tests for airship-seaworthy deployment https://review.openstack.org/604210 | 00:41 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 00:45 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 00:46 |
| *** sgrasley__ has joined #airshipit | 01:13 | |
| *** sgrasley_ has quit IRC | 01:16 | |
| openstackgerrit | Dan Crank proposed openstack/airship-shipyard master: Add relabel_nodes action https://review.openstack.org/599085 | 01:53 |
| *** roman_g has quit IRC | 01:59 | |
| *** felipemonteiro has joined #airshipit | 02:01 | |
| *** felipemonteiro has quit IRC | 02:02 | |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 02:24 |
| *** felipemonteiro has joined #airshipit | 02:28 | |
| *** Bhujay has joined #airshipit | 02:29 | |
| *** Bhujay has quit IRC | 02:30 | |
| *** Bhujay has joined #airshipit | 02:30 | |
| *** felipemonteiro has quit IRC | 02:43 | |
| *** felipemonteiro has joined #airshipit | 03:11 | |
| *** felipemonteiro has quit IRC | 03:30 | |
| *** Bhujay has quit IRC | 03:41 | |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: fix: Pegleg exceptions documentation incorrectly rendering https://review.openstack.org/604644 | 04:28 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: fix: Pegleg exceptions documentation incorrectly rendering https://review.openstack.org/604644 | 04:28 |
| *** Bhujay has joined #airshipit | 04:37 | |
| *** lemko has joined #airshipit | 05:08 | |
| *** cfriesen has quit IRC | 05:17 | |
| *** Bhujay has quit IRC | 06:00 | |
| openstackgerrit | Merged openstack/airship-treasuremap master: New option --out-file, various fixes in logic https://review.openstack.org/603395 | 06:11 |
| *** Bhujay has joined #airshipit | 06:14 | |
| *** Bhujay has quit IRC | 06:15 | |
| *** Bhujay has joined #airshipit | 06:16 | |
| *** Bhujay has quit IRC | 06:32 | |
| *** Bhujay has joined #airshipit | 06:33 | |
| *** roman_g has joined #airshipit | 06:54 | |
| *** TuanVu has joined #airshipit | 07:39 | |
| TuanVu | Hi Team, have a nice day | 07:39 |
| TuanVu | I'm making research on Airship and have a question | 07:40 |
| TuanVu | "What are advantages compared to Triple-O"? | 07:41 |
| TuanVu | As far as I understand, the biggest advantage is that: "all site definitions are completely declarative" | 07:43 |
| TuanVu | is it correct? | 07:43 |
| roman_g | TuanVu: yes, that would be correct | 08:28 |
| roman_g | site definitions, including software configuration, verions, hardware configuration - all declarative | 08:28 |
| TuanVu | roman_g: thanks a lot for your confirmation :) | 08:30 |
| roman_g | check out https://github.com/openstack/airship-treasuremap repository with sample working set of site declarations. | 08:30 |
| roman_g | directories: global, site/airship-seaworthy, and type/foundry/network | 08:31 |
| TuanVu | awesome ! | 08:31 |
| TuanVu | thank you very much | 08:32 |
| TuanVu | besides, are there any other advantages? | 08:32 |
| roman_g | Theoretically, in case you need to modify any part of the configuration, whether that would be config or software version - you commit and merge the change of your site declaration into the Git repository, and your CD system will roll the change out, without your intervention. Practically at the moment it would not work for some heavy changes (e.g. disks partition layout), but works for many other | 08:34 |
| roman_g | things, for example, for software configs and software versions. | 08:34 |
| TuanVu | thank you, roman_g | 08:36 |
| roman_g | Airship (including all components) is made to primarily deploy OpenStack in OpenStack-Helm. OpenStack-Helm is where all OpenStack components (Neutron, Keystone, Heat, Cinder,.. - all of them) are running as (Docker) containers under Kubernetes supervision. | 08:36 |
| TuanVu | thanks for the info, I really appreciate your help :) | 08:37 |
| roman_g | This allows for seamless components upgrade, automatic failower, automatic healing, automatic health-checking of OpenStack components. | 08:37 |
| TuanVu | that's great! | 08:38 |
| roman_g | https://airship-treasuremap.readthedocs.io/en/latest/_images/component_list.png - this is Airship architecture diagram | 08:45 |
| roman_g | The Blue UCP part - Undercloud Platform - this is actually Airship. | 08:45 |
| TuanVu | thanks a lot for your kind help :) | 08:47 |
| roman_g | https://docs.openstack.org/tripleo-docs/latest/_images/physical_view.png https://docs.openstack.org/tripleo-docs/latest/_images/logical_view.png - TripleO architecture diagrams. TripleO deploys UCP as a small OpenStack cluster, consisting of typycally one node. | 08:47 |
| TuanVu | thanks, I'm checking these links | 08:49 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-in-a-bottle master: Minor: remove TODO https://review.openstack.org/605015 | 09:18 |
| *** roman_g has quit IRC | 09:27 | |
| *** roman_g has joined #airshipit | 09:29 | |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-armada master: Minor: drop AT&T from authors https://review.openstack.org/605022 | 09:42 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-shipyard master: Minor: drop AT&T from authors https://review.openstack.org/605023 | 09:42 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-berth master: Fix: various documentation and URL fixes https://review.openstack.org/604425 | 09:45 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-deckhand master: Fix: various documentation and URL fixes https://review.openstack.org/604426 | 09:45 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-divingbell master: Fix: various documentation and URL fixes https://review.openstack.org/604427 | 09:45 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-drydock master: Fix: various documentation and URL fixes https://review.openstack.org/604428 | 09:45 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-pegleg master: Fix: various documentation and URL fixes https://review.openstack.org/604432 | 09:45 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-promenade master: Fix: various documentation and URL fixes https://review.openstack.org/604433 | 09:45 |
| *** sekharvajjula has quit IRC | 10:16 | |
| *** sekharvajjula has joined #airshipit | 10:16 | |
| *** TuanVu has quit IRC | 10:20 | |
| *** Bhujay has quit IRC | 10:31 | |
| *** Bhujay has joined #airshipit | 10:32 | |
| openstackgerrit | Smruti Soumitra Khuntia proposed openstack/airship-drydock master: Update node-labels through Kubernetes Provisioner https://review.openstack.org/593609 | 12:25 |
| *** dims_ has joined #airshipit | 12:26 | |
| *** Bhujay has quit IRC | 12:31 | |
| *** Bhujay has joined #airshipit | 12:32 | |
| *** Bhujay has quit IRC | 12:33 | |
| *** Bhujay has joined #airshipit | 12:33 | |
| openstackgerrit | Dan Crank proposed openstack/airship-shipyard master: Add relabel_nodes action https://review.openstack.org/599085 | 12:35 |
| openstackgerrit | Smruti Soumitra Khuntia proposed openstack/airship-treasuremap master: [WIP] Delivery of default seccomp Profile on each Host on site deployment https://review.openstack.org/602532 | 12:45 |
| *** aaronsheffield has joined #airshipit | 12:48 | |
| openstackgerrit | Merged openstack/airship-pegleg master: fix: Pegleg exceptions documentation incorrectly rendering https://review.openstack.org/604644 | 13:22 |
| *** hemanth_n has joined #airshipit | 13:22 | |
| *** sthussey has joined #airshipit | 13:22 | |
| openstackgerrit | Steve Wilkerson proposed openstack/airship-treasuremap master: OSH-Infra: Update chart configs https://review.openstack.org/603148 | 13:24 |
| openstackgerrit | Merged openstack/airship-treasuremap master: Fix: various documentation and URL fixes https://review.openstack.org/604436 | 13:35 |
| *** munimeha1 has joined #airshipit | 13:37 | |
| openstackgerrit | Merged openstack/airship-armada master: Minor: drop AT&T from authors https://review.openstack.org/605022 | 13:39 |
| openstackgerrit | Merged openstack/airship-shipyard master: Minor: drop AT&T from authors https://review.openstack.org/605023 | 13:56 |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-pegleg master: [WIP] Unify publishing of docs https://review.openstack.org/581706 | 13:58 |
| sthussey | here | 14:00 |
| mattmceuen | I know mark-burnett was a bit under the whether, are you here Mark? | 14:01 |
| mark-burnett | Hey guys, i'm here | 14:01 |
| mark-burnett | #startmeeting airship | 14:02 |
| openstack | Meeting started Tue Sep 25 14:02:05 2018 UTC and is due to finish in 60 minutes. The chair is mark-burnett. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:02 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| hogepodge | Hi | 14:02 |
| *** openstack changes topic to " (Meeting topic: airship)" | 14:02 | |
| openstack | The meeting name has been set to 'airship' | 14:02 |
| mark-burnett | I didn't really prepare an agenda, though: https://etherpad.openstack.org/p/airship-meeting-2018-09-25 | 14:02 |
| mattmceuen | hey mark-burnett hope you're feeling a bit better :) | 14:02 |
| mark-burnett | yeah, a bit better, still a bit of a cold though | 14:02 |
| mark-burnett | :) | 14:02 |
| *** evrardjp has joined #airshipit | 14:04 | |
| mark-burnett | thanks everyone for adding items to the agenda | 14:05 |
| mark-burnett | #topic Meeting TIme | 14:06 |
| *** openstack changes topic to "Meeting TIme (Meeting topic: airship)" | 14:06 | |
| * roman_g o/ and left | 14:06 | |
| mark-burnett | Looks like no change here - we had one or two new additions to the doodle, which look good for the current time. | 14:06 |
| sthussey | Let's close it | 14:07 |
| mark-burnett | Yeah, we can't keep this issue open every week for months | 14:07 |
| sthussey | This thing is just floating and the people asking for it aren't speaking up | 14:07 |
| sthussey | If they really care, they'll ask again | 14:07 |
| mark-burnett | Agree | 14:07 |
| mark-burnett | Ok, let's move on | 14:07 |
| mark-burnett | #topic -2 Reviews | 14:08 |
| *** openstack changes topic to "-2 Reviews (Meeting topic: airship)" | 14:08 | |
| mark-burnett | @portdirect asked that we discuss this again this week | 14:08 |
| mark-burnett | I don't know whether he's here? | 14:08 |
| mark-burnett | well, it seems like we aren't going to have much discussion around this | 14:10 |
| mattmceuen | lol | 14:10 |
| mattmceuen | Channeling my inner pete | 14:10 |
| mark-burnett | two meetings in a row and nothing to say on the topic | 14:10 |
| portdirect | Sorry been held up | 14:11 |
| mattmceuen | "-2 reviews are typically reserved for strong responses when a PS doesn't just need to be changed, it needs to be abandoned" | 14:11 |
| portdirect | Give me 5 if you can | 14:11 |
| mark-burnett | well, we can move on and maybe come back | 14:11 |
| mark-burnett | #topic reducing gerrit bot noise | 14:12 |
| *** openstack changes topic to "reducing gerrit bot noise (Meeting topic: airship)" | 14:12 | |
| mark-burnett | We discussed this briefly last week, but attendance had diminished before we could reasonably vote | 14:12 |
| b-str | So, what would be the options to vote on? | 14:13 |
| mark-burnett | This was originally roman's suggestion, and there didn't seem to be much objection to removing the PS upload notifications | 14:13 |
| mattmceuen | sorry, I missed that one - what are the knobs we can turn down for the bot? | 14:13 |
| mattmceuen | ah I see | 14:13 |
| mark-burnett | I'm not sure whether it's possible to set it to merges only or not | 14:13 |
| evrardjp | Is that a problem? | 14:13 |
| * mark-burnett shrugs | 14:13 | |
| evrardjp | I mean you can configure your client if you don't care. | 14:13 |
| evrardjp | : ) | 14:13 |
| mark-burnett | If everyone configures their client to hide those, then why have them? | 14:14 |
| mattmceuen | FWIW they don't bother me, I find them informative around "what's going on at the moment". But I can easily respect it being distracting / annoying for other folks | 14:14 |
| evrardjp | mark-burnett: not everyone, I like those :) | 14:14 |
| b-str | I don't know if it's possible, but I think a digest summary would be nice, at some interval. | 14:14 |
| evrardjp | mattmceuen: +1 | 14:14 |
| *** roman_g has quit IRC | 14:14 | |
| * portdirect sneaks in through the back door, sits down and hope no one notices him knock over a coffee mug | 14:15 | |
| mark-burnett | I wonder if roman/others would like to try client-side configuration | 14:15 |
| sthussey | I believe the knob is available: https://docs.openstack.org/infra/system-config/irc.html#gerritbot | 14:15 |
| evrardjp | b-str: you can ignore the notifications in your irc client, and ask for a daily digest on gerrit email notifications maybe ? | 14:15 |
| mattmceuen | aha thanks sthussey | 14:15 |
| sthussey | Because of how Gerrit works, I'd be good with turning off PS upload notifications | 14:15 |
| b-str | sure, they don't bother me per se, but I can see where getting 4 notifications for 4 "oops" is a little aggressive reporting. | 14:15 |
| evrardjp | oh it's technically possible, and the feature was added indeed | 14:15 |
| mark-burnett | yeah, hard to separate the "substantial update" vs "4 typo fixes in 1 hour on a single change" | 14:16 |
| sthussey | I think Roman's concern is if someone asks a question and then in 10m it is scrolled off the screen due to PS notifications, it might be hard to get traction with support | 14:16 |
| evrardjp | well yeah, there is no difference between new patches and updates | 14:16 |
| b-str | TBH, put me in the camp of "don't care - not a problem for me" | 14:16 |
| mark-burnett | Yeah, I think that's a valid one, though maybe addressable for him with just ignore | 14:17 |
| sthussey | I personally don't care, but don't find IRC that informative. Gerrit has a web interface, I can easily see what is going on there | 14:17 |
| mark-burnett | That is more digestable, imo too | 14:17 |
| sthussey | I would table it until Roman is here to state his case | 14:17 |
| mark-burnett | I think that's fair | 14:18 |
| mark-burnett | I'll mention the suggestion to him, and he can try it | 14:18 |
| mark-burnett | Should we move back to -2 reviews? | 14:18 |
| b-str | go for it | 14:19 |
| portdirect | sure, this was a comment based on some behavior that we had in the early days - though its fading out now a bit i think | 14:19 |
| mark-burnett | #topic -2 Reviews | 14:19 |
| *** openstack changes topic to "-2 Reviews (Meeting topic: airship)" | 14:19 | |
| portdirect | but its always been my understanding that -2 can essentially be read as "this is a bad idea, we really should never merge this" | 14:19 |
| portdirect | so we should probably be a bit less liberal with them? | 14:20 |
| evrardjp | it is the intent -- but it doesn't prevent a merge | 14:20 |
| evrardjp | -2 should be kept for extremes reason in openstack, so I suppose airship could take this as inspiration | 14:20 |
| aaronsheffield | That seems a bit extreme. Shouldn't folks just read them as 'this will break something.' So is blocks the merge until that is addressed. | 14:21 |
| portdirect | this is my thought | 14:21 |
| sthussey | Gerrit says 'Do not merge'. Seems pretty straightforward. | 14:21 |
| sthussey | People can read into it what they want. | 14:21 |
| portdirect | i'm just offering the perspective of where ive come from in the wider community | 14:21 |
| hogepodge | -1 means something is wrong and you don’t think it should be merged, but needs work | 14:21 |
| hogepodge | -1 workflow will prevent a merge | 14:21 |
| evrardjp | this please have a look at https://docs.openstack.org/project-team-guide/review-the-openstack-way.html | 14:21 |
| portdirect | `-1` would seem pretty approprate for you ned to update or respond to this before merge? | 14:21 |
| portdirect | `The purpose of the -2 vote is to indicate to the submitter that any further time they spend on the change will almost certainly be wasted. ` | 14:22 |
| portdirect | ^ I suppose this is the nub of my argument | 14:22 |
| aaronsheffield | -1 WF doesn't persist between updates to the PS. | 14:22 |
| sthussey | Then Gerrit should be updated to reflect that -2 indicates the PS should be abandoned | 14:23 |
| mattmceuen | That's because you can fix things with PS :) | 14:23 |
| evrardjp | I have certainly used the -2 before for procedural reasons around a release too. | 14:23 |
| b-str | I've been bit by the -1 thing | 14:23 |
| mark-burnett | Yes, sometimes -1's get ignored | 14:23 |
| evrardjp | let's not talk about abandonning, but talking about channeling energy: if needs updates -> -1 | 14:23 |
| mark-burnett | Perhaps that's the root issue? | 14:24 |
| portdirect | evrardjp: +++ | 14:24 |
| evrardjp | I would be happy if airship would follow openstack procedures, as it makes things clearer for contributors | 14:24 |
| sthussey | If a PS will break existing functionality, I think a -2 is warranted | 14:24 |
| b-str | I review after a -1 that didn't address all the -1's things - don't see an issue, but the -1 person sidechannels me about "why not my -1's" | 14:24 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 14:24 |
| evrardjp | sthussey: I do not agree, as this would not pass gating. | 14:24 |
| mark-burnett | in a perfect world that's true | 14:25 |
| sthussey | That may be true when zuul gating can support running larger tests | 14:25 |
| evrardjp | this is further away of the initial conversation :) | 14:25 |
| openstackgerrit | Merged openstack/airship-shipyard master: Update to Airflow 1.10 https://review.openstack.org/603927 | 14:25 |
| mark-burnett | it's not too far i think | 14:25 |
| evrardjp | if you definitely do not want the patch to get in, add -w. | 14:25 |
| portdirect | so I propose one of two things | 14:25 |
| sthussey | Workflow is cleared w/ every PS on a cS | 14:26 |
| portdirect | 1) we adopt the spirit of: https://docs.openstack.org/project-team-guide/review-the-openstack-way.html | 14:26 |
| portdirect | or 2 we write our own | 14:26 |
| evrardjp | sthussey: on purpose: "do not merge in this case, you're breaking x." Then the person fixes its patch to not break x, and review happens again. Why taint the patch? | 14:26 |
| portdirect | otherwise the different style will make it hard for people who have worked on openstack and other projects to come in and participate | 14:26 |
| sthussey | What is tainted? | 14:27 |
| mattmceuen | I would prefer we adopt the openstack way, as we plan to collaborate / integrate with that project quite a bit, and the PTG proved that there is an overlap in user / developer bases | 14:27 |
| sthussey | You remove the -2 and life goes on | 14:27 |
| mark-burnett | I think -2 has been used in order to avoid -1's being ignored | 14:27 |
| evrardjp | portdirect: agreed, one or the other, but I think it is worth documenting | 14:27 |
| mark-burnett | They're easily removed, as sthussey says | 14:27 |
| mark-burnett | But they aren't automatically removed | 14:27 |
| evrardjp | sthussey: I will never review a -2. | 14:27 |
| sthussey | Okay | 14:27 |
| portdirect | neither would i for any project other than here ;) | 14:27 |
| mark-burnett | Does a change need more reviews at that time? | 14:27 |
| evrardjp | well I mean objectively, why would I? | 14:27 |
| mark-burnett | It clearly doesn't | 14:28 |
| mark-burnett | so it's an accurate indicator in this case as well | 14:28 |
| sthussey | I would happily replace -2 with the idea that if a CS gets a -1, it won't be merged until the person giving the -1 provides a +1 or +2 | 14:28 |
| hogepodge | Sometimes explaining why you’re ignoring a -1 in a merge is also polite. It’s the final decision of the cores, but clear communication is better for everyone | 14:29 |
| mattmceuen | If people are ignoring -1s, that's a problem we need to fix | 14:29 |
| evrardjp | agreed with hogepodge | 14:29 |
| portdirect | ++ | 14:29 |
| portdirect | there will be times were a diff of opinion comes in | 14:29 |
| evrardjp | here the idea is: if you want to bring more people to the community be clear at every time. In your contributor's guide (say which practices you have) | 14:29 |
| evrardjp | or in your reviews | 14:29 |
| portdirect | eg: https://review.openstack.org/#/c/604428/5 | 14:29 |
| sthussey | but there have been cases that a -1 review is left and then the author responds to it and the CS is approved and merged before the original reviewer even gets a chance to re-review | 14:30 |
| mattmceuen | We have to weigh: | 14:30 |
| mattmceuen | 1) the chance that things get merged badly if we're not careful | 14:30 |
| mattmceuen | 2) the fact that everyone with experience with -2's interprets them harshly | 14:30 |
| mattmceuen | If we define a non-conventional meaning for -2, we'll be alienating potential contributors | 14:31 |
| evrardjp | 1) git revert | 14:31 |
| mattmceuen | Let's work to be careful with merges | 14:31 |
| mark-burnett | isn't a revert also impolite? | 14:31 |
| mark-burnett | that's the issue here, politeness | 14:31 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Remove Pegleg stub logic from CLI and engine https://review.openstack.org/605091 | 14:31 |
| mark-burnett | or hurt feelings, or however we want to say it | 14:31 |
| sthussey | I'll happily follow whatever is decided, but I come from a perspective of trying to get things done. Feelings don't enter into the equation. | 14:32 |
| evrardjp | 2) as long as there is a policy it would be clearer -- currently there is only an implicit policy which is not the clearest one if I understand the conversation | 14:32 |
| sthussey | Feel free to give my PS a -5 if that is what is warranted | 14:32 |
| aaronsheffield | Zuul can't run full Airship gates, so we need something else to block patch sets, -2 is the only tool available for that at this time. | 14:32 |
| evrardjp | I don't agree | 14:32 |
| portdirect | no - a choesive core team is | 14:32 |
| evrardjp | portdirect: +1 | 14:32 |
| portdirect | *cohesive | 14:32 |
| evrardjp | it's the +w that decides a merge | 14:32 |
| *** lamt has joined #airshipit | 14:33 | |
| evrardjp | if the -2 -1 discussion is talking about merging, I think we are doing it wrong | 14:33 |
| portdirect | Ive seen several cases where there have been 2*+2 and a -1 from a core, often that core who -1 is also the same person to +wf | 14:33 |
| mattmceuen | I don't want to lose contributors for the sake of keeping code out. It's a tradeoff and it's worth the slight risk. | 14:33 |
| portdirect | as they accept that even though they dont like it, others do | 14:34 |
| evrardjp | -2 -1 is about code quality -> -1 update it, -2 go back to drawing board? | 14:34 |
| evrardjp | at least in openstack ters | 14:34 |
| evrardjp | terms* | 14:34 |
| jamesgu__ | according to openstack guideline, -2 should be accompanied by a comment explaining the reason that the change does not fit with the project goals, so that the submitter can understand the reasons and refocus their future contributions more productively. | 14:34 |
| hogepodge | It can be confusing because some rules are cultural (two +2s) and some enforces by the gate (only +1 w needed to merge) | 14:34 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Remove Pegleg stub logic from CLI and engine https://review.openstack.org/605091 | 14:34 |
| sthussey | So it seems like the action item here is likely 'Write the guidelines' | 14:35 |
| evrardjp | I agreed on both assertings of jamesgu__ and hogepodge (again) | 14:35 |
| sthussey | It won't be decided in IRC | 14:35 |
| evrardjp | sthussey: it will be discussed in a review , guess what will happen? :p | 14:35 |
| evrardjp | hahaha :) | 14:35 |
| evrardjp | but yeah maybe worth moving to another topic... | 14:35 |
| sthussey | As stated, it needs to be documented anyway. I believe there is an ongoing effort for governance documentation | 14:36 |
| sthussey | This can just follow under it - Code of Conduct type of thing | 14:36 |
| sthussey | I really doubt it is that big of a thing | 14:37 |
| b-str | I do think that if -1's persissted, it would be clearer. | 14:37 |
| sthussey | In 1000+ reviews of Airship PS, less than 10 are -2 according to stackalytics | 14:37 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Remove Pegleg stub logic from CLI and engine https://review.openstack.org/605091 | 14:37 |
| mattmceuen | are we moving on mark-burnett or wrapping up discussion on this? | 14:38 |
| mark-burnett | Yeah, some good points have been discussed, but it's clear we're not going to settle on something here i think | 14:38 |
| openstackgerrit | Scott Hussey proposed openstack/airship-in-a-bottle master: [WIP] Add apiserver w/ webhook https://review.openstack.org/604918 | 14:39 |
| mark-burnett | #topic Marketing Deadlines | 14:39 |
| *** openstack changes topic to "Marketing Deadlines (Meeting topic: airship)" | 14:39 | |
| mark-burnett | Please note the deadlines and links in the etherpad: https://etherpad.openstack.org/p/airship-meeting-2018-09-25 | 14:39 |
| mark-burnett | I think I should have put this in announcements :> | 14:40 |
| mark-burnett | Ok, on to new business | 14:40 |
| mark-burnett | #topic Pegleg breaking changes | 14:40 |
| *** openstack changes topic to "Pegleg breaking changes (Meeting topic: airship)" | 14:40 | |
| mark-burnett | So the initial pegleg sketch was mostly a stop gap, and frankly full of dubious technical decisions | 14:40 |
| mark-burnett | Felipe has been trying to improve it, and has included some breaking changes that are discussed here: http://paste.openstack.org/show/730723/ | 14:41 |
| mark-burnett | Looks like there's room for discussion around managing breaking/changes, but it feels like it just fits in generally with a versioning conversation | 14:41 |
| mark-burnett | I'm not sure we're at a place where we can solidly version these tools yet, tbh -- surely some of them | 14:42 |
| mattmceuen | I am a big fan of the changes, just a little bit of growing pains - nice work felipe | 14:43 |
| mark-burnett | Yeah, I'm not sure there's much more to talk about here - just an effort to raise awareness about the newer changes | 14:43 |
| mark-burnett | Ok, let's finish up | 14:44 |
| mark-burnett | #topic roundtable | 14:44 |
| *** openstack changes topic to "roundtable (Meeting topic: airship)" | 14:44 | |
| mark-burnett | Anything we missed? | 14:44 |
| sthussey | I think today has gone long enough | 14:45 |
| sthussey | For next meeting possibly open the topic of a versioning strategy for the components | 14:45 |
| mark-burnett | Sure | 14:45 |
| mark-burnett | Thanks all | 14:46 |
| mark-burnett | #endmeeting | 14:46 |
| *** openstack changes topic to "airshipit.org || General Review Dashboard: https://review.openstack.org/#/q/project:%255Eopenstack/airship.*+status:open,n,z" | 14:46 | |
| openstack | Meeting ended Tue Sep 25 14:46:09 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:46 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/airship/2018/airship.2018-09-25-14.02.html | 14:46 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/airship/2018/airship.2018-09-25-14.02.txt | 14:46 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/airship/2018/airship.2018-09-25-14.02.log.html | 14:46 |
| jamesgu__ | hope some are still around between the airship and osh meetings... can I get some eyes on the multi linux distro/opensuse support sepc https://review.openstack.org/#/c/601187/? Many thanks! | 14:49 |
| jamesgu__ | there appears to be formatting issue to fix (zuul gives a -1), but hope I can get some feedback on the content :-) | 14:50 |
| *** cfriesen has joined #airshipit | 14:54 | |
| evrardjp | thanks jamesgu__ | 14:54 |
| *** felipemonteiro has joined #airshipit | 15:00 | |
| *** sgrasley has joined #airshipit | 15:02 | |
| *** Bhujay has quit IRC | 15:03 | |
| openstackgerrit | PRATEEK REDDY DODDA proposed openstack/airship-deckhand master: Have made neccessary changes to avoid dup document https://review.openstack.org/603502 | 15:05 |
| *** sgrasley__ has quit IRC | 15:05 | |
| *** felipemonteiro has quit IRC | 15:06 | |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 15:09 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 15:26 |
| openstackgerrit | Scott Hussey proposed openstack/airship-drydock master: Roll pyghmi back to 1.0.18 https://review.openstack.org/605106 | 15:34 |
| openstackgerrit | Sean Eagan proposed openstack/airship-armada master: Parallelize unsequenced chart group deployments https://review.openstack.org/597180 | 15:35 |
| *** hemanth_n has quit IRC | 15:38 | |
| openstackgerrit | Scott Hussey proposed openstack/airship-in-a-bottle master: Support maas mobility https://review.openstack.org/604411 | 15:57 |
| openstackgerrit | Scott Hussey proposed openstack/airship-in-a-bottle master: Support setting node domains https://review.openstack.org/604412 | 15:57 |
| openstackgerrit | Scott Hussey proposed openstack/airship-in-a-bottle master: Add Pegleg schemas to site definition https://review.openstack.org/604413 | 15:57 |
| *** felipemonteiro has joined #airshipit | 16:08 | |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 16:09 |
| openstackgerrit | Sreejith Punnapuzha proposed openstack/airship-promenade master: [WIP] Implements: etcd backup https://review.openstack.org/600493 | 16:28 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-shipyard master: Rollback to revision 0 instead of clearing Deckhand DB https://review.openstack.org/592742 | 16:29 |
| openstackgerrit | Steve Wilkerson proposed openstack/airship-treasuremap master: OSH-Infra: Update chart configs https://review.openstack.org/603148 | 16:30 |
| openstackgerrit | Steve Wilkerson proposed openstack/airship-treasuremap master: OSH-Infra: Update chart configs https://review.openstack.org/603148 | 16:38 |
| openstackgerrit | Merged openstack/airship-armada master: Fix: git commit id labels on images https://review.openstack.org/604214 | 16:45 |
| openstackgerrit | Matt McEuen proposed openstack/airship-specs master: Spec: Pegleg encryption and decryption https://review.openstack.org/603473 | 16:53 |
| *** roman_g has joined #airshipit | 17:14 | |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Implement global lint and lint by site logic https://review.openstack.org/604871 | 17:18 |
| *** felipemonteiro has quit IRC | 17:28 | |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Implement global lint and lint by site logic https://review.openstack.org/604871 | 17:41 |
| openstackgerrit | Sean Eagan proposed openstack/airship-armada master: Clean up and refactor wait logic https://review.openstack.org/602364 | 17:51 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Implement global lint and lint by site logic https://review.openstack.org/604871 | 17:58 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Implement global lint and lint by site logic https://review.openstack.org/604871 | 18:01 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: docs: Elaborate on document layering in documentation https://review.openstack.org/578554 | 18:10 |
| *** lemko has quit IRC | 18:12 | |
| openstackgerrit | Merged openstack/airship-shipyard master: Minor: docs location fix https://review.openstack.org/602771 | 18:47 |
| openstackgerrit | Nikita Koshikov proposed openstack/airship-divingbell master: Add new apparmor daemonset https://review.openstack.org/599025 | 19:07 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add DAG and operator to test site https://review.openstack.org/603236 | 19:16 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 19:16 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add DAG and operator to test site https://review.openstack.org/603236 | 19:16 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 19:26 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Tidy up collect output file names for cloned repos https://review.openstack.org/604818 | 19:28 |
| openstackgerrit | PRATEEK REDDY DODDA proposed openstack/airship-deckhand master: Have made neccessary changes to avoid dup document https://review.openstack.org/603502 | 19:30 |
| openstackgerrit | Sean Eagan proposed openstack/airship-armada master: Parallelize unsequenced chart group deployments https://review.openstack.org/597180 | 19:40 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 19:49 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add DAG and operator to test site https://review.openstack.org/603236 | 19:49 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 19:58 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 20:01 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add DAG and operator to test site https://review.openstack.org/603236 | 20:01 |
| openstackgerrit | Felipe Monteiro proposed openstack/airship-shipyard master: Rollback to revision 0 instead of clearing Deckhand DB https://review.openstack.org/592742 | 20:04 |
| openstackgerrit | Scott Hussey proposed openstack/airship-promenade master: [WIP] Extend webhook-enabled apiserver chart https://review.openstack.org/603887 | 20:05 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 20:13 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add DAG and operator to test site https://review.openstack.org/603236 | 20:13 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: actions: Add Shipyard action to test site https://review.openstack.org/603237 | 20:13 |
| openstackgerrit | Purnendu proposed openstack/airship-specs master: Adding specs for Spyglass https://review.openstack.org/605227 | 20:26 |
| openstackgerrit | Scott Hussey proposed openstack/airship-in-a-bottle master: [WIP] Add apiserver w/ webhook https://review.openstack.org/604918 | 20:46 |
| openstackgerrit | Merged openstack/airship-shipyard master: Add relabel_nodes action https://review.openstack.org/599085 | 20:50 |
| *** aaronsheffield has quit IRC | 20:51 | |
| openstackgerrit | Merged openstack/airship-maas master: Add file package to sstream-cache image build https://review.openstack.org/604571 | 20:52 |
| openstackgerrit | Merged openstack/airship-drydock master: Update node-labels through Kubernetes Provisioner https://review.openstack.org/593609 | 20:55 |
| openstackgerrit | Scott Hussey proposed openstack/airship-specs master: External facing K8s API w/ Keystone auth/z https://review.openstack.org/588363 | 21:05 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 21:07 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 21:08 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add DAG and operator to test site https://review.openstack.org/603236 | 21:08 |
| openstackgerrit | Merged openstack/airship-drydock master: Roll pyghmi back to 1.0.18 https://review.openstack.org/605106 | 21:08 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 21:14 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add DAG and operator to test site https://review.openstack.org/603236 | 21:14 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: actions: Add Shipyard action to test site https://review.openstack.org/603237 | 21:14 |
| openstackgerrit | Nikita Koshikov proposed openstack/airship-divingbell master: Add new apparmor daemonset https://review.openstack.org/599025 | 21:16 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add get_releases to Armada base operator https://review.openstack.org/603235 | 21:18 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: plugins: Add DAG and operator to test site https://review.openstack.org/603236 | 21:18 |
| openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: actions: Add Shipyard action to test site https://review.openstack.org/603237 | 21:18 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Basic tests for airship-seaworthy deployment https://review.openstack.org/604210 | 21:35 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 21:36 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 21:37 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Set Keystone admin endpoint to match public https://review.openstack.org/604947 | 21:41 |
| openstackgerrit | Anthony Bellino proposed openstack/airship-armada master: Network Policy https://review.openstack.org/595266 | 21:50 |
| *** munimeha1 has quit IRC | 22:03 | |
| openstackgerrit | Scott Hussey proposed openstack/airship-drydock master: (fix) Fix missing parameter for node pkg list https://review.openstack.org/605246 | 22:06 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Basic tests for airship-seaworthy deployment https://review.openstack.org/604210 | 22:07 |
| *** felipemonteiro has joined #airshipit | 23:03 | |
| *** felipemonteiro has quit IRC | 23:41 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
