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

feat: partial walk #990

Draft
wants to merge 46 commits into
base: develop
Choose a base branch
from
Draft

feat: partial walk #990

wants to merge 46 commits into from

Conversation

ajasnosz
Copy link
Collaborator

@ajasnosz ajasnosz commented Apr 5, 2024

Description

Introducing change to the default walk process. The default walk to poll would be profile with SNMPv2-MIB instead of full walk tree. The full walk option can be enabled by setting new flag in values.yaml

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Refactor/improvement
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

manual test

Checklist

  • My commit message is conventional
  • I have run pre-commit on all files before creating the PR
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have checked my code and corrected any misspellings

ajasnosz and others added 19 commits March 8, 2024 11:01
feat: update ci-docker-deployment.yaml

feat: update ci-docker-deployment.yaml

feat: update ci-docker-deployment.yaml

fix: change ci-docker-deployment.yaml to work on branch push

feat: add asset upload to the CI

fix: run pre-commit

feat: add changelog update

feat: delete unnecessary images from documentation

feat: changes in documentation after review

feat: more updates after PR review

fix: rename variables in manage_secrets.py and fix typos in documentation

fix typo in documentation
# [1.11.0-beta.1](v1.10.0...v1.11.0-beta.1) (2024-03-14)

### Bug Fixes

* update .releaserc ([#979](#979)) ([2bcfedd](2bcfedd))

### Features

* add docker compose deployment configuration ([#973](#973)) ([461774a](461774a))
* chore(deps): update dependency pytest to v8.1.1

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: ajasnosz <ajasnosz@splunk.com>
* chore: update jinja2 and pycryptodomex

* test: test [run-int-tests]
….2 (#980)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
….4.2 (#982)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…958)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
# [1.11.0-beta.2](v1.11.0-beta.1...v1.11.0-beta.2) (2024-03-27)

### Bug Fixes

* fix formatting of Debug Splunk Connect for SNMP documentation ([#988](#988)) ([5a944e4](5a944e4))
* fix: improve error messages
@ajasnosz ajasnosz force-pushed the feat/partial-walk branch from 5b1a192 to 8ad5343 Compare April 12, 2024 11:28
wojtekzyla and others added 10 commits April 12, 2024 14:00
fix: change line formatting

fix: add new line character at the end
# [1.11.0-beta.3](v1.11.0-beta.2...v1.11.0-beta.3) (2024-04-12)

### Bug Fixes

* improve mkdocs.yml file formatting ([#994](#994)) ([ead94b3](ead94b3))
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
renovate bot added 3 commits April 17, 2024 10:03
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@ajasnosz ajasnosz force-pushed the feat/partial-walk branch from b08b80c to e80837b Compare April 17, 2024 11:00
ajasnosz and others added 9 commits April 18, 2024 09:44
…ating to SC4SNMP-UI (#998)

fix: update wrong variable name

[run-ui-tests]

fix: update CHANGELOG

fix: update CHANGELOG
# [1.11.0-beta.4](v1.11.0-beta.3...v1.11.0-beta.4) (2024-04-19)

### Bug Fixes

* fix a bug with values.yaml not being loaded to the UI while migrating to SC4SNMP-UI ([#998](#998)) ([3c9273b](3c9273b))
# [1.11.0-beta.5](v1.11.0-beta.4...v1.11.0-beta.5) (2024-04-22)

### Bug Fixes

* fix sonarqube issues with f-string and regex ([#1001](#1001)) ([c415d0e](c415d0e))
# [1.11.0-beta.6](v1.11.0-beta.5...v1.11.0-beta.6) (2024-04-25)

### Bug Fixes

* update microk8s to v1.30 ([#1004](#1004)) ([642c539](642c539))
@ajasnosz ajasnosz force-pushed the feat/partial-walk branch from 1984885 to 7075179 Compare April 25, 2024 12:11
@ajasnosz ajasnosz force-pushed the feat/partial-walk branch from 1602ba5 to c845843 Compare April 26, 2024 10:50
Base automatically changed from develop to main July 24, 2024 12:40
@ajasnosz ajasnosz changed the base branch from main to develop July 24, 2024 13:36
Base automatically changed from develop to main October 16, 2024 10:03
@ajasnosz ajasnosz changed the base branch from main to develop October 16, 2024 10:29
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.

3 participants