Skip to content

Conversation

@vladak
Copy link
Member

@vladak vladak commented Mar 22, 2019

mirror.py was always good candidate for refactoring. I used the opportunity to add the capability of mirroring multiple projects in one run. The intention is to use it in the Docker script (OpenGrok/docker#27).

This change also adds --all switch mirrors all indexed projects. This is mutually exclusive with project arguments of the script.

@vladak
Copy link
Member Author

vladak commented Mar 22, 2019

Another hopefully positive side effect is that configuration is now checked before the run.

@vladak vladak closed this Mar 22, 2019
@vladak vladak reopened this Mar 22, 2019
Vladimir Kotal added 2 commits March 22, 2019 16:17
@coveralls
Copy link

coveralls commented Mar 22, 2019

Pull Request Test Coverage Report for Build 4183

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 73.6%

Totals Coverage Status
Change from base Build 4137: 0.0%
Covered Lines: 33605
Relevant Lines: 45659

💛 - Coveralls
@tulinkry
Copy link
Contributor

tulinkry commented Apr 5, 2019

How exactly you mirror multiple projects at one? I still see a sequential loop in there.

@vladak
Copy link
Member Author

vladak commented Apr 5, 2019

the main loop is in main():

 for project_name in projects: if args.batch: get_batch_logger(logdir, project_name, args.loglevel, args.backupcount, get_class_basename()) project_result = mirror_project(config, project_name, args.incoming, args.uri, source_root) if project_result == 1: ret = 1

it should be possible to take this one step further and make it parallelized so that it can be used outside of opengrok-sync.

@vladak vladak merged commit f3fa009 into oracle:master Apr 8, 2019
@tulinkry
Copy link
Contributor

tulinkry commented Apr 8, 2019

There are 11 unresolved comments and you merged it? Any plan to resolve them?

@vladak
Copy link
Member Author

vladak commented Apr 8, 2019

I believe all of them were resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants