Skip to content

Commit 3afde4f

Browse files
author
Chance Zibolski
authored
Merge pull request #401 from chancez/remove_tectonic
*: Remove all things related to install on Tectonic
2 parents feae315 + 3c17d79 commit 3afde4f

28 files changed

+12
-762
lines changed

Documentation/install-metering.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,10 @@ If you do not wish to modify the Operator Metering configuration, a minimal conf
3131

3232
### Prometheus Monitoring Configuration
3333

34-
For installs into Openshift, ensuring Prometheus is installed can be done using Ansible. For installs into Tectonic, the manual installation method configures Metering to use the Prometheus that's installed by default into the tectonic-system namespace.
34+
For Openshift 3.11 or later, Prometheus is installed by default through cluster monitoring in the openshift-monitoring namespace.
3535

36-
If you're not using Openshift or Tectonic, then you will need to use OLM or the manual install method. In this case if you are not using a [kube-prometheus][kube-prometheus] installation, or your Prometheus service is not named `prometheus-k8s` and in the `monitoring` namespace, then you must customize the [prometheus URL config option][configure-prometheus-url] before proceeding.
36+
If you're not using Openshift, then you will need to use OLM or the manual install method.
37+
In this case if you are not using a [kube-prometheus][kube-prometheus] installation, or your Prometheus service is not named `prometheus-k8s` and in the `monitoring` namespace, then you must customize the [prometheus URL config option][configure-prometheus-url] before proceeding.
3738

3839
## Install Methods
3940

Documentation/manual-install.md

Lines changed: 3 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ By default, if it's unset, it will use the `metering` namespace.
66

77
## Install
88

9-
Depending on your Kubernetes platform (regular Kubernetes, Tectonic, or Openshift)
9+
Depending on your Kubernetes platform (regular Kubernetes, or Openshift)
1010

1111
For a standard Kubernetes cluster:
1212

@@ -15,13 +15,6 @@ $ export METERING_NAMESPACE=metering-$USER
1515
$ ./hack/install.sh
1616
```
1717

18-
If your using Tectonic, use tectonic-install.sh:
19-
20-
```
21-
$ export METERING_NAMESPACE=metering-$USER
22-
$ ./hack/tectonic-install.sh
23-
```
24-
2518
If your using Openshift, use openshift-install.sh:
2619

2720
```
@@ -40,13 +33,6 @@ $ export METERING_NAMESPACE=metering-$USER
4033
$ ./hack/uninstall.sh
4134
```
4235

43-
If your using Tectonic, use tectonic-uninstall.sh:
44-
45-
```
46-
$ export METERING_NAMESPACE=metering-$USER
47-
$ ./hack/tectonic-uninstall.sh
48-
```
49-
5036
If your using Openshift, use openshift-uninstall.sh:
5137

5238
```
@@ -80,7 +66,6 @@ $ export METERING_CR_FILE=metering-custom.yaml
8066
Then run the installation script for your platform:
8167

8268
- `./hack/install.sh`
83-
- `./hack/tectonic-install.sh`
8469
- `./hack/openshift-install.sh`
8570

8671
For more details on configuration options, most are documented in the [configuring metering document][configuring-metering].
@@ -103,13 +88,13 @@ spec:
10388
Next, run the install script for your platform (see above).
10489

10590
After running the install script, figure out where your Prometheus pod is running.
106-
By default the `run-reporting-operator-local` Makefile target assumes that the pod is in the `tectonic-system` namespace and can be queried using the label selector `app=prometheus`.
91+
By default the `run-reporting-operator-local` Makefile target assumes that the pod is in the `openshift-monitoring` namespace and can be queried using the label selector `app=prometheus`.
10792

10893
If you're Prometheus is located somewhere, else, you can override the defaults using the environment variables `METERING_PROMETHEUS_NAMESPACE` and `METERING_PROMTHEUS_LABEL_SELECTOR` to the namespace your Prometheus pod is in, and the label selector for querying Prometheus.
10994

11095
Ex (these are the defaults):
11196
```
112-
export METERING_PROMETHEUS_NAMESPACE=tectonic-system
97+
export METERING_PROMETHEUS_NAMESPACE=openshift-monitoring
11398
export METERING_PROMTHEUS_LABEL_SELECTOR=app=prometheus
11499
```
115100

Makefile

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -256,22 +256,15 @@ bin/metering-override-values.yaml: ./hack/render-metering-chart-override-values.
256256
@mkdir -p bin
257257
./hack/render-metering-chart-override-values.sh $(RELEASE_TAG) > $@
258258

259-
CHART_DEPS := bin/tectonic-metering-0.1.0.tgz \
260-
bin/openshift-metering-0.1.0.tgz \
259+
CHART_DEPS := bin/openshift-metering-0.1.0.tgz \
261260
bin/operator-metering-0.1.0.tgz
262261

263262
all-charts: $(CHART_DEPS)
264263

265-
tectonic-metering-chart: bin/tectonic-metering-0.1.0.tgz
266-
267264
openshift-metering-chart: bin/openshift-metering-0.1.0.tgz
268265

269266
operator-metering-chart: bin/operator-metering-0.1.0.tgz
270267

271-
bin/tectonic-metering-0.1.0.tgz: $(shell find charts -type f)
272-
@echo "Packaging tectonic-metering chart dependencies"
273-
@mkdir -p bin && mkdir -p charts/tectonic-metering/charts && hack/yamltojson < charts/tectonic-metering/requirements.yaml | jq '.dependencies[].repository' -r | sed 's|file://||' | xargs -I {} helm package --save=false -d charts/tectonic-metering/charts charts/tectonic-metering/{}
274-
helm package --save=false -d bin charts/tectonic-metering
275268

276269
bin/openshift-metering-0.1.0.tgz: $(shell find charts -type f)
277270
@echo "Packaging openshift-metering chart dependencies"
@@ -296,7 +289,7 @@ metering-manifests:
296289
metering-test-docker \
297290
metering-e2e-docker-build metering-builder-docker-build \
298291
build-reporting-operator reporting-operator-bin reporting-operator-local \
299-
operator-metering-chart tectonic-metering-chart openshift-metering chart \
292+
operator-metering-chart penshift-metering chart \
300293
bin/metering-override-values.yaml \
301294
metering-manifests bill-of-materials.json \
302295
install-kube-prometheus-helm

charts/tectonic-metering/.gitignore

Lines changed: 0 additions & 1 deletion
This file was deleted.

charts/tectonic-metering/.helmignore

Lines changed: 0 additions & 21 deletions
This file was deleted.

charts/tectonic-metering/Chart.yaml

Lines changed: 0 additions & 4 deletions
This file was deleted.

charts/tectonic-metering/requirements.lock

Lines changed: 0 additions & 12 deletions
This file was deleted.

charts/tectonic-metering/requirements.yaml

Lines changed: 0 additions & 12 deletions
This file was deleted.

charts/tectonic-metering/templates/_metadata.yaml

Lines changed: 0 additions & 6 deletions
This file was deleted.

charts/tectonic-metering/values.yaml

Lines changed: 0 additions & 20 deletions
This file was deleted.

0 commit comments

Comments
 (0)