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

Remove Container Service, Batch Launch, and DAX plugins from default list #95

Closed
drmatthews opened this issue Apr 17, 2024 · 1 comment · Fixed by #163
Closed

Remove Container Service, Batch Launch, and DAX plugins from default list #95

drmatthews opened this issue Apr 17, 2024 · 1 comment · Fixed by #163
Assignees
@HChughtai
Labels
work

Comments

@drmatthews
Copy link
Contributor

Short Description

Remove DAX plugin from xnat_plugin_urls in XNAT role defaults

Note that the plugin will need to removed from the servers and tomcat will need restarting for the change to take effect.

Acceptance Criteria

The DAX plugin is removed.

Dependencies

No response

Details

No response
@drmatthews drmatthews mentioned this issue Apr 17, 2024
Closed
@HChughtai HChughtai added the work label Apr 21, 2024
@drmatthews drmatthews self-assigned this Oct 15, 2024
@drmatthews drmatthews assigned HChughtai and unassigned drmatthews Jan 28, 2025
@p-j-smith p-j-smith mentioned this issue Jan 29, 2025
Merged
@p-j-smith
Copy link
Contributor

removing the DAX plugin causes various warnings in the logs. e.g. in cache.log:

2025-02-03 11:35:47,174 [taskScheduler-3] WARN org.nrg.xnat.services.cache.extractors.ActionsExtractor - Element 'proc:genProcData' not found. This may be a data type that was installed previously but can't be located now.

and in security.log:

2025-02-03 11:35:34,058 [main] ERROR org.nrg.xdat.security.ElementSecurity - An error occurred trying to get the field for XML path xsync:xsyncInfoData/project

After discussing with @HChughtai offline, we thought it would be better to remove DAX from the list of default plugins, and keep it as a server-specific plugin where needed.

We should also do this for the Container Service and Batch Launch plugins.

@p-j-smith p-j-smith changed the title Remove DAX plugin Remove Container Service, Batch Launch, and DAX plugins from default list Feb 3, 2025
p-j-smith added a commit that referenced this issue Feb 4, 2025
@p-j-smith
Remove DAX plugin (#163)
d5ab1fc 
Fixes #95 

- remove DAX, Container Service, and Batch Launch plugins from list of
default XNAT plugin
- add these plugins as server-specific plugins in molecule tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HChughtai HChughtai

Labels
work
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove DAX plugin UCL-MIRSG/ansible-collection-infra
3 participants
@drmatthews @HChughtai @p-j-smith