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

fix/Simplify docker images #2446

Merged
merged 2 commits into from
Jul 19, 2023

Conversation

carpawell
Copy link
Member

Closes #2379. Tested with nspcc-dev/neofs-dev-env#272.

Keep only neofs-related or necessary files (SSL certs for that commit).
Closes nspcc-dev#2379.

Signed-off-by: Pavel Karpy <[email protected]>
@codecov
Copy link

codecov bot commented Jul 18, 2023

Codecov Report

Merging #2446 (de73238) into master (e5ac376) will increase coverage by 0.00%.
The diff coverage is n/a.

❗ Current head de73238 differs from pull request most recent head 2cb41e1. Consider uploading reports for the commit 2cb41e1 to get more accurate results

@@           Coverage Diff           @@
##           master    #2446   +/-   ##
=======================================
  Coverage   29.48%   29.49%           
=======================================
  Files         399      399           
  Lines       30268    30268           
=======================================
+ Hits         8925     8927    +2     
+ Misses      20602    20600    -2     
  Partials      741      741           

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

It was denied in nspcc-dev#654 but is useful in general and for docker healthcheck in
particular.

Signed-off-by: Pavel Karpy <[email protected]>
@roman-khimov roman-khimov merged commit 354fddd into nspcc-dev:master Jul 19, 2023
7 of 8 checks passed
@carpawell carpawell deleted the fix/simplify-docker-images branch July 19, 2023 14:10
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.

docker is intendent to run app not OS
4 participants