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

enable dependabot for opensearch repositories #664

Closed
rursprung opened this issue May 6, 2021 · 8 comments
Closed

enable dependabot for opensearch repositories #664

rursprung opened this issue May 6, 2021 · 8 comments
Assignees
Labels
enhancement Enhancement or improvement to existing feature or request

Comments

@rursprung
Copy link
Contributor

Is your feature request related to a problem? Please describe.
i noticed that dependency updates seem to be done manually (see e.g. #661). this means that there's no guarantee that all dependencies are up-to-date and do not contain any known vulnerabilities.
some users (esp. corporate users) will run 3rd party artifact scanning on all software which they run/ship and will block it if it contains known vulnerabilities (regardless of whether there's a known way to exploit it with this software or it's just part of a library which is being pulled in but that part isn't even being used).
keeping dependencies updated with an automated process will prevent unnecessary last-minute manual updates.

Describe the solution you'd like
github has a feature called dependabot which automatically keeps track of the dependencies in a repository and creates pull request to update the dependencies when newer versions become available.

note: this doesn't just apply to this repo here but instead to all opensearch-project repos.

Describe alternatives you've considered
manual updates don't scale (things are being overlooked, etc.).
of course dependabot (and any other such feature) only works if dependencies make use of semver.
for major releases of dependencies the change must still be done manually since more adaptations will be needed (but who knows, maybe one day this can be automated away as well? 😃)

Additional context
see the dependabot documentation

@rursprung rursprung added the enhancement Enhancement or improvement to existing feature or request label May 6, 2021
@CEHENKLE
Copy link
Member

CEHENKLE commented May 6, 2021

This is a really good point. Let me turn it around and ask -- is there any reason why we wouldn't turn on updating? @hyandell?

@hyandell
Copy link
Member

hyandell commented May 6, 2021

Please turn them on :)

@rursprung
Copy link
Contributor Author

i think this needs to be looked at first by somebody with access to the repository settings.
based on the documentation this should be enabled by default if the settings are correct: https://docs.github.com/en/code-security/supply-chain-security/configuring-dependabot-security-updates#supported-repositories

GitHub automatically enables Dependabot security updates for every repository that meets these prerequisites.
[..]

it can still be enabled manually by providing a .github/dependabot.yml file, but it seems that this shouldn't be necessary

@dblock
Copy link
Member

dblock commented May 7, 2021

I assigned this to @CEHENKLE, much thanks!

@CEHENKLE
Copy link
Member

Thanks @rursprung again for the callout. We've enabled dependency graph, dependabot alerts and dependabot security updates on all OpenSearch repos at the org level, This should impact all current repos, as well as all new repos created.

I'm not 100% sure how it'll work for a complex multi-project gradle build without some more work, but it gives us a target to hit.

Thanks again,
/C

@rursprung
Copy link
Contributor Author

@CEHENKLE: are you sure that this worked? i haven't seen a single PR from dependabot being opened on this repo and can't imagine that there wouldn't be any dependency which it'd want to update (unless you're all super fast in updating to the latest versions of all dependencies at all times when they come out?)

@CEHENKLE
Copy link
Member

@rursprung Hm, that's weird. Let me take a look at that today. Thanks for the ping!

@setiah
Copy link
Contributor

setiah commented Jul 28, 2021

Dependabot is enabled on OpenSearch and Dashboards repos and I could see dependabot based alerts and PRs on OpenSearch-Dashboards repository (see here), however, I don't know yet why we don't have such alerts on OpenSearch.

I was thinking it could be due to missing dependabot configuration file, as described here - "You enable Dependabot version updates by checking a configuration file into your repository." (reference) but then I didn't see any such configuration file in Dashboards repo as well. Will try to dig more on this later

ritty27 pushed a commit to ritty27/OpenSearch that referenced this issue May 12, 2024
…t#664)

* Adding workflow to increment version after release on the 2.x branch

Signed-off-by: Vacha Shah <vachshah@amazon.com>

* Added CHANGELOG

Signed-off-by: Vacha Shah <vachshah@amazon.com>

---------

Signed-off-by: Vacha Shah <vachshah@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement or improvement to existing feature or request
Projects
None yet
Development

No branches or pull requests

5 participants