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

fix:remove osgi from karaf fixes Issue #1891 #1902

Merged
merged 1 commit into from
Sep 30, 2020

Conversation

davecramer
Copy link
Member

Currently we have no automated way to test this

@davecramer
Copy link
Member Author

@steinarb can you check this works on your end. I checked it out and it was fine.

you can build it and push it to your local repo using
./gradlew publishToMavenLocal -Ppgjdbc.version=1.0.0-dev-master -PskipJavadoc

then in karaf specify 1.0.0-dev-master-SNAPSHOT as the jdbc version.

Thanks!

@steinarb
Copy link
Contributor

steinarb commented Sep 30, 2020

@steinarb can you check this works on your end. I checked it out and it was fine.

you can build it and push it to your local repo using
./gradlew publishToMavenLocal -Ppgjdbc.version=1.0.0-dev-master -PskipJavadoc

then in karaf specify 1.0.0-dev-master-SNAPSHOT as the jdbc version.

Looks good!

Here's what I did:

  1. Installed the feature over an ssh connection. The connection wasn't terminated
    karaf@root()> feature:repo-add mvn:org.postgresql/postgresql/1.0.0-dev-master-SNAPSHOT/xml/features
    Adding feature url mvn:org.postgresql/postgresql/1.0.0-dev-master-SNAPSHOT/xml/features
    karaf@root()> feature:install postgresql
    karaf@root()>
    
  2. Used the snapshot in one of my karaf applications https://github.com/steinarb/authservice
  3. Stop karaf, delete the data directory, restart karaf (to lose the already installed postgresql feature)
  4. Installed authservice in karaf without any ill effects (used an ssh connection to the console here also)
  5. Did a smoke test of the installed authservice (verified that I could log in and changed my first name)
  6. Stopped and started karaf (without deleting or adding anything) and logged in again and verified that my first name was still changed
  7. Did a manual visual check of the feature.xml file in the local maven repo and it looks good to me (just loading the transaction-api feature and the bundle itself)

So it looks good! Thanks for fixing it!

@davecramer davecramer merged commit 01dbbac into pgjdbc:master Sep 30, 2020
@davecramer davecramer deleted the fixkaraf branch September 30, 2020 22:17
davecramer added a commit to davecramer/pgjdbc that referenced this pull request Oct 1, 2020
davecramer added a commit to davecramer/pgjdbc that referenced this pull request Oct 2, 2020
davecramer added a commit that referenced this pull request Oct 6, 2020
* fix: avoid removal type annotations on "this" so the source archive is buildable

"this" type annotations are Java 8+, so we no longer need to remove them.

* fix: PgTokenizer was ignoring last empty token (#1882)

* fix: PgTokenizer was ignoring last empty token fixes #1881

* fix: handle smallserial correctly fixes #1897 (#1899)

* feat: add smallserial metadata (#899)

PostgreSQL 9.2 adds a SMALLSERIAL data type, this reports the correct metadata information when a column is a smallserial (int2 with sequence), similar to how a serial or bigserial data types are reported.

* fix:remove osgi from karaf fixes Issue #1891 (#1902)

* Change default of gssEncMode to ALLOW. PostgreSQL can deal with PREFER but there are cloud providers that did not implement the protocol properly. Using PREFER seems to cause more problems than it solves

Co-authored-by: Vladimir Sitnikov <sitnikov.vladimir@gmail.com>
Co-authored-by: Jorge Solorzano <jorsol@gmail.com>
davecramer added a commit that referenced this pull request Oct 8, 2020
* fix: avoid removal type annotations on "this" so the source archive is buildable

"this" type annotations are Java 8+, so we no longer need to remove them.

* fix: PgTokenizer was ignoring last empty token (#1882)


* feat: add smallserial metadata (#899)

PostgreSQL 9.2 adds a SMALLSERIAL data type, this reports the correct metadata information when a column is a smallserial (int2 with sequence), similar to how a serial or bigserial data types are reported.

* fix:remove osgi from karaf fixes Issue #1891 (#1902)

* Change default of gssEncMode to ALLOW. PostgreSQL can deal with PREFER but there are cloud providers that did not implement the protocol properly

* update CHANGELOG and posts for 42.2.17

* fix up CHANGELOG

* Update README, gssEncMode for release

Co-authored-by: Vladimir Sitnikov <sitnikov.vladimir@gmail.com>
Co-authored-by: Jorge Solorzano <jorsol@gmail.com>
davecramer added a commit to davecramer/pgjdbc that referenced this pull request Nov 16, 2020
* fix: avoid removal type annotations on "this" so the source archive is buildable

"this" type annotations are Java 8+, so we no longer need to remove them.

* fix: PgTokenizer was ignoring last empty token (pgjdbc#1882)

* feat: add smallserial metadata (pgjdbc#899)

PostgreSQL 9.2 adds a SMALLSERIAL data type, this reports the correct metadata information when a column is a smallserial (int2 with sequence), similar to how a serial or bigserial data types are reported.

* fix:remove osgi from karaf fixes Issue pgjdbc#1891 (pgjdbc#1902)

* Change default of gssEncMode to ALLOW. PostgreSQL can deal with PREFER but there are cloud providers that did not implement the protocol properly

* update CHANGELOG and posts for 42.2.17

* fix up CHANGELOG

* Update README, gssEncMode for release

Co-authored-by: Vladimir Sitnikov <sitnikov.vladimir@gmail.com>
Co-authored-by: Jorge Solorzano <jorsol@gmail.com>
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

2 participants