14:29:41 #startmeeting Pulp Triage 2018-06-08 14:29:41 #info dawalker has joined triage 14:29:42 Meeting started Fri Jun 8 14:29:41 2018 UTC and is due to finish in 60 minutes. The chair is dawalker. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:29:42 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:29:42 The meeting name has been set to 'pulp_triage_2018_06_08' 14:29:42 dawalker: dawalker has joined triage 14:29:46 !here 14:29:46 #info jortel has joined triage 14:29:46 jortel: jortel has joined triage 14:29:46 !here 14:29:46 #info dkliban has joined triage 14:29:47 dkliban: dkliban has joined triage 14:29:59 !here 14:29:59 #info dalley has joined triage 14:29:59 dalley: dalley has joined triage 14:30:04 #info ttereshc has joined triage 14:30:04 !here 14:30:04 ttereshc: ttereshc has joined triage 14:30:15 !here 14:30:15 #info asmacdo has joined triage 14:30:16 asmacdo: asmacdo has joined triage 14:30:17 !here 14:30:17 #info bizhang has joined triage 14:30:18 bizhang: bizhang has joined triage 14:30:20 Alright, we have a lot of bugs today, so let's get started! 14:30:20 #info daviddavis has joined triage 14:30:20 !here 14:30:21 daviddavis: daviddavis has joined triage 14:30:21 !next 14:30:23 #topic When the checksum type of the upstream repository changed, syncing a "on_demand" repository will fail - http://pulp.plan.io/issues/3736 14:30:23 dawalker: 10 issues left to triage: 3736, 3737, 3738, 3739, 3742, 3743, 3745, 3747, 3748, 3749 14:30:25 Issue #3736 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:30:26 When the checksum type of the upstream repository changed, syncing a "on_demand" repository will fail - http://pulp.plan.io/issues/3736 14:30:36 ttereshc, what's the update on this? 14:30:41 it's a bug 14:30:51 !propose accept 14:30:51 #idea Proposed for #3736: Leave the issue as-is, accepting its current state. 14:30:51 ttereshc: Proposed for #3736: Leave the issue as-is, accepting its current state. 14:30:52 !here 14:30:52 #info ipanova has joined triage 14:30:52 ipanova: ipanova has joined triage 14:31:05 +1 14:31:06 !here 14:31:06 #info bmbouter has joined triage 14:31:06 bmbouter: bmbouter has joined triage 14:31:34 +1 14:31:42 !accept 14:31:42 #agreed Leave the issue as-is, accepting its current state. 14:31:42 dawalker: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:31:44 #topic [tests only] memory error in unittests when mock.MagicMock(parent='anything') is used - http://pulp.plan.io/issues/3737 14:31:44 dawalker: 9 issues left to triage: 3737, 3738, 3739, 3742, 3743, 3745, 3747, 3748, 3749 14:31:45 Issue #3737 [ASSIGNED] (amacdona@redhat.com) - Priority: Normal | Severity: Medium 14:31:46 [tests only] memory error in unittests when mock.MagicMock(parent='anything') is used - http://pulp.plan.io/issues/3737 14:31:56 +1 14:32:02 it's assigned 14:32:12 !propose accept 14:32:12 #idea Proposed for #3737: Leave the issue as-is, accepting its current state. 14:32:12 asmacdo: Proposed for #3737: Leave the issue as-is, accepting its current state. 14:32:19 does it mean it's already kinda on the sprint? 14:32:32 seems that way 14:32:53 !accept 14:32:53 #agreed Leave the issue as-is, accepting its current state. 14:32:53 dawalker: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:32:54 #topic update plugin_template to work with latest pulpcore - http://pulp.plan.io/issues/3738 14:32:55 dawalker: 8 issues left to triage: 3738, 3739, 3742, 3743, 3745, 3747, 3748, 3749 14:32:56 Issue #3738 [ASSIGNED] (amacdona@redhat.com) - Priority: Normal | Severity: Medium 14:32:57 update plugin_template to work with latest pulpcore - http://pulp.plan.io/issues/3738 14:33:16 accept and add to sprint 14:33:20 This one is also assigned 14:33:28 !propose accept and add to sprint 14:33:28 dawalker: propose accept Propose accepting the current issue in its current state. 14:33:28 i'm doing this as part of the bootstrap docker work 14:33:30 isn't 3737 just a PSA? 14:33:32 ah, ok 14:33:46 maybe it should be closed, since there's no actions to take for it 14:33:48 bizhang: no, it also needs changes 14:33:52 !accept 14:33:52 dawalker: No action proposed, nothing to accept. 14:33:54 asmacdo, ack 14:34:01 I think we should delete plugin_template 14:34:06 it's crazy out of date 14:34:08 +1 14:34:10 not anymore 14:34:13 ive got it working 14:34:24 !propose accept 14:34:24 #idea Proposed for #3738: Leave the issue as-is, accepting its current state. 14:34:24 dawalker: Proposed for #3738: Leave the issue as-is, accepting its current state. 14:34:27 !accept 14:34:27 #agreed Leave the issue as-is, accepting its current state. 14:34:27 dawalker: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:34:28 dawalker: 7 issues left to triage: 3739, 3742, 3743, 3745, 3747, 3748, 3749 14:34:29 #topic Migrations failure during vagrant up - http://pulp.plan.io/issues/3739 14:34:29 Issue #3739 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:34:30 Migrations failure during vagrant up - http://pulp.plan.io/issues/3739 14:34:40 wait, I don't think we agreed on 3738 14:34:49 yup 14:35:11 Oh, ok, since it was assigned as the previous, I moved on, sorry. 14:35:23 dawalker, you can go back with !issue 3738 14:35:31 !issue 3738 14:35:31 #topic update plugin_template to work with latest pulpcore - http://pulp.plan.io/issues/3738 14:35:32 Issue #3738 [ASSIGNED] (amacdona@redhat.com) - Priority: Normal | Severity: Medium 14:35:33 update plugin_template to work with latest pulpcore - http://pulp.plan.io/issues/3738 14:35:33 the template is really useful for bootstrapping plugins 14:35:33 thanks ttereshc 14:35:48 I guess my concern is that it'll get out of date again 14:35:53 it probably will 14:35:59 i have some ideas for this 14:36:01 that and also that pulp2 is the main priority 14:36:37 well, it didnt take long to bring it back up to date enough to function 14:36:54 I'm tempted to wait-and-see then 14:37:03 and it took less time to fix it and use it to create the bootstrapped docker than it would have to write from scratch 14:37:57 anyway, maybe we can discuss in another way. if it's fixed for now, we should probably just continue with triage. 14:38:11 agreed 14:38:23 +1, i can put together my ideas for the template and send to pulp-dev 14:38:30 ok, sounds good. thanks asmacdo 14:39:08 !propose other pulp-dev discussion 14:39:08 #idea Proposed for #3738: pulp-dev discussion 14:39:08 dawalker: Proposed for #3738: pulp-dev discussion 14:39:49 +1 14:39:51 +1 14:39:52 +1 14:39:56 !accept 14:39:56 #agreed pulp-dev discussion 14:39:57 dawalker: Current proposal accepted: pulp-dev discussion 14:39:58 dawalker: 7 issues left to triage: 3739, 3742, 3743, 3745, 3747, 3748, 3749 14:39:58 #topic Migrations failure during vagrant up - http://pulp.plan.io/issues/3739 14:39:59 Issue #3739 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:40:00 Migrations failure during vagrant up - http://pulp.plan.io/issues/3739 14:40:30 accept and add to sprint 14:40:46 !propose accept and add to sprint 14:40:46 dawalker: propose accept Propose accepting the current issue in its current state. 14:40:59 +1 14:41:03 +1 14:41:21 +1 14:41:22 +1 14:41:25 !accept 14:41:25 dawalker: No action proposed, nothing to accept. 14:41:44 !propose other accept and add to sprint 14:41:44 #idea Proposed for #3739: accept and add to sprint 14:41:44 dawalker: Proposed for #3739: accept and add to sprint 14:41:46 !accept 14:41:46 #agreed accept and add to sprint 14:41:46 dawalker: Current proposal accepted: accept and add to sprint 14:41:47 dawalker: 6 issues left to triage: 3742, 3743, 3745, 3747, 3748, 3749 14:41:47 #topic Developer ansible installer migrations are not idempotent when used after systemd - http://pulp.plan.io/issues/3742 14:41:48 Issue #3742 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:41:49 Developer ansible installer migrations are not idempotent when used after systemd - http://pulp.plan.io/issues/3742 14:42:43 same area as 3739, maybe we should mark them as related 14:43:17 sounds reasonable 14:43:31 asmacdo, when do you hit it? 14:43:40 when you run a second time 14:43:49 asmacdo, if you re-run playbook for the second time? 14:43:51 manually? 14:44:05 i think i did a vagrant halt, vagrant up 14:44:12 but it should also happen on vagrant provision 14:44:30 how do you mark them as related? 14:44:43 !propose accept 14:44:43 #idea Proposed for #3742: Leave the issue as-is, accepting its current state. 14:44:43 ttereshc: Proposed for #3742: Leave the issue as-is, accepting its current state. 14:44:47 is 3739 the parent? 14:44:57 there is a button "Related issues" --------------------------------------------------------------------------------> [Add Button] 14:45:23 yeah, it's outside of editing the issue itself 14:45:25 Ah, I see it now. Was assuming it was in the edit 14:45:45 +1 14:46:39 +1 14:46:57 +1 14:46:59 !accept 14:46:59 #agreed Leave the issue as-is, accepting its current state. 14:46:59 dawalker: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:47:01 #topic Make 3.0-dev the new master branch - http://pulp.plan.io/issues/3743 14:47:01 dawalker: 5 issues left to triage: 3743, 3745, 3747, 3748, 3749 14:47:03 Issue #3743 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:47:04 Make 3.0-dev the new master branch - http://pulp.plan.io/issues/3743 14:47:13 this is done 14:47:17 i forgot to close it 14:47:19 pulp-python 14:47:28 !propose other convert to task 14:47:28 #idea Proposed for #3743: convert to task 14:47:28 ttereshc: Proposed for #3743: convert to task 14:47:38 +1 task 14:47:40 oh yeah 14:47:44 +1 14:47:45 +1 14:47:51 +1 14:47:52 +1 14:47:54 +1 14:47:59 +1 14:48:01 !accept 14:48:01 #agreed convert to task 14:48:01 dawalker: Current proposal accepted: convert to task 14:48:02 dawalker: 4 issues left to triage: 3745, 3747, 3748, 3749 14:48:03 #topic Applicability task fails if consumer is removed in the middle of the task - http://pulp.plan.io/issues/3745 14:48:03 Issue #3745 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:48:04 Applicability task fails if consumer is removed in the middle of the task - http://pulp.plan.io/issues/3745 14:49:09 it's very easy to fix and few customers are affected by that 14:49:19 we should add to sprint 14:49:20 it can wait the next sprint plannin though 14:49:40 +1 to add to the sprint if no one objects 14:49:45 +1 14:49:47 +1 14:49:54 #idea Proposed for #3745: accept and add to sprint 14:49:54 !propose other accept and add to sprint 14:49:54 +1 14:49:54 dawalker: Proposed for #3745: accept and add to sprint 14:49:58 #agreed accept and add to sprint 14:49:58 !accept 14:49:58 dawalker: Current proposal accepted: accept and add to sprint 14:49:59 +1 14:50:00 dawalker: 3 issues left to triage: 3747, 3748, 3749 14:50:00 #topic Add a config option to the importer to accept foreign layers - http://pulp.plan.io/issues/3747 14:50:01 Issue #3747 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:50:02 Add a config option to the importer to accept foreign layers - http://pulp.plan.io/issues/3747 14:50:13 this is a story 14:50:20 story +1 14:50:31 !propose other convert to story 14:50:31 #idea Proposed for #3747: convert to story 14:50:31 dawalker: Proposed for #3747: convert to story 14:50:33 +1 14:50:43 there have been some changes, we need to adapt them as well 14:51:04 since it will be a story, it will be ungroomed, so i think thats ok 14:51:12 yep 14:51:21 +1 14:51:23 +1 14:51:34 !accept 14:51:34 #agreed convert to story 14:51:34 dawalker: Current proposal accepted: convert to story 14:51:35 dawalker: 2 issues left to triage: 3748, 3749 14:51:35 #topic documentation fails to mention that 'name' must be unique - http://pulp.plan.io/issues/3748 14:51:36 Issue #3748 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:51:37 documentation fails to mention that 'name' must be unique - http://pulp.plan.io/issues/3748 14:51:57 !propose other accept and add to sprint 14:51:57 #idea Proposed for #3748: accept and add to sprint 14:51:57 ttereshc: Proposed for #3748: accept and add to sprint 14:51:57 this is probably more general than the description 14:52:06 +1 14:52:24 im betting that if uniqueness is not on those fields at that endpoint, its not included on every endpoint 14:52:55 so not just name, but all unique fields 14:52:55 agreed. this is likely a widespread problem. 14:53:05 asmacdo, mind leaving a comment? 14:53:08 +1 add to sprint, ill comment 14:53:15 ty 14:53:16 +1 14:53:23 !accept 14:53:23 #agreed accept and add to sprint 14:53:23 dawalker: Current proposal accepted: accept and add to sprint 14:53:24 dawalker: 1 issues left to triage: 3749 14:53:24 #topic Skip upload of the foreign layers - http://pulp.plan.io/issues/3749 14:53:25 Issue #3749 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:53:26 Skip upload of the foreign layers - http://pulp.plan.io/issues/3749 14:53:41 i just checked we already cover this use case 14:53:49 we can close it 14:54:11 !propose other close as WORKFORME 14:54:11 #idea Proposed for #3749: close as WORKFORME 14:54:11 ttereshc: Proposed for #3749: close as WORKFORME 14:54:17 !propose other close as WORKSFORME 14:54:17 #idea Proposed for #3749: close as WORKSFORME 14:54:17 ttereshc: Proposed for #3749: close as WORKSFORME 14:54:35 +1 14:54:44 +1 14:54:57 !accept 14:54:57 #agreed close as WORKSFORME 14:54:57 dawalker: Current proposal accepted: close as WORKSFORME 14:54:58 dawalker: No issues to triage. 14:55:10 Yay! 14:55:38 !end 14:55:45 :) 14:55:45 !end 14:55:45 #endmeeting