cherry-pick: Reconcile master

.devcontainer/devcontainer.json

@@ -1,7 +1,25 @@
 {
-    "build": { "dockerfile": "Dockerfile" },
-    "hostRequirements": {"cpus": 4},
-    "onCreateCommand": ".devcontainer/pre-build.sh",
-    "workspaceMount": "source=${localWorkspaceFolder},target=${localEnv:HOME}/go/src/github.com/argoproj/argo-workflows,type=bind",
-    "workspaceFolder": "${localEnv:HOME}/go/src/github.com/argoproj/argo-workflows"
-}
+  "image": "mcr.microsoft.com/vscode/devcontainers/base:ubuntu",
+  "appPort": 8080,
+  "features": {
+    "ghcr.io/devcontainers/features/go:1": {
+      "version": "1.20"
+    },
+    "ghcr.io/devcontainers/features/node:1": {
+      "version": "16"
+    },
+    "ghcr.io/devcontainers/features/docker-in-docker:2": {},
+    "ghcr.io/devcontainers/features/python:1": {}
+  },
+  "forwardPorts": [9000, 9090, 2746, 8080],
+  "hostRequirements": {
+    "cpus": 4
+  },
+  "onCreateCommand": ".devcontainer/pre-build.sh",
+  "workspaceMount": "source=${localWorkspaceFolder},target=/home/vscode/go/src/github.com/argoproj/argo-workflows,type=bind",
+  "workspaceFolder": "/home/vscode/go/src/github.com/argoproj/argo-workflows",
+  "remoteEnv": {
+    "PATH": "${containerEnv:PATH}:/home/vscode/go/bin",
+    "GOPATH": "/home/vscode/go"
+  }
+}

.devcontainer/pre-build.sh

@@ -1,13 +1,33 @@
 #!/usr/bin/env sh
 set -eux
 
+# Add hosts
+sudo bash -c 'echo "127.0.0.1 dex" >> /etc/hosts'
+sudo bash -c 'echo "127.0.0.1 minio" >> /etc/hosts'
+sudo bash -c 'echo "127.0.0.1 postgres" >> /etc/hosts'
+sudo bash -c 'echo "127.0.0.1 mysql" >> /etc/hosts'
+sudo bash -c 'echo "127.0.0.1 azurite" >> /etc/hosts'
+
 # install kubernetes
 wget -q -O - https://raw.githubusercontent.com/k3d-io/k3d/main/install.sh | bash
 k3d cluster get k3s-default || k3d cluster create --wait
 k3d kubeconfig merge --kubeconfig-merge-default
 
+# install kubectl
+curl -LO https://dl.k8s.io/release/v1.26.0/bin/linux/amd64/kubectl
+chmod +x ./kubectl
+sudo mv ./kubectl /usr/local/bin/kubectl
+kubectl cluster-info
+
 # install kit
 curl -q https://raw.githubusercontent.com/kitproj/kit/main/install.sh | sh
 
-# do time consuming tasks, e.g. download deps and initial build
-kit build
+# install protocol buffer compiler (protoc)
+sudo apt update
+sudo apt install -y protobuf-compiler
+
+# Make sure go path is owned by vscode
+sudo chown -R vscode:vscode /home/vscode/go
+
+# download dependencies and do first-pass compile
+CI=1 kit pre-up

.dockerignore

@@ -3,6 +3,9 @@
 *.iml
 *.md
 *.yaml
+.devcontainer
+# The .git folder gets modified every time git runs preventing caching of builds. Once removed, we must pass GIT_COMMIT as Docker build-args.
+.git
 .github
 .idea
 .run
@@ -12,9 +15,12 @@ coverage.out
 dist
 docs
 examples
+logs
 manifests
 plugins
 sdks
+site
+tasks.yaml
 test/e2e
 ui/dist
 ui/node_modules

.github/dependabot.yml

@@ -25,4 +25,6 @@ updates:
     ignore:
       - dependency-name: raw-loader
       - dependency-name: style-loader
+      - dependency-name: react-router-dom
+      - dependency-name: "@types/react-router-dom"
     open-pull-requests-limit: 10

.github/pull_request_template.md

@@ -1,14 +1,38 @@
+<!--
+
+### Before you open your PR 
+
+- Run `make pre-commit -B` to fix codegen and lint problems (build will fail).
+- [Signed-off your commits](https://github.com/apps/dco/) (otherwise the DCO check will fail).
+- Used [a conventional commit message](https://www.conventionalcommits.org/en/v1.0.0/).
+
+
+### When you open your PR
+
+- PR title format should also conform to [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/).
+- "Fixes #" is in both the PR title (for release notes) and this description (to automatically link and close the issue).
+- Create the PR as draft.
+- Once builds are green, mark your PR "Ready for review".
+
+When changes are requested, please address them and then dismiss the review to get it reviewed again.
+
+-->
+
+
+<!-- Does this PR fix an issue -->
+
 Fixes #TODO
 
-Please do not open a pull request until you have checked ALL of these:
+### Motivation
+
+<!-- TODO: Say why you made your changes. -->
+
+### Modifications
+
+<!-- TODO: Say what changes you made. -->
+
+<!-- TODO: Attach screenshots if you changed the UI. -->
 
-* [ ] Create the PR as draft .
-* [ ] Run `make pre-commit -B` to fix codegen and lint problems.
-* [ ] Sign-off your commits (otherwise the DCO check will fail).
-* [ ] Use [a conventional commit message](https://www.conventionalcommits.org/en/v1.0.0/) (otherwise the commit message check will fail).
-* [ ] "Fixes #" is in both the PR title (for release notes) and this description (to automatically link and close the issue).
-* [ ] Add unit or e2e tests. Say how you tested your changes. If you changed the UI, attach screenshots.
-* [ ] Github checks are green.
-* [ ] Once required tests have passed, mark your PR "Ready for review".
+### Verification
 
-If changes were requested, and you've made them, dismiss the review to get it reviewed again.
+<!-- TODO: Say how you tested your changes. -->

.github/workflows/changelog.yaml

@@ -25,7 +25,7 @@ jobs:
       - run: git tag -l 'v*'
       # avoid invoking `make` to reduce the risk of a Makefile bug failing this workflow
       - run: ./hack/changelog.sh > CHANGELOG.md
-      - uses: peter-evans/create-pull-request@v4
+      - uses: peter-evans/create-pull-request@v5
         with:
           title: 'docs: updated CHANGELOG.md'
           commit-message: 'docs: updated CHANGELOG.md'

.github/workflows/ci-build.yaml

@@ -23,9 +23,9 @@ jobs:
     timeout-minutes: 10
     steps:
       - uses: actions/checkout@v3
-      - uses: actions/setup-go@v3
+      - uses: actions/setup-go@v4
         with:
-          go-version: "1.19"
+          go-version: "1.20"
           cache: true
       - run: make test STATIC_FILES=false GOTEST='go test -p 20 -covermode=atomic -coverprofile=coverage.out'
       # engineers just ignore this in PRs, so lets not even run it
@@ -87,22 +87,22 @@ jobs:
           - test: test-python-sdk
             profile: minimal
           - test: test-executor
-            install_k3s_version: v1.21.2+k3s1
+            install_k3s_version: v1.25.11-k3s1
             profile: minimal
           - test: test-corefunctional
-            install_k3s_version: v1.21.2+k3s1
+            install_k3s_version: v1.25.11-k3s1
             profile: minimal
           - test: test-functional
-            install_k3s_version: v1.21.2+k3s1
+            install_k3s_version: v1.25.11-k3s1
             profile: minimal
     steps:
       - name: Install socat
         # needed by Kubernetes v1.25
         run: sudo apt-get -y install socat
       - uses: actions/checkout@v3
-      - uses: actions/setup-go@v3
+      - uses: actions/setup-go@v4
         with:
-          go-version: "1.19"
+          go-version: "1.20"
           cache: true
       - uses: actions/setup-java@v3
         if: ${{matrix.test == 'test-java-sdk'}}
@@ -117,7 +117,12 @@ jobs:
           cache: pip
       - name: Install and start K3S
         run: |
-          curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=${{matrix.install_k3s_version}} INSTALL_K3S_CHANNEL=stable INSTALL_K3S_EXEC=--docker K3S_KUBECONFIG_MODE=644 sh -
+          if ! echo "${{ matrix.install_k3s_version }}" | egrep '^v[0-9]+\.[0-9]+\.[0-9]+\+k3s1$'; then
+            export INSTALL_K3S_VERSION=v1.27.2+k3s1
+          else
+            export INSTALL_K3S_VERSION=${{ matrix.install_k3s_version }}
+          fi
+          curl -sfL https://get.k3s.io | INSTALL_K3S_CHANNEL=stable INSTALL_K3S_EXEC=--docker K3S_KUBECONFIG_MODE=644 sh -
           until kubectl --kubeconfig=/etc/rancher/k3s/k3s.yaml cluster-info ; do sleep 10s ; done
           cp /etc/rancher/k3s/k3s.yaml /home/runner/.kubeconfig
           echo "- name: fake_token_user" >> $KUBECONFIG
@@ -140,25 +145,18 @@ jobs:
           echo '127.0.0.1 azurite'  | sudo tee -a /etc/hosts
       - run: make install PROFILE=${{matrix.profile}} STATIC_FILES=false
         name: Install manifests
-      - run: make controller $(go env GOPATH)/bin/goreman STATIC_FILES=false
+      - run: make controller kit STATIC_FILES=false
         name: Build controller
       - run: make cli STATIC_FILES=false
         if: ${{matrix.test == 'test-api' || matrix.test == 'test-cli' || matrix.test == 'test-java-sdk' || matrix.test == 'test-python-sdk'}}
         name: Build CLI
-      - run:  ./hack/port-forward.sh
-        name: Start port forward
-      - run: make start PROFILE=${{matrix.profile}} AUTH_MODE=client STATIC_FILES=false LOG_LEVEL=info API=${{matrix.test == 'test-api' || matrix.test == 'test-cli' || matrix.test == 'test-java-sdk' || matrix.test == 'test-python-sdk'}} UI=false LOGS=false > /tmp/argo.log 2>&1 &
+      - run: make start PROFILE=${{matrix.profile}} AUTH_MODE=client STATIC_FILES=false LOG_LEVEL=info API=${{matrix.test == 'test-api' || matrix.test == 'test-cli' || matrix.test == 'test-java-sdk' || matrix.test == 'test-python-sdk'}} UI=false > /tmp/argo.log 2>&1 &
         name: Start controller/API
-      - run: make wait
+      - run: make wait API=${{matrix.test == 'test-api' || matrix.test == 'test-cli' || matrix.test == 'test-java-sdk' || matrix.test == 'test-python-sdk'}}
         timeout-minutes: 4
         name: Wait for controller to be up
       - name: Run tests ${{matrix.test}}
-        # https://github.com/marketplace/actions/retry-step
-        uses: nick-fields/[email protected]
-        with:
-          timeout_minutes: 20
-          max_attempts: 2
-          command: make ${{matrix.test}} E2E_SUITE_TIMEOUT=20m STATIC_FILES=false
+        run: make ${{matrix.test}} E2E_SUITE_TIMEOUT=20m STATIC_FILES=false
       - if: ${{ failure() }}
         name: MinIO/MySQL deployment
         run: |
@@ -202,9 +200,9 @@ jobs:
       GOPATH: /home/runner/go
     steps:
       - uses: actions/checkout@v3
-      - uses: actions/setup-go@v3
+      - uses: actions/setup-go@v4
         with:
-          go-version: "1.19"
+          go-version: "1.20"
           cache: true
       - name: Install protoc
         run: |
@@ -235,9 +233,9 @@ jobs:
       GOPATH: /home/runner/go
     steps:
       - uses: actions/checkout@v3
-      - uses: actions/setup-go@v3
+      - uses: actions/setup-go@v4
         with:
-          go-version: "1.19"
+          go-version: "1.20"
           cache: true
       - run: make lint STATIC_FILES=false
       - run: git diff --exit-code
@@ -252,7 +250,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: "19"
+          node-version: "16"
           cache: yarn
           cache-dependency-path: ui/yarn.lock
       - run: yarn --cwd ui install

.github/workflows/dependabot-reviewer.yml

@@ -15,7 +15,7 @@ jobs:
     steps:
       - name: Dependabot metadata
         id: metadata
-        uses: dependabot/fetch-metadata@v1.3.6
+        uses: dependabot/fetch-metadata@v1.6.0
         with:
           github-token: "${{ secrets.GITHUB_TOKEN }}"
       - name: Approve PR

.github/workflows/gh-pages.yaml.norun

@@ -23,9 +23,9 @@ jobs:
       - uses: actions/setup-python@v4
         with:
           python-version: 3.9
-      - uses: actions/setup-go@v3
+      - uses: actions/setup-go@v4
         with:
-          go-version: '1.19'
+          go-version: '1.20'
       - uses: actions/setup-node@v3
         with:
           node-version: "19"

.github/workflows/pr.yaml

@@ -0,0 +1,17 @@
+name: PR
+
+on:
+  pull_request_target:
+    types:
+      - opened
+      - edited
+      - reopened
+      - synchronize
+
+jobs:
+  title-check:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: amannn/action-semantic-pull-request@v5
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}