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

Release version 1.5.0 #318

Merged
merged 195 commits into from
Sep 25, 2023
Merged

Release version 1.5.0 #318

merged 195 commits into from
Sep 25, 2023

Conversation

indy-independence
Copy link
Member

Version 1.5.0

New features:

  • Add commit confirm modes: mode 0 - no commit confirm (previous behavior), mode 1 - confirm each device individually
    after it has commited, mode 2 - confirm devices only when all devices in the job has comitted successfully
  • Add API to query configured API settings
  • New settings for:
  • users: username, ssh_key, password_hash etc to handle device user accounts
  • dot1x_multi_host, poe_reboot_maintain
  • prefix_sets and routing_policies to describe policies for router peerings etc
  • Sync history events, save what event caused devices to become unsynchronized
  • API to query running config

Bug fixes:

  • Don't return duplicates of neighbor entries
  • Fix error "Error in repository: HEAD is a detached" when refreshing settings repo
  • Mark init neighbors as unsync if they have local changes
  • Log events was not always sent from redis to websocket

indy-independence and others added 30 commits January 15, 2023 15:01
…n, so we can get job_id into the confirmed version
…ce confirm, 2=next job id with confirm (platform dependent?)
…on confirm_devices to run as second job after syncto, to confirm all devices. works with Arista if EOS-driver is changed to not discard changes at close()
…e 2 instead of closing connection immediately. warn if syncing more than 50 devices, connection timeouts might cause issues?
The PostgreSQL test database is usually initialized through some SQL
fixture file. This fixture doesn't necessarily include all the latest
schema changes.

This ensures that once the test PostgreSQL database is up and running,
alembic is used to apply all the available migrations up to the head
revision.
Ensure SQL schema is updated before test run
…ncto and test scheduler use that one with scope=session to only have one scheduler. rework test_scheduler to pytest format.
indy-independence and others added 25 commits September 8, 2023 10:22
…empty diff,

otherwise wrong message is displayed and job lock is released too early.
Add sync event immediately when device is unreachable after commit
Bumps [gitpython](https://github.com/gitpython-developers/GitPython) from 3.1.34 to 3.1.35.
- [Release notes](https://github.com/gitpython-developers/GitPython/releases)
- [Changelog](https://github.com/gitpython-developers/GitPython/blob/main/CHANGES)
- [Commits](gitpython-developers/GitPython@3.1.34...3.1.35)

---
updated-dependencies:
- dependency-name: gitpython
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
…ng" using X-Fields header instead of custom query params, hopefully a bit more standard and more flexible

Example use curl -H "X-Fields: available_variables{interfaces},hostname" to only retreive hostname and variables regarding interfaces
…sh_ok

Bugfix.confirmed 2 failed after push ok
@indy-independence indy-independence added this to the v1.5.0 milestone Sep 25, 2023
@sonarcloud
Copy link

sonarcloud bot commented Sep 25, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
6.2% 6.2% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Patch coverage: 70.21% and project coverage change: +0.67% 🎉

Comparison is base (1b59651) 64.78% compared to head (ea46777) 65.46%.
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #318      +/-   ##
==========================================
+ Coverage   64.78%   65.46%   +0.67%     
==========================================
  Files          65       66       +1     
  Lines        6806     7325     +519     
==========================================
+ Hits         4409     4795     +386     
- Misses       2397     2530     +133     
Files Changed Coverage Δ
src/cnaas_nms/devicehandler/erase.py 0.00% <0.00%> (ø)
src/cnaas_nms/run.py 16.23% <0.00%> (-1.04%) ⬇️
src/cnaas_nms/api/mgmtdomain.py 43.47% <25.80%> (-3.93%) ⬇️
src/cnaas_nms/db/git.py 56.91% <36.84%> (-1.64%) ⬇️
src/cnaas_nms/devicehandler/update.py 77.77% <50.00%> (-0.30%) ⬇️
src/cnaas_nms/devicehandler/init_device.py 53.00% <52.17%> (-0.19%) ⬇️
src/cnaas_nms/api/app.py 73.94% <66.66%> (-0.62%) ⬇️
src/cnaas_nms/devicehandler/sync_devices.py 73.08% <70.53%> (+0.01%) ⬆️
src/cnaas_nms/db/device.py 74.41% <73.07%> (+0.80%) ⬆️
src/cnaas_nms/api/linknet.py 69.51% <75.00%> (+0.07%) ⬆️
... and 16 more

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@indy-independence indy-independence merged commit 01b8c79 into master Sep 25, 2023
5 of 6 checks passed
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.

2 participants