Skip to content

Improve support for description field on CiliumNetworkPolicy

Arthur Evstifeev requested to merge cnp-description into master

What does this MR do?

This commit fixes position of the description field in CiliumNetworkPolicy resource to be top level as per specification. Current implementation assumed that description field is a part of the spec payload which is not true. This change resulted in introduction of the #resource method and related changes in the common mixin. I have also spotted erroneous compact! call and added a spec that ensures related issue is being checked.

related to #223623 (closed)

To observe existing erroneous behavior:

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Arthur Evstifeev

Merge request reports