Skip to content
This repository has been archived by the owner on Jul 16, 2020. It is now read-only.

Weekly Meeting 2016 07 21

Kristen Carlson Accardi edited this page Jul 21, 2016 · 4 revisions

Agenda

##Minutes

#ciao-project Meeting

Meeting started by kristenc at 16:18:13 UTC. The full logs are available at ciao-project/2016/ciao-project.2016-07-21-16.18.log.html .

Meeting summary

  • bug scrub (kristenc, 16:18:22)

  • opens (kristenc, 16:18:49)

  • bug scrub (kristenc, 16:21:04)

    • LINK: https://github.com/01org/ciao/issues/343 (kristenc, 16:22:06)
    • AGREED: P2 bug fixes are lower priority than P1 features. (kristenc, 16:23:50)
    • New Gatekeepers start today. (kristenc, 16:28:29)
    • Sprint 2 started this week. (kristenc, 16:29:05)
  • full end to end testing on Travis (kristenc, 16:37:26)

Meeting ended at 16:44:16 UTC.

Action Items

  • mcastelino to make github issue to track status of travis end to end testing.

Action Items, by person

  • mcastelino
    • mcastelino to make github issue to track status of travis end to end testing.
  • UNASSIGNED
    • (none)

People Present (lines said)

  • kristenc (43)
  • markusry (34)
  • tcpepper (11)
  • mcastelino (8)
  • mrkz (3)
  • ciaomtgbot (2)

Generated by MeetBot_ 0.1.4

.. _MeetBot: http://wiki.debian.org/MeetBot

###Full IRC Log

16:18:13 <kristenc> #startmeeting
16:18:13 <ciaomtgbot> Meeting started Thu Jul 21 16:18:13 2016 UTC.  The chair is kristenc. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:18:13 <ciaomtgbot> Useful Commands: #action #agreed #help #info #idea #link #topic.
16:18:22 <kristenc> #topic bug scrub
16:18:25 <markusry> o/
16:18:29 <mcastelino> o/
16:18:32 <mrkz> o/
16:18:33 <kristenc> o/
16:18:42 <tcpepper> o/
16:18:46 <kristenc> oops.
16:18:49 <kristenc> #topic opens
16:19:23 <kristenc> my open is to report for the records that a problem with the releases has been noted and will be addressed ASAP.
16:19:42 <kristenc> anyone have any other opens?
16:20:22 <mrkz> I have none
16:21:00 <kristenc> ok - I'm calling opens complete.
16:21:04 <kristenc> #topic bug scrub
16:21:15 <kristenc> looks like tim has got a list going.
16:21:59 <kristenc> we seem to have no P1 bugs to talk about.
16:22:06 <kristenc> #link https://github.com/01org/ciao/issues/343
16:22:25 <markusry> All the P2 are mine and were discussed last week
16:22:29 <markusry> Nothing has changed
16:22:57 <kristenc> IMO a P2 bug fix is a lower priority than a P1 feature for the next sprint. Agree?
16:23:07 * tcpepper agrees
16:23:12 <markusry> I'm not currently working on them
16:23:50 <kristenc> #agreed P2 bug fixes are lower priority than P1 features.
16:24:31 <markusry> kristenc: Agreed as well
16:24:44 <markusry> Shall I move them or do we need to check with Amy first?
16:25:09 <kristenc> markusry, move them where?
16:25:15 <markusry> to Sprint3?
16:25:17 <kristenc> they don't have milestones on them, right?
16:25:27 <kristenc> I would just clear any milestone.
16:25:33 <markusry> 16 and 17 do
16:25:46 <kristenc> ah - well then I'd remove the milestone.
16:26:01 <kristenc> they get done when you have bandwidth, or they get elevated to P1.
16:26:11 <markusry> Actually, 3 of them are sprint 2
16:26:31 <markusry> Okay, I'll discuss with Amy tomorrow
16:26:50 <markusry> But there very unlikely to get fixed in Sprint 2
16:26:54 <markusry> they're I mean
16:27:17 <kristenc> yeah.
16:27:43 <kristenc> ok then - we are done with our bug scrub.
16:28:25 <markusry> I think so.
16:28:29 <kristenc> #info New Gatekeepers start today.
16:28:46 <kristenc> tcpepper and Samuel are the new gatekeepers.
16:28:57 <tcpepper> ACK
16:29:05 <kristenc> #info Sprint 2 started this week.
16:29:15 <markusry> tcpepper: About
16:29:18 <markusry> .link https://github.com/01org/ciao/pull/363
16:29:37 <markusry> I wasn't planning to add any unit tests to this, as the code is itself test code that is disabled by default
16:30:02 <markusry> It's not even compiled into the binaries that are used to compute our coverage statistics
16:31:10 <tcpepper> sure, but...it's present and has been broken for a while.  if it's going to stay, a test keeps it working.  if it's been broken for such a long time, we could also remove it
16:31:51 <markusry> tcpepper: What's broken?
16:31:59 <tcpepper> oh I was thinking this was the cpuprofiling one
16:32:01 <tcpepper> n/m
16:32:25 <markusry> Well its tracing not cpuprofiling
16:32:32 <tcpepper> yeah
16:32:32 <markusry> But cpu profiling is not broken
16:32:35 <markusry> It works fine
16:32:41 <markusry> in launcher at leastr
16:32:43 <markusry> least
16:33:10 <tcpepper> ah /me sees that one merged 9 days ago
16:33:12 <markusry> But it doesn't tell you which parts of your code is blocking on a syscall
16:33:22 <markusry> Which is why I added the tracing
16:34:14 <tcpepper> well meh.  I simply worry that we're adding misc diagnostics over time, they're not normally used, they're not well documented or tested.  Sounds a lot like technical debt to me.
16:34:59 <markusry> Well they're tested when they're used.
16:35:40 <markusry> This code is never going to change
16:35:50 <tcpepper> at any rate, I've registered my meta-worry about it and just pressed the merge button
16:35:58 <markusry> Okay, thanks.
16:35:58 <tcpepper> we should think about these things
16:36:09 <kristenc> ok - does anyone have any other topics, or shall we end this meeting?
16:36:29 <mcastelino> One open
16:36:43 <kristenc> mcastelino, ok.
16:36:48 <mcastelino> obedmr: will you be able to add your tests to travis?
16:36:55 <mcastelino> container only at least
16:37:09 <mcastelino> obedmr has way to do full end to end testing on Travis
16:37:17 <markusry> How long does it take?
16:37:26 <mcastelino> 2 mins I think
16:37:26 <kristenc> #topic  full end to end testing on Travis
16:37:36 <markusry> obedmr: That's great by the way
16:37:42 <kristenc> yes - great!
16:37:51 <mcastelino> this is single VM launch containers only
16:37:53 <mrkz> mcastelino: obed is AFK at the moment if I'm not mistaken
16:37:56 <markusry> Is it 100    16:38:36 <kristenc> mcastelino, maybe the thing to do is to put in a github issues that says "support full end to end testing on Travis" and assign it to obedmr
16:38:41 <mcastelino> yes
16:38:42 <kristenc> then he can update the status there.
16:38:48 <markusry> I think there's already an issue
16:39:07 <kristenc> anyone know the issue #?
16:39:10 <markusry> #link https://github.com/01org/ciao/issues/334
16:39:54 <kristenc> mcastelino, does that cover it? It's not assigned to obed.
16:40:37 <kristenc> that one is more about making sure all the bat tests work.
16:40:48 <kristenc> so maybe a bit of a different thing than what obed got working.
16:41:12 <markusry> RIght, the title is a bit confusing
16:41:38 <kristenc> we could edit the title to make it more explicit.
16:42:06 <kristenc> there - I did it :).
16:42:15 <kristenc> Make bat tests work on travis with single machine setup
16:42:31 <kristenc> so - now I think mcastelino should enter a separate issue for whatever obed did.
16:42:49 <kristenc> and we can make the bat tests dependent on it.
16:43:40 <kristenc> #action mcastelino to make github issue to track status of travis end to end testing.
16:43:44 <kristenc> anything else?
16:44:13 <kristenc> ok - time to end the meeting.
Clone this wiki locally