feat: add ingress chart

[kharkevich]

No description provided.

[Copilot]

Pull Request Overview

This PR adds a new Helm chart for managing application-level ingress, including configuration defaults, rendering templates, and basic documentation.

• Define configurable fields in values.yaml for hosts, services, TLS, and rewrite rules.
• Implement templates/ingress.yml to generate an Ingress resource per service with class-specific annotations.
• Provide chart metadata in Chart.yaml and a minimal readme.md.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

File	Description
charts/application-ingress/values.yaml	Introduce default values for ingressClass (commented), hosts, services, TLS settings
charts/application-ingress/templates/ingress.yml	Render per-service Ingress resources with class-specific defaults and overrides
charts/application-ingress/Chart.yaml	Add chart metadata, versioning, and dependency on common chart
charts/application-ingress/readme.md	Add basic usage instructions

Comments suppressed due to low confidence (1)

charts/application-ingress/Chart.yaml:2

• The chart name (ingress) does not match the directory name (application-ingress), which may confuse tooling and users. Consider renaming the chart to application-ingress for consistency.

name: ingress

[Copilot]

Pull Request Overview

This PR introduces a new Helm chart for managing application ingress resources, allowing per-service rules, host definitions, and optional TLS settings.

• Add configurable defaults and examples in values.yaml for hosts, services, and TLS
• Implement an ingress.yml template with class-specific annotations, path mappings, and TLS support
• Provide basic chart metadata (Chart.yaml) and a minimal README

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

File	Description
charts/application-ingress/values.yaml	Add default values and comments for ingress chart configuration
charts/application-ingress/templates/ingress.yml	Implement Ingress resource templating with class-specific defaults
charts/application-ingress/readme.md	Add placeholder documentation linking to values.yaml
charts/application-ingress/Chart.yaml	Define chart metadata and dependencies

Comments suppressed due to low confidence (1)

charts/application-ingress/Chart.yaml:2

• [nitpick] The chart folder is named application-ingress, but Chart.yaml uses name: ingress. Aligning these names improves clarity and consistency for consumers.

name: ingress

[Copilot]

The explicitTLS value is defined as a boolean but used as a secretName. Consider splitting the flag and secret name into separate values (e.g., tls.enabled and tls.secretName) or changing its type to a string.

Suggested change

	{{- if $.Values.explicitTLS }}
	secretName: {{ $.Values.explicitTLS }}
	{{- if $.Values.tls.enabled }}
	secretName: {{ $.Values.tls.secretName }}

[Copilot]

[nitpick] The solution.component key is defined in values.yaml but not used in the ingress template. Remove unused values or document its purpose to avoid confusion.

Suggested change

	solution:
	component: ingress
	# Removed unused solution.component key