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

CRAN Down #83

Closed
joeHickson opened this issue Feb 18, 2021 · 9 comments
Closed

CRAN Down #83

joeHickson opened this issue Feb 18, 2021 · 9 comments

Comments

@joeHickson
Copy link

The cran dataverse package is no longer available for download
https://cran.r-project.org/web/packages/dataverse/index.html

@djbrooke
Copy link

Thanks @joeHickson for the report and @pdurbin for the ping. From a quick look, I think I see the issue.

@wibeasley @kuriwaki - about a year ago, we added token expiration after a year, so it's likely that the token from https://github.com/IQSS/dataverse-client-r/blob/master/inst/constants.yml just needs to be re-created. Let me know if there's anything I can do to help with this.

@kuriwaki
Copy link
Member

I thought this is about #77 so token expiration sounds right. Thank you @djbrooke. So you think I or @wibeasley can create a fresh token and replace the yaml?

@djbrooke
Copy link

Hey @kuriwaki -- based on the errors in https://cran-archive.r-project.org/web/checks/2021/2021-02-16_check_results_dataverse.html it seemed likely to me. I'd guess replacing the token and rerunning the tests would be a likely fix. I'm not sure if this resubmits to CRAN automatically or if there's some other step.

If it's not the token expiration, it would at least allow us to rule something out. :)

@kuriwaki
Copy link
Member

The SSL certificate problem: certificate has expired error in your link only happens in r-patched-solaris-x86, which is consistent with the issue I linked to (though even then I couldn't understand why that would happen in Solaris but not in all the linux/windows/mac OS they test on).

We would need to resubmit to the CRAN team through their form, so the earliest we can fix this is next week I think.

It'd be great if we could test this before we resubmit so we can make sure it'll pass. I'm not the maintainer so it seems I cannot use r-hub to test on Solaris easily.

@kuriwaki
Copy link
Member

Separately, the error log reports a host of errors in previously unproblematic places, for other OS. It looks like there is a simple mixup of the two of the Datasets in our test dataverse. Our tests use contents[[1]] to call a dataset, and maybe the order of the datasets returned in get_dataverse() changed underneath so we got the wrong one.

actual <- dataset_files(contents[[1]])

That said, I'm not sure why this would suddenly happen, since we did not change the test data. Also, devtools::check() returns no errors on my local.

kuriwaki added a commit that referenced this issue Feb 23, 2021
Rather than using [[1]] and [[2]], which may be unstable. #83 (comment)
@kuriwaki
Copy link
Member

I implemented a quick lookup in the tests to avoid the [[1]] / [[2]] ambiguity (09eb25a). This is on top of @wibeasley's edits early on to avoid the expiration issue (ff5a80e).

I ran devtools::check() locally and I get no errors or warnings. Just one NOTE that says "Package suggested but not available for checking: ‘UNF’"

wibeasley added a commit that referenced this issue Feb 23, 2021
wibeasley added a commit that referenced this issue Feb 25, 2021
@kuriwaki
Copy link
Member

kuriwaki commented Mar 4, 2021

Update here: The 0.3.5 resubmission passed CRAN's auto-checks today and is now under manual inspection.

@kuriwaki
Copy link
Member

kuriwaki commented Mar 8, 2021

It's back up as 0.3.7 with a couple of improvements. Thanks @wibeasley!

@kuriwaki kuriwaki closed this as completed Mar 8, 2021
@wibeasley
Copy link
Contributor

The R-hub problems aren't not really related to CRAN, but needed to be solved first.
r-hub/rhub#367 (comment)

rhub::check(
  env_vars=c(R_COMPILE_AND_INSTALL_PACKAGES = "always")
)

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

No branches or pull requests

4 participants