Skip to content
This repository has been archived by the owner on Dec 16, 2024. It is now read-only.

Separate policy controller #644

Merged
merged 1 commit into from
Nov 2, 2023
Merged

Separate policy controller #644

merged 1 commit into from
Nov 2, 2023

Conversation

maleck13
Copy link
Contributor

partially fixes
#639

@maleck13 maleck13 requested a review from mikenairn October 26, 2023 14:25
mikenairn
mikenairn reviewed Oct 26, 2023
View reviewed changes
.vscode/settings.json Outdated Show resolved Hide resolved
@maleck13 maleck13 temporarily deployed to e2e-internal October 27, 2023 15:53 — with GitHub Actions Inactive
@maleck13
Copy link
Contributor Author

e2e pass here https://github.com/Kuadrant/multicluster-gateway-controller/actions/runs/6669439243

@mikenairn mikenairn force-pushed the separate-policy-controller branch from e821885 to 9922fde Compare November 1, 2023 17:05
@mikenairn mikenairn temporarily deployed to e2e-internal November 1, 2023 18:57 — with GitHub Actions Inactive
@maleck13 maleck13 force-pushed the separate-policy-controller branch from 9922fde to b9e13ce Compare November 2, 2023 09:21
@maleck13 maleck13 temporarily deployed to e2e-internal November 2, 2023 09:21 — with GitHub Actions Inactive
@maleck13 maleck13 force-pushed the separate-policy-controller branch from b9e13ce to 21e7399 Compare November 2, 2023 09:27
@maleck13 maleck13 temporarily deployed to e2e-internal November 2, 2023 13:41 — with GitHub Actions Inactive
@maleck13
add new main
7a42f30 
integration tests split and working

add ocm enabled flag

add image build action

add deployment to kustomize

fix metrics ports for local-setup

e2e

update readme and some make changes
@maleck13 maleck13 force-pushed the separate-policy-controller branch from 090f9ab to 7a42f30 Compare November 2, 2023 14:36
@mikenairn
Copy link
Member

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm label Nov 2, 2023
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 2, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: maleck13, mikenairn

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@maleck13 maleck13 enabled auto-merge November 2, 2023 14:52
@maleck13 maleck13 disabled auto-merge November 2, 2023 14:52
@maleck13 maleck13 enabled auto-merge November 2, 2023 14:53
@maleck13 maleck13 disabled auto-merge November 2, 2023 14:53
@maleck13 maleck13 merged commit a75da13 into main Nov 2, 2023
8 of 9 checks passed
@maksymvavilov maksymvavilov mentioned this pull request Nov 8, 2023
Closed
2 tasks
@mikenairn mikenairn mentioned this pull request Nov 15, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mikenairn mikenairn mikenairn left review comments

Assignees

@mikenairn mikenairn

Labels
approved lgtm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maleck13 @mikenairn