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

Splitting KVMS into multiple files - Apigee X/Hybrid #201

Closed
dpalanivel opened this issue Apr 15, 2024 · 5 comments
Closed

Splitting KVMS into multiple files - Apigee X/Hybrid #201

dpalanivel opened this issue Apr 15, 2024 · 5 comments

Comments

@dpalanivel
Copy link

dpalanivel commented Apr 15, 2024

This is similar to Issue #150 where splitting kvm configs into multiple files is not being recognized by the plugin.
Only kvms.json is recognized and processed.

The structure of the folder/files:

image

kvm-deploy.log
Snippet of logs from CD pipeline is attached for further investigation.

Additional information:

  • Version: 2.7.0
  • Maven command: mvn install -P${apigeeDeployEnvironment} -Dapigee.config.options=${apigeeConfigDeployOption} -Dapigee.config.dir=./ -Dorg=${apigeeOrg} -DauthType=${authType} -Dbearer=$(gcloud auth print-access-token)

@ssvaidyanathan Could you please review and help getting the fix from 1.5.3 cascaded to 2.x release?
@kurtkanaskie @rsrikant1982 FYI please.

@ssvaidyanathan
Copy link
Collaborator

Thanks.. Will look into this

ssvaidyanathan added a commit that referenced this issue Apr 15, 2024
ssvaidyanathan added a commit that referenced this issue Apr 15, 2024
@ssvaidyanathan
Copy link
Collaborator

Just released v2.8.0 with this feature. Should be available in the Public repo in a few hours
This should work not only for KVMs, but for other entities as well (as long as you have the right prefix needed)
Try it out and let me know. Will close the issue after that

@dpalanivel
Copy link
Author

@ssvaidyanathan there is a null pointer exception when the plugin tries to read config files. Shared details via email which either Kurt or Ankur might have shared with you. Could you please take a look?

ssvaidyanathan added a commit that referenced this issue Apr 15, 2024
ssvaidyanathan added a commit that referenced this issue Apr 15, 2024
@ssvaidyanathan
Copy link
Collaborator

@dpalanivel - I just released a patch v2.8.1 with the NPE fix. Please try it out as soon as its available in the Public repo. Sorry that was missed. Was a border test case.

@dpalanivel
Copy link
Author

@ssvaidyanathan All set and 2.8.1 worked. I truly appreciate the quick help on this. Thanks again.

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

No branches or pull requests

2 participants