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

If SDK config file has inline comments all plug-ins will fail (Functions, Data Science, Resource Manager) #28

Open
jeliker opened this issue May 29, 2024 · 4 comments

Comments

@jeliker
Copy link

jeliker commented May 29, 2024

v1.0.5

I had ~/.oci/config that looked like this that caused the error

[mytenancy]
user=ocid1.user.oc1..abcdefghijklmnop
fingerprint=00:DE:AD:BE:EF:00
tenancy=ocid1.tenancy.oc1..abcdefghijklmnop
region=us-ashburn-1
key_file=~/.oci/mykey.pem

[mytenancy-user2] #same as mytenancy but user2 OCID
user=ocid1.user.oc1..zyxwvutsrqponmlk
fingerprint=00:DE:AD:BE:EF:00
tenancy=ocid1.tenancy.oc1..abcdefghijklmnop
region=us-ashburn-1
key_file=~/.oci/mykey.pem

When I changed above inline comment to this all was well:

[mytenancy]
user=ocid1.user.oc1..abcdefghijklmnop
fingerprint=00:DE:AD:BE:EF:00
tenancy=ocid1.tenancy.oc1..abcdefghijklmnop
region=us-ashburn-1
key_file=~/.oci/mykey.pem

#same as mytenancy but user2 OCID
[mytenancy-user2]
user=ocid1.user.oc1..zyxwvutsrqponmlk
fingerprint=00:DE:AD:BE:EF:00
tenancy=ocid1.tenancy.oc1..abcdefghijklmnop
region=us-ashburn-1
key_file=~/.oci/mykey.pem

Here is the error. I could not find the means to get any sort of output for debugging. Glad to know if there is a way (for future reference). As I say, once I moved the comment line above to its own line instead of inline with the profile name it worked.

image

@jeliker
Copy link
Author

jeliker commented May 31, 2024

@Admin661 if you mean has the plug-in been changed to no longer show the error I reported then I speculate "no" it is not solved.

I am working around the issue by modifying my SDK config file like I described. I hope that the config file parser/loader is changed to not cause such mysterious errors based on comments or other simple formatting issues. Took a while to track down the errors problem!

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
@jeliker and others