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

EH-1684: put herätepalvelu in internal VPC #277

Merged
merged 2 commits into from
May 30, 2024

Conversation

pkalliok
Copy link
Contributor

No description provided.

@pkalliok
Copy link
Contributor Author

AMIS-stackin deploy vaikutti lupaavalta. Paljon VPC-attribuutteja päivittyi ja AMISherateEmailHandlerin kokeiluajo sen jälkeen tuotti seuraavat lokirivit:

START RequestId: ac7b31a7-7b2e-4842-82bb-46ebc4b76ecb Version: $LATEST
{
[...]  "message": "Lambdaa handleSendAMISEmails kutsuttiin ajastetusti säännöillä  (RequestId: ac7b31a7-7b2e-4842-82bb-46ebc4b76ecb )",
[...]
}
{
[...]  "message": "Aiotaan käsitellä 1 lähetettävää viestiä.",
[...]
}
{
[...]  "message": "Käsitellään heräte: {:kyselytyyppi tutkinnon_suorittaneet, :request-id cdebe647-3c4e-46b4-8177-e28a81df31e9, :kyselylinkki https://arvovastaus-dev.csc.fi/v/H4YMK4, :voimassa-loppupvm 2024-06-04, :hankintakoulutuksen-toteuttaja , :suorituskieli fi, :sahkoposti antti.alimaki@kauppatar.fii, :toimija_oppija 1.2.246.562.10.86077555174/1.2.246.562.24.24582754588, :lahetyspvm 2024-05-06, :osaamisala , :heratepvm 2024-04-30, :herate-source sqs_viesti_ehoksista, :lahetystila ei_lahetetty, :sms-lahetyspvm 2024-05-06, :tallennuspvm 2024-05-06, :muistutukset 2, :oppilaitos 1.2.246.562.10.919116475010, :toimipiste-oid , :viestintapalvelu-id 6468708, :tyyppi_kausi tutkinnon_suorittaneet/2023-2024, :1.-muistutus-lahetetty 2024-05-11, :opiskeluoikeus-oid 1.2.246.562.15.86861814530, :alkupvm 2024-05-06, :koulutustoimija 1.2.246.562.10.86077555174, :tutkintotunnus 334103, :oppija-oid 1.2.246.562.24.24582754588, :sms-lahetystila phone-invalid, :ehoks-id 44961, :rahoituskausi 2023-2024, :puhelinnumero +358966547, :2.-muistutus-lahetetty 2024-05-16}",
[...]
}
{
[...]  "message": "Lähetetty sähköposti id 6470639 , tallennetaan tietokantaan",
[...]
}
END RequestId: ac7b31a7-7b2e-4842-82bb-46ebc4b76ecb
REPORT RequestId: ac7b31a7-7b2e-4842-82bb-46ebc4b76ecb	Duration: 1807.87 ms	Billed Duration: 1808 ms	Memory Size: 1024 MB	Max Memory Used: 292 MB	
XRAY TraceId: 1-66582c0e-2e82bbc637dc7c0456e8cc4b	SegmentId: 02eb24b11c26f459	Sampled: true	

@pkalliok pkalliok marked this pull request as ready for review May 30, 2024 08:33
@pkalliok pkalliok requested review from tuoruu and jkorri May 30, 2024 11:33
Copy link

@jkorri jkorri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hyvältä näyttää

@pkalliok
Copy link
Contributor Author

Jes, tääkin on testattu pallerossa jo (https://opetushallitus.slack.com/archives/C02CA43KWRZ/p1717071177154479?thread_ts=1716375010.817459&cid=C02CA43KWRZ) niin mergeen vaan.

Kiitos katselmoinnista <3

@pkalliok pkalliok merged commit e160a41 into master May 30, 2024
2 checks passed
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