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.

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

[Helm Chart] Fix Network Policies, address issues with Values #325

Merged
merged 5 commits into from
Sep 1, 2023
Merged

[Helm Chart] Fix Network Policies, address issues with Values #325

merged 5 commits into from
Sep 1, 2023

Conversation

i5okie
Copy link
Contributor

@i5okie i5okie commented Aug 31, 2023 •
edited
Loading

  • Fix issues with network policies
    • policy does not get created due to file formatting.
    • policy allows communication to the agent via the http port and not the admin port.
  • Clean up Values file
    • replace image.tag value with empty string to use appVersion in the Chart.yaml by default
    • update appVersion to 2.0.0-beta1
    • remove OpenShift specific annotations

@i5okie
Update app image
9307550 
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@i5okie
Separate agent ingress into its own file
4b431b6 
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@i5okie
controller talks to the agent via admin port
96c1c25 
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@i5okie
Cleanup values
b2c627a 
Signed-off-by: Ivan Polchenko <2119240+i5okie@users.noreply.github.com>
@i5okie
Merge branch '2.0-development' into feature/helm-chart-fixes
0c949ae
@i5okie i5okie requested a review from esune August 31, 2023 23:49
@i5okie i5okie self-assigned this Aug 31, 2023
@esune esune merged commit 78cb9e6 into bcgov:2.0-development Sep 1, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@esune esune esune approved these changes

Assignees

@i5okie i5okie

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@i5okie @esune