Skip to content

feat(network): implement optional cryostat egress network policy #1051

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

Merged
andrewazores merged 18 commits into from
Aug 28, 2025
Merged

feat(network): implement optional cryostat egress network policy #1051

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
0329a9c
rename
andrewazores Jan 22, 2025
33abb3d
property for enabling egress policy
andrewazores Jan 22, 2025
e7806b9
rename
andrewazores Jan 22, 2025
7dc5375
implement core egress policy
andrewazores Jan 22, 2025
760f278
format
andrewazores Jan 22, 2025
037de2b
fixup! implement core egress policy
andrewazores Jan 23, 2025
46ab9e6
fix struct member name for disabling ingress networkpolicies in test
andrewazores Aug 21, 2025
db77506
remove accidentally committed files
andrewazores Aug 21, 2025
9e373a8
rename property
andrewazores Aug 26, 2025
f4c13f1
deprecated comment
andrewazores Aug 26, 2025
cb0b79f
optional
andrewazores Aug 26, 2025
f20d00a
bundle
andrewazores Aug 26, 2025
8105d51
create egress policy using EndpointSlices, not old-style Endpoints
andrewazores Aug 26, 2025
07a7c4b
split rules
andrewazores Aug 26, 2025
812dcea
disable sample app agent startup, increase memory limit
andrewazores Aug 26, 2025
6d27230
update error message
andrewazores Aug 26, 2025
9430d52
style
andrewazores Aug 28, 2025
2e7facb
cleanup, remove 'default' namespace egress rule
andrewazores Aug 28, 2025
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
11 changes: 10 additions & 1 deletion api/v1beta2/cryostat_types.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -444,10 +444,19 @@ type NetworkPoliciesList struct {
}

type NetworkPolicyConfig struct {
// Disable the NetworkPolicy for a given service.
// Disable the NetworkPolicies (Ingress and Egress) for a given service.
// Deprecated: use IngressDisabled and EgressEnabled instead.
// +optional
// +operator-sdk:csv:customresourcedefinitions:type=spec,displayName="Disable NetworkPolicy creation",xDescriptors={"urn:alm:descriptor:com.tectonic.ui:booleanSwitch"}
Disabled *bool `json:"disabled,omitempty"`
// Disable the NetworkPolicy for ingress to a given pod. Enabled by default.
// +optional
// +operator-sdk:csv:customresourcedefinitions:type=spec,displayName="Disable ingress NetworkPolicy creation",xDescriptors={"urn:alm:descriptor:com.tectonic.ui:booleanSwitch"}
IngressDisabled *bool `json:"ingressDisabled,omitempty"`
// Enable the NetworkPolicy for egress from a given pod. Disabled by default.
// +optional
// +operator-sdk:csv:customresourcedefinitions:type=spec,displayName="Enable egress NetworkPolicy creation",xDescriptors={"urn:alm:descriptor:com.tectonic.ui:booleanSwitch"}
EgressEnabled *bool `json:"egressEnabled,omitempty"`
}

// NetworkConfiguration provides customization for how to expose a Cryostat
Expand Down
10 changes: 10 additions & 0 deletions api/v1beta2/zz_generated.deepcopy.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

50 changes: 45 additions & 5 deletions bundle/manifests/cryostat-operator.clusterserviceversion.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ metadata:
capabilities: Seamless Upgrades
categories: Monitoring, Developer Tools
containerImage: quay.io/cryostat/cryostat-operator:4.1.0-dev
createdAt: "2025-06-12T15:56:01Z"
createdAt: "2025-08-26T15:38:48Z"
description: JVM monitoring and profiling tool
operatorframework.io/initialization-resource: |-
{
Expand Down Expand Up @@ -183,26 +183,66 @@ spec:
- description: Options to customize the NetworkPolicy objects created for Cryostat's various Services.
displayName: Network Policies
path: networkPolicies
- description: Disable the NetworkPolicy for a given service.
- description: 'Disable the NetworkPolicies (Ingress and Egress) for a given service. Deprecated: use IngressDisabled and EgressEnabled instead.'
displayName: Disable NetworkPolicy creation
path: networkPolicies.coreConfig.disabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for a given service.
- description: Enable the NetworkPolicy for egress from a given pod. Disabled by default.
displayName: Enable egress NetworkPolicy creation
path: networkPolicies.coreConfig.egressEnabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for ingress to a given pod. Enabled by default.
displayName: Disable ingress NetworkPolicy creation
path: networkPolicies.coreConfig.ingressDisabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: 'Disable the NetworkPolicies (Ingress and Egress) for a given service. Deprecated: use IngressDisabled and EgressEnabled instead.'
displayName: Disable NetworkPolicy creation
path: networkPolicies.databaseConfig.disabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for a given service.
- description: Enable the NetworkPolicy for egress from a given pod. Disabled by default.
displayName: Enable egress NetworkPolicy creation
path: networkPolicies.databaseConfig.egressEnabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for ingress to a given pod. Enabled by default.
displayName: Disable ingress NetworkPolicy creation
path: networkPolicies.databaseConfig.ingressDisabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: 'Disable the NetworkPolicies (Ingress and Egress) for a given service. Deprecated: use IngressDisabled and EgressEnabled instead.'
displayName: Disable NetworkPolicy creation
path: networkPolicies.reportsConfig.disabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for a given service.
- description: Enable the NetworkPolicy for egress from a given pod. Disabled by default.
displayName: Enable egress NetworkPolicy creation
path: networkPolicies.reportsConfig.egressEnabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for ingress to a given pod. Enabled by default.
displayName: Disable ingress NetworkPolicy creation
path: networkPolicies.reportsConfig.ingressDisabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: 'Disable the NetworkPolicies (Ingress and Egress) for a given service. Deprecated: use IngressDisabled and EgressEnabled instead.'
displayName: Disable NetworkPolicy creation
path: networkPolicies.storageConfig.disabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Enable the NetworkPolicy for egress from a given pod. Disabled by default.
displayName: Enable egress NetworkPolicy creation
path: networkPolicies.storageConfig.egressEnabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for ingress to a given pod. Enabled by default.
displayName: Disable ingress NetworkPolicy creation
path: networkPolicies.storageConfig.ingressDisabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Options to configure the Cryostat deployments and pods metadata
displayName: Operand metadata
path: operandMetadata
Expand Down
48 changes: 44 additions & 4 deletions bundle/manifests/operator.cryostat.io_cryostats.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5890,29 +5890,69 @@ spec:
service.
properties:
disabled:
description: Disable the NetworkPolicy for a given service.
description: |-
Disable the NetworkPolicies (Ingress and Egress) for a given service.
Deprecated: use IngressDisabled and EgressEnabled instead.
type: boolean
egressEnabled:
description: Enable the NetworkPolicy for egress from a given
pod. Disabled by default.
type: boolean
ingressDisabled:
description: Disable the NetworkPolicy for ingress to a given
pod. Enabled by default.
type: boolean
type: object
databaseConfig:
description: NetworkPolicy configuration for the database service.
properties:
disabled:
description: Disable the NetworkPolicy for a given service.
description: |-
Disable the NetworkPolicies (Ingress and Egress) for a given service.
Deprecated: use IngressDisabled and EgressEnabled instead.
type: boolean
egressEnabled:
description: Enable the NetworkPolicy for egress from a given
pod. Disabled by default.
type: boolean
ingressDisabled:
description: Disable the NetworkPolicy for ingress to a given
pod. Enabled by default.
type: boolean
type: object
reportsConfig:
description: NetworkPolicy configuration for the cryostat-reports
service.
properties:
disabled:
description: Disable the NetworkPolicy for a given service.
description: |-
Disable the NetworkPolicies (Ingress and Egress) for a given service.
Deprecated: use IngressDisabled and EgressEnabled instead.
type: boolean
egressEnabled:
description: Enable the NetworkPolicy for egress from a given
pod. Disabled by default.
type: boolean
ingressDisabled:
description: Disable the NetworkPolicy for ingress to a given
pod. Enabled by default.
type: boolean
type: object
storageConfig:
description: NetworkPolicy configuration for the storage service.
properties:
disabled:
description: Disable the NetworkPolicy for a given service.
description: |-
Disable the NetworkPolicies (Ingress and Egress) for a given service.
Deprecated: use IngressDisabled and EgressEnabled instead.
type: boolean
egressEnabled:
description: Enable the NetworkPolicy for egress from a given
pod. Disabled by default.
type: boolean
ingressDisabled:
description: Disable the NetworkPolicy for ingress to a given
pod. Enabled by default.
type: boolean
type: object
type: object
Expand Down
48 changes: 44 additions & 4 deletions config/crd/bases/operator.cryostat.io_cryostats.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5877,29 +5877,69 @@ spec:
service.
properties:
disabled:
description: Disable the NetworkPolicy for a given service.
description: |-
Disable the NetworkPolicies (Ingress and Egress) for a given service.
Deprecated: use IngressDisabled and EgressEnabled instead.
type: boolean
egressEnabled:
description: Enable the NetworkPolicy for egress from a given
pod. Disabled by default.
type: boolean
ingressDisabled:
description: Disable the NetworkPolicy for ingress to a given
pod. Enabled by default.
type: boolean
type: object
databaseConfig:
description: NetworkPolicy configuration for the database service.
properties:
disabled:
description: Disable the NetworkPolicy for a given service.
description: |-
Disable the NetworkPolicies (Ingress and Egress) for a given service.
Deprecated: use IngressDisabled and EgressEnabled instead.
type: boolean
egressEnabled:
description: Enable the NetworkPolicy for egress from a given
pod. Disabled by default.
type: boolean
ingressDisabled:
description: Disable the NetworkPolicy for ingress to a given
pod. Enabled by default.
type: boolean
type: object
reportsConfig:
description: NetworkPolicy configuration for the cryostat-reports
service.
properties:
disabled:
description: Disable the NetworkPolicy for a given service.
description: |-
Disable the NetworkPolicies (Ingress and Egress) for a given service.
Deprecated: use IngressDisabled and EgressEnabled instead.
type: boolean
egressEnabled:
description: Enable the NetworkPolicy for egress from a given
pod. Disabled by default.
type: boolean
ingressDisabled:
description: Disable the NetworkPolicy for ingress to a given
pod. Enabled by default.
type: boolean
type: object
storageConfig:
description: NetworkPolicy configuration for the storage service.
properties:
disabled:
description: Disable the NetworkPolicy for a given service.
description: |-
Disable the NetworkPolicies (Ingress and Egress) for a given service.
Deprecated: use IngressDisabled and EgressEnabled instead.
type: boolean
egressEnabled:
description: Enable the NetworkPolicy for egress from a given
pod. Disabled by default.
type: boolean
ingressDisabled:
description: Disable the NetworkPolicy for ingress to a given
pod. Enabled by default.
type: boolean
type: object
type: object
Expand Down
60 changes: 56 additions & 4 deletions config/manifests/bases/cryostat-operator.clusterserviceversion.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -203,26 +203,78 @@ spec:
various Services.
displayName: Network Policies
path: networkPolicies
- description: Disable the NetworkPolicy for a given service.
- description: 'Disable the NetworkPolicies (Ingress and Egress) for a given
service. Deprecated: use IngressDisabled and EgressEnabled instead.'
displayName: Disable NetworkPolicy creation
path: networkPolicies.coreConfig.disabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for a given service.
- description: Enable the NetworkPolicy for egress from a given pod. Disabled
by default.
displayName: Enable egress NetworkPolicy creation
path: networkPolicies.coreConfig.egressEnabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for ingress to a given pod. Enabled
by default.
displayName: Disable ingress NetworkPolicy creation
path: networkPolicies.coreConfig.ingressDisabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: 'Disable the NetworkPolicies (Ingress and Egress) for a given
service. Deprecated: use IngressDisabled and EgressEnabled instead.'
displayName: Disable NetworkPolicy creation
path: networkPolicies.databaseConfig.disabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for a given service.
- description: Enable the NetworkPolicy for egress from a given pod. Disabled
by default.
displayName: Enable egress NetworkPolicy creation
path: networkPolicies.databaseConfig.egressEnabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for ingress to a given pod. Enabled
by default.
displayName: Disable ingress NetworkPolicy creation
path: networkPolicies.databaseConfig.ingressDisabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: 'Disable the NetworkPolicies (Ingress and Egress) for a given
service. Deprecated: use IngressDisabled and EgressEnabled instead.'
displayName: Disable NetworkPolicy creation
path: networkPolicies.reportsConfig.disabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for a given service.
- description: Enable the NetworkPolicy for egress from a given pod. Disabled
by default.
displayName: Enable egress NetworkPolicy creation
path: networkPolicies.reportsConfig.egressEnabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for ingress to a given pod. Enabled
by default.
displayName: Disable ingress NetworkPolicy creation
path: networkPolicies.reportsConfig.ingressDisabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: 'Disable the NetworkPolicies (Ingress and Egress) for a given
service. Deprecated: use IngressDisabled and EgressEnabled instead.'
displayName: Disable NetworkPolicy creation
path: networkPolicies.storageConfig.disabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Enable the NetworkPolicy for egress from a given pod. Disabled
by default.
displayName: Enable egress NetworkPolicy creation
path: networkPolicies.storageConfig.egressEnabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Disable the NetworkPolicy for ingress to a given pod. Enabled
by default.
displayName: Disable ingress NetworkPolicy creation
path: networkPolicies.storageConfig.ingressDisabled
x-descriptors:
- urn:alm:descriptor:com.tectonic.ui:booleanSwitch
- description: Options to configure the Cryostat deployments and pods metadata
displayName: Operand metadata
path: operandMetadata
Expand Down
10 changes: 6 additions & 4 deletions config/samples/sample-app.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,11 @@ spec:
- image: quay.io/redhat-java-monitoring/quarkus-cryostat-agent:latest
imagePullPolicy: Always
name: quarkus-test
env:
- name: JAVA_OPTS_APPEND
value: -Djava.util.logging.manager=org.jboss.logmanager.LogManager -Dcom.sun.management.jmxremote.port=9097 -Dcom.sun.management.jmxremote.ssl=false -Dcom.sun.management.jmxremote.authenticate=false
- name: QUARKUS_HTTP_HOST
value: 0.0.0.0
ports:
- containerPort: 10010
protocol: TCP
Expand All @@ -27,12 +32,9 @@ spec:
protocol: TCP
name: jfr-jmx
resources:
requests:
cpu: 200m
memory: 96Mi
limits:
cpu: 500m
memory: 192Mi
memory: 256Mi
securityContext:
allowPrivilegeEscalation: false
capabilities:
Expand Down
Loading