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

Dev convert to restpy #139

Open
wants to merge 7 commits into
base: dev_convertToRestpy
Choose a base branch
from

Conversation

srsettur
Copy link

RestAPI to RestPy migration test runStatus1669378734.docx

Please find the PR targeted to dev_convertToRestpy branch rather to master. Attached the UT.
There is a failure observed while running mplsNgpf.py, this is been tracked and requires 9.30 API server release.

@hubogee
Copy link
Contributor

hubogee commented Dec 8, 2022

Hi Sreekanth,

It appears to me that the sample scripts were not ran, tested and verified because many of the scripts are failing. I stopped verifying after the followings. I will verify them after you fix these and please verify the rest of the scripts.

  • All sample scripts have IxNetRestApiException that was not imported in the try/except clause:
    except(IxNetRestApiException, Exception, KeyboardInterrupt) as errMsg:

  • bulkConfiguration.py

  • egressTrackingNgpy.py

  • isisNgpf.py

  • lacpOspfNgpy.py

  • jsonConfigImportExportSamples.py

    • the sys.path.insert(0, "../Modules/Main") is incorrect
    • Why is /Main added?

@srsettur
Copy link
Author

Hi @hubogee ,
I used 'git fetch origin pull/139/head:PR_139' to get the 139 PR specific changes. This created a new branch with name PR_139. I tried to run isisNgpf.py. I don't see any issues.
Can you please help me to understand, if I am missing something here.

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