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

Dropped PCL support for CML versions below 2.6.0 #156

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

tmikuska
Copy link
Contributor

@tmikuska tmikuska commented Oct 3, 2024

Dropped topology support for versions below 0.1.0

@coveralls
Copy link

coveralls commented Oct 3, 2024

Pull Request Test Coverage Report for Build 11373068995

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 75.891%

Totals Coverage Status
Change from base Build 9893757736: 0.0%
Covered Lines: 1958
Relevant Lines: 2580

💛 - Coveralls


report: coverage
coverage html
coverage report
open htmlcov/index.html

test: ## run tests quickly with the default Python
python -W ignore::DeprecationWarning setup.py test
pytest tests/v2/*
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed both tests and coverage for compatibility with newer versions of some python modules such as setuptools, etc..

@@ -103,7 +103,7 @@ def get_alt_id(self):
def get_alt_title(self):
return "Other Lab"

def get_cml23_id(self):
def get_cml24_id(self):
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/virl2_client/models/lab.py:1463: DeprecationWarning: Labs created in older CML releases (schema version 0.0.5 or lower) are deprecated. Use labs with schema version 0.1.0 or higher.
warnings.warn(

I bumped test labs only to schema version 0.1.0 which was used in CML 2.4.0 to prevent the warning. Could be bumped to 2.6.0 or changes could be removed if there's any reason to prevent the support for 0.0.3 schema version.

from unittest.mock import patch
except ImportError:
from mock import patch # noqa
from . import BaseCMLTest
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The import was missing and everything else was removed due to dropped support for CML 2.3.

"""
return response

@staticmethod
def get_sys_info(req, ctx=None):
response = {"version": "2.4.0+build.3", "ready": True}
response = {"version": "2.6.0+build.5", "ready": True}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are going to drop support for CML releases below 2.6.0 in the next version of virl2 client (2.8.0).

Copy link
Collaborator

@xorrkaz xorrkaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No objections to this, but can you add a blurb to the README's prereq that states CML 2.6.0+ is required?

Dropped topology support for versions below 0.1.0
@xorrkaz xorrkaz changed the base branch from master to joe-dev October 17, 2024 14:46
@xorrkaz xorrkaz merged commit e9f87f4 into CiscoDevNet:joe-dev Oct 17, 2024
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.

3 participants