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

Re-enable Postgres tests and remove MariaDB #2370

Merged
merged 5 commits into from
Oct 4, 2023

Conversation

pavannd1
Copy link
Contributor

@pavannd1 pavannd1 commented Oct 2, 2023

This reverts commit 06be966.

Additionally, this PR removes MariaDB tests. We will add it back once we have enough resources to run all the tests.

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test

@pavannd1
Copy link
Contributor Author

pavannd1 commented Oct 2, 2023

Looks like the mongo-db tests are failing now.

@akankshakumari393
Copy link
Contributor

if we disable one of the app, the integration test works, I tried disabling MySQL, the test passed commit . https://github.com/kanisterio/kanister/actions/runs/6390938725/job/17345776852, maybe we should upgrade the test infrastructure.

@akankshakumari393
Copy link
Contributor

I have reverted the commit, that I made in this PR to test the CI.

build/integration-test.sh Outdated Show resolved Hide resolved
@pavannd1 pavannd1 changed the title Re-enable Postgres tests Re-enable Postgres tests and remove MariaDB Oct 3, 2023
@pavannd1 pavannd1 added the kueue label Oct 3, 2023
@mergify mergify bot merged commit 3eeb8d8 into master Oct 4, 2023
@mergify mergify bot deleted the revert-2367-temp-disable-pg-16 branch October 4, 2023 03:17
leuyentran pushed a commit that referenced this pull request Oct 18, 2023
* Revert "Temporarily disable Postgres tests (#2367)"

This reverts commit 06be966.

* Disable mysql app from integration test

Signed-off-by: Akanksha Kumari <[email protected]>

* Revert "Disable mysql app from integration test"

This reverts commit 7eb9ec4.

* Remove Maria DB

---------

Signed-off-by: Akanksha Kumari <[email protected]>
Co-authored-by: Akanksha Kumari <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants