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

Prepis konfigurace orchestratoru po dokonceni jobu #40

Closed
ErikZigo opened this Issue Aug 22, 2017 · 5 comments

Comments

Projects
None yet
1 participant
@ErikZigo
Contributor

ErikZigo commented Aug 22, 2017

Vychazi z komunikace keboola/kbc-ui#1095

Jak se to da nasimulovat:

  1. vytvorim orchestraci a nastavim notifikacni email
  2. pustim orchestraci
  3. behem procesovani jobu zmenim v konfiguraci orchestrace notifikacni email
  4. po dokonceni jobu bude orchestrace mit v konfiguraci notifikacni email z bodu 1

@ErikZigo ErikZigo added the bug label Aug 22, 2017

@ErikZigo ErikZigo self-assigned this Aug 22, 2017

@ErikZigo

This comment has been minimized.

Show comment
Hide comment
@ErikZigo
Contributor

ErikZigo commented Aug 22, 2017

@ErikZigo

This comment has been minimized.

Show comment
Hide comment
@ErikZigo

ErikZigo Aug 22, 2017

Contributor

Je to zpusobene tim, ze orchestrator po dokonceni jobu uklada k orchestraci jeho metadata. Pritom zrejme prepise i to nastaveni.
Nastesti to vypada, ze se to tyka opravdu jen tech notifikacnich emailu.

Contributor

ErikZigo commented Aug 22, 2017

Je to zpusobene tim, ze orchestrator po dokonceni jobu uklada k orchestraci jeho metadata. Pritom zrejme prepise i to nastaveni.
Nastesti to vypada, ze se to tyka opravdu jen tech notifikacnich emailu.

@ErikZigo

This comment has been minimized.

Show comment
Hide comment
@ErikZigo

ErikZigo Aug 23, 2017

Contributor

Tak jeste stejny problem by nastal, kdyby nekdo v prubehu behu orchestrace zmenil jeji token

Contributor

ErikZigo commented Aug 23, 2017

Tak jeste stejny problem by nastal, kdyby nekdo v prubehu behu orchestrace zmenil jeji token

@ErikZigo

This comment has been minimized.

Show comment
Hide comment
@ErikZigo

ErikZigo Aug 23, 2017

Contributor

Problem je v listeneru https://github.com/keboola/orchestrator-bundle/blob/master/Listener/DoctrineEntityListener.php ktery po nacteni zaznamu z DB dekoduje nektery fieldy.
A pred ulozenim je zase enkoduje.

ORM si pak mysli, ze doslo k jejich zmene a ulozi je taky do DB

Contributor

ErikZigo commented Aug 23, 2017

Problem je v listeneru https://github.com/keboola/orchestrator-bundle/blob/master/Listener/DoctrineEntityListener.php ktery po nacteni zaznamu z DB dekoduje nektery fieldy.
A pred ulozenim je zase enkoduje.

ORM si pak mysli, ze doslo k jejich zmene a ulozi je taky do DB

@ErikZigo

This comment has been minimized.

Show comment
Hide comment
@ErikZigo

ErikZigo Aug 23, 2017

Contributor

Asi upustim od dekodovani/decryptovani v tom EntityListeneru, kde to je ale krasne na jednom miste. A udelam to na prasaka na vice mistech v kodu.

Contributor

ErikZigo commented Aug 23, 2017

Asi upustim od dekodovani/decryptovani v tom EntityListeneru, kde to je ale krasne na jednom miste. A udelam to na prasaka na vice mistech v kodu.

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