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

Add deploy/zyxel_gs1900.sh #5043

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

coderjoe
Copy link

@coderjoe coderjoe commented Mar 7, 2024

Add support for deploying to the Zyxel GS1900 line of switches as long as those switches are running at least firmware V2.80.

Tested on a Zyxel GS1900-8 and GS1900-24E

Resolves #5042

@coderjoe
Copy link
Author

coderjoe commented Mar 8, 2024

I'm now using this version locally for my switches. Please let me know if there is anything you'd like me to do to improve the PR. No rush. :)

@coderjoe
Copy link
Author

A bug has occurred on one of my servers with a different version of sed. Working on a fix now.

@coderjoe
Copy link
Author

Most recent commit should have resolved the issue I found.

deploy/zyxel_gs1900.sh Outdated Show resolved Hide resolved
@coderjoe coderjoe marked this pull request as draft March 27, 2024 01:49
Copy link
Author

@coderjoe coderjoe left a comment

Choose a reason for hiding this comment

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

These are the differences between the two implementations and why I was using curl rather than _post. I have not been able to get _post to work.

deploy/zyxel_gs1900.sh Outdated Show resolved Hide resolved
deploy/zyxel_gs1900.sh Outdated Show resolved Hide resolved
@coderjoe coderjoe force-pushed the feature/deploy-to-zyxel-gs1900-switches branch from 90a91a2 to 2bedbad Compare April 6, 2024 17:30
@coderjoe
Copy link
Author

coderjoe commented Apr 6, 2024

I've updated for the shellcheck and shfmt linting in the previous run, have removed the defunct curl errors (now that curl is not a requirement) and have squashed the commit to clean up history a bit.

@coderjoe
Copy link
Author

coderjoe commented Apr 6, 2024

I've verified that the updated commits continue to work in all of my deployed environments.
Thank you very much @Neilpang for your help/guidance on wget support.

I don't think I would have figured it out without your suggestion. :)

@coderjoe coderjoe marked this pull request as ready for review April 6, 2024 17:56
Add support for deploying to the Zyxel GS1900 line of switches as long
as those switches are running at least firmware V2.80.

Tested on a Zyxel GS1900-8 and GS1900-24E

Resolves acmesh-official#5042
@coderjoe coderjoe force-pushed the feature/deploy-to-zyxel-gs1900-switches branch from 8646c3b to 721e57c Compare April 7, 2024 14:29
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.

None yet

2 participants