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: mxpc throws an error because parameter clustertag is unknown #42

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

henobi
Copy link

@henobi henobi commented Feb 7, 2024

Hi,

I could not connect my cluster to the Mendix for Private Cloud Platform when deploying this solution.
I did some research in the logs and found the root cause in the parameter clusterTag because it does not exist (anymore?) in mxpc-cli. After I removed the paramter I was able to connect to the Mendix platform as expected.

Therefore my suggestion would be to remote this parameter.

Best,
Hendrik

@henobi henobi requested a review from a team as a code owner February 7, 2024 08:24
@walkline
Copy link
Contributor

walkline commented Feb 7, 2024

Hi @henobi. The clusterTag parameter was added to mxpc-cli starting from version 2.11.0 and is still present in the latest version. This parameter is hidden.
Is there a chance that you tried to use a version of mxpc-cli older than 2.11.0?
Also, is it possible to share the error log message?

@henobi
Copy link
Author

henobi commented Feb 12, 2024

Hello @walkline,
thank you for answer! I tried to reproduce the issue with version 2.14.0 and can confirm that it is working.

Initially, I used the code from the example folder with the default version in the code which is 2.10.0: https://github.com/aws-ia/terraform-aws-mendix-private-cloud/blob/main/examples/basic/variables.tf#L29

So that explains why it was not working.

What do you think about setting the version to 2.14.0 here?

Best regards,

Hendrik

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.

2 participants