Skip to content

Conversation

@kazeborja
Copy link

No description provided.

@kazeborja
Copy link
Author

Not yet ready for review, but guess will be nice to get some early indications in case something seems odd :D

@kazeborja kazeborja marked this pull request as ready for review June 8, 2023 08:18
@kazeborja
Copy link
Author

Comming back to this old PR (that I forgot to follow up), the changes have been running in WLCG for a while already so things should be more or less fine.

Few things to have into consideration as they may break running deployments:

  • Configuration file has now some extra properties (protocol and wlcg)
  • The WLCG producer and type are configurable and need to be specified in the config file
  • Having the logs in the same folder had some issues while running in docker, so I moved them to a "logs" directory

Please let me know any comments you might have on this :)

@djw8605
Copy link
Member

djw8605 commented Nov 3, 2023

Hi Borja. There's some conflicts with this pull request (it has been a while since it was created). If you can resolve the conflicts, I would be happy to merge this.

@kazeborja
Copy link
Author

Hey Derek, missed the comment, I'm adapting it to the new changes but will wait to push things here until we sort out the current situation with the backlog (as probably caused by some of my changes :D)

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