gophercloud · mandre · Jul 16, 2024 · Jul 16, 2024 · Jul 16, 2024 · Jul 16, 2024
diff --git a/internal/acceptance/openstack/compute/v2/services_test.go b/internal/acceptance/openstack/compute/v2/services_test.go
@@ -67,6 +67,7 @@ func TestServicesListWithOpts(t *testing.T) {

 func TestServicesUpdate(t *testing.T) {
 	clients.RequireAdmin(t)
+	t.Skip("We need to have more than one compute node for this to not conflict")

 	client, err := clients.NewComputeV2Client()
 	th.AssertNoErr(t, err)

diff --git a/internal/acceptance/openstack/compute/v2/volumeattach_test.go b/internal/acceptance/openstack/compute/v2/volumeattach_test.go
@@ -3,11 +3,13 @@
 package v2

 import (
+	"context"
 	"testing"

 	"github.com/gophercloud/gophercloud/v2/internal/acceptance/clients"
 	bs "github.com/gophercloud/gophercloud/v2/internal/acceptance/openstack/blockstorage/v2"
 	"github.com/gophercloud/gophercloud/v2/internal/acceptance/tools"
+	"github.com/gophercloud/gophercloud/v2/openstack/placement/v1/resourceproviders"
 	th "github.com/gophercloud/gophercloud/v2/testhelper"
 )

@@ -20,6 +22,34 @@ func TestVolumeAttachAttachment(t *testing.T) {
 	blockClient, err := clients.NewBlockStorageV3Client()
 	th.AssertNoErr(t, err)

+	// TEMP: Check inventory to figure out what's missing
+	placementClient, err := clients.NewPlacementV1Client()
+	th.AssertNoErr(t, err)
+	placementClient.Microversion = "1.37"
+
+	allPages, err := resourceproviders.List(placementClient, resourceproviders.ListOpts{}).AllPages(context.TODO())
+	th.AssertNoErr(t, err)
+
+	allResourceProviders, err := resourceproviders.ExtractResourceProviders(allPages)
+	th.AssertNoErr(t, err)
+
+	for _, r := range allResourceProviders {
+		inv, err := resourceproviders.GetInventories(context.TODO(), placementClient, r.UUID).Extract()
+		th.AssertNoErr(t, err)
+
+		for k, v := range inv.Inventories {
+			t.Logf("resource %s (%s) inventory: %s=%+v", r.UUID, r.Name, k, v)
+		}
+
+		use, err := resourceproviders.GetUsages(context.TODO(), placementClient, r.UUID).Extract()
+		th.AssertNoErr(t, err)
+
+		for k, v := range use.Usages {
+			t.Logf("resource %s (%s) usage: %s=%d", r.UUID, r.Name, k, v)
+		}
+	}
+	// TEMP: Check inventory to figure out what's missing
+
 	server, err := CreateServer(t, client)
 	th.AssertNoErr(t, err)
 	defer DeleteServer(t, client, server)

diff --git a/openstack/compute/v2/aggregates/requests.go b/openstack/compute/v2/aggregates/requests.go
@@ -1,5 +1,7 @@
 package aggregates

+// Martin was here
+
 import (
 	"context"
 	"strconv"

diff --git a/script/stackenv b/script/stackenv
@@ -35,8 +35,8 @@ echo export OS_DOMAIN_ID=default >> openrc

 _FLAVOR_ID=99
 _FLAVOR_ALT_ID=98
-openstack flavor create m1.acctest --id "$_FLAVOR_ID" --ram 512 --disk 10 --vcpu 1 --ephemeral 10
-openstack flavor create m1.resize --id "$_FLAVOR_ALT_ID" --ram 512 --disk 11 --vcpu 1 --ephemeral 10
+openstack flavor create m1.acctest --id "$_FLAVOR_ID" --ram 256 --disk 10 --vcpu 1 --ephemeral 10
+openstack flavor create m1.resize --id "$_FLAVOR_ALT_ID" --ram 256 --disk 11 --vcpu 1 --ephemeral 10
 openstack keypair create magnum
 _NETWORK_ID=$(openstack network show private -c id -f value)
 _SUBNET_ID=$(openstack subnet show private-subnet -c id -f value)
-Original file line number
+Diff line change
@@ -1,5 +1,7 @@
     package aggregates
+    // Martin was here
             Copy link

  
      
    
  

  
      

  
    Contributor


      

  

  
    
      

      
            stephenfin
  

      

      

      


        Jul 16, 2024


      
    

  


        
      
  
  
  
    

    There was a problem hiding this comment.


  

 
  
    

    Choose a reason for hiding this comment

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

    
      
      


  


  
    
      Pure vandalism...
    
  
  


    

        
      
  
  
    
  
  
  
    
    Sorry, something went wrong.
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.


  
  


          
      
  
    
    
            kayrus reacted with laugh emoji
      
        
            
    All reactions
  


          
              
                😄
                  1 reaction
     import (
     	"context"
     	"strconv"
-          Expand Down