Skip to content

Commit d00b047

Browse files
author
Théo Penavaire
authored
Add postprocessing package publish pipelines (#3433)
* Add postprocessing package publish pipelines * Add reference to pp package in Smoke Tests * Explicitly list physics module as pp. dep * Mark assembly as test assembly * Mark Editor.Tests assembly as test assembly * Add specific ppv2 publish job (tmp) * Fix naming and dependencies * Add --package-path to upm ci calls * Update pointer to gfx-sdet-tools repo * Update pointer to gfx-sdet-tools repo * Apply yml changes * Tmp exclude ppv2 package from publish all (project context) * Remove ppv2 from smoke test dependencies * Update pointer reference to sdet tools
1 parent 5fd7fb9 commit d00b047

File tree

6 files changed

+138
-8
lines changed

6 files changed

+138
-8
lines changed

Diff for: .yamato/_tmp_postprocessing.yml

+119
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,119 @@
1+
2+
# From https://github.com/Unity-Technologies/PostProcessing/blob/v2/.yamato/upm-ci.yml
3+
4+
name: postprocessing
5+
test_editors:
6+
- trunk
7+
- 2021.1
8+
- 2020.2
9+
- 2019.4
10+
- 2018.4
11+
test_platforms:
12+
- name: win
13+
type: Unity::VM
14+
image: package-ci/win10:stable
15+
flavor: b1.large
16+
- name: mac
17+
type: Unity::VM::osx
18+
image: package-ci/mac:stable
19+
flavor: m1.mac
20+
---
21+
{{ name }}_pack:
22+
name: {{ name }} - Pack
23+
agent:
24+
type: Unity::VM
25+
image: package-ci/win10:stable
26+
flavor: b1.large
27+
commands:
28+
- npm install upm-ci-utils@stable -g --registry https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-npm
29+
- upm-ci package pack --package-path com.unity.postprocessing
30+
artifacts:
31+
packages:
32+
paths:
33+
- "upm-ci~/**/*"
34+
35+
{% for editor in test_editors %}
36+
{% for platform in test_platforms %}
37+
{{ name }}_test_{{ platform.name }}_{{ editor }}:
38+
name : {{ name }} - Test {{ editor }} on {{ platform.name }}
39+
agent:
40+
type: {{ platform.type }}
41+
image: {{ platform.image }}
42+
flavor: {{ platform.flavor}}
43+
commands:
44+
- npm install upm-ci-utils@stable -g --registry https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-npm
45+
- upm-ci package test --unity-version {{ editor }} --package-path com.unity.postprocessing --extra-utr-arg="--compilation-errors-as-warnings"
46+
artifacts:
47+
logs:
48+
paths:
49+
- "upm-ci~/test-results/**/*"
50+
dependencies:
51+
- .yamato/_tmp_postprocessing.yml#{{ name }}_pack
52+
{% endfor %}
53+
{% endfor %}
54+
55+
{{ name }}_test_trigger:
56+
name: {{ name }} - Tests Trigger
57+
agent:
58+
type: Unity::VM
59+
image: package-ci/win10:stable
60+
flavor: b1.large
61+
commands:
62+
- dir
63+
artifacts:
64+
logs:
65+
paths:
66+
- "upm-ci~/test-results/**/*"
67+
packages:
68+
paths:
69+
- "upm-ci~/packages/**/*"
70+
dependencies:
71+
- .yamato/_tmp_postprocessing.yml#{{ name }}_pack
72+
{% for editor in test_editors %}
73+
{% for platform in test_platforms %}
74+
- .yamato/_tmp_postprocessing.yml#{{ name }}_test_{{platform.name}}_{{editor}}
75+
{% endfor %}
76+
{% endfor %}
77+
78+
{{ name }}_publish:
79+
name: {{ name }} - Publish to Internal Registry
80+
agent:
81+
type: Unity::VM
82+
image: package-ci/win10:stable
83+
flavor: b1.large
84+
commands:
85+
- npm install upm-ci-utils@stable -g --registry https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-npm
86+
- upm-ci package publish --package-path com.unity.postprocessing
87+
artifacts:
88+
artifacts:
89+
paths:
90+
- "upm-ci~/packages/*.tgz"
91+
dependencies:
92+
- .yamato/_tmp_postprocessing.yml#{{ name }}_pack
93+
{% for editor in test_editors %}
94+
{% for platform in test_platforms %}
95+
- .yamato/_tmp_postprocessing.yml#{{ name }}_test_{{ platform.name }}_{{ editor }}
96+
{% endfor %}
97+
{% endfor %}
98+
99+
100+
{{ name }}_publish_dry_run:
101+
name: {{ name }} - Publish to Internal Registry [dry-run]
102+
agent:
103+
type: Unity::VM
104+
image: package-ci/win10:stable
105+
flavor: b1.large
106+
commands:
107+
- npm install upm-ci-utils@stable -g --registry https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-npm
108+
- upm-ci package publish --dry-run --package-path com.unity.postprocessing
109+
artifacts:
110+
artifacts:
111+
paths:
112+
- "upm-ci~/packages/*.tgz"
113+
dependencies:
114+
- .yamato/_tmp_postprocessing.yml#{{ name }}_pack
115+
{% for editor in test_editors %}
116+
{% for platform in test_platforms %}
117+
- .yamato/_tmp_postprocessing.yml#{{ name }}_test_{{ platform.name }}_{{ editor }}
118+
{% endfor %}
119+
{% endfor %}

Diff for: .yamato/config/_packages.metafile

+9-4
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# all_package_ci needs dependencies for package_test_dependencies (currently commented out due to breaking)
22
packages:
3-
- name: Core
3+
- name: Core
44
id: core
55
packagename: com.unity.render-pipelines.core
66
dependencies:
@@ -54,9 +54,14 @@ packages:
5454
- core
5555
- config
5656
hascodependencies: 1
57+
# - name: Postprocessing
58+
# id: postprocessing
59+
# packagename: com.unity.postprocessing
60+
# dependencies:
61+
# - postprocessing
5762
platforms:
5863
- name: Win
5964
- name: OSX
60-
agent_pack: package_ci_win_large
61-
agent_publish: package_ci_win_large
62-
agent_publish_all: package_ci_ubuntu_large
65+
agent_pack: package_ci_win_large
66+
agent_publish: package_ci_win_large
67+
agent_publish_all: package_ci_ubuntu_large

Diff for: .yamato/script/gfx_sdet_tools_revision.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
c7cd8d4e10f7d878552ed95f29efa8bb5fe4f581
1+
60c73738f40dc3e0ca0957fdae0164f97edeafb5

Diff for: com.unity.postprocessing/Tests/Editor/Unity.Postprocessing.Editor.Tests.asmdef

+4-2
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,7 @@
1515
"overrideReferences": false,
1616
"precompiledReferences": [],
1717
"autoReferenced": true,
18-
"defineConstraints": []
19-
}
18+
"defineConstraints": [
19+
"UNITY_INCLUDE_TESTS"
20+
]
21+
}

Diff for: com.unity.postprocessing/Tests/Runtime/Unity.Postprocessing.Runtime.Tests.asmdef

+4-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,10 @@
66
"optionalUnityReferences": [
77
"TestAssemblies"
88
],
9+
"defineConstraints": [
10+
"UNITY_INCLUDE_TESTS"
11+
],
912
"includePlatforms": [],
1013
"excludePlatforms": [],
1114
"allowUnsafeCode": true
12-
}
15+
}

Diff for: com.unity.postprocessing/package.json

+1
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,6 @@
55
"unity": "2018.4",
66
"description": "The post-processing stack (v2) comes with a collection of effects and image filters you can apply to your cameras to improve the visuals of your games.",
77
"dependencies": {
8+
"com.unity.modules.physics": "1.0.0"
89
}
910
}

0 commit comments

Comments
 (0)