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.

Sign up for GitHub

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

Jump to bottom

Scheduled polling v1 #78

Merged
merged 60 commits into from
Jul 16, 2024
Merged

Scheduled polling v1 #78

merged 60 commits into from
Jul 16, 2024

Conversation

somesylvie
Copy link
Contributor

@somesylvie somesylvie commented Jul 15, 2024 •
edited by saquino0827
Loading

Start on scheduled polling

This PR creates an Azure Function that, based on a schedule configured by a CRON expression, enqueues a message to a new queue. Later PRs will read this message and use it to kick off polling a specific external site

Issue

CDCgov/trusted-intermediary#1080

Checklist

  • I have added tests to cover my changes
  • I have added logging where useful (with appropriate log level)
  • I have added documentation and comments where required
  • I have updated the documentation accordingly

Note: You may remove items that are not applicable

somesylvie and others added 30 commits July 8, 2024 12:49
@somesylvie @saquino0827
try adding webjob resource
77c83fd 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie @saquino0827 @pluckyswan
add webjob script?
2f7b800 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @saquino0827 @pluckyswan
update pathing
5d117fe 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @saquino0827 @pluckyswan
switch to source file not dir
63a093a 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @saquino0827 @pluckyswan
try some other command
d7efc52 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @saquino0827 @pluckyswan
try login first
6f8a67e 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @saquino0827 @pluckyswan
take name off command
f45443b 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @saquino0827 @pluckyswan
make the name an empty string
566cbd8 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @saquino0827 @pluckyswan
take out webjob attempt
8f80514 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @saquino0827
Start making an Azure Function for timer control
f72c5fa 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie @saquino0827
update some TODOs
cba6b43 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie
Add a bunch more notes, start TF for new queue
ffd27d8
@somesylvie
more TODO notes
e503a12
@jherrflexion
Azure func terraform WIP
05a067d
@jherrflexion @saquino0827
@somesylvie @pluckyswan
Re-configure polling function app function
c9b76c4 
Co-authored-by: saquino0827 <saquino@flexion.us>
Co-authored-by: Sylvie <sschuresko@flexion.us>
Co-authored-by: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@jherrflexion @saquino0827 @somesylvie
Attempting to fix Terraform errors
fc540e5 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: Sylvie <38440028+somesylvie@users.noreply.github.com>
@jherrflexion @saquino0827 @somesylvie
Attempting fix for Terraform errors
59445d0 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: Sylvie <38440028+somesylvie@users.noreply.github.com>
@jherrflexion @saquino0827 @somesylvie
WIP functions Terraform
8ff0219 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: Sylvie <38440028+somesylvie@users.noreply.github.com>
@saquino0827 @pluckyswan @jherrflexion
Add template for function app function deployment
010ff29 
Co-authored-by: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
Co-authored-by: James Herr <jherr@flexion.us>
@pluckyswan @saquino0827
@jherrflexion @somesylvie
created github workflow for node js function app
d76a64b 
Co-authored-by: saquino0827 <saquino@flexion.us>
Co-authored-by: James Herr <jherr@flexion.us>
Co-authored-by: Sylvie <sschuresko@flexion.us>
@pluckyswan @saquino0827
@somesylvie @jherrflexion
updated function workflow call
d65a8eb 
Co-authored-by: saquino0827 <saquino@flexion.us>
Co-authored-by: Sylvie <sschuresko@flexion.us>
Co-authored-by: James Herr <jherr@flexion.us>
@pluckyswan @saquino0827 @somesylvie
updated call to function workflow
a211db8 
Co-authored-by: saquino0827 <saquino@flexion.us>
Co-authored-by: Sylvie <sschuresko@flexion.us>
@saquino0827 @jherrflexion
@pluckyswan @somesylvie
Update github functions workflow with internal function publish profi…
874f6a9 
…le secret

Co-authored-by: James Herr <jherr@flexion.us>
Co-authored-by: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
Co-authored-by: Sylvie <sschuresko@flexion.us>
@saquino0827
Merge branch 'main' into scheduling
2dd8760
@somesylvie @pluckyswan
re-remove the unneeded TF
b047f88 
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @pluckyswan
copy changes out of yesterday's internal branch
a21dca9 
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @pluckyswan
put application insights in the right section
3b227f6 
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@somesylvie @pluckyswan
change a setting to cause terraform changes
22b1a7d 
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@jherrflexion @saquino0827
@somesylvie @pluckyswan
Added GetPublishProfile step
d4b1aeb 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: Sylvie <38440028+somesylvie@users.noreply.github.com>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@jherrflexion @saquino0827
@somesylvie @pluckyswan
Renamed function step
e44e743 
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: Sylvie <38440028+somesylvie@users.noreply.github.com>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
somesylvie and others added 6 commits July 15, 2024 13:51
@somesylvie @jcrichlake
@jherrflexion @saquino0827
first try at a health check
f018414 
Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com>
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie @jcrichlake
@jherrflexion @saquino0827
just look for function name
51a88e4 
Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com>
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie @jcrichlake
@jherrflexion @saquino0827
let's break this again
d49c44a 
Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com>
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie @jcrichlake
@jherrflexion @saquino0827
turn off actual enqueueing for now, add some comments for later, remo…
29e3093 
…ve the not-working health check attempt

Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com>
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie @jcrichlake
@jherrflexion @saquino0827
README, ADR, and some cleanup
b864f28 
Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com>
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie @jcrichlake
@jherrflexion @saquino0827
do some cleanup
752b314 
Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com>
Co-Authored-By: Samuel Aquino <saquino@flexion.us>
@somesylvie somesylvie mentioned this pull request Jul 15, 2024
Closed
46 tasks
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jul 16, 2024

Quality Gate Passed Quality Gate passed

Issues
10 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@somesylvie somesylvie merged commit 85bf105 into main Jul 16, 2024
13 checks passed
@somesylvie somesylvie deleted the scheduling branch July 16, 2024 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saquino0827 saquino0827 saquino0827 left review comments

@pluckyswan pluckyswan pluckyswan approved these changes

@jherrflexion jherrflexion jherrflexion approved these changes

@jcrichlake jcrichlake jcrichlake approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@somesylvie @saquino0827 @pluckyswan @jherrflexion @jcrichlake