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

[INJIMOB-1599]: Add spring cloud composite properties #1545

Merged
merged 0 commits into from
Aug 20, 2024

Conversation

srikanth716
Copy link

@srikanth716 srikanth716 commented Jul 19, 2024

According to Spring Boot's configuration conventions, spring.profiles.active should not be defined in profile-specific property files like mimoto-default.properties
The spring.profiles.active property is defined in the mosip/inji-config mimoto-default.properties file.
Removing it from the file to get the congfigs from mosip/inji-config

SPRING_CONFIG_GIT_URIS used in application.yml file is env variable.
Value of this variable is json as stringify
"[{\"uri\": \"https://github.com/mosip/inji-config\", \"branch\": \"develop\"}, {\"uri\": \"https://github.com/mosip/mosip-config\", \"branch\": \"develop\"}]"

@srikanth716 srikanth716 changed the title Develop Add spring cloud composite properties Jul 22, 2024
@srikanth716 srikanth716 changed the title Add spring cloud composite properties [INJIMOB-1599]: Add spring cloud composite properties Jul 25, 2024
import io.micrometer.observation.ObservationRegistry;

@Configuration
public class compositeEnvironmentConfig {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

class name should be CompositeEnvironmentConfig

@Bean
public CompositeEnvironmentRepository compositeEnvironmentRepository() {
List<EnvironmentRepository> repositories = new ArrayList<>();
JSONArray GitUrisArr = new JSONArray(gitUris);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename variable toconfigGitUris

server:
port: 51000
servlet:
context-path: ${server.servlet.path}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

swap line# 13 & 14

@Prafulrakhade Prafulrakhade reopened this Jul 26, 2024
@ckm007 ckm007 merged commit 9544c53 into mosip:develop Aug 20, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants