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

adds toggling refresh disable/enable for deactivate/activate operation while updating URL_DOWNLOAD type configs #1240

Merged
merged 4 commits into from
Aug 10, 2024

Conversation

eirsep
Copy link
Member

@eirsep eirsep commented Aug 9, 2024

adds toggling refresh disable/enable for deactivate/activate operation while updating URL_DOWNLOAD type configs

@jowg-amazon
Copy link
Collaborator

LGTM given CI's pass

AWSHurneyt
AWSHurneyt previously approved these changes Aug 10, 2024
@AWSHurneyt
Copy link
Collaborator

Approved assuming CIs pass.

amsiglan
amsiglan previously approved these changes Aug 10, 2024
amsiglan
amsiglan previously approved these changes Aug 10, 2024
AWSHurneyt
AWSHurneyt previously approved these changes Aug 10, 2024
eirsep added 4 commits August 10, 2024 02:11
…n while updating URL_DOWNLOAD type configs

Signed-off-by: Surya Sashank Nistala <[email protected]>
Signed-off-by: Surya Sashank Nistala <[email protected]>
… source config response

Signed-off-by: Surya Sashank Nistala <[email protected]>
@eirsep eirsep merged commit 3e1f59d into opensearch-project:main Aug 10, 2024
16 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 10, 2024
…n while updating URL_DOWNLOAD type configs (#1240)

* adds toggling refresh disable/enable for deactivate/activate operation while updating URL_DOWNLOAD type configs

Signed-off-by: Surya Sashank Nistala <[email protected]>

* add sleep for lock issue

Signed-off-by: Surya Sashank Nistala <[email protected]>

* wait on release lock operation completion before returning update tif source config response

Signed-off-by: Surya Sashank Nistala <[email protected]>

* reset flag after integ tests for SourceConfigWithoutS3RestApiIT

Signed-off-by: Surya Sashank Nistala <[email protected]>

---------

Signed-off-by: Surya Sashank Nistala <[email protected]>
(cherry picked from commit 3e1f59d)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 10, 2024
…n while updating URL_DOWNLOAD type configs (#1240)

* adds toggling refresh disable/enable for deactivate/activate operation while updating URL_DOWNLOAD type configs

Signed-off-by: Surya Sashank Nistala <[email protected]>

* add sleep for lock issue

Signed-off-by: Surya Sashank Nistala <[email protected]>

* wait on release lock operation completion before returning update tif source config response

Signed-off-by: Surya Sashank Nistala <[email protected]>

* reset flag after integ tests for SourceConfigWithoutS3RestApiIT

Signed-off-by: Surya Sashank Nistala <[email protected]>

---------

Signed-off-by: Surya Sashank Nistala <[email protected]>
(cherry picked from commit 3e1f59d)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 10, 2024
…n while updating URL_DOWNLOAD type configs (#1240)

* adds toggling refresh disable/enable for deactivate/activate operation while updating URL_DOWNLOAD type configs

Signed-off-by: Surya Sashank Nistala <[email protected]>

* add sleep for lock issue

Signed-off-by: Surya Sashank Nistala <[email protected]>

* wait on release lock operation completion before returning update tif source config response

Signed-off-by: Surya Sashank Nistala <[email protected]>

* reset flag after integ tests for SourceConfigWithoutS3RestApiIT

Signed-off-by: Surya Sashank Nistala <[email protected]>

---------

Signed-off-by: Surya Sashank Nistala <[email protected]>
(cherry picked from commit 3e1f59d)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
eirsep pushed a commit that referenced this pull request Aug 11, 2024
…n while updating URL_DOWNLOAD type configs (#1240) (#1245)

* adds toggling refresh disable/enable for deactivate/activate operation while updating URL_DOWNLOAD type configs



* add sleep for lock issue



* wait on release lock operation completion before returning update tif source config response



* reset flag after integ tests for SourceConfigWithoutS3RestApiIT



---------


(cherry picked from commit 3e1f59d)

Signed-off-by: Surya Sashank Nistala <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
amsiglan pushed a commit that referenced this pull request Aug 20, 2024
…n while updating URL_DOWNLOAD type configs (#1240) (#1246)

* adds toggling refresh disable/enable for deactivate/activate operation while updating URL_DOWNLOAD type configs



* add sleep for lock issue



* wait on release lock operation completion before returning update tif source config response



* reset flag after integ tests for SourceConfigWithoutS3RestApiIT



---------


(cherry picked from commit 3e1f59d)

Signed-off-by: Surya Sashank Nistala <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
amsiglan pushed a commit that referenced this pull request Aug 20, 2024
…n while updating URL_DOWNLOAD type configs (#1240) (#1244)

* adds toggling refresh disable/enable for deactivate/activate operation while updating URL_DOWNLOAD type configs



* add sleep for lock issue



* wait on release lock operation completion before returning update tif source config response



* reset flag after integ tests for SourceConfigWithoutS3RestApiIT



---------


(cherry picked from commit 3e1f59d)

Signed-off-by: Surya Sashank Nistala <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants