Skip to content
This repository has been archived by the owner on Dec 11, 2022. It is now read-only.

Separate graph imports to standalone workers #198

Merged
merged 1 commit into from
Jun 23, 2017

Conversation

fridex
Copy link
Contributor

@fridex fridex commented Jun 19, 2017

No description provided.

@fridex
Copy link
Contributor Author

fridex commented Jun 19, 2017

@fridex
Copy link
Contributor Author

fridex commented Jun 19, 2017

Will bump worker commit hash once required changes get merged in worker repo.

@jpopelka
Copy link

@fridex there are some conflicts

@fridex fridex force-pushed the separate-graph-sync branch from 31f4246 to ac5329e Compare June 20, 2017 07:45
@fridex
Copy link
Contributor Author

fridex commented Jun 20, 2017

@jpopelka rebased.

@fridex fridex force-pushed the separate-graph-sync branch from ac5329e to c65e6bd Compare June 22, 2017 12:32
@fridex
Copy link
Contributor Author

fridex commented Jun 22, 2017

Fixes:

  • CVE fixes by Jirka
  • Graph sync by Mitesh
  • separate graph importer worker

CC @msrb merge? :)

@msrb
Copy link
Collaborator

msrb commented Jun 22, 2017

@vpavlin will the tooling handle this YAML?

@fridex
Copy link
Contributor Author

fridex commented Jun 22, 2017

@vpavlin will the tooling handle this YAML?

saasherder is using anymarkup which uses PyYAML - this is valid YAML, but worth to double-check.

@msrb
Copy link
Collaborator

msrb commented Jun 22, 2017

The image referenced by the hash failed to build in CI, so do not merge please.

@fridex
Copy link
Contributor Author

fridex commented Jun 23, 2017

The image referenced by the hash failed to build in CI, so do not merge please.

Good point, haven't check it. I think we should automate these checks - opened #211

@vpavlin
Copy link
Contributor

vpavlin commented Jun 23, 2017

Download of the template succeeded

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants