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

Conflicting info when creating configuration #196

Open
NilesDev opened this issue Feb 2, 2023 · 4 comments
Open

Conflicting info when creating configuration #196

NilesDev opened this issue Feb 2, 2023 · 4 comments

Comments

@NilesDev
Copy link

NilesDev commented Feb 2, 2023

I recently got this message when running "lm-cli configurations create ...":

╭─────────────────────────────────────────────────────────────────────────────────────────── REQUEST FAILED ────────────────────────────────────────────────────────────────────────────────────────────╮
│ Couldn't create configuration: Received an error response ({"message":"Configuration id 12 created."}).     

After the command, the configuration was indeed created and looked like any other configuration.

@NilesDev
Copy link
Author

NilesDev commented Feb 2, 2023

One possibility: I did not have permission to add any slurm license counters, that might have been the failure.

@julianaklulo
Copy link
Contributor

Hi @NilesDev, thank you for reporting this.

I see that configuration 12 was created in the staging environment, so the command executed correctly.
I tried to reproduce the error and got the expected answer:

lm-cli configurations create --name test --product test --features '{"test": {"total": 123, "limit": 123}}' --license-servers test:123.123.123:123 --license-server-type test --grace-time 123 --client-id test

╭────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── Configuration creation succeeded. ──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│                                                                                                                                                                                                                                                                                                                                                                                                                               │
│   The configuration was created successfully.                                                                                                                                                                                                                                                                                                                                                                                 │
│                                                                                                                                                                                                                                                                                                                                                                                                                               │
╰───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯

I'll investigate it further and let you know what I find.

@julianaklulo
Copy link
Contributor

Do you remember when you ran the command?
The lm-cli in staging enviroment was updated to version 2.2.17 on Jan 25.
In this version, the response code for the create endpoint was changed from 200 (OK) to 201 (Created), which could trigger the error in the response code validation in lm-cli.

@NilesDev
Copy link
Author

Sorry for late reply, but this was in production so probably the earlier version with the 201 response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants