Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Resolve SSL issues in containers - #48042

Merged
evict merged 1 commit into
mainfrom
vincent/resolve-ssl-issues-base
Feb 22, 2023
Merged

Resolve SSL issues in containers#48042
evict merged 1 commit into
mainfrom
vincent/resolve-ssl-issues-base

Conversation

@evict

@evict evict commented Feb 22, 2023

Copy link
Copy Markdown
Contributor

Replace the old base image with the new updated one. Resolving the SSL issues in new container builds.

Test plan

  • image builds

@cla-bot cla-bot Bot added the cla-signed label Feb 22, 2023
@evict evict changed the title Resolve ssl issues base part 2 Resolve SSL issues in containers Feb 22, 2023
@evict
evict requested review from a team and eseliger February 22, 2023 14:15
@evict evict self-assigned this Feb 22, 2023
@evict evict added the SSDLC label Feb 22, 2023
@evict
evict force-pushed the vincent/resolve-ssl-issues-base branch from 096f635 to 127fb3b Compare February 22, 2023 14:15
@sourcegraph-bot

sourcegraph-bot commented Feb 22, 2023

Copy link
Copy Markdown
Contributor

Codenotify: Notifying subscribers in CODENOTIFY files for diff da6d971...127fb3b.

Notify File(s)
@bobheadxi docker-images/grafana/Dockerfile
docker-images/prometheus/Dockerfile
@efritz cmd/worker/Dockerfile
enterprise/cmd/executor/docker-image/Dockerfile
enterprise/cmd/precise-code-intel-worker/Dockerfile
enterprise/cmd/worker/Dockerfile
@indradhanush cmd/repo-updater/Dockerfile
@keegancsmith cmd/searcher/Dockerfile
cmd/symbols/Dockerfile
@sashaostrikov cmd/repo-updater/Dockerfile
@slimsag cmd/blobstore/Dockerfile

@sourcegraph-bot

sourcegraph-bot commented Feb 22, 2023

Copy link
Copy Markdown
Contributor

Codenotify: Notifying subscribers in OWNERS files for diff da6d971...127fb3b.

Notify File(s)
@mrnugget dev/sg/Dockerfile
@sourcegraph/dev-experience dev/sg/Dockerfile

@evict

evict commented Feb 22, 2023

Copy link
Copy Markdown
Contributor Author
$ trivy image --severity 'HIGH,CRITICAL' us.gcr.io/sourcegraph-dev/gitserver:127fb3bc19fcbfd88c88cd56115cadec28b6f534_200779_candidate                                                                        (main)
2023-02-22T14:32:45.904Z	INFO	Vulnerability scanning is enabled
2023-02-22T14:32:45.904Z	INFO	Secret scanning is enabled
2023-02-22T14:32:45.904Z	INFO	If your scanning is slow, please try '--security-checks vuln' to disable secret scanning
2023-02-22T14:32:45.904Z	INFO	Please see also https://aquasecurity.github.io/trivy/v0.36/docs/secret/scanning/#recommendation for faster secret detection
2023-02-22T14:32:48.717Z	INFO	Detected OS: alpine
2023-02-22T14:32:48.717Z	INFO	Detecting Alpine vulnerabilities...
2023-02-22T14:32:48.722Z	INFO	Number of language-specific files: 1
2023-02-22T14:32:48.722Z	INFO	Detecting gobinary vulnerabilities...

us.gcr.io/sourcegraph-dev/gitserver:127fb3bc19fcbfd88c88cd56115cadec28b6f534_200779_candidate (alpine 3.14.8)

Total: 0 (HIGH: 0, CRITICAL: 0)

@evict
evict merged commit e4fe8cf into main Feb 22, 2023
@evict
evict deleted the vincent/resolve-ssl-issues-base branch February 22, 2023 14:34
github-actions Bot pushed a commit that referenced this pull request Feb 22, 2023
Replace the old base image with the new updated one. Resolving the SSL
issues in new container builds.

(cherry picked from commit e4fe8cf)
coury-clark pushed a commit that referenced this pull request Feb 22, 2023
Backport e4fe8cf from #48042.

Co-authored-by: Vincent <evict@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants