jenkinsci / jenkinsci/ec2-plugin
[JENKINS-65259] 'Check Current Spot Price' shows incorrect price for spot instance if 'AMI ID' field is empty
- Dominant language
- Java
- Stars
- 292
- Forks
- 709
- Avg merge
- 3d 16h
- Merged PRs (30d)
- 4
Description
Hi,
I have started using the AMI filters instead of static AMI configuration: https://github.com/jenkinsci/ec2-plugin/pull/533/commits/759e1fd10fe8ad9881d699fca198b081c8faed74 and noticed that 'Check Current Spot Price' funtionality shows price for on-demand instances if 'AMI ID' is not specified.
STR:
- Configure ec2 plugin and template to use spot configuration;
- Press 'Check Current Spot Price' button and you will see the correct price
- remove the field 'AMI ID'
- Press 'Check Current Spot Price' button one more time
Actual Result:
ec2-plugin shows the price for on-demand instances now...
I attached the screen-shots.
---
Originally reported by aston_r, imported from: 'Check Current Spot Price' shows incorrect price for spot instance if 'AMI ID' field is empty
Raw content of original issue
Hi,
I have started using the AMI filters instead of static AMI configuration: https://github.com/jenkinsci/ec2-plugin/pull/533/commits/759e1fd10fe8ad9881d699fca198b081c8faed74 and noticed that 'Check Current Spot Price' funtionality shows price for on-demand instances if 'AMI ID' is not specified.
STR:
- Configure ec2 plugin and template to use spot configuration;
- Press 'Check Current Spot Price' button and you will see the correct price
- remove the field 'AMI ID'
- Press 'Check Current Spot Price' button one more time
Actual Result:
ec2-plugin shows the price for on-demand instances now...
I attached the screen-shots.
environment
```
jenkins 2.277.1
ec2-plugin 1.56
```
2 attachments
- [ami-id-empty.PNG](https://issues.jenkins.io/secure/attachment/54370/ami-id-empty.PNG)
> 
- [ami-id.PNG](https://issues.jenkins.io/secure/attachment/54369/ami-id.PNG)
> 
Contributor guide
Assessment
This issue has not been assessed yet.