Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package polaris for openSUSE:Factory checked in at 2023-06-22 23:26:17 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/polaris (Old) and /work/SRC/openSUSE:Factory/.polaris.new.15902 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "polaris" Thu Jun 22 23:26:17 2023 rev:18 rq:1094598 version:8.2.3 Changes: -------- --- /work/SRC/openSUSE:Factory/polaris/polaris.changes 2023-06-16 16:54:11.909476076 +0200 +++ /work/SRC/openSUSE:Factory/.polaris.new.15902/polaris.changes 2023-06-22 23:26:44.794051334 +0200 @@ -1,0 +2,7 @@ +Thu Jun 22 05:07:36 UTC 2023 - ka...@b1-systems.de + +- Update to version 8.2.3: + * add test for required fields on builtin checks (#965) + * FWI-4307 - fix checks category (#964) + +------------------------------------------------------------------- Old: ---- polaris-8.2.2.obscpio New: ---- polaris-8.2.3.obscpio ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ polaris.spec ++++++ --- /var/tmp/diff_new_pack.hLGuW0/_old 2023-06-22 23:26:45.566055271 +0200 +++ /var/tmp/diff_new_pack.hLGuW0/_new 2023-06-22 23:26:45.574055311 +0200 @@ -19,7 +19,7 @@ %define __arch_install_post export NO_BRP_STRIP_DEBUG=true Name: polaris -Version: 8.2.2 +Version: 8.2.3 Release: 0 Summary: Validation of best practices in your Kubernetes clusters License: Apache-2.0 ++++++ _service ++++++ --- /var/tmp/diff_new_pack.hLGuW0/_old 2023-06-22 23:26:45.618055536 +0200 +++ /var/tmp/diff_new_pack.hLGuW0/_new 2023-06-22 23:26:45.618055536 +0200 @@ -3,7 +3,7 @@ <param name="url">https://github.com/FairwindsOps/polaris</param> <param name="scm">git</param> <param name="exclude">.git</param> - <param name="revision">8.2.2</param> + <param name="revision">8.2.3</param> <param name="versionformat">@PARENT_TAG@</param> <param name="changesgenerate">enable</param> </service> ++++++ _servicedata ++++++ --- /var/tmp/diff_new_pack.hLGuW0/_old 2023-06-22 23:26:45.646055678 +0200 +++ /var/tmp/diff_new_pack.hLGuW0/_new 2023-06-22 23:26:45.650055699 +0200 @@ -1,6 +1,6 @@ <servicedata> <service name="tar_scm"> <param name="url">https://github.com/FairwindsOps/polaris</param> - <param name="changesrevision">85a439653dd55ae47f992a60768594a3c15503f2</param></service></servicedata> + <param name="changesrevision">4b1d6635e06a52fbaf5d8e4746d94ec6b9313493</param></service></servicedata> (No newline at EOF) ++++++ polaris-8.2.2.obscpio -> polaris-8.2.3.obscpio ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/polaris-8.2.2/checks/metadataAndNameMismatched.yaml new/polaris-8.2.3/checks/metadataAndNameMismatched.yaml --- old/polaris-8.2.2/checks/metadataAndNameMismatched.yaml 2023-06-14 16:51:21.000000000 +0200 +++ new/polaris-8.2.3/checks/metadataAndNameMismatched.yaml 2023-06-21 18:25:44.000000000 +0200 @@ -1,5 +1,6 @@ successMessage: Label app.kubernetes.io/name matches metadata.name failureMessage: Label app.kubernetes.io/name must match metadata.name +category: Reliability target: Controller schema: '$schema': http://json-schema.org/draft-07/schema diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/polaris-8.2.2/checks/priorityClassNotSet.yaml new/polaris-8.2.3/checks/priorityClassNotSet.yaml --- old/polaris-8.2.2/checks/priorityClassNotSet.yaml 2023-06-14 16:51:21.000000000 +0200 +++ new/polaris-8.2.3/checks/priorityClassNotSet.yaml 2023-06-21 18:25:44.000000000 +0200 @@ -1,6 +1,6 @@ successMessage: Priority class has been set failureMessage: Priority class should be set -category: Security +category: Reliability target: PodSpec schema: '$schema': http://json-schema.org/draft-07/schema diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/polaris-8.2.2/docs/admission-controller.md new/polaris-8.2.3/docs/admission-controller.md --- old/polaris-8.2.2/docs/admission-controller.md 2023-06-14 16:51:21.000000000 +0200 +++ new/polaris-8.2.3/docs/admission-controller.md 2023-06-21 18:25:44.000000000 +0200 @@ -53,7 +53,7 @@ the deployment. To enable the mutating webhook, add `--set webhook.mutate=true` to your -Helm instlallation command. +Helm installation command. The following default checks currently have mutation support enabled: * `hostPIDSet` diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/polaris-8.2.2/docs/changelog.md new/polaris-8.2.3/docs/changelog.md --- old/polaris-8.2.2/docs/changelog.md 2023-06-14 16:51:21.000000000 +0200 +++ new/polaris-8.2.3/docs/changelog.md 2023-06-21 18:25:44.000000000 +0200 @@ -6,6 +6,10 @@ --- +## 8.1.1 +* Add category for `metadataAndNameMismatched`. +* Fix category for `priorityClassNotSet`. + ## 8.1.0 * Add `insights-host` global flag to configure Fairwinds Insights host (defaults to `https://insights.fairwinds.com`). * Add new `auth` sub-commands be able to authenticate on Polaris using Fairwinds Insights credentials diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/polaris-8.2.2/docs/checks/reliability.md new/polaris-8.2.3/docs/checks/reliability.md --- old/polaris-8.2.2/docs/checks/reliability.md 2023-06-14 16:51:21.000000000 +0200 +++ new/polaris-8.2.3/docs/checks/reliability.md 2023-06-21 18:25:44.000000000 +0200 @@ -16,8 +16,8 @@ `pullPolicyNotAlways` | `warning` | Fails when an image pull policy is not `always`. `priorityClassNotSet` | `warning` | Fails when a priorityClassName is not set for a pod. `deploymentMissingReplicas` | `warning` | Fails when there is only one replica for a deployment. -`missingPodDisruptionBudget` | `warning` -`metadataAndNameMismatched` | `warning` +`missingPodDisruptionBudget` | `warning` | Fails when PDB is missing. +`metadataAndNameMismatched` | `warning` | Fails when label `app.kubernetes.io/name` and `metadata.name` mismatch `topologySpreadConstraint` | `warning` | Fails when there is no topology spread constraint on the pod ## Background @@ -70,4 +70,4 @@ - [Kubernetes Docs: Configure Liveness and Readiness Probes](https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-probes/) - [Utilizing Kubernetes Liveness and Readiness Probes to Automatically Recover From Failure](https://medium.com/spire-labs/utilizing-kubernetes-liveness-and-readiness-probes-to-automatically-recover-from-failure-2fe0314f2b2e) - [Kubernetes Liveness and Readiness Probes: How to Avoid Shooting Yourself in the Foot](https://blog.colinbreck.com/kubernetes-liveness-and-readiness-probes-how-to-avoid-shooting-yourself-in-the-foot/) -- [Topology Spread Cosntraints](https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/) \ No newline at end of file +- [Topology Spread Constraints](https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/) \ No newline at end of file diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/polaris-8.2.2/pkg/config/checks.go new/polaris-8.2.3/pkg/config/checks.go --- old/polaris-8.2.2/pkg/config/checks.go 2023-06-14 16:51:21.000000000 +0200 +++ new/polaris-8.2.3/pkg/config/checks.go 2023-06-21 18:25:44.000000000 +0200 @@ -22,7 +22,6 @@ var ( // BuiltInChecks contains the checks that come pre-installed w/ Polaris BuiltInChecks = map[string]SchemaCheck{} - schemaBox = (*packr.Box)(nil) // We explicitly set the order to avoid thrash in the // tests as we migrate toward JSON schema checkOrder = []string{ @@ -72,7 +71,7 @@ ) func init() { - schemaBox = packr.New("Schemas", "../../checks") + schemaBox := packr.New("Schemas", "../../checks") for _, checkID := range checkOrder { contents, err := schemaBox.Find(checkID + ".yaml") if err != nil { diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/polaris-8.2.2/pkg/config/checks_test.go new/polaris-8.2.3/pkg/config/checks_test.go --- old/polaris-8.2.2/pkg/config/checks_test.go 1970-01-01 01:00:00.000000000 +0100 +++ new/polaris-8.2.3/pkg/config/checks_test.go 2023-06-21 18:25:44.000000000 +0200 @@ -0,0 +1,16 @@ +package config + +import ( + "testing" + + "github.com/stretchr/testify/assert" +) + +func TestRequiredFieldsOnBuiltInChecks(t *testing.T) { + for _, v := range BuiltInChecks { + assert.NotEmpty(t, v.SuccessMessage) + assert.NotEmpty(t, v.FailureMessage) + assert.NotEmpty(t, v.Category) + assert.NotEmpty(t, v.Target) + } +} diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/polaris-8.2.2/test/mutation_test.go new/polaris-8.2.3/test/mutation_test.go --- old/polaris-8.2.2/test/mutation_test.go 2023-06-14 16:51:21.000000000 +0200 +++ new/polaris-8.2.3/test/mutation_test.go 2023-06-21 18:25:44.000000000 +0200 @@ -19,7 +19,7 @@ hostNetworkSet: danger hostPortSet: warning deploymentMissingReplicas: warning - priorityClassNotSet: ignore + priorityClassNotSet: warning runAsRootAllowed: danger cpuRequestsMissing: warning cpuLimitsMissing: warning ++++++ polaris.obsinfo ++++++ --- /var/tmp/diff_new_pack.hLGuW0/_old 2023-06-22 23:26:45.926057107 +0200 +++ /var/tmp/diff_new_pack.hLGuW0/_new 2023-06-22 23:26:45.930057127 +0200 @@ -1,5 +1,5 @@ name: polaris -version: 8.2.2 -mtime: 1686754281 -commit: 85a439653dd55ae47f992a60768594a3c15503f2 +version: 8.2.3 +mtime: 1687364744 +commit: 4b1d6635e06a52fbaf5d8e4746d94ec6b9313493 ++++++ vendor.tar.gz ++++++ /work/SRC/openSUSE:Factory/polaris/vendor.tar.gz /work/SRC/openSUSE:Factory/.polaris.new.15902/vendor.tar.gz differ: char 5, line 1