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 scale with windows nodes in cluster issue 4908 #5917

Closed
wants to merge 39 commits into from
Closed
Show file tree
Hide file tree
Changes from 37 commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
7943f09
fix scale with windows nodes in cluster issue 4908
Jun 27, 2024
ff0c353
review fixes
Jul 2, 2024
a6d9f73
Add additional manifest checks to detect missing CRDs & CRs (#5921)
joelsmith Jul 1, 2024
c49ad9b
Add Splunk scaler (#5905)
circa10a Jul 1, 2024
d7ee1b5
Update estimated release dates (#5942)
zroubalik Jul 3, 2024
d60b371
Update github.com/hashicorp/go-retryablehttp for CVE-2024-6104 (#5945)
joelsmith Jul 8, 2024
5a9a823
chore(deps): update actions/upload-artifact digest to 0b2256b (#5950)
renovate[bot] Jul 8, 2024
10547f1
Provide CloudEvents around the management of ScaledObjects resources …
SpiritZhou Jul 10, 2024
8db4090
fix scaler and tests
Aug 1, 2024
32966e1
Add username from env mysql scaler (#5952)
Indresh2410 Jul 22, 2024
8412f6b
Return the error if TLS config fails in Rabbitmq scaler (#5964)
semihbkgr Jul 22, 2024
6697ce1
chore(deps): update github actions (#5957)
renovate[bot] Jul 22, 2024
1e01186
Migrate old Azure SDKs into to use `github.com/Azure/azure-sdk-for-go…
JorTurFer Jul 24, 2024
b0d891a
feat: add TLS support for IBM MQ scaler (#5976)
rickbrouwer Jul 25, 2024
0051955
feat: GCP Pub/Sub scaler add configurable fallback value when no metr…
Caislear Jul 25, 2024
f338ae3
fix: Exclude test files from semgrep scan (#5982)
JorTurFer Jul 25, 2024
f647037
fix: use AMD machines again for semgrep workflow (#5983)
JorTurFer Jul 25, 2024
01715b9
add core logic to support access token in postgres scaler (#5589)
Ferdinanddb Jul 29, 2024
9a55f34
Add missing TestNamespace to ScaledJob validation test template (#5989)
jkyros Jul 29, 2024
c5702b7
chore: Use CNCF runner to execute PR e2e tests (#5992)
JorTurFer Jul 29, 2024
0764461
Add option to the Datadog scaler to use the Cluster Agent as proxy (#…
arapulido Jul 29, 2024
9cc1663
chore: Update supported versions (#5993)
JorTurFer Jul 30, 2024
5d58ced
fix: Revert equinix change (#5999)
JorTurFer Jul 30, 2024
1ccbe6a
Update GetAzureQueueLength in azure storage queue scaler to consider …
leodip Jul 30, 2024
b5865c6
Add eagerScalingStrategy for ScaledJob (#5872)
junekhan Jul 30, 2024
d69b00f
[BUG-5656] Annotate Jobs with parent ScaledJob generation (#5876)
josefkarasek Jul 30, 2024
26e9672
chore: Update e2e image (#6000)
JorTurFer Jul 30, 2024
f5b549f
[BUG-5922] Report failing ScaledJob triggers in status (#5916)
josefkarasek Jul 30, 2024
f886fca
chore: build with keda-tools:1.22.5 (#5971)
pauldotyu Jul 30, 2024
2bc0d63
remove unused leader election parameters of metrics adapter (#5986)
lantingchiang Jul 30, 2024
07d89b4
feat: Dynatrace scaler (#5685)
cyrilico Jul 30, 2024
4eb4eed
chore: Disable temporally AzPipeline WI e2e (#6003)
JorTurFer Jul 30, 2024
71785c1
chore: Prepare main branch for v2.14.1 (#6007)
JorTurFer Jul 31, 2024
18ed515
Remove deprecated cortexOrgId in prometheus scaler (#5990)
dttung2905 Jul 31, 2024
409d9a9
fix: reassign err to prevent invalid return in KedaProvider `GetExter…
maxcao13 Jul 31, 2024
32ee3a3
fix
Aug 1, 2024
eea7dcd
Signed-off-by: Doofus100500 <[email protected]>
Aug 1, 2024
eec6acd
Merge branch 'main' into fix-selenium-scaler
Doofus100500 Aug 1, 2024
21a9e1a
Merge branch 'main' into fix-selenium-scaler
Doofus100500 Aug 1, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
52 changes: 19 additions & 33 deletions .devcontainer/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
# Licensed under the MIT License. See https://go.microsoft.com/fwlink/?linkid=2090316 for license information.
#-------------------------------------------------------------------------------------------------------------

FROM golang:1.21.9
FROM golang:1.22.5

# Avoid warnings by switching to noninteractive
ENV DEBIAN_FRONTEND=noninteractive
Expand All @@ -16,41 +16,32 @@ ARG USERNAME=vscode
ARG USER_UID=1000
ARG USER_GID=$USER_UID

ENV GO111MODULE=auto

# Configure apt, install packages and tools
RUN apt-get update \
&& apt-get -y install --no-install-recommends apt-utils dialog unzip 2>&1 \
#
# Verify git, process tools, lsb-release (common in install instructions for CLIs) installed
&& apt-get -y install git iproute2 procps lsb-release \
#
# Install gocode-gomod
&& go get -x -d github.com/stamblerre/gocode 2>&1 \
&& go build -o gocode-gomod github.com/stamblerre/gocode \
&& mv gocode-gomod $GOPATH/bin/ \
#
# Install Go tools
&& go get -u -v \
github.com/mdempsky/gocode \
github.com/uudashr/gopkgs/cmd/gopkgs \
github.com/ramya-rao-a/go-outline \
github.com/acroca/go-symbols \
github.com/godoctor/godoctor \
golang.org/x/tools/cmd/gorename \
github.com/rogpeppe/godef \
github.com/zmb3/gogetdoc \
github.com/haya14busa/goplay/cmd/goplay \
github.com/sqs/goreturns \
github.com/josharian/impl \
github.com/davidrjenni/reftools/cmd/fillstruct \
github.com/fatih/gomodifytags \
github.com/cweill/gotests/... \
golang.org/x/tools/cmd/goimports \
golang.org/x/lint/golint \
github.com/alecthomas/gometalinter 2>&1 \
github.com/mgechev/revive \
github.com/derekparker/delve/cmd/dlv 2>&1 \
&& go install github.com/uudashr/gopkgs/v2/cmd/gopkgs@latest \
&& go install github.com/ramya-rao-a/go-outline@latest \
&& go install github.com/acroca/go-symbols@latest \
&& go install github.com/godoctor/godoctor@latest \
&& go install golang.org/x/tools/cmd/gorename@latest \
&& go install github.com/rogpeppe/godef@latest \
&& go install github.com/zmb3/gogetdoc@latest \
&& go install github.com/haya14busa/goplay/cmd/goplay@latest \
&& go install github.com/sqs/goreturns@latest \
&& go install github.com/josharian/impl@latest \
&& go install github.com/davidrjenni/reftools/cmd/fillstruct@latest \
&& go install github.com/fatih/gomodifytags@latest \
&& go install github.com/cweill/gotests/...@latest \
&& go install golang.org/x/tools/cmd/goimports@latest \
&& go install golang.org/x/lint/golint@latest \
&& go install github.com/golangci/golangci-lint/cmd/golangci-lint@latest \
&& go install github.com/mgechev/revive@latest \
&& go install github.com/go-delve/delve/cmd/dlv@latest \
&& go install honnef.co/go/tools/cmd/staticcheck@latest \
&& go install golang.org/x/tools/gopls@latest \
# Protocol Buffer Compiler
Expand All @@ -61,8 +52,6 @@ RUN apt-get update \
&& mv $HOME/.local/bin/protoc /usr/local/bin/protoc \
&& mv $HOME/.local/include/ /usr/local/bin/include/ \
&& protoc --version \
# Install golangci-lint
&& curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.55.2 \
#
# Create a non-root user to use if preferred - see https://aka.ms/vscode-remote/containers/non-root-user.
&& groupadd --gid $USER_GID $USERNAME \
Expand Down Expand Up @@ -91,9 +80,6 @@ RUN apt-get update \
&& apt-get clean -y \
&& rm -rf /var/lib/apt/lists/*
Doofus100500 marked this conversation as resolved.
Show resolved Hide resolved

# Enable go modules
ENV GO111MODULE=on

ENV OPERATOR_RELEASE_VERSION=v1.26.0
RUN ARCH=$(case $(uname -m) in x86_64) echo -n amd64 ;; aarch64) echo -n arm64 ;; *) echo -n $(uname -m) ;; esac) \
&& OS=$(uname | awk '{print tolower($0)}') \
Expand Down
1 change: 1 addition & 0 deletions .github/ISSUE_TEMPLATE/3_bug_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ body:
label: KEDA Version
description: What version of KEDA that are you running?
options:
- "2.14.1"
- "2.14.0"
- "2.13.1"
- "2.13.0"
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/fossa.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
- uses: actions/setup-go@cdcb36043654635271a94b9a6d1392de5bb323a7 # v5
- uses: actions/setup-go@0a12ed9d6a96ab950c8f026ed9f722fe0da7ef32 # v5
with:
go-version: "1.21"
go-version: "1.22"
- run: go version
- name: Get branch name
id: branch-name
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/main-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
id-token: write # needed for signing the images with GitHub OIDC Token **not production ready**

# keda-tools is built from github.com/test-tools/tools/Dockerfile
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
steps:
- name: Check out code
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
Expand Down Expand Up @@ -41,7 +41,7 @@ jobs:
key: ${{ runner.os }}-go-build-cache-${{ hashFiles('**/go.sum') }}

- name: Go modules sync
run: go mod tidy -compat=1.21
run: go mod tidy -compat=1.22

- name: Test
run: make test
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/pr-e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ jobs:
needs: triage
runs-on: ubuntu-latest
name: Build images
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
if: needs.triage.outputs.run-e2e == 'true'
steps:
- name: Set status in-progress
Expand Down Expand Up @@ -148,7 +148,7 @@ jobs:
needs: [triage, build-test-images]
runs-on: e2e
name: Execute e2e tests
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
if: needs.triage.outputs.run-e2e == 'true'
steps:
- name: Set status in-progress
Expand Down Expand Up @@ -190,7 +190,7 @@ jobs:
AWS_RUN_IDENTITY_TESTS: true
AZURE_RUN_WORKLOAD_IDENTITY_TESTS: true
GCP_RUN_IDENTITY_TESTS: true
ENABLE_OPENTELEMETRY : true
ENABLE_OPENTELEMETRY: true
E2E_IMAGE_TAG: ${{ needs.triage.outputs.image_tag }}
TEST_CLUSTER_NAME: keda-e2e-cluster-pr
COMMENT_BODY: ${{ github.event.comment.body }}
Expand Down Expand Up @@ -254,7 +254,7 @@ jobs:
details_url: https://github.com/${{github.repository}}/actions/runs/${{github.run_id}}

- name: Upload test logs
uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4
uses: actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b # v4
with:
name: e2e-test-logs
path: "${{ github.workspace }}/**/*.log"
Expand Down
24 changes: 14 additions & 10 deletions .github/workflows/pr-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
validate:
name: validate - ${{ matrix.name }}
runs-on: ${{ matrix.runner }}
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
strategy:
matrix:
include:
Expand All @@ -19,12 +19,16 @@ jobs:
- runner: ubuntu-latest
name: amd64
steps:

- name: Check out code
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
with:
fetch-depth: 1

- name: Install prerequisites
run: |
apt-get update
apt-get install -y jq python3-yaml

- name: Register workspace path
run: git config --global --add safe.directory "$GITHUB_WORKSPACE"

Expand All @@ -50,7 +54,7 @@ jobs:
key: ${{ runner.os }}-go-build-cache-${{ hashFiles('**/go.sum') }}

- name: Go modules sync
run: go mod tidy -compat=1.21
run: go mod tidy -compat=1.22

- name: Verify generated Clientset is up to date
run: make clientset-verify
Expand All @@ -73,9 +77,9 @@ jobs:
validate-dockerfiles:
name: validate-dockerfiles - ${{ matrix.name }}
runs-on: ${{ matrix.runner }}
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
strategy:
matrix:
matrix:
include:
- runner: ARM64
name: arm64
Expand Down Expand Up @@ -104,9 +108,9 @@ jobs:
validate-dev-container:
name: Validate dev-container - ${{ matrix.name }}
runs-on: ${{ matrix.runner }}
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
strategy:
matrix:
matrix:
include:
- runner: ARM64
name: arm64
Expand Down Expand Up @@ -134,12 +138,12 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
- uses: actions/setup-python@82c7e631bb3cdc910f68e0081d67478d79c6982d # v5.1.0
- uses: actions/setup-python@39cd14951b08e74b54015e9e001cdefcf80e669f # v5.1.1
with:
python-version: 3.x
- uses: actions/setup-go@cdcb36043654635271a94b9a6d1392de5bb323a7 # v5
- uses: actions/setup-go@0a12ed9d6a96ab950c8f026ed9f722fe0da7ef32 # v5
with:
go-version: "1.21"
go-version: "1.22"
- name: Get golangci
run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.55.2
- uses: pre-commit/[email protected]
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/release-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
id-token: write # needed for signing the images with GitHub OIDC Token **not production ready**

# keda-tools is built from github.com/test-tools/tools/Dockerfile
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
steps:
- name: Check out code
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
Expand Down Expand Up @@ -41,7 +41,7 @@ jobs:
key: ${{ runner.os }}-go-build-cache-${{ hashFiles('**/go.sum') }}

- name: Go modules sync
run: go mod tidy -compat=1.21
run: go mod tidy -compat=1.22

- name: Login to GitHub Container Registry
uses: docker/login-action@v3
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/static-analysis-codeql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
codeQl:
name: Analyze CodeQL Go
runs-on: ubuntu-latest
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
if: (github.actor != 'dependabot[bot]')
steps:
- name: Checkout repository
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/static-analysis-semgrep.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: "Semgrep"

on:
push:
branches: [ "main" ]
branches: ["main"]
pull_request_target: {}

concurrency:
Expand Down Expand Up @@ -30,7 +30,7 @@ jobs:
apk add github-cli
gh pr checkout ${{ github.event.number }}

- run: semgrep ci --sarif --output=semgrep.sarif
- run: semgrep ci --exclude=test --exclude=test --sarif --output=semgrep.sarif
env:
SEMGREP_APP_TOKEN: ${{ secrets.SEMGREP_APP_TOKEN }}

Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/template-main-e2e-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
name: Run e2e test
runs-on: ARM64
# keda-tools is built from github.com/test-tools/tools/Dockerfile
container: ghcr.io/kedacore/keda-tools:1.21.9
container: ghcr.io/kedacore/keda-tools:1.22.5
concurrency: e2e-tests
steps:
- name: Check out code
Expand Down Expand Up @@ -47,7 +47,7 @@ jobs:
NODE_POOL_SIZE: 1

- name: Upload test logs
uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4
uses: actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b # v4
if: ${{ always() }}
with:
name: e2e-test-logs
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/template-smoke-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@ jobs:
runs-on: ${{ inputs.runs-on }}
steps:
- name: Setup Go
uses: actions/setup-go@cdcb36043654635271a94b9a6d1392de5bb323a7 # v5
uses: actions/setup-go@0a12ed9d6a96ab950c8f026ed9f722fe0da7ef32 # v5
with:
go-version: "1.21"
go-version: "1.22"

- name: Install prerequisites
run: |
Expand All @@ -44,7 +44,7 @@ jobs:
run: make smoke-test

- name: Upload test logs
uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4
uses: actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b # v4
if: ${{ always() }}
with:
name: smoke-test-logs ${{ inputs.runs-on }}-${{ inputs.kubernetesVersion }}
Expand Down
4 changes: 0 additions & 4 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,10 +80,6 @@ issues:
- path: azure/azure_app_insights.go
linters:
- bodyclose
# We need to use Golang 1.18 before update the SDK due to it's a prerrequisite
- linters:
- staticcheck
text: "SA1019: \"github.com/Azure/azure-sdk-for-go/services/preview/monitor/mgmt/2018-03-01/insights"
# Exclude for stan_scaler and nats_jetstream_scaler, reason:
# pkg/scalers/nats_jetstream_scaler.go:109: 109-153 lines are duplicate of `pkg/scalers/stan_scaler.go:83-127` (dupl)
- path: nats_jetstream_scaler.go
Expand Down
Loading
Loading