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

Omwappi 1545 x dial server patches 2401 sprint #106

Open
wants to merge 3 commits into
base: 2401_sprint
Choose a base branch
from

Conversation

margret-jaison-infosys
Copy link

1)ARRISAPP-69 wait for application state responses (LibertyGlobal#16)

It is now possible to specify the amount of time
xdial server waits for rtremote status update resposes
with XDIAL_WAIT_FOR_RTREMOTE_STATE_RESPONSE_MS.
This can be used to make sure that up-to-date application
states are returned (instead of cached ones)

2)OMWAPPI-1342 xdial: create GDialApp if app exists (LibertyGlobal#23)

if app instances are created outside of xdial, we might
still be missing GDialApp instance.
do not fetch app state asynchronously in case
xdial is configured to wait for remote state responses
anyway
if current app state is 'stopped' or 'hidden'
always return 201 on POST requests
3)OMWAPPI-1360 introduce XDIAL_MAX_STATE_VALUE_AGE_MS (LibertyGlobal#24)

To limit synchronous app state polling (that takes time),
XDIAL_MAX_STATE_VALUE_AGE_MS env variable can be set to
specify how long the last state update is still
considered 'current'; until this time passes the last
cached state is returned and the remote state is not queried.

ONEMPERS-642: stop and hide results with 404 fixed (https://github.com/rdkcentral/xdialserver/pull/17)(rtcache.cpp)
cache objects marked as markUnevictable(, true) to not remove them from cache

It is now possible to specify the amount of time
xdial server waits for rtremote status update resposes
with XDIAL_WAIT_FOR_RTREMOTE_STATE_RESPONSE_MS.
This can be used to make sure that up-to-date application
states are returned (instead of cached ones)
- if app instances are created outside of xdial, we might
  still be missing GDialApp instance.
- do not fetch app state asynchronously in case
  xdial is configured to wait for remote state responses
  anyway
- if current app state is 'stopped' or 'hidden'
  always return 201 on POST requests
@margret-jaison-infosys margret-jaison-infosys force-pushed the OMWAPPI-1545-XDialServer-patches-2401_sprint branch from dabbf3e to 3b54fbb Compare April 3, 2024 09:48
@pradeeptakdas
Copy link

Internal ticket created for tracking https://ccp.sys.comcast.net/browse/RDKCOM-4665

@viveksinghnarwaria
Copy link

Hi @margret-jaison-infosys , Can you please bring the change to the latest sprint branch 24Q2_sprint ?

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.

None yet

3 participants