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

Missing JSON output for service-version clone command #1353

Open
Tasteful opened this issue Dec 3, 2024 · 1 comment
Open

Missing JSON output for service-version clone command #1353

Tasteful opened this issue Dec 3, 2024 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@Tasteful
Copy link

Tasteful commented Dec 3, 2024

The service-version clone command missing the JSON output and to use it in scripts the output need to be parsed.

Using service-version list command directly after cloning a new version is not always putting the version in the end that will make it hard to find or if the list returns stale data.

This is from our log files where we using the clone follow with the list command.

2024-12-03 03:16:30.1042 Console.Internal | SUCCESS: Cloned service XXXX version 1460 to version 1461
2024-12-03 03:16:31.1049 Console.Internal | Found fastly version 1460
2024-12-03 03:16:31.1051 Console.Internal | Add domain DOMAIN to service
2024-12-03 03:16:31.9595 Console.Internal |
2024-12-03 03:16:31.9598 Console.Internal | ERROR: service version 1460 is locked.
2024-12-03 03:16:31.9606 Console.Internal |
2024-12-03 03:16:31.9607 Console.Internal | Repeat the command with the --autoclone flag to allow the version to be cloned

I think the relative code for this is this part:

text.Success(out, "Cloned service %s version %d to version %d", fastly.ToValue(ver.ServiceID), c.Input.ServiceVersion, fastly.ToValue(ver.Number))

@kpfleming kpfleming added bug Something isn't working good first issue Good for newcomers labels Dec 4, 2024
@kpfleming
Copy link
Contributor

Absolutely correct, that command does not pay any attention to the request for JSON output, and it should.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants