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

added nat to docker builds #998

Merged
merged 17 commits into from
Mar 20, 2023
Merged

added nat to docker builds #998

merged 17 commits into from
Mar 20, 2023

Conversation

shahnawaz-creator
Copy link
Collaborator

Changes

Fixes

Tests

Tasks to complete before merging PR:

  • Ensure system tests are passing. If not Run them manually to check for any regressions 📋
  • Do any new system tests need added to test this change? do any existing system tests need updated? If so create a PR at 0chain/system_test
  • Merge your system tests PR to master AFTER merging this PR

Associated PRs (Link as appropriate):

  • 0chain:
  • gosdk:
  • system_test:
  • zboxcli:
  • zwalletcli:
  • Other: ...

@codecov-commenter
Copy link

codecov-commenter commented Mar 18, 2023

Codecov Report

Merging #998 (998781e) into swagger (67b5871) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff            @@
##           swagger     #998   +/-   ##
========================================
  Coverage    27.74%   27.74%           
========================================
  Files           77       77           
  Lines         8089     8089           
========================================
  Hits          2244     2244           
  Misses        5554     5554           
  Partials       291      291           
Flag Coverage Δ
Unit-Tests 27.74% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@peterlimg
Copy link
Member

@shahnawaz-creator blobber building process seems got stuck

@boddumanohar boddumanohar merged commit ed73837 into swagger Mar 20, 2023
@boddumanohar boddumanohar deleted the fix/docker-nat branch March 20, 2023 09:06
dabasov pushed a commit that referenced this pull request Mar 23, 2023
* init swagger

* swagger doc for rename handler

* add new endpoint

* Check swagger generation

* swagger generation

* Add swagger endpoints

* update Dockerfile for blobber_base

* remove /swagger.yaml registration

* create a new mux instead of the default mux

* clean up

* setup handler

* added nat to docker builds (#998)

added nat to docker builds

* added host network (#1005)

---------

Co-authored-by: shahnawaz-creator <117025384+shahnawaz-creator@users.noreply.github.com>
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.

4 participants