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

fix(dynamicAccounts): Included java files in source set #5090

Merged
merged 4 commits into from
Nov 11, 2020

Conversation

german-muzquiz
Copy link
Contributor

@german-muzquiz german-muzquiz commented Nov 10, 2020

The module clouddriver-configserver is misconfigured. It doesn't detect the source files in src/main/java. As a result, the jar file clouddriver-configserver.jar produced in builds lacks any java classes.

This can be easily verified if whithout this fix, you introduce a compilation error in CloudConfigRefreshConfig and run ./gradlew build -x test, as it builds successfully.

Closes spinnaker/spinnaker#6128

@german-muzquiz german-muzquiz added the backport-candidate Add to PRs to designate release branch patch candidates. label Nov 10, 2020
@german-muzquiz
Copy link
Contributor Author

cc @dogonthehorizon backport candidate for 1.23

@german-muzquiz german-muzquiz added the ready to merge Approved and ready for a merge label Nov 11, 2020
@mergify mergify bot added the auto merged Merged automatically by a bot label Nov 11, 2020
@mergify mergify bot merged commit c0c90db into spinnaker:master Nov 11, 2020
@german-muzquiz german-muzquiz deleted the fix/configserver branch November 11, 2020 15:14
@dogonthehorizon
Copy link
Member

@Mergifyio backport release-1.23.x

@dogonthehorizon dogonthehorizon removed the backport-candidate Add to PRs to designate release branch patch candidates. label Nov 16, 2020
mergify bot pushed a commit that referenced this pull request Nov 16, 2020
* fix(dynamicAccounts): Compilation error for testing

Proves that the build of clouddriver-configserver doesn't include java source files

* fix(dynamicAccounts): Included java files in source set

* fix(dynamicAccounts): Removed compilation error

(cherry picked from commit c0c90db)
@mergify
Copy link
Contributor

mergify bot commented Nov 16, 2020

Command backport release-1.23.x: success

Backports have been created

mergify bot added a commit that referenced this pull request Nov 16, 2020
* fix(dynamicAccounts): Compilation error for testing

Proves that the build of clouddriver-configserver doesn't include java source files

* fix(dynamicAccounts): Included java files in source set

* fix(dynamicAccounts): Removed compilation error

(cherry picked from commit c0c90db)

Co-authored-by: German Muzquiz <35276119+german-muzquiz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Approved and ready for a merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clouddriver not refreshing Spring Cloud Config properties
3 participants