:wave: I was testing out `flytectl` with `DeviceFl...
# ask-the-community
n
👋 I was testing out
flytectl
with
DeviceFlow
auth. It seems like it doesn't properly record the expiration so it requires a re-auth on every request. Has anyone else ran into this issue?
Copy code
Error: failed to refresh token using cached token. Error: oauth2: token expired and refresh token is not set
Actually I just found https://github.com/flyteorg/flyte/pull/4319 which seems to have a fix for it but hasn't gotten much attention.
c
This issue may also be related? https://github.com/flyteorg/flyte/issues/3288
n
Yeah, seems to be issue related to that PR I referenced. Is there a process to help get this more attention from the maintainers of the repository? Seems like it would be a good fix to get included in an upcoming release.
c
Ah yeah just realized that the PR is referring to the same issue. 🙂
d
@Nick Tate can you give the Issue a 👍🏽 ? we're trying to establish a lightweight mechanism for the community to signal their priorities and this is one of them. I just also gave it a label that will help continue the discussion with the maintainers
ah the PR, I just requested reviews, sorry
n
Thank you 🙇
y
will merge the change once tests pass
n
Thanks for getting this merged in 🙇 One question I had was around the release process. I see that
flytectl
is now bundled in the
flyte
monorepo but we still have the separate repository for
flytectl
as well that looks active (and possibly what the new releases are based off of).
If we cut based off of the standalone repo, I think will need to upgrade the
flytesdlib
dependency to incorporate this change. Happy to do that, but just wanted to verify the correct repository the releases were built off of first.
d
@Eduardo Apolinario (eapolinario)
e
I'm working on a change to allow us to release flytectl from the monorepo. I hope to get it out by EOW. Once that is merged I'm going to port the remaining changes from the flytectl repo and archive it to decrease the confusion.
The first PR is out: https://github.com/flyteorg/flyte/pull/5354. After that's merged I'm going to work on another PR to actually generate the release. I hope to get it out by Monday 10/13.
This is out.
flytectl
is now released from the monorepo and the original repo is archived. Starting from version
v0.8.20
all releases will come from the monorepo.