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.

Sign up for GitHub

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

Jump to bottom

Update ios tests #42509

Merged
merged 2 commits into from Jul 9, 2018
Merged

Update ios tests #42509

merged 2 commits into from Jul 9, 2018

Conversation

Qalthos
Copy link
Contributor

@Qalthos Qalthos commented Jul 9, 2018

SUMMARY

Should fix DCI issues

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ios

ANSIBLE VERSION
2.7

@Qalthos Qalthos requested a review from trishnaguha July 9, 2018 13:13
@ansibot
Copy link
Contributor

ansibot commented Jul 9, 2018

cc @privateip @rcarrillocruz
click here for bot help

@ansibot ansibot added affects_2.7 This issue/PR affects Ansible v2.7 bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. support:community This issue/PR relates to code supported by the Ansible community. support:network This issue/PR relates to code supported by the Ansible Network Team. test This PR relates to tests. labels Jul 9, 2018
@Qalthos Qalthos merged commit 399d13d into ansible:devel Jul 9, 2018
@Qalthos Qalthos deleted the ios-tests branch July 9, 2018 14:12
Qalthos added a commit to Qalthos/ansible that referenced this pull request Jul 9, 2018
@Qalthos
[2.6] Update ios tests (ansible#42509)
e098b12 
* 192.168.0.1/24 conflicts with management IP

* Images don't have the hostnames we expect
(cherry picked from commit 399d13d)

Co-authored-by: Nathaniel Case <this.is@nathanielca.se>
Qalthos added a commit to Qalthos/ansible that referenced this pull request Jul 9, 2018
@Qalthos
[2.5] Update ios tests (ansible#42509)
50a80fe 
* 192.168.0.1/24 conflicts with management IP

* Images don't have the hostnames we expect
(cherry picked from commit 399d13d)

Co-authored-by: Nathaniel Case <this.is@nathanielca.se>
@bcoca bcoca removed the needs_triage Needs a first human triage before being processed. label Jul 9, 2018
mattclay pushed a commit that referenced this pull request Jul 17, 2018
@Qalthos @mattclay
[2.6] Update ios tests (#42509) (#42516)
d9f5eb6 
* [2.6] Update ios tests (#42509)

* 192.168.0.1/24 conflicts with management IP

* Images don't have the hostnames we expect
(cherry picked from commit 399d13d)

Co-authored-by: Nathaniel Case <this.is@nathanielca.se>

* ios test cleanup pt 2 (#42570)

* Remove local from ios_config to avoid needing to add provider

* ios_smoke should test all with local to pick up the slack

(cherry picked from commit 381263a)
nitzmahone pushed a commit that referenced this pull request Jul 23, 2018
@Qalthos @nitzmahone
[2.5] Update ios tests (#42509) (#42515)
dc37b41 
* [2.5] Update ios tests (#42509)

* 192.168.0.1/24 conflicts with management IP

* Images don't have the hostnames we expect
(cherry picked from commit 399d13d)

Co-authored-by: Nathaniel Case <this.is@nathanielca.se>

* ios test cleanup pt 2 (#42570)

* Remove local from ios_config to avoid needing to add provider

* ios_smoke should test all with local to pick up the slack

(cherry picked from commit 381263a)
@dagwieers dagwieers added ios Cisco IOS community cisco Cisco technologies networking Network category labels Feb 27, 2019
@ansible ansible locked and limited conversation to collaborators Jul 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@trishnaguha trishnaguha Awaiting requested review from trishnaguha

Assignees
No one assigned
Labels
affects_2.7 This issue/PR affects Ansible v2.7 bug This issue/PR relates to a bug. cisco Cisco technologies ios Cisco IOS community networking Network category support:community This issue/PR relates to code supported by the Ansible community. support:network This issue/PR relates to code supported by the Ansible Network Team. test This PR relates to tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Qalthos @ansibot @dagwieers @bcoca