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

Fix build issues #286

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Fix build issues #286

wants to merge 18 commits into from

Conversation

movence
Copy link
Contributor

@movence movence commented Jan 24, 2025

Revs:

  • Move operator build to Github runner by syncing Makefile and Dockerfile with the upstream
  • Drop all hacky changes since they are not needed anymore

Build and deploy test run: https://github.com/aws/amazon-cloudwatch-agent-operator/actions/runs/13055448538

--

  • Use ubuntu:latest for TargetAllocator certificates to make it consistent with CWA
  • Drop date change for TargetAllocator

Build is failing due to expired certificate with opencensus and not being able to fetch alpine for certs

Example error:

// cert
#12 [builder  5/12] RUN go mod download
#12 233.1 go: [email protected]: unrecognized import path "go.opencensus.io": https fetch: Get "https://go.opencensus.io/?go-get=1": tls: failed to verify certificate: x509: certificate has expired or is not yet valid: current time 2025-01-23T20:15:07Z is after 2025-01-21T03:43:04Z

// fetching alpine for certs
 > [linux/amd64 certificates 2/2] RUN apk --no-cache add ca-certificates:
0.096 fetch https://dl-cdn.alpinelinux.org/alpine/v3.18/main/x86_64/APKINDEX.tar.gz
0.161 485BF0EAC27F0000:error:0A000086:SSL routines:tls_post_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1889:

Description of changes: Original changes that are not needed anymore

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@lisguo
Copy link
Contributor

lisguo commented Jan 28, 2025

Seems like dotnet integ test is failing?

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