Skip to content

Commit 63655f2

Browse files
committed
chore(cirrus): include fml files in docker image
1 parent 8b3bb29 commit 63655f2

File tree

10 files changed

+108
-3
lines changed

10 files changed

+108
-3
lines changed

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ jetstream_config:
9898
rm -Rf experimenter/experimenter/segments/metric-hub-main/.script/
9999

100100
feature_manifests: build_dev
101-
$(COMPOSE_RUN) -e GITHUB_BEARER_TOKEN=$(GITHUB_BEARER_TOKEN) experimenter /experimenter/bin/manifest-tool.py fetch $(FETCH_ARGS)
101+
$(COMPOSE_RUN) --no-deps -e GITHUB_BEARER_TOKEN=$(GITHUB_BEARER_TOKEN) experimenter /experimenter/bin/manifest-tool.py fetch $(FETCH_ARGS)
102102

103103
install_nimbus_cli: ## Install Nimbus client
104104
mkdir -p $(CLI_DIR)
@@ -283,7 +283,7 @@ CIRRUS_COVERAGE_JSON := $(TEST_RESULTS_DIR)/$(TEST_FILE_PREFIX)unit__coverage.js
283283
CIRRUS_JUNIT_XML := $(TEST_RESULTS_DIR)/$(TEST_FILE_PREFIX)integration__results.xml
284284

285285
cirrus_build: build_megazords
286-
$(CIRRUS_ENABLE) $(DOCKER_BUILD) --target deploy -f cirrus/server/Dockerfile -t cirrus:deploy cirrus/server/
286+
$(CIRRUS_ENABLE) $(DOCKER_BUILD) --target deploy -f cirrus/server/Dockerfile -t cirrus:deploy --build-context=fml=experimenter/experimenter/features/manifests/ cirrus/server/
287287

288288
cirrus_build_test: build_megazords
289289
$(CIRRUS_ENABLE) $(COMPOSE_TEST) build cirrus

cirrus/server/Dockerfile

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,9 @@ COPY --from=python-builder /usr/local/bin /usr/local/bin
5151

5252
ENV PYTHONPATH=$PYTHONPATH:/application-services
5353

54+
# Copy fml files
55+
COPY --from=fml . /fml/
56+
5457
# Copy all other files into the container's working directory
5558
COPY . .
5659

docker-compose-cirrus.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ version: "3"
33
services:
44
cirrus:
55
build:
6+
additional_contexts:
7+
- fml=experimenter/experimenter/features/manifests/
68
context: cirrus/server/
79
dockerfile: Dockerfile
810
env_file: .env

docker-compose-test.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@ services:
2222
cirrus:
2323
env_file: .env.test
2424
build:
25+
additional_contexts:
26+
- fml=experimenter/experimenter/features/manifests/
2527
context: cirrus/server/
2628
dockerfile: Dockerfile
2729
working_dir: /cirrus
28-

experimenter/experimenter/features/manifests/apps.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,3 +133,11 @@ firefox_desktop:
133133
- "release"
134134
- "esr115"
135135
- "esr128"
136+
137+
experimenter_cirrus:
138+
slug: "experimenter"
139+
repo:
140+
type: "github"
141+
name: "mozilla/experimenter"
142+
default_branch: relud-cirrus-fml
143+
fml_path: "experimenter/experimenter/features/manifests/experimenter/nimbus.fml.yaml"
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
---
2+
version: 1.0.0
3+
about:
4+
description: Nimbus Feature Manifest for Experimenter Web testing
5+
channels:
6+
- dev
7+
features:
8+
example-feature:
9+
description: An example feature.
10+
variables:
11+
emoji:
12+
description: An emoji to show for this treatment.
13+
type: Option<String>
14+
default: ❤️
15+
enabled:
16+
description: If the feature is enabled.
17+
type: Boolean
18+
default: false
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
---
2+
example-feature:
3+
description: An example feature.
4+
hasExposure: true
5+
exposureDescription: ""
6+
variables:
7+
emoji:
8+
type: string
9+
description: An emoji to show for this treatment.
10+
enabled:
11+
type: boolean
12+
description: If the feature is enabled.
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
about:
2+
description: Nimbus Feature Manifest for Experimenter Web testing
3+
channels:
4+
- dev
5+
- staging
6+
- production
7+
features:
8+
example-feature:
9+
description: An example feature.
10+
variables:
11+
emoji:
12+
description: An emoji to show for this treatment.
13+
type: Option<String>
14+
default: ❤️
15+
enabled:
16+
description: If the feature is enabled.
17+
type: Boolean
18+
default: false
19+
defaults:
20+
- channel: dev
21+
value: { "enabled": false }
22+
- channel: staging
23+
value: { "enabled": false }
24+
- channel: production
25+
value: { "enabled": false }
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
---
2+
version: 1.0.0
3+
about:
4+
description: Nimbus Feature Manifest for Experimenter Web testing
5+
channels:
6+
- production
7+
features:
8+
example-feature:
9+
description: An example feature.
10+
variables:
11+
emoji:
12+
description: An emoji to show for this treatment.
13+
type: Option<String>
14+
default: ❤️
15+
enabled:
16+
description: If the feature is enabled.
17+
type: Boolean
18+
default: false
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
---
2+
version: 1.0.0
3+
about:
4+
description: Nimbus Feature Manifest for Experimenter Web testing
5+
channels:
6+
- staging
7+
features:
8+
example-feature:
9+
description: An example feature.
10+
variables:
11+
emoji:
12+
description: An emoji to show for this treatment.
13+
type: Option<String>
14+
default: ❤️
15+
enabled:
16+
description: If the feature is enabled.
17+
type: Boolean
18+
default: false

0 commit comments

Comments
 (0)