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

Bulk Walk Maximum Repetitions may sometimes be ignored #5239

Closed
thomasbuesser opened this issue Feb 16, 2023 · 9 comments
Closed

Bulk Walk Maximum Repetitions may sometimes be ignored #5239

thomasbuesser opened this issue Feb 16, 2023 · 9 comments
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Milestone

Comments

@thomasbuesser
Copy link

Describe the bug

If you change the value for Bulk Walk Maximum Repetitions on the SNMP-Options Page the value is not written to the DB

To Reproduce

Steps to reproduce the behavior:

  1. Go to '/cacti/automation_snmp.php?action=item_edit&item_id=4&id=1'

  2. Change 'Bulk Walk Maximum Repetitions' from Auto to '1 Repetition'

  3. Save

  4. Reopen Item

Expected behavior

1 Repetition should be saved

@thomasbuesser thomasbuesser added bug Undesired behaviour unverified Some days we don't have a clue labels Feb 16, 2023
@TheWitness
Copy link
Member

You did not state your Cacti version.

@thomasbuesser
Copy link
Author

sorry, Version 1.2.23 - fresh installed on freebsd

@TheWitness
Copy link
Member

Okay, thanks for the confirmation.

@TheWitness
Copy link
Member

Can you update to the 1.2.x branch and see if the problem persists? I believe this was recently addressed. I was able to save to 1 and then check back. Going to let it simmer a few polling cycles and re-indexes and see if it changes back to auto though. Yea, it's working as expected on the 1.2.x branch. Sorry for the inconvenience, and thanks for reporting.

@TheWitness TheWitness added duplicate Duplicate of another issue resolved A fixed issue and removed unverified Some days we don't have a clue labels Feb 19, 2023
@TheWitness
Copy link
Member

Duplicate bug is here: #5145

@thomasbuesser
Copy link
Author

i will... give me a few days for testing

@thomasbuesser
Copy link
Author

tried it: no success

the form-field bulk_walk_size is not processed in automation_snmp.php --> isset_request_var('save_component_automation_snmp_item')

@TheWitness
Copy link
Member

Oh geeze I completely missed that it was for automation. Let me look into that.

@TheWitness TheWitness reopened this Feb 20, 2023
@TheWitness TheWitness added confirmed Bug is confirm by dev team and removed duplicate Duplicate of another issue labels Feb 20, 2023
@TheWitness TheWitness added this to the v1.2.24 milestone Feb 20, 2023
@TheWitness
Copy link
Member

Okay, I've done the fix now. Thanks for continuing to pester me ;)

@TheWitness TheWitness added porting required Requires porting to develop and removed porting required Requires porting to develop labels Feb 20, 2023
netniV pushed a commit that referenced this issue Feb 24, 2023
Fixing #5238 - SQL Issues due to non-visible chars
Fixing #5232 Poller Stacking
Fixing #5236 - Edit Graph Template
Fixing #5243 - Export columns missing
Fixing Issue #5239 - Automation SNMP Bulk Size
Fixing #5241 - Polling distribution not working
Fixing #5247 and QA on #5241 - Polling Issues
@netniV netniV changed the title SNMP-Options - Bulk Walk Maximum Repetitions ignored Bulk Walk Maximum Repetitions may sometimes be ignored Feb 27, 2023
@github-actions github-actions bot locked and limited conversation to collaborators May 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Projects
None yet
Development

No branches or pull requests

2 participants