Skip to content

Commit eecd2a1

Browse files
authored
check in status for lastModifiedTime pipelineResources (#182)
Issue #, if available: Description of changes: Check status field for lastModifiedtime in K8S side check By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent a6341c5 commit eecd2a1

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

test/e2e/tests/test_pipeline.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ def test_pipeline_succeeded(self, pipeline):
156156
)
157157
assert old_pipeline_last_modified_time != pipeline_desc["LastModifiedTime"]
158158
assert (
159-
resource["spec"].get("lastModifiedTime", None)
159+
resource["status"].get("lastModifiedTime")
160160
!= old_pipeline_last_modified_time
161161
)
162162

test/e2e/tests/test_pipeline_execution.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ def test_pipeline_execution_succeeded(self, pipeline_execution):
177177
)
178178

179179
old_pipeline_last_modified_time = pipeline_execution_desc["LastModifiedTime"]
180-
180+
181181
assert k8s.get_resource_arn(resource) == pipeline_execution_arn
182182

183183
self._assert_pipeline_execution_status_in_sync(
@@ -220,7 +220,7 @@ def test_pipeline_execution_succeeded(self, pipeline_execution):
220220
!= pipeline_execution_desc["LastModifiedTime"]
221221
)
222222
assert (
223-
resource["spec"].get("lastModifiedTime", None)
223+
resource["status"].get("lastModifiedTime")
224224
!= old_pipeline_last_modified_time
225225
)
226226

0 commit comments

Comments
 (0)