Skip to content

Comments

Update VPN setup instructions and add image#304

Merged
CarlosLopezElorduy merged 5 commits intomainfrom
293-vpn-setup-irbcluster02
Jan 30, 2026
Merged

Update VPN setup instructions and add image#304
CarlosLopezElorduy merged 5 commits intomainfrom
293-vpn-setup-irbcluster02

Conversation

@CarlosLopezElorduy
Copy link
Member

This pull request significantly updates the VPN setup documentation for IRB, focusing on a new, streamlined process using FortiClient. The instructions are now clearer and more step-by-step, with outdated details about two-factor authentication and previous Linux setup methods removed. The new version also adds guidance for terminal usage and includes helpful links and images.

Major documentation update and simplification:

  • Rewrote the VPN setup instructions to focus on installing and configuring FortiClient, providing a clear step-by-step process with direct download links and visuals.
  • Removed outdated sections related to two-factor authentication setup and the previous OpenConnect-based Linux instructions, consolidating the guide around the new FortiClient workflow.

Linux-specific improvements:

  • Added specific terminal commands for Ubuntu users to configure DNS resolution for the VPN, replacing previous troubleshooting steps.
  • Provided detailed instructions for importing the VPN configuration file and managing connections through both the FortiClient GUI and terminal commands.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the IRB VPN setup documentation by replacing the previous OpenConnect-based instructions with new FortiClient-based instructions. The changes remove outdated two-factor authentication setup details and Linux-specific OpenConnect troubleshooting, replacing them with streamlined FortiClient installation and configuration steps.

Changes:

  • Replaced OpenConnect VPN setup instructions with FortiClient-based workflow
  • Added FortiClient installation guide with download links and terminal commands for Ubuntu DNS configuration
  • Included new screenshot image showing the FortiClient download page and added terminal command reference table

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 9 comments.

File Description
docs/IRB/VPN.md Complete rewrite of VPN setup instructions, replacing OpenConnect with FortiClient installation and configuration steps
docs/assets/images/forticlient_web.png New PNG image showing FortiClient download page (binary file)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@CarlosLopezElorduy
Copy link
Member Author

@copilot address the PR comments, and also lint the markdown

Copy link
Contributor

Copilot AI commented Jan 23, 2026

@CarlosLopezElorduy I've opened a new pull request, #312, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: CarlosLopezElorduy <107858804+CarlosLopezElorduy@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
and separeted the two command lines
Copilot AI review requested due to automatic review settings January 29, 2026 11:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

@msguixe msguixe left a comment

Choose a reason for hiding this comment

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

I have made a new commit changing 'VPN-Nexica' to 'VPN-nexica'.
Also, I have separated the bash commands into two separated lines, so it is clear that they need to be executed separated.
The rest looks good.

@msguixe msguixe self-requested a review January 29, 2026 14:54
Added troubleshooting note for VPN connection issues.
Copilot AI review requested due to automatic review settings January 29, 2026 15:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

@msguixe msguixe left a comment

Choose a reason for hiding this comment

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

I'va added a note stating to restart the computer i the connection is stuck.
We can now pull request.

@CarlosLopezElorduy CarlosLopezElorduy merged commit be26e8b into main Jan 30, 2026
2 checks passed
@CarlosLopezElorduy CarlosLopezElorduy deleted the 293-vpn-setup-irbcluster02 branch January 30, 2026 08:40
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.

BBGwiki | [update] VPN Setup Configuration for IRBCluster02

3 participants