Skip to content

A/B upgrade- Added wrapper script #67

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

Closed
wants to merge 21 commits into from
Closed

Conversation

ipsita-npg
Copy link
Contributor

Pull Request Template

Description

Please include a summary of the changes and the related issue. List any dependencies
that are required for this change.

Fixes # (issue)

Any Newly Introduced Dependencies

Please describe any newly introduced 3rd party dependencies in this change.
List their name, license information, and how they are used in the project.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions
so we can reproduce. Please also list any relevant details for your test configuration.

Checklist

  • I agree to use the APACHE-2.0 license for my code changes
  • I have not introduced any 3rd party dependency changes
  • I have performed a self-review of my code

@ipsita-npg ipsita-npg changed the title A/B upgrade- Added wrapper script A/B upgrade- Added wrapper script [ WIP ] Jun 6, 2025
@ipsita-npg ipsita-npg requested a review from vthiruveedula June 6, 2025 10:44
@ipsita-npg ipsita-npg changed the base branch from main to feature-enhancement-3.1 June 9, 2025 05:26
@ipsita-npg ipsita-npg marked this pull request as ready for review June 10, 2025 05:25
@ipsita-npg ipsita-npg changed the title A/B upgrade- Added wrapper script [ WIP ] A/B upgrade- Added wrapper script Jun 10, 2025
@ipsita-npg ipsita-npg force-pushed the add-wrapper-script-upgrade branch from ffa6a63 to 77f50ad Compare June 10, 2025 16:50
@ipsita-npg ipsita-npg force-pushed the add-wrapper-script-upgrade branch from 4a386b7 to 5302c63 Compare June 10, 2025 18:26
@ipsita-npg ipsita-npg changed the base branch from feature-enhancement-3.1 to main June 11, 2025 05:47
@ipsita-npg ipsita-npg force-pushed the add-wrapper-script-upgrade branch from cad8ddd to 755d3b0 Compare June 11, 2025 11:17
@ipsita-npg ipsita-npg force-pushed the add-wrapper-script-upgrade branch from 3c92af3 to 55b935e Compare June 11, 2025 11:38
@ipsita-npg ipsita-npg requested a review from krishnajs as a code owner June 11, 2025 14:23
Copy link
Contributor

@vthiruveedula vthiruveedula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ipsita-npg ipsita-npg mentioned this pull request Jun 12, 2025
3 tasks
@@ -1 +1 @@
3.0.3
3.0.5-dev
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why it is bumped with +2?

Copy link
Contributor Author

@ipsita-npg ipsita-npg Jun 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

- Extract the contents of `sen-installation-files.tar.gz`

```bash
tar -xzf sen-installation-files.tar.gz
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

double check once with Anand sen is allowed here?

Copy link
Contributor Author

@ipsita-npg ipsita-npg Jun 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

forbidden word removal sen to standalone is already done and merged in branch [sn-emt-uOS-integration]

https://github.com/open-edge-platform/edge-microvisor-toolkit-standalone-node/pull/65/files#diff-980f2a5f9e7cd4e60cb5ddc65d569efa78db04f6c2d75d787329cc7c2dd03735

In order to merge into mainline, this PR #67 was created The word hasn't been changed to sync with the manline. once we rebase the branch "sn-emt-uOS-integration" to" mainline", forbidden word removal will be taken care

if os-update-tool.sh -c; then
echo "Commit update successful."
else
echo "Failed to commit update."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these echo logs will to banner?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes , these echo logs are part of banner

Copy link

This pull request is stale because it has been open 30 days with no activity. Make a comment or update the PR to avoid closing PR after 15 days.

@ipsita-npg ipsita-npg closed this Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants