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

[IOS-XE] Support of SSID with space #832

Open
FlavienJP opened this issue Mar 25, 2024 · 6 comments
Open

[IOS-XE] Support of SSID with space #832

FlavienJP opened this issue Mar 25, 2024 · 6 comments
Assignees

Comments

@FlavienJP
Copy link

Hello,

We are using Genie in conjonction with PyATS for testing, but we saw that Genie was unable to work with SSID that is using space within it.

We have made some changes to our local library that has corrected the genie behaviour to support this configuration.

class ShowWlanSummary(ShowWlanSummarySchema):

wlan_info_capture = re.compile(

Before :
^(?P<wlan_id>\d+)\s+(?P<profile_name>\S+)\s+(?P<ssid>\S+)\s+(?P<wlan_status>\S+)\s+(?P<status_security>.*$)

After :
^(?P<wlan_id>\d+)\s+(?P<profile_name>\S+)\s+(?P<ssid>.*?)(?:\s{2,})\s+(?P<wlan_status>\S+)\s+(?P<status_security>.*$)

This enhance the SSID support with maxmium 2 following spaces in the title (in my opinion, 1 is enough).

class ShowWlanAllSchema(MetaParser):

Before :
"radio_policy": str,

After :
Optional("radio_policy"): str,

Base on our testings, the radio policy is not visible in the CLI extract when using DNAC, it's preferable to mark it as optional.

class ShowWlanAll(ShowWlanAllSchema):

p_name_ssid = re.compile(r"^Network\s+Name\s+\(SSID\)\s+:\s+(?P<value>\S+)$")

Before :
p_name_ssid = re.compile(r"^Network\s+Name\s+\(SSID\)\s+:\s+(?P<value>\S+)$")

After :
p_name_ssid = re.compile(r"^Network\s+Name\s+\(SSID\)\s+:\s+(?P<value>.*)$")

Then end of the line is containing the SSID (no more information) so we can add .* to catch everything.

@ParimiNeeraja
Copy link

ParimiNeeraja commented Mar 26, 2024

Hello ,

Can you please share me the outputs of two command show wlan summary and show wlan all.

@FlavienJP
Copy link
Author

Hello @ParimiNeeraja,

I can first provide you the summary.

raw = device.execute("show wlan summary")
2024-03-26 17:38:20,677: %UNICON-INFO: +++ demowlc with via 'cli': executing command 'show wlan summary' +++
show wlan summary
Number of WLANs: 2
ID   Profile Name                     SSID                             Status Security                                                                                             
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
17   CUST_Global_NF_6f7f68b3     CUST Administrative                     UP     [WPA2][PSK][AES],MAC Filtering                                                                       
18   CUSTC_Global_NF_24c59b82    CUST Factory                 UP     [WPA2][802.1x][AES]                                                                                  
demowlc#

Regards,

@Neeraja1bala
Copy link

Thanks for providing output.
Can you please provide the output for show wlan all

@Neeraja1bala
Copy link

Hello @FlavienJP ,

The above PR got approve from reviewers. Kindly merge the PR.

@Neeraja1bala
Copy link

Hello @FlavienJP ,

One reviewer was commented in PR. Please follow up the PR.

@Neeraja1bala
Copy link

Hello @FlavienJP ,

PR got merged. Changes will reflected in next version.

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

No branches or pull requests

3 participants