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

build: update build image to 98a1276049a3cf4bebc6a0343217444a8675baf8 #760

Closed
wants to merge 1 commit into from

Conversation

mattklein123
Copy link
Member

No description provided.

@mattklein123
Copy link
Member Author

@htuch

@htuch
Copy link
Member

htuch commented Apr 13, 2017

LGTM, let me know if there are any issues.

@mattklein123
Copy link
Member Author

@htuch looks like coverage is failing: https://travis-ci.org/lyft/envoy/jobs/221897553

@htuch
Copy link
Member

htuch commented Apr 13, 2017

Will look into this early tomorrow if that's OK.

@mattklein123
Copy link
Member Author

Yup no problem.

htuch added a commit to htuch/envoy that referenced this pull request Apr 14, 2017
After envoyproxy#747, we no longer have the BUILD_DISTINCT=1 set of dependencies in the CI image, so fix the
gcovr path to solve the failures in envoyproxy#760.
htuch added a commit to htuch/envoy that referenced this pull request Apr 14, 2017
After envoyproxy#747, we no longer have the BUILD_DISTINCT=1 set of dependencies in the CI image, so fix the
gcovr path to solve the failures in envoyproxy#760.

Also fixed a snafu in configs/configgen.sh that I hit when running Docker as non-root. It was trying
to peek at the user's home directory, with no passwd entry.
@mattklein123
Copy link
Member Author

Closing in favor of #769

@mattklein123 mattklein123 deleted the update_build_image branch April 14, 2017 23:53
mattklein123 pushed a commit that referenced this pull request Apr 17, 2017
After #747, we no longer have the BUILD_DISTINCT=1 set of dependencies in the CI image, so fix the
gcovr path to solve the failures in #760.

Also fixed a snafu in configs/configgen.sh that I hit when running Docker as non-root. It was trying
to peek at the user's home directory, with no passwd entry.
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Description: in preparation of cutting v0.3 I went ahead and audited inline TODOs.

Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Description: in preparation of cutting v0.3 I went ahead and audited inline TODOs.

Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.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.

2 participants