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 allow inbound echo requests + configure SSH keys only when needed #152

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

Conversation

DeBuXer
Copy link

@DeBuXer DeBuXer commented Dec 5, 2024

Two small adjustments:

  1. Allowing ICMP requests did not work because the variable $fwmgcalPolicy was not defined. As a result, executing it produced the following error message:
[2024-12-05 13:44 +01:00] * Enabling Ping

LocalPolicy        : System.__ComObject
CurrentProfileType : 1

[2024-12-05 13:44 +01:00] - Enable Allow Inbound Echo Requests
The property 'AllowInboundEchoRequest' cannot be found on this object. Verify that the property exists and can be set.
At C:\Program Files (x86)\OpenNebula\context.ps1:916 char:12
+     $ret = $pro.IcmpSettings.AllowInboundEchoRequest = $true
+            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : PropertyNotFound
The property 'AllowInboundEchoRequest' cannot be found on this object. Verify that the property exists and can be set.
At C:\Program Files (x86)\OpenNebula\context.ps1:916 char:12
+     $ret = $pro.IcmpSettings.AllowInboundEchoRequest = $true
+            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : PropertyNotFound

[2024-12-05 13:44 +01:00]   ... Failed
  1. When the variable $context["SSH_PUBLIC_KEY"] is empty, it is unnecessary to call the functions any further.

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.

1 participant