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

meraki_vlan - Small integration test fixes for bad variable names #42287

Merged
merged 1 commit into from
Jul 4, 2018

Conversation

kbreit
Copy link
Contributor

@kbreit kbreit commented Jul 4, 2018

SUMMARY

Fixed typos in variable names

ISSUE TYPE
  • Bug Fix Request
COMPONENT NAME

meraki_vlan

ANSIBLE VERSION
ansible 2.7.0.dev0 (meraki/util_construct_any e5bcb4d159) last updated 2018/07/03 22:36:37 (GMT -500)
  config file = None
  configured module search path = ['/Users/kbreit/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /Users/kbreit/Documents/Programming/ansible/lib/ansible
  executable location = /Users/kbreit/Documents/Programming/ansible/bin/ansible
  python version = 3.5.4 (default, Feb 25 2018, 14:56:02) [GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.39.2)]

@ansibot
Copy link
Contributor

ansibot commented Jul 4, 2018

@ansibot ansibot added affects_2.7 This issue/PR affects Ansible v2.7 bug This issue/PR relates to a bug. meraki Cisco Meraki community needs_triage Needs a first human triage before being processed. networking Network category support:community This issue/PR relates to code supported by the Ansible community. test This PR relates to tests. labels Jul 4, 2018
@kbreit
Copy link
Contributor Author

kbreit commented Jul 4, 2018

bot_status

@ansibot
Copy link
Contributor

ansibot commented Jul 4, 2018

Components

test/integration/targets/meraki_vlan/tasks/main.yml
support: community
maintainers: dagwieers kbreit

Metadata

waiting_on: maintainer
changes_requested_by: null
needs_info: False
needs_revision: False
needs_rebase: False
merge_commits: []
too many files or commits: False
mergeable_state: clean
shippable_status: success
maintainer_shipits (module maintainers): 1
community_shipits (namespace maintainers): 0
ansible_shipits (core team members): 0
shipit_actors (maintainers or core team members): kbreit
shipit_actors_other: []
automerge: automerge shipit test failed

click here for bot help

@kbreit
Copy link
Contributor Author

kbreit commented Jul 4, 2018

shipit

@kbreit
Copy link
Contributor Author

kbreit commented Jul 4, 2018

bot_status

@ansibot
Copy link
Contributor

ansibot commented Jul 4, 2018

Components

test/integration/targets/meraki_vlan/tasks/main.yml
support: community
maintainers: dagwieers kbreit

Metadata

waiting_on: maintainer
changes_requested_by: null
needs_info: False
needs_revision: False
needs_rebase: False
merge_commits: []
too many files or commits: False
mergeable_state: clean
shippable_status: success
maintainer_shipits (module maintainers): 1
community_shipits (namespace maintainers): 0
ansible_shipits (core team members): 0
shipit_actors (maintainers or core team members): kbreit
shipit_actors_other: []
automerge: automerge shipit test failed

click here for bot help

@gundalow gundalow removed the needs_triage Needs a first human triage before being processed. label Jul 4, 2018
@gundalow gundalow merged commit c3c86a0 into ansible:devel Jul 4, 2018
@gundalow
Copy link
Contributor

gundalow commented Jul 4, 2018

Merged into devel, if needed please raise a backport PR for other branches

@kbreit kbreit deleted the meraki/test-vlan-fix branch February 1, 2019 04:13
@dagwieers dagwieers added the cisco Cisco technologies label Feb 22, 2019
@ansible ansible locked and limited conversation to collaborators Jul 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.7 This issue/PR affects Ansible v2.7 bug This issue/PR relates to a bug. cisco Cisco technologies meraki Cisco Meraki community networking Network category support:community This issue/PR relates to code supported by the Ansible community. test This PR relates to tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants