Skip to content

Conversation

kumaraksh1
Copy link
Contributor

@kumaraksh1 kumaraksh1 commented Aug 17, 2025

Context

Azure App Service has started providing sidecar deployment facility and GA is also done way back. Now adding changes to support deployment using VSTS pipeline.
https://msazure.visualstudio.com/Antares/_workitems/edit/33618137/

Task Name

AzureWebAppV1


Description

Azure App Service has started providing sidecar deployment facility and GA is also done way back. Now adding changes to support deployment using VSTS pipeline.


Risk Assessment (Low / Medium / High)

LOW


Change Behind Feature Flag (Yes / No)

Can this change be behine feature flag, if not why?
No, GA is already done, its just about a deployment client change.

Tech Design / Approach

  • Design has been written and reviewed. - Done
  • Any architectural decisions, trade-offs, and alternatives are captured. - Done

Documentation Changes Required (Yes/No)

Indicate whether related documentation needs to be updated.- Not needed right now.

  • User guides, API specs, system diagrams, or runbooks are updated. - Not needed right now.

Unit Tests Added or Updated (Yes / No)

Manual Test and end to end testing.


Additional Testing Performed

End to end testing is done
Blessed + Sidecar
https://dev.azure.com/kumaraksh/Test/_build/results?buildId=63&view=logs&j=d915e05c-da5f-5455-5678-bf36e6b1ce6e&t=f1b9fd0b-0572-5107-8345-bf15f753fda3
Only Blessed
https://dev.azure.com/kumaraksh/Test/_build/results?buildId=65&view=logs&j=d915e05c-da5f-5455-5678-bf36e6b1ce6e&t=f1b9fd0b-0572-5107-8345-bf15f753fda3

Logging Added/Updated (Yes/No)

  • Appropriate log statements are added with meaningful messages. - Yes
  • Logging does not expose sensitive data. - Yes
  • Log levels are used correctly (e.g., info, warn, error). - Yes

Telemetry Added/Updated (Yes/No)

  • Custom telemetry (e.g., counters, timers, error tracking) is added as needed. NA
  • Events are tagged with proper metadata for filtering and analysis. NA
  • Telemetry is validated in staging or test environments. NA

Rollback Scenario and Process (Yes/No)

  • Rollback plan is documented. - Just need to revert this PR.

Dependency Impact Assessed and Regression Tested (Yes/No)

  • All impacted internal modules, APIs, services, and third-party libraries are analyzed. - Yes
  • Results are reviewed and confirmed to not break existing functionality. - Yes

Checklist

  • Related issue linked (if applicable)
  • Task version was bumped — see versioning guide
  • Verified the task behaves as expected

@kumaraksh1 kumaraksh1 marked this pull request as ready for review August 28, 2025 08:53
@kumaraksh1 kumaraksh1 requested review from manolerazvan and a team as code owners August 28, 2025 08:53
@manolerazvan
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@kumaraksh1 kumaraksh1 force-pushed the users/kumaraksh1/sitecontainers-changes-blessed branch from 10d526c to eeeb873 Compare August 29, 2025 13:48
@manolerazvan manolerazvan self-requested a review August 29, 2025 14:07
@manolerazvan
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

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