-
Notifications
You must be signed in to change notification settings - Fork 669
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
Support SL Quote API #554
Comments
Any update as to when will this feature be released? |
Hi, Any Updates? |
Thank you very much for finally adding the feature to order using quote_id via Pull request #2433 . We are able to provision a server. However, there are few issues we are facing while provisioning the server using this code.
To start with we are creating a Virtual server with the following simple code before we start using complicated terraform scripts.
|
@cb-ankit-gupta Thanks for testing and sharing out the results.... We will move the fix to master ..Is it possible for you test If i provide you binary on my fork branch before releasing the provider...With this binary i can confirm the sshkey, imageid should be resolved... On the discount may be we need to check with some classic Infrastructure team...Once you confirm the other two issues resolved |
sure, I can test the same. |
Thanks Which version of the terraform are you using |
Hi, Unfortunately, I am not able to provision the servers using this Binary. I am getting the following error on apply
Versions:
|
on renaming the binary to
output:
|
@cb-ankit-gupta I tested on my enivronment(darwin mac) I can run it without any issues
|
@cb-ankit-gupta I tested on linux its working as expected
|
I have cleared the .terraform folder in the folder where I run the code. I am still not able to provision the server with the Binary you have given. Terraform plugin folder
Terraform code
Respose
However, the same code, same steps. I am at least able to provision the server with the earlier binary for which I had raised the issues. |
On removing GCS backend and changing to the local backend I got a crash. Hope this crash log helps you. https://github.com/cb-ankit-gupta/terraform-provider-ibm-issue-554/blob/master/crash-1.log |
Thanks for sharing the log but I don't get much from the log why its failing at the order of the receipt..I tried same template multiple times from our end
I can see the order goes successfully |
From your log can you confirm the post and get calls are showing the valid responses ..I don't see in your log may be you deleted them |
I did not delete anything. I have just redacted the values using |
I have found the issue. I am able to provision the server when I am giving just 1 SSH key. However, on giving 3 SSH keys I am getting the above error. Following code is working and provisioning the server.
Following code is not working and giving error.
|
@cb-ankit-gupta Thanks I will debug more wt is the issue with multiple keys....But with the new driver does the machines are applicable with discounts? |
no, they are still without discounts.
|
https://github.com/IBM-Cloud/terraform-provider-ibm/releases/tag/v1.23.0
|
This Works, Earlier
|
Sure, I will check v1.23.0 and update you shortly. |
Hi @hkantare, hope the logs were useful to find out why the discounts are not applied. |
@cb-ankit-gupta Yes I followd with classic infrastructure API team with the log shared by you..
Regarding first point- As per their comments they have some concer whether the account has appropriate discounts configured on the account, https://github.com/hkantare/terraform-provider-ibm/releases/tag/v1.23.1 |
Hi @hkantare FIXED
NOT FIXED
|
Thanks for update |
@cb-ankit-gupta |
Some customers receiving a discount on Softlayer IaaS services have to use the Quote API in order to get the proper billing. The IBM Cloud Provider should support using the Quotes API.
https://sldn.softlayer.com/reference/services/SoftLayer_Billing_Order_Quote/
The text was updated successfully, but these errors were encountered: