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.

Sign up for GitHub

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

Jump to bottom

The security CR's management.connector config is wrongly handled #37

Closed
gaohoward opened this issue Oct 23, 2023 · 0 comments
Closed

The security CR's management.connector config is wrongly handled #37

gaohoward opened this issue Oct 23, 2023 · 0 comments
Assignees
@gaohoward
Labels
bug Something isn't working

Comments

@gaohoward
Copy link
Collaborator

Describe the bug
When you deploy a security CR configuring connector it doesnt work.
example:

apiVersion: broker.amq.io/v1alpha1
kind: ActiveMQArtemisSecurity
metadata:
  name: demo-jmx
spec:
  applyToCrNames: 
  - demo
  securitySettings:
    management:
      connector:
        secured: false

The deployment will fail with init container error. The reason is that the connector's attributes are not correctly handled in the security apply script.

To Reproduce
Steps to reproduce the behavior:

  1. Get X version of the source
  2. '...'

Platform (please complete the following information):

  • OS: [e.g. Fedora]
  • Version [e.g. 32]
  • Container build tool [e.g. Podman]

Expected behavior
Optional, use if not obvious. A clear and concise description of what you expected to happen.

Screenshots
Optional, add screenshots to help explain your problem.

Additional context
Optional. Add any other context about the problem here.
@gaohoward gaohoward added the bug Something isn't working label Oct 23, 2023
gaohoward pushed a commit to gaohoward/activemq-artemis-broker-init-image that referenced this issue Oct 23, 2023
@howardgao
[artemiscloud#37] The security CR's management.connector config is wr…
372c328 
…ongly handled
gaohoward pushed a commit to gaohoward/activemq-artemis-broker-init-image that referenced this issue Oct 24, 2023
@howardgao
[artemiscloud#37] The security CR's management.connector config is wr…
1fc3794 
…ongly handled
brusdev pushed a commit that referenced this issue Oct 24, 2023
@howardgao @brusdev
[#37] The security CR's management.connector config is wrongly handled
7b045d5
rh-messaging-ci pushed a commit to rh-messaging/activemq-artemis-broker-init-image that referenced this issue Oct 26, 2023
@howardgao
[artemiscloud#37] The security CR's management.connector config is wr…
625af04 
…ongly handled

(cherry picked from commit 7b045d5)

downstream: ENTMQBR-8538
brusdev added a commit to rh-messaging/activemq-artemis-broker-init-image that referenced this issue Nov 6, 2023
@brusdev
Revert "[artemiscloud#37] The security CR's management.connector conf…
638e286 
…ig is wrongly handled"

This reverts commit 625af04.
brusdev pushed a commit to rh-messaging/activemq-artemis-broker-init-image that referenced this issue Nov 15, 2023
@howardgao @brusdev
[artemiscloud#37] The security CR's management.connector config is wr…
3a1e516 
…ongly handled

(cherry picked from commit 7b045d5)

downstream: ENTMQBR-8538
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gaohoward gaohoward

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gaohoward @RoddieKieley