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

Add aap-eda mime type for uploads #449

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

hsong-rh
Copy link
Contributor

What?

Add aap-eda to the upload mime types

https://issues.redhat.com/browse/AAP-30055

@dehort
Copy link
Contributor

dehort commented Sep 11, 2024

@hsong-rh This change looks fine. Do you know how big the uploads will be? How frequently will the uploads be sent? How many hosts will be sending the uploads and over what time period?

@dehort
Copy link
Contributor

dehort commented Sep 11, 2024

/retest

@hsong-rh
Copy link
Contributor Author

@hsong-rh This change looks fine. Do you know how big the uploads will be? How frequently will the uploads be sent? How many hosts will be sending the uploads and over what time period?

@dehort We will use the insights-analytics-collector repo to collect and upload data. The maximum file size is 100MB. For a big table, it may be split into multiple files. The collection will be scheduled every 4 hours. So far we only have one scheduler host.

@hsong-rh
Copy link
Contributor Author

@dehort Any suggestions for this PR to be merged?

@dehort
Copy link
Contributor

dehort commented Sep 16, 2024

/retest

@dehort
Copy link
Contributor

dehort commented Sep 16, 2024

@hsong-rh right now you only have upload coming from one host. Is that right? How many hosts do you expect to be uploading these files in the future?

@hsong-rh
Copy link
Contributor Author

@hsong-rh right now you only have upload coming from one host. Is that right? How many hosts do you expect to be uploading these files in the future?

Yes, there is only one host now. For the future, I am not sure, @Ladas may have answers.

@dehort dehort merged commit 3a1103f into RedHatInsights:master Sep 16, 2024
9 checks passed
@Ladas
Copy link
Contributor

Ladas commented Sep 16, 2024

@dehort the estimate is that max few hundreds of hosts would be sending data. (up to ±1500 longer term)

By default they will send data every 4h. The payload size per host is yet to be determined. (shouldn't be large from start)

@Ladas
Copy link
Contributor

Ladas commented Sep 17, 2024

@dehort quick question, should this be autoreleased on Stage already? (and we need manual prod deploy scheduled?)

@dehort
Copy link
Contributor

dehort commented Sep 17, 2024

@Ladas It should have been autoreleased to Stage, but it looks like it didn't. I'll look into getting this pushed out to stage tomorrow.

@hsong-rh
Copy link
Contributor Author

@dehort thank you, please let us know when it's pushed

@dehort
Copy link
Contributor

dehort commented Sep 18, 2024

@Ladas @hsong-rh this change has been deployed to stage. I am not sure why it didn't work previously. 🤷

@hsong-rh
Copy link
Contributor Author

@Ladas @hsong-rh this change has been deployed to stage. I am not sure why it didn't work previously. 🤷

@dehort It worked now, I saw the log record in the Kibana board. thank you!

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.

3 participants