Skip to content

Remove aaS entry from software listOD#1881

Merged
roidelapluie merged 1 commit intomasterfrom
richih/2
Feb 22, 2021
Merged

Remove aaS entry from software listOD#1881
roidelapluie merged 1 commit intomasterfrom
richih/2

Conversation

@RichiH
Copy link
Member

@RichiH RichiH commented Feb 22, 2021

Signed-off-by: Richard Hartmann richih@richih.org

@RichiH RichiH requested a review from brancz February 22, 2021 07:58
Copy link
Member

@roidelapluie roidelapluie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am blocking this to make sure we first notice them about this and explain them why we do this. I think it should also be part of the commit message.

This is a list of software projects and products implemented our standard, not offers based on said software.

Initial internal discussion on if we want to carry a list of offerings has started, but not gotten anywhere yet.

Brian remembers digging to see if it was based on Cortex and merging because it was not obviously based on software listed already. As such, this commits reverts a mistake.

Signed-off-by: Richard Hartmann <richih@richih.org>
@roidelapluie
Copy link
Member

What about newrelic?

@RichiH
Copy link
Member Author

RichiH commented Feb 22, 2021

I disagree that we are required to keep third parties informed about changes in all cases, but we can do so. Do you have direct contact information? CC @ComradeAkko in any case.

New Relic is a distinct piece of software. That there is only one offering by a commercial vendor stands to reason. What would your suggestion be?

@roidelapluie
Copy link
Member

The split could be : software you install vs software you consume.

@RichiH
Copy link
Member Author

RichiH commented Feb 22, 2021

That change would be out of scope for the current PR, though. My main intention is to make the list correct as it is defined right now.

@roidelapluie
Copy link
Member

LGTM, and we have now pinged them via github.

@RichiH RichiH removed the request for review from brancz February 22, 2021 08:39
@roidelapluie roidelapluie merged commit 319b49f into master Feb 22, 2021
@roidelapluie roidelapluie deleted the richih/2 branch February 22, 2021 08:39
@roidelapluie
Copy link
Member

That change would be out of scope for the current PR, though. My main intention is to make the list correct as it is defined right now.

From that perspective that pull request is unambihuous and non controversial. Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants