13:02:36 <mvollmer> #startmeeting meeting 13:02:37 <zodbot> Meeting started Mon Oct 17 13:02:36 2016 UTC. The chair is mvollmer. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:02:37 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic. 13:02:37 <zodbot> The meeting name has been set to 'meeting' 13:02:45 <andreasn> .hello andreasn 13:02:47 <mvollmer> .hello mvo 13:02:48 <zodbot> andreasn: andreasn 'Andreas Nilsson' <anilsson@redhat.com> 13:02:51 <zodbot> mvollmer: mvo 'Marius Vollmer' <marius.vollmer@gmail.com> 13:02:55 <mvollmer> #topic Agenda 13:03:31 <larsu> .hello larsu 13:03:32 <zodbot> larsu: larsu 'Lars Karlitski' <lars@karlitski.net> 13:03:37 <dperpeet> .hello dperpeet 13:03:38 <zodbot> dperpeet: dperpeet 'None' <dperpeet@redhat.com> 13:03:41 <dperpeet> * atomic images 13:03:44 <dperpeet> * outreachy 13:04:06 <mvollmer> * small storage list pattern update 13:04:58 <larsu> * docker redesign 13:05:14 <mvollmer> let's start 13:05:20 <mvollmer> #topic atomic images 13:05:46 <dperpeet> we are nearing getting more current atomic images again 13:05:49 <dperpeet> via pull request https://github.com/cockpit-project/cockpit/pull/5152 13:06:01 <dperpeet> it's a rebase of stef's work with some patches 13:06:15 <dperpeet> that includes the fedora-atomic image 13:06:29 <dperpeet> I've been looking at rhel-atomic, but that isn't done yet (tests failing) 13:06:41 <dperpeet> and continuous atomic currently doesn't work at all 13:07:23 <dperpeet> that's because kubernetes has been removed from continuous atomic builds https://github.com/cockpit-project/cockpit/pull/4942 13:07:42 <dperpeet> is this going to happen on the other atomic systems as well? 13:08:02 <mvollmer> do we need to update them all together? 13:08:12 <dperpeet> no 13:08:19 <dperpeet> the first pr I listed updates fedora-atomic 13:08:32 <dperpeet> rhel-atomic is another pr and blocked 13:08:42 <mvollmer> right 13:08:47 <dperpeet> continuous-atomic is closed, since we need to install things differently 13:10:13 <dperpeet> well, let's get fedora-atomic fixed pretty soon at least 13:10:25 <dperpeet> the image isn't all that current, but better than what we have now 13:10:49 <mvollmer> and it moves us from f23 to f24, right? 13:11:07 <dperpeet> right 13:11:23 <dperpeet> and makes sure to download the correct base image 13:11:27 <mvollmer> should we mark this "priority" maybe? 13:12:08 <dperpeet> good idea, done 13:13:31 <mvollmer> cool 13:13:52 <mvollmer> i think stuff that blocks image updates should be automatically priority 13:14:20 <mvollmer> we didn't run the storage tests on f25 for some time 13:14:24 <dperpeet> makes sense 13:14:29 <mvollmer> and now that we finally can, we find the bugs 13:15:30 <mvollmer> alright, next? 13:16:13 <mvollmer> #topic outreachy 13:16:28 <larsu> mclasen was asking here about outreachy on the weekend 13:16:37 <dperpeet> application deadline is today 13:16:50 <larsu> but I didn't know specifics. did he read you dperpeet or stefw? 13:17:40 <dperpeet> mclasen did not contact me about outreachy yet 13:18:20 <larsu> he wanted to know if you have candidates ready 13:18:55 <mvollmer> I thought they have already started .... :-) 13:19:21 <andreasn> we have three potential candidates who already did contributions on issues 13:19:26 <stefw> i think dperpeet is working on this ... there was third possible last minute applicant ... but it may be too late for that 13:19:27 <dperpeet> well, what I wanted to say: thanks to everyone who has contributed to cockpit so far, I'm looking forward to seeing who actually applies 13:19:47 <dperpeet> I think we may have four or five applicants 13:20:05 <mvollmer> nice 13:20:11 <github> [cockpit] stefwalter opened pull request #5170: ws: Fix testing race with ssh-agent unix socket (master...ssh-agent-socket-race) https://git.io/vPPjw 13:20:17 <stefw> dperpeet, are all applicants for design tasks? 13:20:19 <dperpeet> but we will only be able to take one UX project and at most one dev project 13:20:50 <dperpeet> stefw, I can get back to that question next week once we know who has actually applied 13:20:54 <stefw> ok 13:21:03 <dperpeet> but all the contributions towards cockpit have been from UX applicants so far 13:21:22 <dperpeet> and as with GSoC, we value prior contributions very highly when looking at applications 13:21:58 <stefw> agree 13:21:58 <dperpeet> andreasn and I discussed the possibility of taking more than one UX project 13:22:07 <dperpeet> but we decided that we couldn't, with good conscience 13:22:25 <dperpeet> we want to provide good mentorshop 13:22:30 <dperpeet> *mentorship / mentoring 13:22:34 <andreasn> I think next round could be a good opportunity to do several, but this round it would be best to focus on one 13:23:10 <dperpeet> but I'm really happy to see more than one good candidate! 13:23:32 <dperpeet> the good news is that Outreachy is twice a year 13:23:57 <dperpeet> end of topic 13:24:23 <mvollmer> #topic small storage list pattern update 13:24:31 <mvollmer> I am still working on that 13:24:45 <mvollmer> had to stop myself from doing too much redesign already now 13:24:51 <andreasn> I didn't get around to review the latest changes yet 13:24:57 <mvollmer> but I am quite happy where this is going 13:25:19 <mvollmer> andreasn, I start to think that the Format button is hard to find in its new place 13:25:40 <mvollmer> but I am already getting used to the new design 13:25:45 <andreasn> could be. I want to try it out in action 13:25:51 <andreasn> but I'm open to moving it if needed 13:26:19 <mvollmer> e.g., it's not immediatly visible after creating a new logical volume 13:26:32 <andreasn> right, it breaks the logical flow a bit 13:26:40 <github> [cockpit] stefwalter pushed 3 new commits to master: https://git.io/vPXe2 13:26:40 <github> cockpit/master 28d202a Marius Vollmer: test: Update fedora-25 to fix storaged SELinux issues... 13:26:41 <github> cockpit/master 57b4841 Marius Vollmer: test: Expect to be able to manage Docker storage on fedora-25... 13:26:41 <github> cockpit/master 36abf2b Marius Vollmer: storaged: Allow for more creative rounding in the storage stack... 13:26:58 <andreasn> hard to tell what is the next logical step 13:27:47 <andreasn> so yeah, it's a concern 13:27:55 <andreasn> I'll take a look at the PR again 13:28:02 <mvollmer> but this new design gives us so many more options for the future 13:28:09 <mvollmer> really nice, IMO 13:28:35 <mvollmer> end of topic 13:28:48 <mvollmer> #topic docker redesign 13:29:20 <larsu> I think it is coming along quite nicely 13:29:29 <larsu> and I want to get the current state in, if nobody objects 13:29:48 <andreasn> what's the PR? 13:29:51 <larsu> there are a couple of things from the new design missing (cards on top, additional tabs) 13:29:57 <larsu> but those are not regressions 13:30:02 <andreasn> right 13:30:15 <andreasn> I think the cards are more of the atomic scan step 13:30:16 <larsu> the one regression that dperpeet brought up would be the "danger mode", which is replaced by selection mode 13:30:34 <larsu> but I'd like to postpone that, because it needs quite some changes to the listing view as well 13:30:44 <larsu> (again, if nobody objects) 13:30:55 <larsu> andreasn: the pr is 4952 13:30:58 <andreasn> thanks 13:30:58 <github> [cockpit] stefwalter opened pull request #5171: test: Fix race where sosreport takes too long (master...race-sosreport-too-slow) https://git.io/vPXvJ 13:31:00 <stefw> danger mode has always been dubious 13:31:13 <stefw> in particular because rows disappeared as you were clicking buttons 13:31:17 <dperpeet> I wouldn't consider that change a blocker 13:31:17 <stefw> which really is risky 13:31:31 <larsu> you can still delete containers from the overview, but you need to expand the rows 13:31:33 <dperpeet> I think we need more top-down design to decide what we want to do there 13:31:38 <andreasn> yeah 13:31:40 <stefw> dperpeet, yup 13:31:53 <dperpeet> once we have a listing, we can tinker with it in a separate pr 13:31:54 <larsu> stefw: I agree, and dperpeet had a good idea to alleviate that with simply waiting a while before deleting the container 13:32:11 <larsu> which needs further design thought 13:32:16 <dperpeet> oh, my press and hold button idea :) 13:32:19 <stefw> if we do add a multi-select/danger mode to the listing view ... we should do it properly 13:32:30 <stefw> and likely not change/remove rows while in that mode 13:32:36 <dperpeet> yeah 13:32:41 <stefw> just gray them out or something like that ... with a status of what happened 13:33:02 <stefw> ok ... larsu so no blockers on this? 13:33:10 <stefw> why is it listed as blocking on a closed pull request? 13:33:20 <larsu> it's blocking on the listing view one 13:33:25 <stefw> which is closed 13:33:33 <stefw> 24 days ago? 13:33:43 <stefw> https://github.com/cockpit-project/cockpit/pull/5043 13:33:47 <dperpeet> yeah, larsu do you mean a different pr? 13:34:12 <larsu> oh yeah, how did that happen 13:34:20 <larsu> I mean 5156 13:34:31 <stefw> i can merge that 13:34:32 <stefw> looks good 13:34:35 <larsu> cool, thakns 13:35:02 <dperpeet> stefw, did you try the playground on #5156? 13:35:23 <dperpeet> this changes the basics of our select pattern implementation 13:35:36 <stefw> i guess jump in on the review then 13:35:38 <dperpeet> and changes API in lib/cockpit-components-select 13:35:49 <dperpeet> yes, like stated earlier I want to try this out 13:35:52 <stefw> those are fine for API changes 13:36:10 <stefw> oh i guess your comment on the PR got hidden 13:36:13 <stefw> Github things :S 13:36:14 * larsu think he caught everything using that 13:36:27 <dperpeet> no, when we discussed this earlier today on irc :) 13:36:28 <larsu> but yeah, please have another look 13:36:42 <dperpeet> I can merge it once tests pass 13:36:45 <dperpeet> and I've looked 13:36:48 <larsu> stefw: that issue is mostly fixed with the new review system 13:36:53 <larsu> dperpeet: thanks! 13:37:20 <larsu> if you find this to not be ready, I could also amend the docker one to use the old way of doing things 13:37:44 <larsu> anyway, getting this in so that I can base the image scanning stuff on top of that would be great 13:37:53 <larsu> otherwise, end of topic from my side 13:38:11 <stefw> yes, too many blockers here 13:38:15 <dperpeet> yes, let's get it in soon 13:38:17 <stefw> that's why i'm trying to jump in on the review 13:40:32 <dperpeet> mvollmer, end of topic? 13:40:42 <mvollmer> yeah 13:41:01 <mvollmer> #topic any other buisness 13:41:07 <mvollmer> *bushness 13:42:29 <mvollmer> alright 13:42:34 <mvollmer> thanks everyone! 13:42:44 <mvollmer> #endmeeting