Skip to content
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

Adjust the server_containerized module to kubernetes refactoring #1747

Merged
merged 1 commit into from
Nov 15, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
3 changes: 0 additions & 3 deletions main.tf.libvirt-testsuite.example
Original file line number Diff line number Diff line change
Expand Up @@ -97,9 +97,6 @@ module "cucumber_testsuite" {

# Override where to get the containers from
# container_repository = "registry.opensuse.org/systemsmanagement/uyuni/master/containers/uyuni"

# Override where to get the helm chart from
# helm_chart_url = "oci://registry.opensuse.org/systemsmanagement/uyuni/master/charts/uyuni/server-helm"
}
proxy = {
name = "proxy"
Expand Down
3 changes: 0 additions & 3 deletions main.tf.libvirt.example
Original file line number Diff line number Diff line change
Expand Up @@ -54,9 +54,6 @@ module "server" {

// To define a specific container image tag to install
// container_tag =

// To override the URL to the helm chart to install
// helm_chart_url = "oci://registry.opensuse.org/systemsmanagement/uyuni/master/servercontainer/charts/uyuni/server"
}

module "minion" {
Expand Down
1 change: 0 additions & 1 deletion modules/cucumber_testsuite/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,6 @@ module "server_containerized" {
runtime = lookup(local.runtimes, "server_containerized", "podman")
container_repository = lookup(local.container_repositories, "server_containerized", "")
container_tag = lookup(local.container_tags, "server_containerized", "")
helm_chart_url = lookup(local.helm_chart_urls, "server_containerized", "")
auto_accept = false
download_private_ssl_key = false
disable_firewall = false
Expand Down
1 change: 0 additions & 1 deletion modules/server_containerized/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ module "server_containerized" {
container_runtime = var.runtime
container_repository = var.container_repository
container_tag = var.container_tag
helm_chart_url = var.helm_chart_url
cc_username = var.base_configuration["cc_username"]
cc_password = var.base_configuration["cc_password"]
mirror = var.base_configuration["mirror"]
Expand Down
5 changes: 0 additions & 5 deletions modules/server_containerized/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,6 @@ variable "container_tag" {
default = ""
}

variable "helm_chart_url" {
description = "Where to get the helm chart from. Uses the released one by default."
default = ""
}

variable "product_version" {
description = "One of: 4.3-released, 4.3-nightly, 4.3-pr, 4.3-beta, 4.3-build_image, 4.3-VM-nightly, 4.3-VM-released, 5.0-nightly, 5.0-released, head, uyuni-master, uyuni-released, uyuni-pr"
type = string
Expand Down
1 change: 0 additions & 1 deletion salt/server_containerized/chart-values.yaml

This file was deleted.

6 changes: 0 additions & 6 deletions salt/server_containerized/install_k3s.sls
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,3 @@ helm_install:
- refresh: True
- name: helm
{% endif %}

chart_values_file:
file.managed:
- name: /root/chart-values.yaml
- source: salt://server_containerized/chart-values.yaml
- template: jinja
8 changes: 3 additions & 5 deletions salt/server_containerized/mgradm.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,10 @@ tag: {{ grains.get('container_tag') }}
{%- set mirror_hostname = grains.get('server_mounted_mirror') if grains.get('server_mounted_mirror') else grains.get('mirror') %}
{%- if mirror_hostname %}
mirror: /srv/mirror
{% endif -%}
{% set helm_chart_default = 'oci://registry.opensuse.org/uyuni/server-helm' %}
helm:
{% endif %}
kubernetes:
uyuni:
chart: {{ grains.get("helm_chart_url") | default(helm_chart_default, true) }}
values: /root/chart-values.yaml
namespace: uyuni
{%- if grains.get("java_debugging") %}
debug:
java: true
Expand Down
Loading