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

get mononoke green in github CI #974

Closed
wants to merge 1 commit into from
Closed

Conversation

ahornby
Copy link
Contributor

@ahornby ahornby commented Oct 23, 2024

Summary:

  • mononoke unit tests are failing, fix them
  • test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list

changes done to make this easier:

  • python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
  • add a bit more info to the "unknown python exception" message from hg
  • reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

Test Plan:

Local run of unittest

./build/fbcode_builder/getdeps.py --allow-system-packages build --src-dir=. mononoke
./build/fbcode_builder/getdeps.py --allow-system-packages test--src-dir=. mononoke

Before fails:

failures:
    store::test::test_find_abandoned_requests
    store::test::test_get_stats
    store::test::test_mark_as_new

After, passes.

Local run of integration test

# first build sapling cli & install system deps
./build/fbcode_builder/getdeps.py --allow-system-packages build --src-dir=. sapling
./build/fbcode_builder/getdeps.py install-system-deps --recursive mononoke_integration
# then test mononoke
./build/fbcode_builder/getdeps.py --allow-system-packages build --no-deps --src-dir=. mononoke_integration
./build/fbcode_builder/getdeps.py --allow-system-packages test --src-dir=. mononoke_integration --num-jobs=4

Before, fails

AssertionError: The test groups contain not existing tests: ['cross_repo/test-cross-repo-mononoke-git-sot-switch.t'] in /home/runner/work/sapling/sapling/eden/mononoke/tests/integration

After, passes

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ahornby has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ahornby has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ahornby has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ahornby has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening


Test Plan:
**Local run of unittest**
```
./build/fbcode_builder/getdeps.py --allow-system-packages build --src-dir=. mononoke
./build/fbcode_builder/getdeps.py --allow-system-packages test--src-dir=. mononoke
```

**Local run of integration test**
```
# first build sapling cli & install system deps
./build/fbcode_builder/getdeps.py --allow-system-packages build --src-dir=. sapling
./build/fbcode_builder/getdeps.py install-system-deps --recursive mononoke_integration
# then test mononoke
./build/fbcode_builder/getdeps.py --allow-system-packages build --no-deps --src-dir=. mononoke_integration
./build/fbcode_builder/getdeps.py --allow-system-packages test --src-dir=. mononoke_integration --num-jobs=4
```

Before, fails
```
AssertionError: The test groups contain not existing tests: ['cross_repo/test-cross-repo-mononoke-git-sot-switch.t'] in /home/runner/work/sapling/sapling/eden/mononoke/tests/integration
```

After, lets see the CI

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby
@facebook-github-bot
Copy link
Contributor

@ahornby has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D64827811

facebook-github-bot pushed a commit to facebook/folly that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebookexperimental/rust-shed that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebook/fbthrift that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebook/mvfst that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebookexperimental/edencommon that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebook/openr that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebookincubator/hsthrift that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebookincubator/katran that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebook/wangle that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebook/fboss that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebookincubator/fizz that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebook/watchman that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebook/proxygen that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebookexperimental/moxygen that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
@facebook-github-bot
Copy link
Contributor

@ahornby merged this pull request in 630ab16.

facebook-github-bot pushed a commit to facebook/fb303 that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
facebook-github-bot pushed a commit to facebook/hhvm that referenced this pull request Oct 24, 2024
Summary:
X-link: facebookincubator/zstrong#1018

* test-cross-repo-mononoke-git-sot.t deleted, remove from exclusion list
* exclude flaky integration tests

changes done to make this easier:
 * python-click wasn't found in sapling dir, added package mappings for rpm and deb so that we can pick up the installed version
 * add a bit more info to the "unknown python exception" message from hg
 * reduce paths set by getdeps, no point listing non-existing dir, makes it easier to see what is happening

X-link: facebook/sapling#974

Reviewed By: quark-zju

Differential Revision: D64827811

Pulled By: ahornby

fbshipit-source-id: 82b4fa224d8ce957ef85e5ecf1e220f71c93e6c4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants