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

Updated command - slcli account bandwidth-pools #1943

Merged
merged 1 commit into from May 22, 2023

Conversation

edsonarios
Copy link
Contributor

Issue: #1937

softlayer-python/ (issue1933) $ slcli account bandwidth-pools
                                                  Bandwidth Pools
┌─────────┬──────────────────────┬─────────────────┬─────────┬────────────┬───────────────┬─────────────────┬──────┐
│ Id      │ Name                 │ Region          │ Devices │ Allocation │ Current Usage │ Projected Usage │ Cost │
├─────────┼──────────────────────┼─────────────────┼─────────┼────────────┼───────────────┼─────────────────┼──────┤

Copy link
Contributor

@caberos caberos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@allmightyspiff allmightyspiff linked an issue May 18, 2023 that may be closed by this pull request
@ramkishor-ch
Copy link
Contributor

Hi @edsonarios,

When I tried to test the code changes, I am getting an error.

Example:
./slcli account bandwidth-pools Bandwidth Pools
Usage: slcli account bandwidth-pools [OPTIONS]

Try 'slcli account bandwidth-pools --help' for help.

Error: Got unexpected extra arguments (Bandwidth Pools)

Can you please fix it.

Thanks,
Ramkishor Chaladi.

@edsonarios
Copy link
Contributor Author

Hi @ramkishor-ch can you upload a screenshot with the error please, I tested and works without problem.
image
image

Thanks.

@ramkishor-ch
Copy link
Contributor

Hi @edsonarios,

Actually I have miss typed it, Now It is working fine.

I have tested the code changes.

Please find the screenshots.

Screen Shot: 1
Screenshot 2023-05-19 at 6 48 48 PM

Screen Shot: 2
Screenshot 2023-05-19 at 6 45 15 PM

Thanks,
Ramkishor Chaladi.

Copy link
Contributor

@ramkishor-ch ramkishor-ch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
I have "tested the changes".
Approving the Pull Request.
Thanks,
Ramkishor Chaladi.

@allmightyspiff allmightyspiff merged commit 2db4e3e into softlayer:master May 22, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Updated command slcli account bandwidth-pools
4 participants