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

ec2_elb_lb: Numeric parameter stickiness['expiration'] passed to module as string and fails test #27309

Closed
hackndoes opened this issue Jul 26, 2017 · 2 comments · Fixed by #27328
Closed

ec2_elb_lb: Numeric parameter stickiness['expiration'] passed to module as string and fails test #27309

hackndoes opened this issue Jul 26, 2017 · 2 comments · Fixed by #27328
Labels
affects_2.3 This issue/PR affects Ansible v2.3 aws bug This issue/PR relates to a bug. cloud deprecated This issue/PR relates to a deprecated module. module This issue/PR relates to a module.

Comments

@hackndoes
Copy link

ISSUE TYPE
  • Bug Report
COMPONENT NAME

ec2_elb_lb.py

ANSIBLE VERSION
ansible 2.3.1.0
  config file = 
  configured module search path = Default w/o overrides
  python version = 2.7.13 (default, Jan  5 2017, 16:03:42) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.42.1)]

also exists on 2.2.0.0
CONFIGURATION

standard

OS / ENVIRONMENT

MacOS/Linux

SUMMARY

setting stickiness.expiration to 0 in the module should be converted to None.
it doesn't because the test stickiness['expiration'] is not 0 fails since stickiness['expiration'] is "0" for some odd reason. this makes the value sent to aws a 0 instead of a None which is not legal on amazon side as value for expiration.

STEPS TO REPRODUCE

simply try to create an ELB using the ec2_elb_lb module

https://gist.github.com/hackndoes/925e461767d4b26a0e6d817d558070ed

https://gist.github.com/hackndoes/619b9000dd75c558b352f2709520d3ff

EXPECTED RESULTS

creation of LB will succeed

ACTUAL RESULTS

inside first gist
@ansibot
Copy link
Contributor

ansibot commented Jul 26, 2017

cc @jarv
click here for bot help

@ansibot ansibot added affects_2.3 This issue/PR affects Ansible v2.3 aws bug_report cloud deprecated This issue/PR relates to a deprecated module. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. labels Jul 26, 2017
@gundalow gundalow changed the title Numeric parameter stickiness['expiration'] passed to module as string and fails test ec2_elb_lb: Numeric parameter stickiness['expiration'] passed to module as string and fails test Jul 26, 2017
@gundalow gundalow removed the needs_triage Needs a first human triage before being processed. label Jul 26, 2017
@ansibot
Copy link
Contributor

ansibot commented Jul 26, 2017

cc @jsdalton
click here for bot help

@s-hertel s-hertel mentioned this issue Jul 26, 2017
Merged
@ansibot ansibot added bug This issue/PR relates to a bug. and removed bug_report labels Mar 7, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
affects_2.3 This issue/PR affects Ansible v2.3 aws bug This issue/PR relates to a bug. cloud deprecated This issue/PR relates to a deprecated module. module This issue/PR relates to a module.
Projects
None yet
Milestone
No milestone
3 participants
@gundalow @hackndoes @ansibot