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

v1.5.0 release candidate #812

Merged
merged 10 commits into from
Jun 5, 2024
Merged

v1.5.0 release candidate #812

merged 10 commits into from
Jun 5, 2024

Conversation

simonpcouch
Copy link
Collaborator

Closes #811, closes #807. Two notes:

  • I wasn't sure whether to make this release patch or minor, and
  • Needs a version bump in DESCRIPTION before submission.

Depending on where the Snowflake migration is at, these may be the first GHA to fail. The offending tests use:

  pwd <- Sys.getenv("ODBC_PWD_SNOWFLAKE")
  if (nchar(pwd) == 0) {
    skip("Secret ODBC_PWD_SNOWFLAKE not available.")
  }

...so will not be an issue on CRAN.

```
✖ Error: vignettes/develop.Rmd:25:181 404: Not Found
Posit employees have access to the [Posit Professional drivers](https://docs.posit.co/pro-drivers/) and the infrastructure used to build and test them in the [rstudio/pro-drivers](https://github.com/rstudio/pro-drivers) repo. The Posit Professional drivers are a set of drivers vendored from Magnitude Simba that support many of the most popular DBMS, including SQL Server, Oracle, Redshift, Databricks, Snowflake, etc. The repository they're developed in contains tooling to spin up a number of databases in docker containers to test against.
                                                                                                                                                                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
✖ Error: vignettes/develop.Rmd:35:47 404: Not Found
Among other things, the [rstudio/pro-drivers](https://github.com/rstudio/pro-drivers) repo defines a `MAKE` tool for setting up and tearing down databases in docker containers. Ensure that you have a docker daemon running (i.e. Docker Desktop open) and, if you're on macOS aarch64, have `Settings > Use Rosetta for x86_64/amd64 emulation on Apple Silicon` enabled. To start a container for a given `dbms`, run `MAKE dist=none DB=dbms up`, and **tear it down** with `MAKE dist=none DB=db down`. To see available `dbms` options, see the names of `.yml` files in the `docker-compose` directory. Find connection details for each database in `docker/shared/odbc.ini`.
                                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
✖ Error: vignettes/develop.Rmd:316:60 404: Not Found
If you're a Posit employee, you should have access to the [https://github.com/rstudio/warehouse](https://github.com/rstudio/warehouse) repository. Follow the instructions there to get access to the internal Redshift cluster. Access to the cluster is only enabled through Posit's internal Workbench instance, where the professional drivers will already be installed.
                                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
✖ Error: README.md:22:10 403: Forbidden
[Oracle](https://www.oracle.com/database),
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
@simonpcouch simonpcouch requested a review from hadley May 29, 2024 13:40
@simonpcouch simonpcouch changed the title v1.4.3 release candidates v1.4.3 release candidate May 29, 2024
@hadley
Copy link
Member

hadley commented May 29, 2024

Probably should be a minor release, I think.

@simonpcouch simonpcouch changed the title v1.4.3 release candidate v1.5.0 release candidate May 29, 2024
@simonpcouch
Copy link
Collaborator Author

You got it. Still needs the DESCRIPTION version bump to 1.5.0 when you submit.

@hadley
Copy link
Member

hadley commented May 31, 2024

And submitted 😄

@simonpcouch
Copy link
Collaborator Author

Seeing:

   ══ Failed tests ════════════════════════════════════════════════════════════════
    ── Failure ('test-utils.R:89:3'): set_odbcsysini() works (#791) ────────────────
    identical(Sys.getenv("ODBCSYSINI"), "") is not FALSE

Will try to see if I can reproduce / get some more information out of macbuilder.

mac-builder appears to be down today (possibly as a result of CRAN being down earlier this morning?) and I'm [unable to replicate the failures on R-hub](https://github.com/r-hub2/indirect-dragonfly-odbc/actions/runs/9350831269). Will skip those tests on CRAN for this release.
@simonpcouch
Copy link
Collaborator Author

simonpcouch commented Jun 3, 2024

mac-builder appears to be down today (possibly as a result of CRAN being down earlier this morning?) and I'm unable to replicate the failures on R-hub. Suggesting that we skip those tests on CRAN for this release.

EDIT: forgot to mention, those R-hub builds are with:

test_that("set_odbcsysini() works (#791)", {
  skip_if(is_windows())

  # expect_false(identical(Sys.getenv("ODBCSYSINI"), ""))
  path <- dirname(odbcListConfig()[["drivers"]])
  Sys.setenv(ODBCSYSINI = path)

  expect_equal(Sys.getenv("ODBCSYSINI"), path)
})

@hadley
Copy link
Member

hadley commented Jun 3, 2024

Resubmitted

@simonpcouch simonpcouch merged commit 9924af6 into main Jun 5, 2024
14 of 16 checks passed
@simonpcouch simonpcouch deleted the v1.4.3-rc branch June 5, 2024 19:11
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.

Release odbc 1.5.0 CRAN NOTE on installed package size on macOS
2 participants