Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: fluxcd/flagger
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v1.24.0
Choose a base ref
...
head repository: fluxcd/flagger
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v1.24.1
Choose a head ref
  • 16 commits
  • 15 files changed
  • 3 contributors

Commits on Oct 24, 2022

  1. Bump actions/checkout from 2 to 3

    Bumps [actions/checkout](https://github.com/actions/checkout) from 2 to 3.
    - [Release notes](https://github.com/actions/checkout/releases)
    - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
    - [Commits](actions/checkout@v2...v3)
    
    ---
    updated-dependencies:
    - dependency-name: actions/checkout
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored Oct 24, 2022
    Copy the full SHA
    9b56445 View commit details
  2. Bump github/codeql-action from 1 to 2

    Bumps [github/codeql-action](https://github.com/github/codeql-action) from 1 to 2.
    - [Release notes](https://github.com/github/codeql-action/releases)
    - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
    - [Commits](github/codeql-action@v1...v2)
    
    ---
    updated-dependencies:
    - dependency-name: github/codeql-action
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored Oct 24, 2022
    Copy the full SHA
    f1f4e68 View commit details
  3. Bump docker/login-action from 1 to 2

    Bumps [docker/login-action](https://github.com/docker/login-action) from 1 to 2.
    - [Release notes](https://github.com/docker/login-action/releases)
    - [Commits](docker/login-action@v1...v2)
    
    ---
    updated-dependencies:
    - dependency-name: docker/login-action
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored Oct 24, 2022
    Copy the full SHA
    b3112a5 View commit details
  4. Bump goreleaser/goreleaser-action from 2 to 3

    Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 2 to 3.
    - [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
    - [Commits](goreleaser/goreleaser-action@v2...v3)
    
    ---
    updated-dependencies:
    - dependency-name: goreleaser/goreleaser-action
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored Oct 24, 2022
    Copy the full SHA
    bdbd1fb View commit details
  5. Bump docker/build-push-action from 2 to 3

    Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 2 to 3.
    - [Release notes](https://github.com/docker/build-push-action/releases)
    - [Commits](docker/build-push-action@v2...v3)
    
    ---
    updated-dependencies:
    - dependency-name: docker/build-push-action
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] authored Oct 24, 2022
    Copy the full SHA
    f9f8d7e View commit details
  6. Merge pull request #1299 from fluxcd/dependabot/github_actions/docker…

    …/build-push-action-3
    
    Bump docker/build-push-action from 2 to 3
    stefanprodan authored Oct 24, 2022
    Copy the full SHA
    66364bb View commit details
  7. Merge pull request #1295 from fluxcd/dependabot/github_actions/action…

    …s/checkout-3
    
    Bump actions/checkout from 2 to 3
    stefanprodan authored Oct 24, 2022
    Copy the full SHA
    26ec719 View commit details
  8. Merge pull request #1296 from fluxcd/dependabot/github_actions/github…

    …/codeql-action-2
    
    Bump github/codeql-action from 1 to 2
    stefanprodan authored Oct 24, 2022
    Copy the full SHA
    85a3b7c View commit details
  9. Merge pull request #1297 from fluxcd/dependabot/github_actions/docker…

    …/login-action-2
    
    Bump docker/login-action from 1 to 2
    stefanprodan authored Oct 24, 2022
    Copy the full SHA
    19d4e52 View commit details
  10. Merge pull request #1298 from fluxcd/dependabot/github_actions/gorele…

    …aser/goreleaser-action-3
    
    Bump goreleaser/goreleaser-action from 2 to 3
    stefanprodan authored Oct 24, 2022
    Copy the full SHA
    ec44f64 View commit details

Commits on Oct 25, 2022

  1. fix(gloo): Use correct route table name in case service name was over…

    …written
    
    Signed-off-by: mdolinin <dmo.builder@gmail.com>
    mdolinin committed Oct 25, 2022
    Copy the full SHA
    4c0b2be View commit details
  2. fix(gloo): Update Gloo to the latest stable version

    Signed-off-by: mdolinin <dmo.builder@gmail.com>
    mdolinin committed Oct 25, 2022
    Copy the full SHA
    3d3e051 View commit details
  3. fix(gloo): Update tests to not check gateway deployment. Was removed …

    …from >1.12.x
    
    Signed-off-by: mdolinin <dmo.builder@gmail.com>
    mdolinin committed Oct 25, 2022
    Copy the full SHA
    647f624 View commit details

Commits on Oct 26, 2022

  1. Merge pull request #1300 from mdolinin/fix/gloo-non-default-service-n…

    …ame-corretly-get-routes
    
    fix(gloo): Use correct route table name in case service name was overwritten
    stefanprodan authored Oct 26, 2022
    Copy the full SHA
    a7df345 View commit details
  2. Release v1.24.1

    Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
    stefanprodan committed Oct 26, 2022
    Copy the full SHA
    50f9255 View commit details
  3. Merge pull request #1301 from fluxcd/release-v1.24.1

    Release v1.24.1
    stefanprodan authored Oct 26, 2022
    Copy the full SHA
    381c19b View commit details
2 changes: 1 addition & 1 deletion .github/workflows/build.yaml
Original file line number Diff line number Diff line change
@@ -17,7 +17,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v2
uses: actions/checkout@v3
- name: Restore Go cache
uses: actions/cache@v3.0.11
with:
2 changes: 1 addition & 1 deletion .github/workflows/e2e.yaml
Original file line number Diff line number Diff line change
@@ -35,7 +35,7 @@ jobs:
- keda
steps:
- name: Checkout
uses: actions/checkout@v2
uses: actions/checkout@v3
- name: Setup Kubernetes
uses: engineerd/setup-kind@v0.5.0
with:
2 changes: 1 addition & 1 deletion .github/workflows/helm.yaml
Original file line number Diff line number Diff line change
@@ -12,7 +12,7 @@ jobs:
permissions:
contents: write
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Publish Helm charts
uses: stefanprodan/helm-gh-pages@v1.6.0
with:
4 changes: 2 additions & 2 deletions .github/workflows/push-ld.yml
Original file line number Diff line number Diff line change
@@ -29,7 +29,7 @@ jobs:
id: buildx
uses: docker/setup-buildx-action@v2
- name: Login to GitHub Container Registry
uses: docker/login-action@v1
uses: docker/login-action@v2
with:
registry: ghcr.io
username: fluxcdbot
@@ -43,7 +43,7 @@ jobs:
tags: |
type=raw,value=${{ steps.prep.outputs.VERSION }}
- name: Publish image
uses: docker/build-push-action@v2
uses: docker/build-push-action@v3
with:
push: true
builder: ${{ steps.buildx.outputs.name }}
4 changes: 2 additions & 2 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
@@ -49,7 +49,7 @@ jobs:
tags: |
type=raw,value=${{ steps.prep.outputs.VERSION }}
- name: Publish image
uses: docker/build-push-action@v2
uses: docker/build-push-action@v3
with:
push: true
builder: ${{ steps.buildx.outputs.name }}
@@ -93,7 +93,7 @@ jobs:
cosign sign ghcr.io/fluxcd/flagger-manifests:${{ steps.prep.outputs.VERSION }}
- uses: anchore/sbom-action/download-syft@v0
- name: Create release and SBOM
uses: goreleaser/goreleaser-action@v2
uses: goreleaser/goreleaser-action@v3
with:
version: latest
args: release --release-notes=notes.md --rm-dist --skip-validate
10 changes: 5 additions & 5 deletions .github/workflows/scan.yml
Original file line number Diff line number Diff line change
@@ -17,7 +17,7 @@ jobs:
permissions:
security-events: write
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Run FOSSA scan and upload build data
uses: fossa-contrib/fossa-action@v1
with:
@@ -30,12 +30,12 @@ jobs:
security-events: write
steps:
- name: Checkout repository
uses: actions/checkout@v2
uses: actions/checkout@v3
- name: Initialize CodeQL
uses: github/codeql-action/init@v1
uses: github/codeql-action/init@v2
with:
languages: go
- name: Autobuild
uses: github/codeql-action/autobuild@v1
uses: github/codeql-action/autobuild@v2
- name: Perform CodeQL Analysis
uses: github/codeql-action/analyze@v1
uses: github/codeql-action/analyze@v2
13 changes: 13 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -2,6 +2,19 @@

All notable changes to this project are documented in this file.

## 1.24.1

**Release date:** 2022-10-26

This release comes with a fix to Gloo routing when a custom service name id used.

In addition, the Gloo ingress end-to-end testing was updated to Gloo Helm chart v1.12.31.

#### Fixes:

- fix(gloo): Use correct route table name in case service name was overwritten
[#1300](https://github.com/fluxcd/flagger/pull/1300)

## 1.24.0

**Release date:** 2022-10-23
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -260,7 +260,7 @@ When submitting bug reports please include as many details as possible:
Here is a list of good entry points into our community, how we stay in touch and how you can meet us as a team.
- Slack: Join in and talk to us in the `#flagger` channel on [CNCF Slack](https://slack.cncf.io/).
- Meetings: We run weekly, public meetings - join one of the upcoming dev meetings from the [Flux calendar](https://fluxcd.io/#calendar).
- Public meetings: We run weekly meetings - join one of the upcoming dev meetings from the [Flux calendar](https://fluxcd.io/#calendar).
- Blog: Stay up to date with the latest news on [the Flux blog](https://fluxcd.io/blog/).
- Mailing list: To be updated on Flux and Flagger progress regularly, please [join the flux-dev mailing list](https://lists.cncf.io/g/cncf-flux-dev).

2 changes: 1 addition & 1 deletion artifacts/flagger/deployment.yaml
Original file line number Diff line number Diff line change
@@ -22,7 +22,7 @@ spec:
serviceAccountName: flagger
containers:
- name: flagger
image: ghcr.io/fluxcd/flagger:1.24.0
image: ghcr.io/fluxcd/flagger:1.24.1
imagePullPolicy: IfNotPresent
ports:
- name: http
4 changes: 2 additions & 2 deletions charts/flagger/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
name: flagger
version: 1.24.0
appVersion: 1.24.0
version: 1.24.1
appVersion: 1.24.1
kubeVersion: ">=1.19.0-0"
engine: gotpl
description: Flagger is a progressive delivery operator for Kubernetes
2 changes: 1 addition & 1 deletion charts/flagger/values.yaml
Original file line number Diff line number Diff line change
@@ -2,7 +2,7 @@

image:
repository: ghcr.io/fluxcd/flagger
tag: 1.24.0
tag: 1.24.1
pullPolicy: IfNotPresent
pullSecret:

2 changes: 1 addition & 1 deletion kustomize/base/flagger/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -9,4 +9,4 @@ resources:
images:
- name: ghcr.io/fluxcd/flagger
newName: ghcr.io/fluxcd/flagger
newTag: 1.24.0
newTag: 1.24.1
7 changes: 3 additions & 4 deletions pkg/router/gloo.go
Original file line number Diff line number Diff line change
@@ -170,9 +170,8 @@ func (gr *GlooRouter) GetRoutes(canary *flaggerv1.Canary) (
mirrored bool,
err error,
) {
apexName := canary.Spec.TargetRef.Name
primaryName := fmt.Sprintf("%s-%s-primaryupstream-%v", canary.Namespace, canary.Spec.TargetRef.Name, canary.Spec.Service.Port)

apexName, _, _ := canary.GetServiceNames()
primaryUpstreamName := fmt.Sprintf("%s-%s-primaryupstream-%v", canary.Namespace, apexName, canary.Spec.Service.Port)
routeTable, err := gr.glooClient.GatewayV1().RouteTables(canary.Namespace).Get(context.TODO(), apexName, metav1.GetOptions{})
if err != nil {
err = fmt.Errorf("RouteTable %s.%s get query error: %w", apexName, canary.Namespace, err)
@@ -185,7 +184,7 @@ func (gr *GlooRouter) GetRoutes(canary *flaggerv1.Canary) (
}

for _, dst := range routeTable.Spec.Routes[0].Action.Destination.Destinations {
if dst.Destination.Upstream.Name == primaryName {
if dst.Destination.Upstream.Name == primaryUpstreamName {
primaryWeight = int(dst.Weight)
canaryWeight = 100 - primaryWeight
return
2 changes: 1 addition & 1 deletion pkg/version/version.go
Original file line number Diff line number Diff line change
@@ -16,5 +16,5 @@ limitations under the License.

package version

var VERSION = "1.24.0"
var VERSION = "1.24.1"
var REVISION = "unknown"
3 changes: 1 addition & 2 deletions test/gloo/install.sh
Original file line number Diff line number Diff line change
@@ -2,7 +2,7 @@

set -o errexit

GLOO_VER="1.11.13"
GLOO_VER="1.12.31"
REPO_ROOT=$(git rev-parse --show-toplevel)

mkdir -p ${REPO_ROOT}/bin
@@ -15,7 +15,6 @@ helm upgrade -i gloo gloo/gloo --version ${GLOO_VER} \
--set discovery.enabled=false

kubectl -n gloo-system rollout status deployment/gloo
kubectl -n gloo-system rollout status deployment/gateway
kubectl -n gloo-system get all

echo '>>> Installing Flagger'