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

Failing acceptance tests #1456

Closed
nikimanoledaki opened this issue Feb 16, 2022 · 1 comment
Closed

Failing acceptance tests #1456

nikimanoledaki opened this issue Feb 16, 2022 · 1 comment
Labels
area/testing bug Something isn't working

Comments

@nikimanoledaki
Copy link

nikimanoledaki commented Feb 16, 2022

Pended the acceptance test SmokeTestLong - Verify that gitops can deploy an app with long name, which was failing here with:

• Failure [302.259 seconds]
Weave GitOps Add App Tests
/home/runner/work/***/***/test/acceptance/test/add_tests.go:29
  @skipOnNightly SmokeTestLong - Verify that gitops can deploy a single workload to multiple clusters with app manifests in config repo (Bug #810) [It]
  /home/runner/work/***/***/test/acceptance/test/add_tests.go:1404

  Unexpected error:
      <*errors.errorString | 0xc000139ab0>: {
          s: "Failed to delete app test-app-dvnup739",
      }
      Failed to delete app test-app-dvnup739
  occurred

  /home/runner/work/***/***/test/acceptance/test/add_tests.go:1522

Its failure was blocking one of the checks that are required for branch merging, which was in turn blocking #1445 from being merged.

Seems related to #1449 and other v2 work.

Edit : [Gini]
Another test seems to be failing when I tried to fix the broken main #1458
https://github.com/weaveworks/weave-gitops/runs/5221157992?check_suite_focus=true

• Failure [155.910 seconds]
Weave GitOps Add App Tests
/home/runner/work/***/***/test/acceptance/test/add_tests.go:29
  @gitlabTests Test3 - Verify that gitops can deploy and delete a gitlab app in a subgroup [It]
  /home/runner/work/***/***/test/acceptance/test/add_tests.go:1169

  Unexpected error:
      <*errors.errorString | 0xc00043f260>: {
          s: "Failed to delete app test-app-idejv7a5",
      }
      Failed to delete app test-app-idejv7a5
  occurred

  /home/runner/work/***/***/test/acceptance/test/add_tests.go:1236
------------------------------

This ticket needs some investigation on why these tests fail. There's a suggestion about increasing wait time by @rokshana-b #1458 (comment) , should be considered as part of this investigation. These all seem to be failing in the same file/suite.

@nikimanoledaki nikimanoledaki added the bug Something isn't working label Feb 16, 2022
@nikimanoledaki nikimanoledaki changed the title Pended failing acceptance test SmokeTestLong - Verify that gitops can deploy an app with long name Failing acceptance test SmokeTestLong - Verify that gitops can deploy an app with long name Feb 16, 2022
@nikimanoledaki nikimanoledaki added the severity/medium low < medium < high < critical label Feb 16, 2022
@Himangini Himangini removed the severity/medium low < medium < high < critical label Feb 16, 2022
@Himangini Himangini changed the title Failing acceptance test SmokeTestLong - Verify that gitops can deploy an app with long name Failing acceptance tests Feb 16, 2022
@rokshana-b
Copy link
Contributor

Closing this issue since these tests no longer exist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/testing bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants