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

67 name file is not defined is shown in some checks #68

Merged
merged 2 commits into from
Aug 14, 2023

Conversation

monrog2
Copy link
Collaborator

@monrog2 monrog2 commented Aug 14, 2023

switchover to open() and supress pexpect futurewarning in py3.8 env (ACI 6.0(2))

@monrog2 monrog2 linked an issue Aug 14, 2023 that may be closed by this pull request
@monrog2
Copy link
Collaborator Author

monrog2 commented Aug 14, 2023

6.0(2) tests passed
pytest on py2 ACI versions passed

admin@apic1:techsupport> python aci-preupgrade-validation-script.py
    ==== 2023-08-14T17-52-35+0000, Script Version v1.6.1  ====

!!!! Check https://github.com/datacenter/ACI-Pre-Upgrade-Validation-Script for Latest Release !!!!

To use a non-default Login Domain, enter apic#DOMAIN\\USERNAME
Enter username for APIC login          : admin
Enter password for corresponding User  : 

Checking current APIC version (switch nodes are assumed to be on the same version)...6.0(2h)

Gathering APIC Versions from Firmware Repository...

[1]: aci-apic-dk9.5.2.3g.bin
[2]: aci-apic-dk9.6.0.2h.bin

What is the Target Version?     : 1

You have chosen version "aci-apic-dk9.5.2.3g.bin"

Collecting VPC Node IDs...

[Check  1/47] APIC Target version image and MD5 hash... 
              Checking apic1......                                                                                                    DONE

@monrog2 monrog2 merged commit 5254892 into master Aug 14, 2023
@monrog2 monrog2 deleted the 67-name-file-is-not-defined-is-shown-in-some-checks branch August 14, 2023 18:43
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.

"name 'file' is not defined" is shown in some checks.
1 participant