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

[bugfix] gcc environment for ARM #2334

Merged
merged 1 commit into from Feb 15, 2024
Merged

Conversation

kpango
Copy link
Collaborator

@kpango kpango commented Feb 8, 2024

Description

Related Issue

Versions

  • Go Version: 1.22.0
  • Docker Version: 20.10.8
  • Kubernetes Version: v1.29.1
  • NGT Version: 2.1.6

Checklist

Special notes for your reviewer

@vdaas-ci
Copy link
Collaborator

vdaas-ci commented Feb 8, 2024

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 🍱 /format - format codes and add licenses
  • /gen-test - generate test codes
  • 🏷️ /label - add labels
  • 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch from 764e457 to 7c7ea29 Compare February 8, 2024 07:09
Copy link

cloudflare-pages bot commented Feb 8, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 92d3966
Status: ✅  Deploy successful!
Preview URL: https://427d80b7.vald.pages.dev
Branch Preview URL: https://bugfix-docker-gcc-environmen.vald.pages.dev

View logs

@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch 2 times, most recently from 02e647a to 9856ee4 Compare February 8, 2024 07:17
@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch from 9856ee4 to 1572340 Compare February 8, 2024 07:24
hlts2
hlts2 previously approved these changes Feb 8, 2024
Copy link
Contributor

@hlts2 hlts2 left a comment

Choose a reason for hiding this comment

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

LGTM!

@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch 2 times, most recently from c5d0932 to 49d31e6 Compare February 8, 2024 08:21
@github-actions github-actions bot added size/L and removed size/M labels Feb 8, 2024
@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch 4 times, most recently from 5c688f9 to e52428b Compare February 8, 2024 09:35
@hlts2 hlts2 self-assigned this Feb 8, 2024
@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch from e52428b to 6f65704 Compare February 8, 2024 09:50
dockers/agent/core/faiss/Dockerfile Outdated Show resolved Hide resolved
dockers/tools/cli/loadtest/Dockerfile Outdated Show resolved Hide resolved
kmrmt
kmrmt previously approved these changes Feb 13, 2024
@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch 3 times, most recently from e353001 to 605fc10 Compare February 13, 2024 05:12
@hlts2 hlts2 requested a review from ykadowak February 13, 2024 08:30
@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch 2 times, most recently from b239793 to d6f06c1 Compare February 14, 2024 07:59
@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch from d6f06c1 to f00f2b2 Compare February 14, 2024 07:59
@github-actions github-actions bot removed the size/L label Feb 14, 2024
@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch 7 times, most recently from dc2d4fc to c771290 Compare February 14, 2024 09:55
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
go mod download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${TARGETARCH}" \
--mount=type=cache,target="${HOME}/.cache/go-build",id="go-build-${TARGETARCH}" \
make GOARCH=${TARGETARCH} GOOS=${TARGETOS} go/download
RUN --mount=type=cache,target="${GOPATH}/pkg",id="go-build-${BUILDARCH}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 [hadolint] <SC2086> reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.

@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch 5 times, most recently from 0214272 to 19d9ee2 Compare February 15, 2024 02:25
Signed-off-by: kpango <kpango@vdaas.org>
@kpango kpango force-pushed the bugfix/docker/gcc-environment-for-arm branch from 19d9ee2 to 92d3966 Compare February 15, 2024 02:25
Copy link
Contributor

@hlts2 hlts2 left a comment

Choose a reason for hiding this comment

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

LGTM!

@kpango kpango merged commit 16fb09f into main Feb 15, 2024
146 checks passed
@kpango kpango deleted the bugfix/docker/gcc-environment-for-arm branch February 15, 2024 04:26
This was referenced Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants