-
Notifications
You must be signed in to change notification settings - Fork 19
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
Closes: #371 - invalid lineliness probe for web process in latest release of app #372
Merged
Merged
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
4db390c
Closes: #371 - invalid lineliness probe for web process in latest rel…
whitej6 7559fd7
updates for feedback
whitej6 22c151b
resolve failing issues with pre-commit hooks
whitej6 78d1491
adding verbose flag to find the offending error
whitej6 9b16f3a
attempting to fix a kubescape failure
whitej6 1aefb6b
attempting to fix a kubescape failure
whitej6 6c32a0c
helm docs
whitej6 7536d09
update ci to see if compliance threshold addresses
whitej6 50255ff
fix CI
whitej6 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,15 +12,15 @@ | |
# Node ports to expose if service.type is NodePort | ||
nodePorts: | ||
# -- (int) Node port for Nautobot http choose port in Kubernetes `--service-node-port-range` typically 30000-32767 | ||
http: # yamllint disable-line rule:empty-values | ||
http: # yamllint disable-line rule:empty-values | ||
# -- (int) Node port for Nautobot https choose port in Kubernetes `--service-node-port-range` typically 30000-32767 | ||
https: # yamllint disable-line rule:empty-values | ||
https: # yamllint disable-line rule:empty-values | ||
# -- (int) Node port for Nautobot uwsgimetrics choose port in Kubernetes `--service-node-port-range` typically 30000-32767 | ||
uwsgimetrics: # yamllint disable-line rule:empty-values | ||
uwsgimetrics: # yamllint disable-line rule:empty-values | ||
# -- (IP) IP address to use as the clusterIP | ||
clusterIP: # yamllint disable-line rule:empty-values | ||
clusterIP: # yamllint disable-line rule:empty-values | ||
# -- (IP) IP address to use as the loadBalancerIP | ||
loadBalancerIP: # yamllint disable-line rule:empty-values | ||
loadBalancerIP: # yamllint disable-line rule:empty-values | ||
# -- ([]CIDR) List of allowed CIDRs to access the load balancer default 0.0.0.0/0, cloud provider dependent | ||
loadBalancerSourceRanges: [] | ||
# -- (`Cluster`, `Local`) Kubernetes externalTrafficPolicy valid values: `Cluster` or `Local` | ||
|
@@ -41,7 +41,7 @@ | |
# -- Nautobot image name, common to all deployments | ||
repository: "nautobot/nautobot" | ||
# -- Nautobot image tag, common to all deployments | ||
tag: "2.0.5-py3.11" | ||
tag: "2.1.9-py3.11" | ||
# -- [Kubernetes image pull policy](https://kubernetes.io/docs/concepts/containers/images/), common to all deployments valid values: `Always`, `Never`, or `IfNotPresent` | ||
pullPolicy: "Always" | ||
# -- ([]string) List of secret names to be used as image [pull secrets](https://kubernetes.io/docs/tasks/configure-pod-container/pull-image-private-registry/), common to all deployments | ||
|
@@ -60,12 +60,14 @@ | |
livenessProbe: | ||
# -- Enable the Nautobot Liveness Probe | ||
enabled: true | ||
httpGet: | ||
path: "/api/" | ||
port: "http" | ||
exec: | ||
command: | ||
- "bash" | ||
- "-c" | ||
- "nautobot-server health_check" | ||
initialDelaySeconds: 3 | ||
periodSeconds: 10 | ||
timeoutSeconds: 5 | ||
periodSeconds: 15 | ||
timeoutSeconds: 10 | ||
failureThreshold: 3 | ||
successThreshold: 1 | ||
|
||
|
@@ -117,7 +119,7 @@ | |
allowPrivilegeEscalation: false | ||
# -- Enable the Nautobot container security context | ||
enabled: true | ||
readOnlyRootFilesystem: false # Must be set to false due to jobs directory | ||
readOnlyRootFilesystem: false # Must be set to false due to jobs directory | ||
runAsUser: 999 | ||
runAsGroup: 999 | ||
capabilities: | ||
|
@@ -415,9 +417,9 @@ | |
# -- [[ref](https://nautobot.readthedocs.io/en/stable/docker/#nautobot_superuser_email)] Configure an email address for the super user if `nautobot.superUser.enabled` is `true` (NAUTOBOT_SUPERUSER_EMAIL)<sup>[1](#notes)</sup> | ||
email: "[email protected]" | ||
# -- [[ref](https://nautobot.readthedocs.io/en/stable/docker/#nautobot_superuser_name)] User name to use for the super user to be created if `nautobot.superUser.enabled` is `true` (NAUTOBOT_SUPERUSER_NAME)<sup>[1](#notes)</sup> | ||
username: "admin" # NAUTOBOT_SUPERUSER_NAME | ||
username: "admin" # NAUTOBOT_SUPERUSER_NAME | ||
# -- [[ref](https://nautobot.readthedocs.io/en/stable/docker/#nautobot_superuser_password)] Password to use for the super user to be created if `nautobot.superUser.enabled` is `true` (NAUTOBOT_SUPERUSER_NAME), if unset a random password will be generated<sup>[1](#notes)</sup> | ||
password: "" # NAUTOBOT_SUPERUSER_PASSWORD | ||
password: "" # NAUTOBOT_SUPERUSER_PASSWORD | ||
|
||
# -- [[ref](https://docs.nautobot.com/projects/helm-charts/en/stable/advanced-features/init-hook)] Run the Nautobot initialization tasks as a helm hook instead of an initContainer<sup>[1](#notes)</sup> | ||
singleInit: false | ||
|
@@ -436,7 +438,7 @@ | |
bufferSize: 4096 | ||
|
||
# -- ([]map[name,value]string)An array of environment variable objects (`name` and `value` are required) to add to ALL Nautobot and Nautobot Worker related deployments<sup>[1](#notes)</sup> | ||
extraVars: [] # https://docs.nautobot.com/projects/core/en/stable/configuration/optional-settings/ | ||
extraVars: [] # https://docs.nautobot.com/projects/core/en/stable/configuration/optional-settings/ | ||
# e.g. | ||
# extraVars: | ||
# - name: "NAUTOBOT_BANNER_TOP" | ||
|
@@ -557,7 +559,7 @@ | |
allowPrivilegeEscalation: false | ||
# -- Enable the Worker Celery Container security context | ||
enabled: true | ||
readOnlyRootFilesystem: false # Must be set to false due to jobs directory | ||
readOnlyRootFilesystem: false # Must be set to false due to jobs directory | ||
runAsUser: 999 | ||
capabilities: | ||
drop: | ||
|
@@ -697,7 +699,7 @@ | |
beat: | ||
enabled: true | ||
concurrency: 0 | ||
replicaCount: 1 # Beat can have a max replicaCount of 1 | ||
replicaCount: 1 # Beat can have a max replicaCount of 1 | ||
command: | ||
- "nautobot-server" | ||
- "celery" | ||
|
@@ -739,7 +741,7 @@ | |
pathType: "Prefix" | ||
|
||
# -- Override API Version (automatically detected if not set) | ||
apiVersion: # yamllint disable-line rule:empty-values | ||
apiVersion: # yamllint disable-line rule:empty-values | ||
|
||
# -- Ingress Hostname | ||
hostname: "nautobot.local" | ||
|
@@ -989,7 +991,7 @@ | |
type: "RuntimeDefault" | ||
containerSecurityContext: | ||
allowPrivilegeEscalation: false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
capabilities: | ||
drop: | ||
- "ALL" | ||
|
@@ -1076,7 +1078,7 @@ | |
primary: | ||
containerSecurityContext: | ||
allowPrivilegeEscalation: false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
capabilities: | ||
drop: | ||
- "ALL" | ||
|
@@ -1091,7 +1093,7 @@ | |
secondary: | ||
containerSecurityContext: | ||
allowPrivilegeEscalation: false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
capabilities: | ||
drop: | ||
- "ALL" | ||
|
@@ -1130,7 +1132,7 @@ | |
containerSecurityContext: | ||
enabled: true | ||
allowPrivilegeEscalation: false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
seccompProfile: | ||
type: "RuntimeDefault" | ||
capabilities: | ||
|
@@ -1169,7 +1171,7 @@ | |
create: true | ||
containerSecurityContext: | ||
allowPrivilegeEscalation: false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
readOnlyRootFilesystem: false # Must be set to false | ||
seccompProfile: | ||
type: "RuntimeDefault" | ||
capabilities: | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to go to
2.1.0
? Some changes are not just patches. Not breaking, but still: