Skip to content

Print user application log when create/update deployment#5846

Merged
zhoxing-ms merged 14 commits into
Azure:mainfrom
zmssp:asa/deployment-log
Feb 22, 2023
Merged

Print user application log when create/update deployment#5846
zhoxing-ms merged 14 commits into
Azure:mainfrom
zmssp:asa/deployment-log

Conversation

@zmssp
Copy link
Copy Markdown
Contributor

@zmssp zmssp commented Feb 6, 2023

Print user application log when create/update deployment using log stream.

  • Print user app logs when create/update deployment
  • Ignore log stream errors.
  • Print application logs to stderr to avoid potential breaking changes.(Customer may read stdout in their pipelines.)

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

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Feb 6, 2023

spring

@zmssp zmssp marked this pull request as ready for review February 10, 2023 05:46
@zmssp zmssp changed the title Print user application log for faile deployment when create/update deployment Print user application log when create/update deployment Feb 10, 2023
Comment thread src/spring/azext_spring/app.py
@zhoxing-ms zhoxing-ms merged commit 16d049c into Azure:main Feb 22, 2023
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.

4 participants