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

Fixes #21509: Adapt datasource to new test config-repo #489

Conversation

fanf
Copy link
Member

@fanf fanf commented Aug 1, 2022

@fanf fanf requested a review from amousset August 1, 2022 14:27
@fanf fanf changed the base branch from branches/rudder/7.1 to 7.1-next August 1, 2022 14:29
@fanf fanf force-pushed the arch_21509/adapt_datasource_to_new_test_config_repo branch from 1e6075e to 4e51682 Compare August 1, 2022 14:58
@fanf fanf force-pushed the arch_21509/adapt_datasource_to_new_test_config_repo branch from 4e51682 to 54a4faf Compare August 1, 2022 20:05
@Normation-Quality-Assistant
Copy link
Contributor

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder-plugins/pull/489
-- Your faithful QA
Kant merge: "Morality is not the doctrine of how we may make ourselves happy, but how we may make ourselves worthy of happiness."
(https://ci.normation.com/jenkins/job/merge-accepted-pr/57894/console)

@fanf
Copy link
Member Author

fanf commented Aug 2, 2022

OK, merging this PR

5 similar comments
@fanf
Copy link
Member Author

fanf commented Aug 2, 2022

OK, merging this PR

@fanf
Copy link
Member Author

fanf commented Aug 2, 2022

OK, merging this PR

@fanf
Copy link
Member Author

fanf commented Aug 2, 2022

OK, merging this PR

@fanf
Copy link
Member Author

fanf commented Aug 2, 2022

OK, merging this PR

@fanf
Copy link
Member Author

fanf commented Aug 2, 2022

OK, merging this PR

@fanf fanf merged commit bc7d65b into Normation:7.1-next Aug 2, 2022
This pull request was closed.
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.

3 participants