Project

General

Profile

Actions
Copy link

Bug #22400

closed

broken paging logic causes arvados-dispatch-cloud to hang at startup while looking up spot prices

Added by Tom Clegg 5 days ago. Updated 3 days ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Tom Clegg
Category:
Crunch
Target version:
Development 2025-01-08
Story points:
-
Release:
Arvados 3.0.1
Release relationship:
Auto

Description

We have

func (instanceSet *ec2InstanceSet) updateSpotPrices(instances []cloud.Instance) {
...
                if page.NextToken == nil {
                        break
                }

but the NextToken documentation is

        // The token to include in another request to get the next page of items. This                                                                                            
        // value is an empty string ( "" ) or null when there are no more items to return.                                                                                        
        NextToken *string

The web docs also mention this.

Subtasks 1 (0 open — 1 closed)

Task #22403: Review 22400-spot-price-pagingResolvedTom Clegg12/18/2024Actions

Related issues 2 (2 open — 0 closed)

Related to Arvados - Bug #22402: arvados-dispatch-cloud should log warnings when cloud API calls are slowNewTom CleggActions
Related to Arvados - Bug #22401: crunch-run --list (with no additional args) should not read from stdinNewTom CleggActions
Actions
Copy link

Also available in: Atom PDF