IPShield Intial Code Completed #5

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.

Sign up for GitHub

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

Jump to bottom

Merged

msafwankarim merged 17 commits into main from sa-6985-ipshield-operator

Feb 13, 2025

Contributor

msafwankarim commented Feb 7, 2025

No description provided.

msafwankarim added 4 commits

February 6, 2025 21:16


          Initial working code. Need to fix conditions

809f32f


          Fixed issues in conditions

0c8ba54


          Updated Readme

0c54483


          Removed contributing section

msafwankarim marked this pull request as ready for review

February 10, 2025 08:41

msafwankarim added 6 commits

February 10, 2025 17:32


          Pushed test cases and other local changes

fee29f3


          Merge remote-tracking branch 'origin' into sa-6985-ipshield-operator

ad6b3ea


          Completed e2e test cases

5d7c34b


          Add basic unit tests

8ad4f7f


          Added test case to validate configmap is being properly cleaned up

6f96fcd


          Merge branch 'main' into sa-6985-ipshield-operator

5daeaa5

# Conflicts:
#	config/manager/kustomization.yaml

github-actions bot commented Feb 11, 2025

@msafwankarim Image is available for testing. docker pull ghcr.io/stakater/ipshield:SNAPSHOT-PR-5-5daeaa53

Juansasa reviewed

View reviewed changes

internal/controller/routewhitelist_controller.go Show resolved Hide resolved

Juansasa reviewed

View reviewed changes

internal/controller/routewhitelist_controller.go Outdated Show resolved Hide resolved

Juansasa reviewed

View reviewed changes

internal/controller/routewhitelist_controller.go Outdated Show resolved Hide resolved

Juansasa reviewed

View reviewed changes

internal/controller/routewhitelist_controller.go Outdated Show resolved Hide resolved

Juansasa reviewed

View reviewed changes

internal/controller/routewhitelist_controller.go Outdated Show resolved Hide resolved


          Resolved comments

df39b87

github-actions bot commented Feb 12, 2025

@msafwankarim Image is available for testing. docker pull ghcr.io/stakater/ipshield:SNAPSHOT-PR-5-df39b874


          Logged error in case of status update fails

9615d1b

github-actions bot commented Feb 12, 2025

@msafwankarim Image is available for testing. docker pull ghcr.io/stakater/ipshield:SNAPSHOT-PR-5-9615d1b9

Juansasa reviewed

View reviewed changes

cmd/main.go Show resolved Hide resolved

msafwankarim added 4 commits

February 13, 2025 12:53


          Fixed the linting issues and resolved comments

7053b3b


          Removed exportloopref linter

edf191e


          Upgraded linter version

eb9b499


          Ignored fprintf errors

7ee0123

github-actions bot commented Feb 13, 2025

@msafwankarim Image is available for testing. docker pull ghcr.io/stakater/ipshield:0.0.1-SNAPSHOT-PR-5-7ee01231

Juansasa reviewed

View reviewed changes

internal/controller/routewhitelist_controller.go Outdated Show resolved Hide resolved


          Added owner reference to configmap to CRDs

2dc3b0d

github-actions bot commented Feb 13, 2025

@msafwankarim Image is available for testing. docker pull ghcr.io/stakater/ipshield:0.0.1-SNAPSHOT-PR-5-2dc3b0d9

Juansasa reviewed

View reviewed changes

internal/controller/routewhitelist_controller.go Show resolved Hide resolved

Juansasa approved these changes

View reviewed changes

msafwankarim merged commit fdde8d0 into main

1 check passed

msafwankarim deleted the sa-6985-ipshield-operator branch

February 13, 2025 11:34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet