Skip to content

Update bgp-peers.inc.php for timos#15154

Merged
electrocret merged 1 commit intolibrenms:shift-91874from
i4networks:patch-1
Jul 20, 2023
Merged

Update bgp-peers.inc.php for timos#15154
electrocret merged 1 commit intolibrenms:shift-91874from
i4networks:patch-1

Conversation

@i4networks
Copy link
Copy Markdown
Contributor

@i4networks i4networks commented Jul 19, 2023

$address = ipv4.1.xxx.xxx.xxx.xxx -> could be 21 not 15 characters, why this function?

change verified on working on
Nokia SR OS (TiMOS) 22.10.R4

Please give a short description what your pull request is for

DO NOT DELETE THE UNDERLYING TEXT

Please note

Please read this information carefully. You can run ./lnms dev:check to check your code before submitting.

  • Have you followed our code guidelines?
  • If my Pull Request does some changes/fixes/enhancements in the WebUI, I have inserted a screenshot of it.
  • If my Pull Request makes discovery/polling/yaml changes, I have added/updated test data.

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926
After you are done testing, you can remove the changes with ./scripts/github-remove. If there are schema changes, you can ask on discord how to revert.

$address = ipv4.1.xxx.xxx.xxx.xxx -> could be 21 character,
why this function?

change verified on working on 
Nokia SR OS (TiMOS) 22.10.R4
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jul 19, 2023

CLA assistant check
All committers have signed the CLA.

@electrocret electrocret changed the title Update bgp-peers.inc.php Update bgp-peers.inc.php for timos Jul 20, 2023
@electrocret electrocret merged commit 993f35d into librenms:shift-91874 Jul 20, 2023
@electrocret electrocret added the Device 🖥️ New or added device support label Aug 2, 2023
@librenms-bot
Copy link
Copy Markdown

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/23-8-0-changelog/22078/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Device 🖥️ New or added device support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants