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