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

deploy/routeros.sh fixes #3947

Merged
merged 3 commits into from
Feb 22, 2022

Conversation

abiessmann
Copy link
Contributor

Changes to deploy/routeros.sh:

  • have scp and ssh configurable to set specific parameters to those commands
  • delete all three certificates of full chain
  • set owner from ssh user
  • set commented line in script as RouterOS comment

In order to modify ssh/scp commands make them configurable via
environment variables.

Signed-off-by: Andreas Bießmann <andreas@biessmann.org>
As the script is applying the fullchain which includes three certificates,
delete all of them before applying updated certificate.

Signed-off-by: Andreas Bießmann <andreas@biessmann.org>
This set the owner of script to ssh user, have the comment line in script
as real comment and removes policy since this is set from current user,
at least for RouterOS 7.x.

Signed-off-by: Andreas Bießmann <andreas@biessmann.org>
@abiessmann abiessmann force-pushed the deploy_routeros_fixes branch from 472a007 to c46ceb0 Compare February 19, 2022 13:13
@abiessmann
Copy link
Contributor Author

Just saw another change in dev ... @mac-zhou introduced ROUTER_OS_PORT which was not in my base ... need to discuss if we stick with ROUTER_OS_PORT and ROUTER_OS_SSH_CMD/ROUTER_OS_SCP_CMD variables or to kick ROUTER_OS_PORT

@Neilpang Neilpang merged commit e88442c into acmesh-official:dev Feb 22, 2022
@abiessmann abiessmann deleted the deploy_routeros_fixes branch March 17, 2022 13:34
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.

2 participants