Skip to content
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

fix(project): updates subnet when vpn is already enabled #324

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

pallabpain
Copy link
Contributor

@pallabpain pallabpain commented Jun 27, 2024

Description

The rio project features vpn command does not update the subnet if the VPN is already enabled. The reason is a check that returns if the current state is already the desired one.

This commit removes the check and allows an update to happen irrespective and thus allowing the subnet range updates to work.

Wrike Ticket: https://www.wrike.com/open.htm?id=1418307735

Slack Thread: https://rapyuta-robotics.slack.com/archives/CBAP39MGD/p1719467131181389?thread_ts=1719465795.794819&cid=CBAP39MGD

The rio project features vpn command does not update the subnet if the
VPN is already enabled. The reason is a check that returns if the
current state is already the desired one.

This commit removes the check and allows an update to happen
irrespective and thus allowing the subnet range updates to work.

Wrike Ticket: https://www.wrike.com/open.htm?id=1418307735
@pallabpain pallabpain requested review from a team as code owners June 27, 2024 16:32
@pallabpain pallabpain self-assigned this Jun 27, 2024
@pallabpain pallabpain added the 🐛 bug Something isn't working label Jun 27, 2024
Copy link

🤖 Pull Request Artifacts (#9700274730) 🎉

@pallabpain pallabpain merged commit e8d553b into devel Jul 3, 2024
6 checks passed
rr-github-ci-user pushed a commit that referenced this pull request Jul 4, 2024
# [7.6.0](v7.5.0...v7.6.0) (2024-07-04)

### Bug Fixes

* **device:** sets device labels correctly ([#329](#329)) ([a3ac73d](a3ac73d))
* **device:** sets highperf label on virtual device ([#328](#328)) ([1eb7906](1eb7906))
* **project:** updates subnet when vpn is already enabled ([#324](#324)) ([e8d553b](e8d553b))

### Features

* **configtree:** add export command ([6f236ae](6f236ae))
* **configtree:** add support for diff ([7df605e](7df605e))
* **configtree:** add support for merge ([a1d273f](a1d273f))
* **configtree:** add support for milestone ([6ada3f7](6ada3f7))
* **device:** supports onboarding hwil devices via device manifest ([#323](#323)) ([7c257f1](7c257f1))
* **hwil:** implements hwil command ([#319](#319)) ([3e24b0b](3e24b0b))
@rr-github-ci-user
Copy link
Collaborator

🎉 This PR is included in version 7.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pallabpain pallabpain deleted the fix/project-vpn branch October 14, 2024 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants