[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Ping: Regressed XSA-286, was [xen-unstable test] 161917: regressions - FAIL


  • To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Mon, 28 Jun 2021 14:35:11 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NqO9UDVuUossHo7M/+agqwQGG+tXBLJ5Zyyux978lLo=; b=V+CLOLD+HNLFbtdZP7FOE+nmSFW2oeLoTHAYBxTsh+FCFKU8orXZz2XidVjhuGZqyUaOOCJDt5di320Xa4nWNUKzyPV2YosesnamvoBHGQCtD83mgrllTuLRMfr2JviGu5APf2WZk3C3Gsj+1nFZ9SxBYefuyqeNVc63BcwQyjfbkCfQAnDIfkGxQbzVgXlpcGVx4fCJFM1QpwQj+iZi+u4jlzSgOaPDPXBQKCRx5ElrfCValBZTuN+fSXidXnzPTG7E/xb+tAEuJtRDfain1WnGaCIiOXMd3oD2hFiU2OWHIq18H06xVLe7d8yiVQLilb/ah/8ZCIf75vjZ5wmATg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=oDrjMA4WhEUHiIYJkmajRHgmMKHUevW5HCXM0q9wfRnhq8XGYfwmWO7cRRXPjxCbeD3Kla0YqAk13hvR7Sy8mfswodTd9kwTgtYJpN7Il4CrBjRcgtOxXiJFkzCj20zymdQg9qVCEcbARQ4+uPrBdrk8I6QSf1127olUEtGG2rpT8YHTao53yFeUGNf0nS/4pTTAG8+Byt1POHz7nYVj/Mg7e3GjWlzkHv1haqJVLJdCQbFqq4w3knXtE0lYCNK+HNBt8V6dC3x60FwivV/tqWT9HnX4n2eiepG8vDjCJohU4VVpeHujntb0UcEQslWHOXvIRw4dfx6CiZwJBeiVMA==
  • Authentication-results: xenproject.org; dkim=none (message not signed) header.d=none;xenproject.org; dmarc=none action=none header.from=suse.com;
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Roger Pau Monné <roger.pau@xxxxxxxxxx>, "committers@xxxxxxxxxxxxxx" <committers@xxxxxxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>
  • Delivery-date: Mon, 28 Jun 2021 12:35:26 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 17.06.2021 15:05, Ian Jackson wrote:
> On to process:
> 
> Jan Beulich writes ("Re: Regressed XSA-286, was [xen-unstable test] 161917: 
> regressions - FAIL"):
>> On 16.06.2021 17:43, Andrew Cooper wrote:
>>> I am very irritated that you have *twice* recently introduced security
>>> vulnerabilities by bypassing my reviews/objections on patches.
>>
>> I'm sorry, Andrew, but already in my original reply a month ago I did
>> express that I couldn't find any record of you having objected to the
>> changes. It doesn't help that you claim you've objected when you
>> really didn't (which is the impression I get from not finding anything,
>> and which also matches my recollection of what was discussed).
> 
> Andrew, can you provide references to your objections ?
> 
>> I don't think I know which 2nd instance you're referring to, and hence
>> I can't respond to that aspect.
> 
> And, likewise, references for this.
> 
>>> In the case of this revert specifically, I did get agreement on IRC
>>> before reverting.
>>
>> How can I know you did? You didn't even care to reply to my mail from
>> a month ago. And there was no reason to make an emergency out of this
>> and ask on irc. You could have sent mail just like is done for all
>> other normal bug fixes etc. Iirc I was on PTO at that time; it would
>> hence only have been fair to wait until my return.
> 
> I think it would be good practice to copy and paste relevant IRC
> discussions into email in this kind of situation.  That email also
> makes space to properly write down what you are doing, that you
> realise it is controversial, who you have consulted, and why you are
> going ahead.
> 
> I looked at one of the two disputed reverts in Xen,
> cb199cc7de987cfda4659fccf51059f210f6ad34, and it does not have any
> tags indicating approval by anyone else.
> 
> Andy, if you got agreement on IRC, who from ? [1]
> 
> Ian.
> 
> [1] This may well have included me.  I do not reliably record this
> kind of information in my wetware.  That is what we have computers
> for.

Another 11 days have passed without a reply to any of the questions
above. I find it generally inappropriate to try to have controversies
die out by simply not replying, but in a case like this one it is imo
extra bad to do so. In case it hasn't come through clearly before: My
primary goal is not to revert your revert. Instead I'd like want to be
given proper reasons, so I can fully understand parts I may have been
missing so far. But of course I also expect you to correct your views
in case the technical details speak against your original reasoning
(at which point undoing your change may indeed be the necessary
consequence).

And of course all technical aspects aside there remains the process
aspect of this whole situation.

Jan




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.