Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update(proposal): clarify that old drivers are not removed anymore #1738

Merged
merged 1 commit into from
Sep 29, 2021

Conversation

leodido
Copy link
Member

@leodido leodido commented Sep 28, 2021

Signed-off-by: Leonardo Di Donato leodidonato@gmail.com

What type of PR is this?

/kind cleanup

Any specific area of the project related to this PR?

/area proposals

What this PR does / why we need it:

Clarify that the drivers/cleanup job we needed when we were using Bintray to store the Falco drivers is not needed anymore. And that, its related proposal has been superseded by a newer one (already implemented and in place during the last year).

Which issue(s) this PR fixes:

NONE

Special notes for your reviewer:

/milestone 0.30.0

Does this PR introduce a user-facing change?:

docs: clarify that previous Falco drivers will remain available at https://download.falco.org and no automated cleanup is run anymore

Signed-off-by: Leonardo Di Donato <leodidonato@gmail.com>
@poiana
Copy link

poiana commented Sep 28, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: leodido

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link
Member

@leogr leogr left a comment

Choose a reason for hiding this comment

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

Thanks for the clarification.

@poiana poiana added the lgtm label Sep 28, 2021
@araujof araujof mentioned this pull request Sep 28, 2021
13 tasks
@krisnova
Copy link
Contributor

krisnova commented Sep 28, 2021

Can you please add more detail about what the expectation is to a user?

Will Falco break? If so when should we expect it to break after this change? What will we expect Falco to do if/when it breaks? Does Falco now have a log line that will reference this change that is printed to stdout or stderr once the break has occured?

I do not have any issues with the change, however I do not think this does a good job at explaining what might go wrong after this change takes place. Maybe I am missing this documentation somewhere else? My main concern is that people will be affected by this, and we will see GitHub issues like "Falco unable to find driver on {os} and {kernel}" and it will take one of us a large amount of time to connect these new issues back to this change.

We should do what we can to make our end-users as self sufficient as possible, so that they do not need a maintainer to help them understand why Falco stopped working.

@leodido
Copy link
Member Author

leodido commented Sep 28, 2021

Can you please add more detail about what the expectation is to a user?

Will Falco break? If so when should we expect it to break after this change? What will we expect Falco to do if/when it breaks? Does Falco now have a log line that will reference this change that is printed to stdout or stderr once the break has occured?

I do not have any issues with the change, however I do not think this does a good job at explaining what might go wrong after this change takes place. Maybe I am missing this documentation somewhere else? My main concern is that people will be affected by this, and we will see GitHub issues like "Falco unable to find driver on {os} and {kernel}" and it will take one of us a large amount of time to connect these new issues back to this change.

We should do what we can to make our end-users as self sufficient as possible, so that they do not need a maintainer to help them understand why Falco stopped working.

These edits just reflects changes in place for all the past year, Kris.

In fact, it is just a documentation change about the fact that we store drivers at https://download.falco.org and we do not need anymore to delete the previous Falco drivers (like we had to do in the past) because we are free from storage limits and constraints.

There is nothing that can go wrong since this just updates an almost 2 years old proposal for reflecting the new process that has been in place for 6 releases already.

@leogr
Copy link
Member

leogr commented Sep 28, 2021

Can you please add more detail about what the expectation is to a user?

Nothing will change. This is just a further clarification of what is already in place.

Will Falco break? If so when should we expect it to break after this change? What will we expect Falco to do if/when it breaks? Does Falco now have a log line that will reference this change that is printed to stdout or stderr once the break has occurred?

No, Falco will not break, because nothing is changing here.

I do not have any issues with the change, however I do not think this does a good job at explaining what might go wrong after this change takes place. Maybe I am missing this documentation somewhere else? My main concern is that people will be affected by this, and we will see GitHub issues like "Falco unable to find driver on {os} and {kernel}" and it will take one of us a large amount of time to connect these new issues back to this change.

This PR clarifies more that we are keeping all previous prebuilts on S3, available for download at https://download.falco.org/. So we should not see that kind of issues :)

We should do what we can to make our end-users as self sufficient as possible, so that they do not need a maintainer to help them understand why Falco stopped working.

That's exactly what we did when decided to do NOT delete drivers from S3 anymore. So, again I just want to ensure you that this PR it's just a documentation change to clarify that we are already going to the right direction :)

@poiana poiana merged commit 3414ca5 into master Sep 29, 2021
@poiana poiana deleted the leo/proposal-cleanup-superseeded branch September 29, 2021 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants