Skip to content

Conversation

@calvinsID
Copy link
Contributor

@calvinsID calvinsID commented Jun 30, 2020

Description
closes #14099

  • az webapp log deployment show should show actual logs instead of the log info
  • Mark commands as in preview

Testing Guide
az webapp log deployment show

Should have output of a list of logs similar to:

[
{
"log_time": "2020...",
"id": "id",
"message": "log message",
"type": 0,
"details_url": null
},
{
"log_time": "2020...",
"id": "id",
"message": "log message 2",
"type": 0,
"details_url": null
}
]

Rather than an object similar to

{
"active": false,
"author": "N/A",
"author_email": "N/A",
"complete": true,
"deployer": "GITHUB_ZIP_DEPLOY",
"end_time": "2020-06-25T19:04:18.8758562Z",
"id": "id",
"is_readonly": true,
"is_temp": false,
"last_success_end_time": "2020-06-25T19:04:18.8758562Z",
"log_url": "https://calvin-githubactions-webapp.scm.azurewebsites.net/api/deployments/deploymentId/log",
"message": "Created via a push deployment",
"progress": "",
"received_time": "2020-06-25T19:04:00.9811072Z",
"site_name": "calvin-githubactions-webapp",
"start_time": "2020-06-25T19:04:01.061354Z",
"status": 4,
"status_text": "",
"url": "https://calvin-githubactions-webapp.scm.azurewebsites.net/api/deployments/deploymentId"
}


This checklist is used to make sure that common guidelines for a pull request are followed.

@yonzhan
Copy link
Collaborator

yonzhan commented Jun 30, 2020

Webapp

@calvinsID calvinsID marked this pull request as ready for review June 30, 2020 20:45
@calvinsID calvinsID requested a review from panchagnula June 30, 2020 20:45
@calvinsID calvinsID force-pushed the issue-14099-fix-deployment-logs branch from 420ead2 to 96dbdde Compare July 3, 2020 17:26
@qwordy
Copy link
Member

qwordy commented Jul 6, 2020

ready to merge?

@calvinsID
Copy link
Contributor Author

ready to merge?

Ready, thanks!

@panchagnula panchagnula merged commit cdf4117 into Azure:dev Jul 7, 2020
@calvinsID calvinsID deleted the issue-14099-fix-deployment-logs branch October 28, 2020 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

az webapp log deployment show should show actual logs instead of the log info

4 participants