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

Spidercrab: i06t02, i06t03, i06t04, i06t07 #118

Merged
merged 17 commits into from
Apr 24, 2014
Merged

Spidercrab: i06t02, i06t03, i06t04, i06t07 #118

merged 17 commits into from
Apr 24, 2014

Conversation

ktalik
Copy link
Member

@ktalik ktalik commented Apr 14, 2014

  • Description:
    • Integration with Ocean Don Corleone
      • 8b86f02 Merge integration fixes — Konrad Talik
      • 0800daa Spidercrab _check_and_pull_config() from Corleone
    • Integration with tagging module (database labels, ...?)
      • 54118fb NotYetTagged label for Content nodes
  • Time: 3 h
    • Some minor delays related to don_utils.py

Merging this request would close issue #105.

@kudkudak
Copy link
Member

Note: pass node_id to get_configuration and test if it works (use don on ocean-don.no-ip.biz)

@ktalik
Copy link
Member Author

ktalik commented Apr 15, 2014

node_id is now passed to get_running_service and is working.

kudkudak and others added 4 commits April 16, 2014 08:54
Added checking for empty model nodes.
Something can be wrong we neo4j. We
will return to this problem if it occurs
again
Now master and slave Spidercrabs are looping. If there is need to
terminate them after work is done - use `"terminate_on_end": 1` parameter
in config.json.
@ktalik
Copy link
Member Author

ktalik commented Apr 16, 2014

Looping now works as described in 1bd2c1b.

@ktalik ktalik changed the title i06t02: Spidercrab system integration Spidercrab:i06t02, i06t03, i06t04, i06t07 Apr 21, 2014
@ktalik ktalik changed the title Spidercrab:i06t02, i06t03, i06t04, i06t07 Spidercrab: i06t02, i06t03, i06t04, i06t07 Apr 21, 2014
@ktalik
Copy link
Member Author

ktalik commented Apr 21, 2014

Add last fixes and improvements.

This pull request is ready to test and merge. It closes following tasks:

Please comment if there is anything missing.

Improved Spidercrab master and slave functionalities:
* Fix bug of loop in recursion of finding another content source when a
particular source is no longer available on given url
* Improved _news_is_supported() method
* Fixed Corleone config fetching for command line options inside
spidecrab init scripts
* Add logging stats into json file inside logs/ directory
* Add -t option (for stats exporting) to spidercrab init scripts

This commit closes tasks #103 and #106.
@ktalik
Copy link
Member Author

ktalik commented Apr 23, 2014

Spidercrab is passing the test but I am thinking about more restrictive version of this test now.

kudkudak added a commit that referenced this pull request Apr 24, 2014
Spidercrab: i06t02, i06t03, i06t04, i06t07
@kudkudak kudkudak merged commit dd5b1ab into dev Apr 24, 2014
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