Skip to content
Merged
9 changes: 7 additions & 2 deletions rust/stackable-cockpit/src/platform/manifests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,11 @@ pub trait InstallManifestsExt {
) -> Result<(), Error> {
debug!("Installing demo / stack manifests");

let mut parameters = parameters.clone();
// We add the NAMESPACE parameter, so that stacks/demos can use that to render e.g. the
// fqdn service names [which contain the namespace].
parameters.insert("NAMESPACE".to_owned(), product_namespace.to_owned());

for manifest in manifests {
match manifest {
ManifestSpec::HelmChart(helm_file) => {
Expand All @@ -85,7 +90,7 @@ pub trait InstallManifestsExt {
})?;

let helm_chart: helm::Chart = transfer_client
.get(&helm_file, &Template::new(parameters).then(Yaml::new()))
.get(&helm_file, &Template::new(&parameters).then(Yaml::new()))
.await
.context(FileTransferSnafu)?;

Expand Down Expand Up @@ -133,7 +138,7 @@ pub trait InstallManifestsExt {
})?;

let manifests = transfer_client
.get(&path_or_url, &Template::new(parameters))
.get(&path_or_url, &Template::new(&parameters))
.await
.context(FileTransferSnafu)?;

Expand Down
6 changes: 6 additions & 0 deletions rust/stackablectl/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,17 @@ All notable changes to this project will be documented in this file.

## [Unreleased]

### Added

- Pass the namespace as argument `NAMESPACE` to templated manifests.
This should unblock demos to run in all namespaces, as they can template the namespace e.g. for the fqdn of services ([#355]).

### Fixes

- Use `rustls-native-certs` so that `stackablectl` can be used in environments with internal PKI ([#351]).

[#351]: https://github.com/stackabletech/stackable-cockpit/pull/351
[#355]: https://github.com/stackabletech/stackable-cockpit/pull/355

## [24.11.3] - 2025-01-27

Expand Down
Loading