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

Dockerfile change because error: E: The method driver /usr/lib/apt/methods/https… #6258

Merged
merged 2 commits into from
Nov 3, 2018

Conversation

kk580kk
Copy link
Contributor

@kk580kk kk580kk commented Nov 1, 2018

issues: #6256

Got error

E: The method driver /usr/lib/apt/methods/https could not be found.
E: Failed to fetch https://dl.yarnpkg.com/debian/dists/stable/InRelease                 
E: Some index files failed to download. They have been ignored, or old ones used instead. 

when without apt-transport-https
and got warning

debconf: delaying package configuration, since apt-utils is not installed

without apt-utils

… could not be found.

 要提交的变更:
	修改:     contrib/docker/Dockerfile
@codecov-io
Copy link

codecov-io commented Nov 1, 2018

Codecov Report

Merging #6258 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6258      +/-   ##
==========================================
- Coverage   77.02%   77.02%   -0.01%     
==========================================
  Files          64       64              
  Lines        9500     9504       +4     
==========================================
+ Hits         7317     7320       +3     
- Misses       2183     2184       +1
Impacted Files Coverage Δ
superset/utils/core.py 88.12% <0%> (-0.18%) ⬇️
superset/views/core.py 73.93% <0%> (+0.07%) ⬆️

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 59109f6...666a347. Read the comment docs.

@mistercrunch
Copy link
Member

Do we know what actually requires these deps? If so, can we add a comment above the line you added explaining why this is required?

@kk580kk
Copy link
Contributor Author

kk580kk commented Nov 2, 2018

@mistercrunch I see , i will add comment above the line

@kk580kk kk580kk changed the title change because error: E: The method driver /usr/lib/apt/methods/https… Dockerfile change because error: E: The method driver /usr/lib/apt/methods/https… Nov 2, 2018
@mistercrunch mistercrunch merged commit fb13218 into apache:master Nov 3, 2018
bipinsoniguavus pushed a commit to ThalesGroup/incubator-superset that referenced this pull request Dec 26, 2018
…thods/https… (apache#6258)

* change because error: E: The method driver /usr/lib/apt/methods/https could not be found.
 要提交的变更:
	修改:     contrib/docker/Dockerfile

* 	修改:     contrib/docker/Dockerfile
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.34.0 labels Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.34.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants