Skip to content

Conversation

@zaibon
Copy link
Contributor

@zaibon zaibon commented Oct 9, 2019

No description provided.

@zaibon zaibon force-pushed the container_environment branch from 3b8866d to 3ee6ecf Compare October 9, 2019 12:49
@zaibon zaibon changed the title contd: read environment variable from .stratup.toml if it exists contd: read environment variable from .stratup.toml if it exists and ensure /etc/resolv.conf Oct 9, 2019
@codecov
Copy link

codecov bot commented Oct 9, 2019

Codecov Report

Merging #318 into master will decrease coverage by 2.18%.
The diff coverage is 38.33%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #318      +/-   ##
=========================================
- Coverage   41.59%   39.4%   -2.19%     
=========================================
  Files          54      57       +3     
  Lines        3284    3522     +238     
=========================================
+ Hits         1366    1388      +22     
- Misses       1733    1952     +219     
+ Partials      185     182       -3
Impacted Files Coverage Δ
pkg/container/opts.go 0% <ø> (ø)
pkg/container/container.go 0% <0%> (ø)
pkg/container/startup.go 95.83% <95.83%> (ø)
pkg/flist/flist.go 56.35% <0%> (+1.56%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3988a7c...1e452ee. Read the comment docs.

@zaibon zaibon force-pushed the container_environment branch from 1da0675 to f6b5b63 Compare October 9, 2019 14:03
@zaibon zaibon force-pushed the container_environment branch from 9c3f5d1 to 94b0fea Compare October 15, 2019 15:01
@zaibon zaibon requested a review from muhamadazmy October 15, 2019 15:48
@zaibon zaibon force-pushed the container_environment branch 3 times, most recently from 4fbe91b to 3c555e6 Compare October 15, 2019 18:57
Copy link
Member

@muhamadazmy muhamadazmy left a comment

Choose a reason for hiding this comment

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

More few comments

@zaibon zaibon force-pushed the container_environment branch from 3c555e6 to e1b8df9 Compare October 17, 2019 09:58
@zaibon zaibon requested a review from muhamadazmy October 17, 2019 09:58
fixes #319

we extract 3 things from statup.toml:
- entrypoint
- working dir
- enviroment variable
@zaibon zaibon force-pushed the container_environment branch from e1b8df9 to 1e452ee Compare October 17, 2019 12:31
@zaibon zaibon merged commit 58b9940 into master Oct 17, 2019
@zaibon zaibon deleted the container_environment branch October 17, 2019 12:41
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