Uploaded image for project: 'Apache YuniKorn'
  1. Apache YuniKorn
  2. YUNIKORN-886

Fix e2e tests after YUNIKORN-794

    XMLWordPrintableJSON

Details

    Description

      YUNIKORN-794 changed the REST API, which broke the e2e tests:

      /home/runner/work/incubator-yunikorn-k8shim/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt/queue_quota_mgmt_test.go:34
        Verify_Queue_Quota_Allocation [It]
        /home/runner/work/incubator-yunikorn-k8shim/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt/queue_quota_mgmt_test.go:69
      
        Test Panicked
        interface conversion: interface {} is nil, not string
        /opt/hostedtoolcache/go/1.15.15/x64/src/runtime/iface.go:261
      
        Full Stack Trace
        github.com/apache/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt_test.glob..func3.2()
        	/home/runner/work/incubator-yunikorn-k8shim/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt/queue_quota_mgmt_test.go:105 +0x29ff
        github.com/onsi/ginkgo/internal/leafnodes.(*runner).runSync(0xc000038360, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
        	/home/runner/go/pkg/mod/github.com/onsi/ginkgo@v1.16.5/internal/leafnodes/runner.go:113 +0xa3
      [...]
        github.com/apache/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt_test.TestStateAwareAppScheduling(0xc0000ee780)
        	/home/runner/work/incubator-yunikorn-k8shim/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt/queue_quota_mgmt_suite_test.go:83 +0x156
        testing.tRunner(0xc0000ee780, 0x1b75ca8)
        	/opt/hostedtoolcache/go/1.15.15/x64/src/testing/testing.go:1123 +0xef
        created by testing.(*T).Run
        	/opt/hostedtoolcache/go/1.15.15/x64/src/testing/testing.go:1168 +0x2b3
      ------------------------------
      

      Problem is that we expect strings like "maxcapacity", "usedcapacity", etc. but the JSON response was slightly modified.

      Attachments

        Issue Links

          Activity

            People

              pbacsko Peter Bacsko
              pbacsko Peter Bacsko
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: