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

Format yaml using google/yamlfmt & update go version and dependencies #2322

Merged
merged 3 commits into from
Feb 8, 2024
Merged
Show file tree
Hide file tree
Changes from 2 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
96 changes: 48 additions & 48 deletions .fossa.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,51 +22,51 @@ cli:
locator: git+github.com/vdaas/vald
analyze:
modules:
- name: github.com/vdaas/vald/cmd/agent/core/ngt
type: go
target: github.com/vdaas/vald/cmd/agent/core/ngt
path: cmd/agent/core/ngt
- name: github.com/vdaas/vald/cmd/agent/sidecar
type: go
target: github.com/vdaas/vald/cmd/agent/sidecar
path: cmd/agent/sidecar
- name: github.com/vdaas/vald/cmd/discoverer/k8s
type: go
target: github.com/vdaas/vald/cmd/discoverer/k8s
path: cmd/discoverer/k8s
- name: github.com/vdaas/vald/cmd/gateway/filter
type: go
target: github.com/vdaas/vald/cmd/gateway/filter
path: cmd/gateway/filter
- name: github.com/vdaas/vald/cmd/gateway/lb
type: go
target: github.com/vdaas/vald/cmd/gateway/lb
path: cmd/gateway/lb
- name: github.com/vdaas/vald/cmd/manager/index
type: go
target: github.com/vdaas/vald/cmd/manager/index
path: cmd/manager/index
- name: github.com/vdaas/vald/cmd/tools/cli/vdctl
type: go
target: github.com/vdaas/vald/cmd/tools/cli/vdctl
path: cmd/tools/cli/vdctl
- name: github.com/vdaas/vald/hack/helm/schema/gen
type: go
target: github.com/vdaas/vald/hack/helm/schema/gen
path: hack/helm/schema/gen
- name: github.com/vdaas/vald/hack/license/gen
type: go
target: github.com/vdaas/vald/hack/license/gen
path: hack/license/gen
- name: github.com/vdaas/vald/hack/tools/config/agent/core/ngt
type: go
target: github.com/vdaas/vald/hack/tools/config/agent/core/ngt
path: hack/tools/config/agent/core/ngt
- name: github.com/vdaas/vald/hack/tools/config/discoverer/k8s
type: go
target: github.com/vdaas/vald/hack/tools/config/discoverer/k8s
path: hack/tools/config/discoverer/k8s
- name: github.com/vdaas/vald/hack/tools/metrics
type: go
target: github.com/vdaas/vald/hack/tools/metrics
path: hack/tools/metrics
- name: github.com/vdaas/vald/cmd/agent/core/ngt
type: go
target: github.com/vdaas/vald/cmd/agent/core/ngt
path: cmd/agent/core/ngt
- name: github.com/vdaas/vald/cmd/agent/sidecar
type: go
target: github.com/vdaas/vald/cmd/agent/sidecar
path: cmd/agent/sidecar
- name: github.com/vdaas/vald/cmd/discoverer/k8s
type: go
target: github.com/vdaas/vald/cmd/discoverer/k8s
path: cmd/discoverer/k8s
- name: github.com/vdaas/vald/cmd/gateway/filter
type: go
target: github.com/vdaas/vald/cmd/gateway/filter
path: cmd/gateway/filter
- name: github.com/vdaas/vald/cmd/gateway/lb
type: go
target: github.com/vdaas/vald/cmd/gateway/lb
path: cmd/gateway/lb
- name: github.com/vdaas/vald/cmd/manager/index
type: go
target: github.com/vdaas/vald/cmd/manager/index
path: cmd/manager/index
- name: github.com/vdaas/vald/cmd/tools/cli/vdctl
type: go
target: github.com/vdaas/vald/cmd/tools/cli/vdctl
path: cmd/tools/cli/vdctl
- name: github.com/vdaas/vald/hack/helm/schema/gen
type: go
target: github.com/vdaas/vald/hack/helm/schema/gen
path: hack/helm/schema/gen
- name: github.com/vdaas/vald/hack/license/gen
type: go
target: github.com/vdaas/vald/hack/license/gen
path: hack/license/gen
- name: github.com/vdaas/vald/hack/tools/config/agent/core/ngt
type: go
target: github.com/vdaas/vald/hack/tools/config/agent/core/ngt
path: hack/tools/config/agent/core/ngt
- name: github.com/vdaas/vald/hack/tools/config/discoverer/k8s
type: go
target: github.com/vdaas/vald/hack/tools/config/discoverer/k8s
path: hack/tools/config/discoverer/k8s
- name: github.com/vdaas/vald/hack/tools/metrics
type: go
target: github.com/vdaas/vald/hack/tools/metrics
path: hack/tools/metrics
10 changes: 5 additions & 5 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,23 +6,23 @@ labels: "type/bug, priority/medium, team/core"
assignees: ""
---

### Describe the bug:
### Describe the bug

<!-- A clear and concise description of what the bug is. -->

### To Reproduce:
### To Reproduce

<!-- Please describe the steps to reproduce the behavior: -->

### Expected behavior:
### Expected behavior

<!-- A clear and concise description of what you expected to happen. -->

### Environment:
### Environment

<!--- Please change the versions below along with your environment -->

- Go Version: 1.21.6
- Go Version: 1.22.0
- Docker Version: 20.10.8
- Kubernetes Version: v1.29.1
- NGT Version: 2.1.6
8 changes: 4 additions & 4 deletions .github/ISSUE_TEMPLATE/security_issue_report.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,17 +6,17 @@ labels: "type/security, priority/medium, team/core, team/sre"
assignees: ""
---

### Describe the issue:
### Describe the issue

<!-- A clear and concise description of what the issue is. -->

### CVE:
### CVE

### Environment:
### Environment

<!--- Please change the versions below along with your environment -->

- Go Version: 1.21.6
- Go Version: 1.22.0
- Docker Version: 20.10.8
- Kubernetes Version: v1.29.1
- NGT Version: 2.1.6
12 changes: 6 additions & 6 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,34 +1,34 @@
<!--- Provide a general summary of your changes in the Title above -->

### Description:
### Description

<!-- Describe your changes in detail -->
<!-- It would be better to describe the details especially What changed and Why you changed -->

### Related Issue:
### Related Issue

<!-- This project mainly accepts pull requests related to open issues -->
<!-- NOTE: If suggesting a new feature or change, please discuss it in an issue first -->
<!-- NOTE: If fixing a bug, there should be an issue describing it with steps to reproduce -->
<!-- Please link to the issue here: -->

### Versions:
### Versions

<!--- Please change the versions below along with your environment -->

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

### Checklist:
### Checklist

<!-- For completed items, change [ ] to [x]. -->
<!-- NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open. -->

- [ ] I have read the [CONTRIBUTING](https://github.com/vdaas/vald/blob/main/CONTRIBUTING.md) document and completed [our CLA agreement](https://cla-assistant.io/vdaas/vald).
- [ ] I have checked open [Pull Requests](https://github.com/vdaas/vald/pulls) for the similar feature or fixes?

### Special notes for your reviewer:
### Special notes for your reviewer

<!-- Please tell us anything you would like to share to reviewers related this PR -->
3 changes: 0 additions & 3 deletions .github/actions/deploy-chaos-mesh/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,18 +36,15 @@ runs:
echo "version=${CHAOS_MESH_VERSION}" >> $GITHUB_OUTPUT
env:
CHAOS_MESH_VERSION: ${{ inputs.chaos_mesh_version }}

- name: Check k3d
shell: bash
id: check_k3d
run: |
kubectl cluster-info dump
- name: Check Helm
shell: bash
run: |
helm version
- name: Deploy Chaos Mesh
shell: bash
run: |
Expand Down
6 changes: 0 additions & 6 deletions .github/actions/docker-build/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
#
name: "Build Docker images"
description: "A action to build Docker images and publish them"

inputs:
target:
description: "Build target"
Expand Down Expand Up @@ -45,7 +44,6 @@ outputs:
EXTRA_TAGS:
description: "Extra tags"
value: ${{ steps.add_extra_tags.outputs.EXTRA_TAGS }}

runs:
using: "composite"
steps:
Expand All @@ -62,11 +60,9 @@ runs:
echo "ALTER_IMAGE_NAME=${alter_image_name}" >> $GITHUB_OUTPUT
env:
TARGET: ${{ inputs.target }}

- name: Determine tag name
id: determine_tag_name
uses: ./.github/actions/determine-docker-image-tag

- name: Determine platforms
shell: bash
id: determine_platforms
Expand All @@ -86,7 +82,6 @@ runs:
echo "PLATFORMS=${platforms}" >> $GITHUB_OUTPUT
env:
TARGET_PLATFORMS: ${{ inputs.platforms }}

- name: Add extra tags
shell: bash
id: add_extra_tags
Expand All @@ -102,7 +97,6 @@ runs:
IMAGE_NAME: ${{ steps.image_name.outputs.IMAGE_NAME }}
ALTER_IMAGE_NAME: ${{ steps.image_name.outputs.ALTER_IMAGE_NAME }}
PRIMARY_TAG: ${{ steps.determine_tag_name.outputs.PRIMARY_TAG }}

- name: Build and Push
shell: bash
id: build_and_push
Expand Down
6 changes: 0 additions & 6 deletions .github/actions/dump-context/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
#
name: "Dump context to log"
description: "A action to dump context to log"

runs:
using: "composite"
steps:
Expand All @@ -24,31 +23,26 @@ runs:
run: echo $JSON
env:
JSON: ${{toJSON(github)}}

- name: "Dump job context"
shell: bash
run: echo $JSON
env:
JSON: ${{ toJSON(job) }}

- name: "Dump steps context"
shell: bash
run: echo $JSON
env:
JSON: ${{toJSON(steps)}}

- name: "Dump runner context"
shell: bash
run: echo $JSON
env:
JSON: ${{toJSON(runner)}}

- name: "Dump strategy context"
shell: bash
run: echo $JSON
env:
JSON: ${{toJSON(strategy)}}

- name: "Dump matrix context"
shell: bash
run: echo $JSON
Expand Down
6 changes: 0 additions & 6 deletions .github/actions/e2e-deploy-vald-helm-operator/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
#
name: "Deploy Vald using vald-helm-operator for E2E test"
description: "A action to deploy vald using vald-helm-operator for E2E test"

inputs:
require_minio:
description: "If Minio is required, set this to true."
Expand Down Expand Up @@ -49,7 +48,6 @@ outputs:
POD_NAME:
description: "A pod name that waited for"
value: ${{ steps.deploy_vald.outputs.POD_NAME }}

runs:
using: "composite"
steps:
Expand All @@ -59,12 +57,10 @@ runs:
if: ${{ inputs.require_minio == 'true' }}
run: |
make K8S_SLEEP_DURATION_FOR_WAIT_COMMAND=10 k8s/external/minio/deploy
- name: Dump valdrelease
shell: bash
run: |
cat ${{ inputs.valdrelease }}
- name: Deploy vald helm operator from remote charts
shell: bash
id: deploy_vald_helm_operator_remote
Expand All @@ -79,7 +75,6 @@ runs:
env:
DEFAULT_IMAGE_TAG: ${{ inputs.default_image_tag }}
HELM_EXTRA_OPTIONS: ${{ inputs.helm_extra_options }}

- name: Deploy vald helm operator from local charts
shell: bash
id: deploy_vald_helm_operator_local
Expand All @@ -90,7 +85,6 @@ runs:
env:
DEFAULT_IMAGE_TAG: ${{ inputs.default_image_tag }}
HELM_EXTRA_OPTIONS: ${{ inputs.helm_extra_options }}

- name: Deploy vald
shell: bash
id: deploy_vald
Expand Down
6 changes: 0 additions & 6 deletions .github/actions/e2e-deploy-vald-readreplica/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
#
name: "Deploy Vald Read Replica for E2E test"
description: "A action to deploy vald read replica for E2E test"

inputs:
require_minio:
description: "If Minio is required, set this to true."
Expand Down Expand Up @@ -49,7 +48,6 @@ outputs:
POD_NAME:
description: "A pod name that waited for"
value: ${{ steps.get_real_pod_name.outputs.POD_NAME }}

runs:
using: "composite"
steps:
Expand All @@ -59,12 +57,10 @@ runs:
if: ${{ inputs.require_minio == 'true' }}
run: |
make K8S_SLEEP_DURATION_FOR_WAIT_COMMAND=10 k8s/external/minio/deploy
- name: Dump Helm values
shell: bash
run: |
cat ${{ inputs.values }}
- name: Deploy vald read replica from remote charts
shell: bash
id: deploy_vald_readreplica_remote
Expand All @@ -90,7 +86,6 @@ runs:
HELM_EXTRA_OPTIONS: ${{ inputs.helm_extra_options }}
WAIT_FOR_SELECTOR: ${{ inputs.wait_for_selector }}
WAIT_FOR_TIMEOUT: ${{ inputs.wait_for_timeout }}

- name: Deploy vald read replica from local charts
shell: bash
id: deploy_vald_readreplica_local
Expand All @@ -112,7 +107,6 @@ runs:
HELM_EXTRA_OPTIONS: ${{ inputs.helm_extra_options }}
WAIT_FOR_SELECTOR: ${{ inputs.wait_for_selector }}
WAIT_FOR_TIMEOUT: ${{ inputs.wait_for_timeout }}

- name: Get real pod name
shell: bash
id: get_real_pod_name
Expand Down
Loading
Loading