15:31:38 #startmeeting Pulp Triage 2020-11-24 15:31:38 #info fao89 has joined triage 15:31:38 !start 15:31:38 Meeting started Tue Nov 24 15:31:38 2020 UTC. The chair is fao89. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:31:38 Useful Commands: #action #agreed #help #info #idea #link #topic. 15:31:38 The meeting name has been set to 'pulp_triage_2020-11-24' 15:31:38 fao89: fao89 has joined triage 15:32:17 #info daviddavis has joined triage 15:32:17 !here 15:32:17 daviddavis: daviddavis has joined triage 15:32:25 Open floor! 15:32:35 #info ttereshc has joined triage 15:32:35 !here 15:32:35 ttereshc: ttereshc has joined triage 15:32:40 https://hackmd.io/@pulp/triage/edit the agenda is empty 15:32:48 D: 15:32:54 but we have lots of issues at triage query 15:32:58 great 15:33:23 I just need one more person to join, so we can move to triage 15:34:03 #info dalley has joined triage 15:34:03 !here 15:34:03 dalley: dalley has joined triage 15:34:14 https://pulp.plan.io/issues?query_id=143 15:34:37 !next 15:34:38 fao89: 12 issues left to triage: 7890, 7888, 7878, 7875, 7867, 7857, 7856, 7854, 7853, 7848, 7845, 7835 15:34:38 #topic https://pulp.plan.io/issues/7890 15:34:39 RM 7890 - daviddavis - ASSIGNED - Unskip broken filesystem exporter test 15:34:40 https://pulp.plan.io/issues/7890 15:34:41 #info ipanova has joined triage 15:34:41 !here 15:34:42 ipanova: ipanova has joined triage 15:34:58 task 15:35:01 #idea Proposed for #7890: Leave the issue as-is, accepting its current state. 15:35:01 !propose accept 15:35:01 fao89: Proposed for #7890: Leave the issue as-is, accepting its current state. 15:35:12 +1 to task and accept 15:35:24 #idea Proposed for #7890: convert to task 15:35:24 !propose other convert to task 15:35:24 fao89: Proposed for #7890: convert to task 15:35:30 #agreed convert to task 15:35:30 !accept 15:35:30 fao89: Current proposal accepted: convert to task 15:35:31 #topic https://pulp.plan.io/issues/7888 15:35:31 fao89: 10 issues left to triage: 7888, 7878, 7867, 7857, 7856, 7854, 7853, 7848, 7845, 7835 15:35:32 RM 7888 - equipe_serveurs@exane.com - POST - Can't update Fields through Rest API We pages (PUT requests) 15:35:33 https://pulp.plan.io/issues/7888 15:35:37 #info bmbouter has joined triage 15:35:37 !here 15:35:37 bmbouter: bmbouter has joined triage 15:35:43 accept? 15:35:53 #idea Proposed for #7888: Leave the issue as-is, accepting its current state. 15:35:53 !propose accept 15:35:53 fao89: Proposed for #7888: Leave the issue as-is, accepting its current state. 15:35:54 daviddavis: there is a pr up if you can take a look also 15:36:15 daviddavis: I was unsure about all this 15:36:23 I'm probably not a good person to review this. I don't use the html stuff. 15:36:50 but I can try to take a look if no one else can 15:36:56 we initially thought this pr is from the students 15:37:12 I don't think so. this person is based in Paris. 15:37:19 oh I said that it's my mistake 15:37:24 I thought student team 15:37:25 yeah, it's from a bank 15:37:31 so this is significant enough that we need to be sure 15:37:41 #info mikedep333 has joined triage 15:37:41 !here 15:37:41 mikedep333: mikedep333 has joined triage 15:37:54 I'm happy to get on a call for 15-30 min to at least talk it over with anyone else 15:38:35 I can be the rubber duck 15:39:21 sold to me and fabricio! 15:39:40 should I accept? 15:39:42 or skip? 15:40:33 #agreed Leave the issue as-is, accepting its current state. 15:40:33 !accept 15:40:33 fao89: Current proposal accepted: Leave the issue as-is, accepting its current state. 15:40:34 #topic https://pulp.plan.io/issues/7878 15:40:34 fao89: 9 issues left to triage: 7878, 7867, 7857, 7856, 7854, 7853, 7848, 7845, 7835 15:40:36 RM 7878 - daviddavis - POST - Developer setup still refers to pulplift 15:40:37 https://pulp.plan.io/issues/7878 15:40:47 accept and add to sprint 15:40:54 +1 15:40:57 +1 15:40:59 +1 15:41:11 #idea Proposed for #7878: accept and add to sprint 15:41:11 !propose other accept and add to sprint 15:41:11 fao89: Proposed for #7878: accept and add to sprint 15:41:11 #agreed accept and add to sprint 15:41:11 !accept 15:41:12 #topic https://pulp.plan.io/issues/7867 15:41:12 fao89: Current proposal accepted: accept and add to sprint 15:41:13 fao89: 8 issues left to triage: 7867, 7857, 7856, 7854, 7853, 7848, 7845, 7835 15:41:14 RM 7867 - tosif.meman@kindredgroup.com - NEW - Issue #7796 : Pulp Version 2.20 - Mongod and pulp_celerybeat failing randomly 15:41:15 https://pulp.plan.io/issues/7867 15:42:27 skip? 15:42:48 we need to add pulp 2 tag at least 15:43:26 skipping it doesn't help I think... 15:43:39 dalley, bmbouter, any thoughts on the issue? 15:43:59 they have lock on the mongo 15:44:06 bmbouter, is this the one you were already looking into? 15:44:33 they need to remove lock and repair their db 15:45:23 yeah they should cancel all their tasks 15:45:31 and I gotta make this patch today for pulp2 15:45:48 but it won't help them until it's released and that prevents it occuring again 15:45:59 I can post on this issue explaining for them to cancell all tasks 15:46:15 so what should I do? accept the issue? 15:46:28 bmbouter: ty, should we also put it on the sprint? or just accept 15:46:31 although ... is this the same issue? 15:46:41 well maybe neither I'm already fixing it as another issue 15:47:23 I will skip it then 15:47:36 #idea Proposed for #7867: bmbouter will comment on the issue and close as dup if necessary 15:47:36 !propose other bmbouter will comment on the issue and close as dup if necessary 15:47:36 ipanova: Proposed for #7867: bmbouter will comment on the issue and close as dup if necessary 15:47:43 https://pulp.plan.io/issues/5502 15:47:51 so I don't think this is realted 15:47:53 #agreed bmbouter will comment on the issue and close as dup if necessary 15:47:53 !accept 15:47:53 fao89: Current proposal accepted: bmbouter will comment on the issue and close as dup if necessary 15:47:55 #topic https://pulp.plan.io/issues/7857 15:47:55 fao89: 7 issues left to triage: 7857, 7856, 7854, 7853, 7848, 7845, 7835 15:47:56 RM 7857 - jsherril@redhat.com - NEW - 502 proxy error when trying to yum install a package 15:47:57 https://pulp.plan.io/issues/7857 15:48:10 I don't have any info to give that user 15:48:19 can we go back 15:48:31 bmbouter: i agree they are not related 15:49:05 could this be related to django-cleanup? 15:49:27 for 7796? 15:49:28 I don't know if it is possible to go back 15:49:39 I think so daviddavis 15:49:41 no 7857 15:49:45 it's at least a good guess 15:49:58 yea 15:50:28 re 7867, I think I will just link the user to https://docs.mongodb.com/manual/tutorial/transparent-huge-pages/ 15:50:45 when using mongo they should not have that system option enabled 15:50:55 also I'll encourage them to switch to pulp3 15:51:08 we might have this recommendation in our docs 15:51:08 +1 15:51:13 we do 15:51:17 I'll link there too 15:51:29 I'll leave it open tho for 1 more week or close it? 15:51:49 actually I recommend close it, they can repost if necessary, pulp2 is maint mode 15:51:55 bmbouter: close 15:51:56 bmbouter, https://docs.pulpproject.org/en/2.21/user-guide/installation/index.html?highlight=huge 15:52:05 +1 15:53:14 for 7857, should I accpet and add to sprint? 15:54:34 I think we can only encourage to upgrade and see if the issue stays 15:54:35 #idea Proposed for #7857: Leave the issue as-is, accepting its current state. 15:54:35 !propose accept 15:54:35 fao89: Proposed for #7857: Leave the issue as-is, accepting its current state. 15:54:53 what is the resolution do we think for 7857 15:54:53 how far did we backport django-cleanup? 15:55:03 great question! 15:55:08 you always have those great questions 15:55:09 I think all the way 15:55:34 so I'd suggest to upgrade katello to the latest 3.6 15:55:38 ttereshc++ 15:55:38 fao89: ttereshc's karma is now 255 15:55:42 :) 15:55:58 i think have backported to 3.7.3 15:56:38 #idea Proposed for #7857: accept and add to sprint 15:56:38 !propose other accept and add to sprint 15:56:39 fao89: Proposed for #7857: accept and add to sprint 15:57:04 yes django-cleanup was never in 3.6 15:57:11 so this predates django-cleanup 15:57:18 django-cleanup was added in 3.7 15:57:44 ok, so then the reason is different :/ 15:58:06 yup 15:58:27 from comment 1 it sounds like they upgraded 15:58:30 from pulp 2 15:58:39 err no wait 15:58:50 they have pulp-server-2.21.4 15:58:56 but it's maybe not an upgrade 15:59:26 we have 6 more issues to triage 15:59:28 daviddavis, so rpm stuff is pulp 2 here, docker and file are pulp3 15:59:28 we can accept but I'm stumped as to how to approach this 15:59:51 i don't know how we'd reproduce this 16:00:04 i feel like there is not enough info 16:00:09 so is this a mixed environment? 16:00:12 pulp2/pulp3 16:00:21 so they can have only fresh install for rpm and not migration 16:00:26 I guess that's how katello runs maybe? 16:00:49 I believe so 16:01:15 yeah 16:01:30 ack 16:01:38 ipanova, I agree there is not enough info to reproduce but I'mnot sure what to ask for 16:02:18 yeah 16:02:53 is there a change they have upgraded from 3.6 to a version in between that had django-cleanup 16:03:02 s/change/chance 16:03:08 #idea Proposed for #7857: skip, ttereshc will ask jsherrill about the setup 16:03:08 !propose other skip, ttereshc will ask jsherrill about the setup 16:03:08 ttereshc: Proposed for #7857: skip, ttereshc will ask jsherrill about the setup 16:03:20 #agreed skip, ttereshc will ask jsherrill about the setup 16:03:20 !accept 16:03:20 fao89: Current proposal accepted: skip, ttereshc will ask jsherrill about the setup 16:03:21 fao89: 6 issues left to triage: 7856, 7854, 7853, 7848, 7845, 7835 16:03:22 #topic https://pulp.plan.io/issues/7856 16:03:22 RM 7856 - ggainey - NEW - FIPS: Plugin-author doc needs to talk about ALLOWED_CHECKSUMS and Artifact.DIGEST_FIELDS 16:03:24 https://pulp.plan.io/issues/7856 16:03:44 convert to task? 16:03:50 +1 16:03:54 #idea Proposed for #7856: convert to task 16:03:54 !propose other convert to task 16:03:54 fao89: Proposed for #7856: convert to task 16:04:03 #agreed convert to task 16:04:03 !accept 16:04:03 fao89: Current proposal accepted: convert to task 16:04:03 #topic https://pulp.plan.io/issues/7854 16:04:04 fao89: 5 issues left to triage: 7854, 7853, 7848, 7845, 7835 16:04:05 RM 7854 - ggainey - NEW - FIPS: QueryExistingArtifacts stage needs to enforce ALLOWED_ALGORITHMS 16:04:06 https://pulp.plan.io/issues/7854 16:04:22 task? 16:04:39 #idea Proposed for #7854: convert to task 16:04:39 !propose other convert to task 16:04:39 fao89: Proposed for #7854: convert to task 16:04:57 #agreed convert to task 16:04:57 !accept 16:04:57 fao89: Current proposal accepted: convert to task 16:04:58 #topic https://pulp.plan.io/issues/7853 16:04:58 fao89: 4 issues left to triage: 7853, 7848, 7845, 7835 16:04:59 RM 7853 - ggainey - NEW - FIPS: downloader needs to notice if expected-digest-algorithm is FORBIDDEN 16:05:00 https://pulp.plan.io/issues/7853 16:05:42 #idea Proposed for #7853: convert to task 16:05:42 !propose other convert to task 16:05:42 fao89: Proposed for #7853: convert to task 16:06:03 no preference, can be both task or issue, imo 16:06:10 depends how to look at it :) 16:06:32 me too 16:06:47 yep 16:07:01 he put a hackmd, I think he wanted to discuss 16:07:29 the hackmd was from the discussion we already had 16:07:44 #agreed convert to task 16:07:44 !accept 16:07:44 fao89: Current proposal accepted: convert to task 16:07:44 #topic https://pulp.plan.io/issues/7848 16:07:45 fao89: 3 issues left to triage: 7848, 7845, 7835 16:07:46 RM 7848 - daviddavis - NEW - credentials.enc file not being used 16:07:47 https://pulp.plan.io/issues/7848 16:08:16 let's skip. I think we'll end up deleting .travis directories eventually and this will go away. 16:08:16 hmmm 16:08:25 daviddavis: +1 16:08:25 that sounds good 16:08:29 !skip 16:08:30 #topic https://pulp.plan.io/issues/7845 16:08:30 fao89: 2 issues left to triage: 7845, 7835 16:08:31 +1 16:08:32 RM 7845 - mdellweg - ASSIGNED - The view_set parameter on AccessPolicy is not namespacing plugins 16:08:33 https://pulp.plan.io/issues/7845 16:08:58 #idea Proposed for #7845: accept and add to sprint 16:08:58 !propose other accept and add to sprint 16:08:58 fao89: Proposed for #7845: accept and add to sprint 16:09:17 +1 16:09:18 is it a plugin api change/feature? 16:09:26 or is it a bugfix? 16:09:45 a bit of both? 16:10:01 it could certainly lead to bugs 16:10:16 we don't have this flavour yet :) 16:10:22 ok, +1 to accept 16:10:26 x9c4: didn't this get addressed i nyour PR? 16:10:53 https://github.com/pulp/pulpcore/pull/1029 16:11:14 that pr is for this issue 16:11:28 or at least it has CHANGES/7845.bugfix 16:11:30 Yes. but pulpbot didn't revisit the pr. 16:11:43 pro tip: if you delete the commit, it will revisit the pr 16:11:59 s/commit/comment/ 16:12:01 #agreed accept and add to sprint 16:12:01 !accept 16:12:01 fao89: Current proposal accepted: accept and add to sprint 16:12:02 #topic https://pulp.plan.io/issues/7835 16:12:02 fao89: 1 issues left to triage: 7835 16:12:03 RM 7835 - ipanova@redhat.com - NEW - Repair feature should remove unrepairable corrupted file 16:12:04 https://pulp.plan.io/issues/7835 16:12:22 I think this is right 16:12:31 it's a story 16:12:37 agreed 16:12:40 daviddavis++ 16:12:40 x9c4: daviddavis's karma is now 399 16:12:45 #idea Proposed for #7835: convert to story 16:12:45 !propose other convert to story 16:12:45 fao89: Proposed for #7835: convert to story 16:12:59 +1 16:13:02 #agreed convert to story 16:13:02 !accept 16:13:02 fao89: Current proposal accepted: convert to story 16:13:03 fao89: No issues to triage. 16:13:19 +1 16:13:28 #endmeeting 16:13:28 !end