Resolve "Send deployment information in job API"
What does this MR do?
Extend BuildDetailEnity
to include a new key deployment_status
which has all the information about deployments.
What are the relevant issue numbers?
gitlab-org/gitlab-ce#50460
gitlab-org/gitlab-ce#50101
To Do
-
Create issue for environment json schema
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary -
Documentation created/updated -
Tests added for this feature/bug -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the database guides
Closes #50460 (closed)
Edited by Steve Xuereb
Merge request reports
Activity
mentioned in issue #50101 (closed)
- Resolved by Kamil Trzciński
- Resolved by Kamil Trzciński
- Resolved by Kamil Trzciński
- Resolved by Kamil Trzciński
- Resolved by Kamil Trzciński
added 1 commit
- 9bcc9652 - Add deployment information in job API [ci skip]
added 374 commits
-
9bcc9652...d79218fc - 369 commits from branch
master
- ebf45113 - Add artifact information for job controller
- 0c84ddf4 - Add JSON schema validation for job API
- fd4a130b - fixup! Add JSON schema validation for job API
- 99750449 - fixup! Add artifact information for job controller
- 0d4678b0 - Add deployment information in job API
Toggle commit list-
9bcc9652...d79218fc - 369 commits from branch
marked as a Work In Progress from fd4a130b
@ayufan I pushed 3039902a as a final commit, can you take a look again?
Few things to note:
- The json schema for the
environment
will not be added to this MR since the response is qutie big and is something that should have been there before. I will create a seperate issue to add the schema if you agree - For now I have rebased it on top of gitlab-org/gitlab-ce!21328 since that has the changes for the json schema which this MR needed, so when gitlab-org/gitlab-ce!21328 will be merged I will rebase this on top of master again.
Edited by Steve Xuereb- The json schema for the
assigned to @ayufan
- Resolved by Kamil Trzciński
Please register or sign in to reply