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

Automator: merge upstream changes to openshift-service-mesh/sail-operator@main #141

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
82 commits
Select commit Hold shift + click to select a range
3ece647
Add release workflow (#332)
bmangoen Sep 12, 2024
468f457
Fix setup-multi-primary script (#340)
sridhargaddam Sep 16, 2024
7577aaf
Add README file for Helm charts section (#338)
MaxBab Sep 17, 2024
f896eb5
Increase run operator-sdk run bundle timeout in OLM deploy (#345)
fjglira Sep 18, 2024
5aea134
Update latest Istio version to latest commit upstream (#348)
fjglira Sep 19, 2024
9d12b6f
added helm overrides for proxy image as is available for operator (#339)
twhite0 Sep 20, 2024
1dff7d9
Show name of active revision in Istio/RemoteIstio status (#350)
luksa Sep 20, 2024
c5e8021
Add External Controlplane to user docs (#335)
nrfox Sep 20, 2024
bd6e048
Use Kubernetes defaults for imagePullPolicy (#352)
dgn Sep 23, 2024
ab57cdf
Adding E2e multicluster test (#342)
fjglira Sep 23, 2024
005d1c3
Adding wait after olm install in olm test setup (#357)
fjglira Sep 24, 2024
dbf31fe
Automatically adjust default release channel (#353)
dgn Sep 24, 2024
4c64887
Update test labels in the wayofwork documentation (#359)
fjglira Sep 24, 2024
87f9fbd
Add CODE-OF-CONDUCT to the project (#362)
MaxBab Sep 24, 2024
ae7ba93
Fix formatting in the charts readme page (#364)
sridhargaddam Sep 25, 2024
96e2c0e
Add pull request template (#366)
MaxBab Sep 26, 2024
c199a1b
Update helm chart README (#370)
MaxBab Sep 27, 2024
837ffbc
E2E kubectl util refactor (#368)
fjglira Sep 27, 2024
4da701e
Update dependencies and fix transform (#374)
luksa Sep 30, 2024
196dc48
Add ability to specify resources in helm chart (#375)
tjons Sep 30, 2024
2a194f0
Improve pull request template (#376)
MaxBab Sep 30, 2024
e1fcbdb
Implement e2e dualStack control plane tests (#373)
sridhargaddam Sep 30, 2024
c6f30e2
Fix broken url for sample gateway (#377)
sridhargaddam Oct 1, 2024
93cd455
Update/Fix Istioctl links and docs (#349)
longmuir Oct 1, 2024
c7164d4
Adding default value for IP_FAMILY (#378)
FilipB Oct 1, 2024
9de503c
Fix error handling in deployBookinfo (#384)
sridhargaddam Oct 7, 2024
e4ffa47
Implement e2e dualStack data plane tests (#383)
sridhargaddam Oct 7, 2024
baada7c
Clean up code (#392)
luksa Oct 8, 2024
a7f0980
Simplify supportedversion.go and add unit tests (#391)
luksa Oct 8, 2024
2593bcb
Add Helm artifacts publish automation (#385)
MaxBab Oct 8, 2024
a9acf5a
Ensure crd-all.gen.yaml file is found in newer and older Istio versions
luksa Oct 7, 2024
9be8699
Remove reference to istiod-remote chart
luksa Oct 7, 2024
477c4c7
Update Istio versions
luksa Oct 7, 2024
31f96d9
Add generics to EnqueueEventLogger and AdditionNotifierQueue
luksa Oct 7, 2024
747fcfb
Make gen
luksa Oct 7, 2024
1336282
Temporarily disable RemoteIstio tests for 1.24+
luksa Oct 8, 2024
216be06
Print Istio CNI logs when test fails
luksa Oct 8, 2024
a664127
Ensure Primary-Remote e2e test is also skipped for 1.24 pre-release v…
luksa Oct 8, 2024
b43355f
Fix patch-csv.sh so that it finds hub, image, and tag in newer Istio …
luksa Oct 8, 2024
bb67c21
Make patch-csv.sh more readable
luksa Oct 8, 2024
061d116
Don't add istiod-remote chart when updating latest version (#396)
luksa Oct 8, 2024
8e10ad5
Improve kubectl utility in e2e tests (#394)
luksa Oct 8, 2024
68c6585
Add the make clean target
jwendell Oct 8, 2024
ade9a71
Fix the update location for the rbac proxy
jwendell Oct 8, 2024
3656ec9
Replace `manager` with `sail-operator` for the binary name
jwendell Oct 8, 2024
4136a7b
Create GH Action for update-deps (#393)
dgn Oct 9, 2024
fcb8418
Fix flakiness in multi-cluster tests (#400)
luksa Oct 9, 2024
1c56d16
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 9, 2024
845bd13
Use upstream builder image in update-deps workflow (#399)
dgn Oct 9, 2024
5f65f6c
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 10, 2024
b263e62
Optimize multicluster e2e tests (#403)
luksa Oct 10, 2024
cfc54d2
Faster cleanup in dualstack_test.go (#404)
luksa Oct 10, 2024
ae7afa3
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 11, 2024
1f338ff
Documentation for dual-stack support (#407)
sridhargaddam Oct 11, 2024
521379e
Fix e2e test suite names (#409)
luksa Oct 11, 2024
9b90789
Centralize operator installation code in e2e tests (#411)
luksa Oct 11, 2024
9c95e60
Enable verbose logging in operator in e2e tests (#412)
luksa Oct 11, 2024
318fafd
helmified imagePullSecrets and resources (#371)
twhite0 Oct 11, 2024
1804918
Add operatorLogLevel to Operator Helm chart (#410)
luksa Oct 11, 2024
991cac7
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 12, 2024
6ccd682
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 13, 2024
5af04cb
Skip control plane tests for dualStack job (#414)
sridhargaddam Oct 14, 2024
0dd4827
Test coverage workflow (#387)
bmangoen Oct 14, 2024
55817bc
Generate asciidoc API reference (#397)
bmangoen Oct 14, 2024
e1c447d
Add debug info to multicluster e2e tests (#417)
MaxBab Oct 14, 2024
927eaaa
Fix typo in helm-artifacts-publish ENV var command (#419)
MaxBab Oct 14, 2024
e6155da
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 15, 2024
d459300
Make spec.namespace fields immutable (#418)
dgn Oct 15, 2024
bd0660a
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 16, 2024
8896f99
Minor updates to docs (#425)
sridhargaddam Oct 16, 2024
b10e7b7
Clean up gateway sections in README.md (#428)
longmuir Oct 16, 2024
dea3547
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 17, 2024
62d9c90
Honor the `--log-enqueue-events` option in all controllers (#432)
luksa Oct 17, 2024
8d78113
Support metrics on dualStack clusters (#426)
sridhargaddam Oct 17, 2024
4af5c27
Convert all values to pointers (#433)
luksa Oct 17, 2024
fae1cef
Remove `--default-profile` in favor of automatic platform detection (…
luksa Oct 17, 2024
e75d7d7
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 18, 2024
bc065b5
Reduce minimum deletion grace period to 0s (#436)
dgn Oct 18, 2024
79dedbc
Also print version when running `./sail-operator version` (#438)
luksa Oct 18, 2024
411faaf
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 19, 2024
552d34f
Automator: Update dependencies in istio-ecosystem/sail-operator@main …
openshift-service-mesh-bot Oct 20, 2024
73d0583
Automated merge
Oct 21, 2024
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
2 changes: 1 addition & 1 deletion .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "istio build-tools",
"image": "gcr.io/istio-testing/build-tools:master-0aa2afb4bac9a4fd1bfe50a929c077a643066b3a",
"image": "gcr.io/istio-testing/build-tools:master-4759bf88d40172234fc6a0b9e11a4c5f1ea58a90",
"privileged": true,
"remoteEnv": {
"USE_GKE_GCLOUD_AUTH_PLUGIN": "True",
Expand Down
4 changes: 4 additions & 0 deletions .github/codecov.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
ignore:
- "api"
- "hack"
- "tests"
40 changes: 40 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
<!-- Thanks for sending a pull request! Here are some tips for you:

1. If this is your first time, please read our contributor guidelines: https://github.com/istio-ecosystem/sail-operator/blob/main/CONTRIBUTING.md
2. Discuss your changes before you start working on them. You can open a new issue in the [Sail Operator GitHub repository](https://github.com/istio-ecosystem/sail-operator/issues) or start a discussion in the [Sail Operator Discussion](https://github.com/istio-ecosystem/sail-operator/discussions). By this way, you can get feedback from the community and ensure that your changes are aligned with the project goals.
3. If the PR is unfinished, make is as a draft.
-->

#### What type of PR is this?
<!--
In order to minimize the time taken to categorize your PR, add a label accoutring to the PR type defined above.
Please, use the following labels, according to the PR type:
* Enhancement / New Feature - enhancement
* Bug Fix - bug
* Refactor - cleanup/refactor
* Optimization - enhancement
* Test - test-e2e
* Documentation Update - documentation
-->

- [ ] Enhancement / New Feature
- [ ] Bug Fix
- [ ] Refactor
- [ ] Optimization
- [ ] Test
- [ ] Documentation Update

#### What this PR does / why we need it:


#### Which issue(s) this PR fixes:
<!--
*Automatically closes linked issue when PR is merged.
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
Add related issue or PR if exists.
-->
Fixes #

Related Issue/PR #

#### Additional information:
21 changes: 21 additions & 0 deletions .github/workflows/coverage.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
name: Code coverage workflow

on: [push, pull_request]

jobs:
coverage:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Run coverage
run: |
make test.unit \
-e COVERAGE=true
- name: Upload to Codecov
uses: codecov/codecov-action@v4
with:
file: out/coverage.out
token: ${{ secrets.CODECOV_TOKEN }}
fail_ci_if_error: true
32 changes: 32 additions & 0 deletions .github/workflows/helm.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
name: Publish Helm artifact

on:
workflow_dispatch:
inputs:
release_version:
description: "Release version"
required: true

run-name: Publish Helm artifact ${{ inputs.release_version }}

env:
GIT_USER: ${{ secrets.GIT_USER }}
GITHUB_TOKEN: ${{ secrets.GIT_TOKEN }}
VERSION: ${{ inputs.release_version }}

jobs:
helm:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4

- name: Publish Helm artifact to "gh-pages" branch
run: |
make helm-artifacts-publish \
-e GIT_CONFIG_USER_NAME="$GIT_CONFIG_USER_NAME" \
-e GIT_CONFIG_USER_EMAIL="$GIT_CONFIG_USER_EMAIL" \
-e OPERATOR_VERSION=$VERSION
env:
GIT_CONFIG_USER_NAME: "${{ github.actor }}"
GIT_CONFIG_USER_EMAIL: "${{ github.actor_id }}+${{ github.actor }}@users.noreply.github.com"
89 changes: 89 additions & 0 deletions .github/workflows/release.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,89 @@
name: Release workflow

on:
workflow_dispatch:
inputs:
release_version:
description: "Release version"
required: true
bundle_channels:
description: "Bundle channels"
required: true
default: candidates
is_draft_release:
description: "Draft release"
type: boolean
required: false
default: true
is_pre_release:
description: "Pre-release"
type: boolean
required: false
default: false

run-name: Release ${{ inputs.release_version }}

env:
GIT_USER: ${{ secrets.GIT_USER }}
GITHUB_TOKEN: ${{ secrets.GIT_TOKEN }}
VERSION: ${{ inputs.release_version }}

jobs:
release:
runs-on: ubuntu-latest

steps:
- name: Login to quay.io
uses: docker/login-action@v3
with:
registry: quay.io
username: ${{ secrets.QUAY_USER }}
password: ${{ secrets.QUAY_PWD }}

- uses: actions/checkout@v4

- name: Build and push operator image
run: |
make docker-buildx \
-e TAG=$VERSION

- name: Generate bundle metadata
run: |
make bundle \
-e CHANNELS=$CHANNELS
env:
CHANNELS: ${{ inputs.bundle_channels }}

- name: Publish bundle in operatorhub.io
run: |
make bundle-publish \
-e GIT_CONFIG_USER_NAME="$GIT_CONFIG_USER_NAME" \
-e GIT_CONFIG_USER_EMAIL="$GIT_CONFIG_USER_EMAIL" \
-e OPERATOR_VERSION=$VERSION \
-e OPERATOR_HUB=community-operators \
-e OWNER=k8s-operatorhub \
-e FORK=maistra
env:
GIT_CONFIG_USER_NAME: "${{ github.actor }}"
GIT_CONFIG_USER_EMAIL: "${{ github.actor_id }}+${{ github.actor }}@users.noreply.github.com"

- name: Publish bundle in OpenShift OperatorHub
run: |
make bundle-publish \
-e GIT_CONFIG_USER_NAME="$GIT_CONFIG_USER_NAME" \
-e GIT_CONFIG_USER_EMAIL="$GIT_CONFIG_USER_EMAIL" \
-e OPERATOR_VERSION=$VERSION \
-e OWNER=redhat-openshift-ecosystem \
-e FORK=maistra
env:
GIT_CONFIG_USER_NAME: "${{ github.actor }}"
GIT_CONFIG_USER_EMAIL: "${{ github.actor_id }}+${{ github.actor }}@users.noreply.github.com"

- name: Create GitHub release
run: |
make create-gh-release \
-e GH_PRE_RELEASE=$GH_PRE_RELEASE \
-e GH_RELEASE_DRAFT=$GH_RELEASE_DRAFT
env:
GH_PRE_RELEASE: ${{ github.event.inputs.is_pre_release == 'true' }}
GH_RELEASE_DRAFT: ${{ github.event.inputs.is_draft_release == 'true' }}
50 changes: 50 additions & 0 deletions .github/workflows/update-deps.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,50 @@
name: Update-deps workflow

on:
schedule:
- cron: "0 5 * * *" # everyday at 5AM UTC
workflow_dispatch:
inputs:
branch:
description: "Branch to update"
default: "main"
required: true

run-name: update-deps

env:
GIT_USER: ${{ secrets.GIT_USER }}
GH_TOKEN: ${{ secrets.GIT_TOKEN }}
AUTOMATOR_ORG: istio-ecosystem
AUTOMATOR_REPO: sail-operator
AUTOMATOR_BRANCH: ${{ inputs.branch || 'main' }}

jobs:
update-deps:
runs-on: ubuntu-latest
container:
image: gcr.io/istio-testing/build-tools:master-4759bf88d40172234fc6a0b9e11a4c5f1ea58a90
options: --entrypoint ''

steps:
- uses: actions/checkout@v4
with:
repository: istio/test-infra
ref: master

# this is a workaround for a permissions issue when using the istio build container
- run: git config --system --add safe.directory /__w/sail-operator/sail-operator

- name: Run Automator
run: |
./tools/automator/automator.sh \
--org=$AUTOMATOR_ORG \
--repo=sail-operator \
--branch=$AUTOMATOR_BRANCH \
'--title=Automator: Update dependencies in $AUTOMATOR_ORG/$AUTOMATOR_REPO@$AUTOMATOR_BRANCH' \
--labels=auto-merge \
--email=openshiftservicemeshbot@gmail.com \
--modifier=update_deps \
--token-env \
--cmd='BUILD_WITH_CONTAINER=0 ./tools/update_deps.sh' \
--signoff
7 changes: 7 additions & 0 deletions CODE-OF-CONDUCT.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
# Sail Operator Community Code of Conduct

As contributors and maintainers of this project, and in the interest of fostering an open and welcoming community, we pledge to respect all people who contribute through reporting issues, posting feature requests, updating documentation, submitting pull requests or patches, and other activities.

All members of the Sail Operator community must abide by the [CNCF Code of Conduct](https://github.com/cncf/foundation/blob/main/code-of-conduct.md).
Only by respecting each other can we develop a productive, collaborative community.

14 changes: 2 additions & 12 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,25 +25,15 @@ If you want to contribute to the Sail Operator project, you can follow some rule
- Discuss your changes before you start working on them. You can open a new issue in the [Sail Operator GitHub repository](https://github.com/istio-ecosystem/sail-operator/issues) or start a discussion in the [Sail Operator Discussion](https://github.com/istio-ecosystem/sail-operator/discussions). By this way, you can get feedback from the community and ensure that your changes are aligned with the project goals.
- Use of Labels: We use labels in the issues to help us track the progress of the issues. You can use the labels to help you understand the status of the issue and what is needed to move forward. Those labels are:
- `backport/backport-handled`: Use this label to indicate that the issue has been backported to the appropriate branches.
- `testing`: Use this label to indicate that the issue is related to testing. Can be used in combination with other labels to mark the proper testing type, for example: `testing/e2e`, `testing/unit`, `testing/integration`.
- `test`: Use this label to indicate that the issue is related to test or add `test-needed` when a issue needs a test to be added related. Can be used in combination with other labels to mark the proper test type, for example: `test-e2e`, `test-unit`, `test-integration`.
- `good first issue`: Use this label to indicate that the issue is a good first issue for new contributors.
- `help wanted`: Use this label to indicate that the issue needs help from the community.
- `enhancement`: Use this label to indicate that the issue is an enhancement related to a new feature or improvement.
- Pull Requests: When you open a pull request, you can follow this template to help you provide the necessary information to the maintainers:
- **What type of PR is this?**
- **What this PR does / why we need it:**
- **Which issue(s) this PR fixes:** (Mark with Fixes #12345, with this the issue will be autoclosed when the PR is merged)
- **Special notes for your reviewer:**
- **Does this PR introduce a user-facing change?**
- **Additional documentation:**
- **Does this PR introduce a breaking change?**
- **Other information:**
- Labels: You can use the labels to help you track the status of the PR. The labels are the same as the issue labels. Additionally, you can use the `cleanup/refactor` to indicate that the PR is a cleanup or refactor of the codebase. Having the label just helps with filtering pull requests. It also is a hint that this work does not need an entry in the changelog

## Community meetings

This is not defined yet. We are working on defining the community meetings and how the community can participate in them. We will update this section once we have more information.

## Security Issues

If you find a security issue in the Sail Operator project, please refer to the [Security Policy](https://github.com/istio-ecosystem/sail-operator/security/policy) for more information on how to report security issues. Please do not report security issues in the public GitHub repository.
If you find a security issue in the Sail Operator project, please refer to the [Security Policy](https://github.com/istio-ecosystem/sail-operator/security/policy) for more information on how to report security issues. Please do not report security issues in the public GitHub repository.
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ FROM registry.access.redhat.com/ubi9/ubi-minimal:latest

ARG TARGETOS TARGETARCH

ADD out/${TARGETOS:-linux}_${TARGETARCH:-amd64}/manager /manager
ADD out/${TARGETOS:-linux}_${TARGETARCH:-amd64}/sail-operator /sail-operator
ADD resources /var/lib/sail-operator/resources

USER 65532:65532
WORKDIR /
ENTRYPOINT ["/manager"]
ENTRYPOINT ["/sail-operator"]
Loading
Loading