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

chore: remove docker #990

Closed
wants to merge 2 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@

/build-logic/** @hashgraph/release-engineering @hashgraph/release-engineering-managers @hashgraph/devops-ci
/charts/** @hashgraph/release-engineering @hashgraph/release-engineering-managers @hashgraph/devops-ci
/docker/** @hashgraph/release-engineering @hashgraph/release-engineering-managers @hashgraph/devops-ci
/fullstack-core/** @hashgraph/release-engineering
/fullstack-examples/** @hashgraph/release-engineering
/fullstack-gradle-plugin/** @hashgraph/release-engineering @hashgraph/devops-ci
Expand Down
5 changes: 0 additions & 5 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,3 @@ updates:
schedule:
interval: "daily"
open-pull-requests-limit: 15
- package-ecosystem: "docker"
directory: "/docker"
schedule:
interval: "daily"
open-pull-requests-limit: 5
55 changes: 0 additions & 55 deletions .github/workflows/flow-deploy-release-artifact.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -115,60 +115,6 @@ jobs:
ossrh-user-name: ${{ secrets.OSSRH_USER_NAME }}
ossrh-user-password: ${{ secrets.OSSRH_USER_PASSWORD }}

publish-docker-image:
name: Publish / Docker Image
runs-on: solo-linux-medium
needs:
- prepare-release
steps:
- name: Checkout repository
uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1

- name: Login to GitHub Container Registry
uses: docker/login-action@9780b0c442fbb1117ed29e0efdff1e18412f7567 # v3.3.0
with:
registry: ${{ env.REGISTRY }}
username: ${{ github.actor }}
password: ${{ secrets.GITHUB_TOKEN }}

- name: Set up Docker Qemu
uses: docker/setup-qemu-action@49b3bc8e6bdd4a60e6116a5414239cba5943d3cf # v3.2.0

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@8026d2bc3645ea78b0d2544766a1225eb5691f89 # v3.7.0

- name: Build Docker Image (ubi8-init-dind)
uses: docker/build-push-action@4f58ea79222b3b9dc2c8bbdd6debcef730109a75 # v6.9.0
with:
context: docker/ubi8-init-dind
platforms: linux/amd64, linux/arm64
push: ${{ github.event.inputs.dry-run-enabled != 'true' }}
tags: ghcr.io/${{ github.repository }}/ubi8-init-dind:${{ needs.prepare-release.outputs.version }}

- name: Build Docker Image (ubi8-init-java17)
uses: docker/build-push-action@4f58ea79222b3b9dc2c8bbdd6debcef730109a75 # v6.9.0
with:
context: docker/ubi8-init-java17
platforms: linux/amd64, linux/arm64
push: ${{ github.event.inputs.dry-run-enabled != 'true' }}
tags: ghcr.io/${{ github.repository }}/ubi8-init-java17:${{ needs.prepare-release.outputs.version }}

- name: Build Docker Image (ubi8-init-java21)
uses: docker/build-push-action@4f58ea79222b3b9dc2c8bbdd6debcef730109a75 # v6.9.0
with:
context: docker/ubi8-init-java21
platforms: linux/amd64, linux/arm64
push: ${{ github.event.inputs.dry-run-enabled != 'true' }}
tags: ghcr.io/${{ github.repository }}/ubi8-init-java21:${{ needs.prepare-release.outputs.version }}

- name: Build Docker Image (kubectl-bats)
uses: docker/build-push-action@4f58ea79222b3b9dc2c8bbdd6debcef730109a75 # v6.9.0
with:
context: docker/kubectl-bats
platforms: linux/amd64, linux/arm64
push: ${{ github.event.inputs.dry-run-enabled != 'true' }}
tags: ghcr.io/${{ github.repository }}/kubectl-bats:${{ needs.prepare-release.outputs.version }}

publish-helm-charts:
name: Publish / Helm Charts
runs-on: solo-linux-medium
Expand Down Expand Up @@ -212,7 +158,6 @@ jobs:
runs-on: solo-linux-medium
needs:
- publish-maven-central
- publish-docker-image
- publish-helm-charts
steps:
- name: Checkout Code
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/flow-pull-request-checks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -59,12 +59,6 @@ jobs:
gradle-cache-username: ${{ secrets.GRADLE_CACHE_USERNAME }}
gradle-cache-password: ${{ secrets.GRADLE_CACHE_PASSWORD }}

chart-tests:
name: FS Helm Chart Tests
uses: ./.github/workflows/zxc-helm-chart-tests.yaml
with:
custom-job-label: Standard

Comment on lines -62 to -67
Copy link
Contributor

Choose a reason for hiding this comment

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

this is not docker related.

Copy link
Contributor

Choose a reason for hiding this comment

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

are you just removing this to avoid having to update the tests to point to the new solo container repo?

codecov:
name: CodeCov
uses: ./.github/workflows/zxc-code-analysis.yaml
Expand Down
8 changes: 0 additions & 8 deletions .github/workflows/support/ci/ci-values.yaml

This file was deleted.

126 changes: 0 additions & 126 deletions .github/workflows/support/ci_test.sh
Copy link
Contributor

Choose a reason for hiding this comment

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

not docker related, is this being removed to avoid updates? does it still run correctly without the solo container repo changes?

This file was deleted.

3 changes: 2 additions & 1 deletion .github/workflows/support/dev-cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,5 @@ name: fst # this is overridden if CLUSTER_NAME env var is set. Check .env file
nodes:
- role: control-plane
labels:
fullstack-scheduling.io/role: network
fullstack-scheduling.io/role: network

62 changes: 0 additions & 62 deletions .github/workflows/support/local-node/config.template
Copy link
Contributor

Choose a reason for hiding this comment

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

not docker related, is this being removed to avoid updates? does it still run correctly without the solo container repo changes?

This file was deleted.

Copy link
Contributor

Choose a reason for hiding this comment

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

not docker related, is this being removed to avoid updates? does it still run correctly without the solo container repo changes?

This file was deleted.

Copy link
Contributor

Choose a reason for hiding this comment

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

not docker related, is this being removed to avoid updates? does it still run correctly without the solo container repo changes?

This file was deleted.

Copy link
Contributor

Choose a reason for hiding this comment

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

not docker related, is this being removed to avoid updates? does it still run correctly without the solo container repo changes?

This file was deleted.

Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Loading
Loading