-
Notifications
You must be signed in to change notification settings - Fork 228
Azure Container Instances document #5101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Azure Container Instances document #5101
Conversation
|
@sachin-sumologic, I added the new article to the index page, the product list article, and the left nav bar. I also added a logo, since your PR didn't have one. See commit "Add article to left nav". |
jpipkin1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sachin-sumologic, I made a few minor changes (see commit "Updates from review").
You need to complete a "to do" you left in the article.
kimsauce
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Double-checking if this doc needs a CID?
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
himanshu219
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's put all the files
Dockerfile and yaml file in S3 in same folder as screenshots
This will allow us to update them later
Head branch was pushed to by a user without write access
amee-sumo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Incorporated your comments.
JV0812
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Purpose of this pull request
This pull request...
Select the type of change
Ticket (if applicable)
SUMO-255285